• Eric Biggers's avatar
    fscrypt: new helper functions for ->symlink() · 76e81d6d
    Eric Biggers authored
    Currently, filesystems supporting fscrypt need to implement some tricky
    logic when creating encrypted symlinks, including handling a peculiar
    on-disk format (struct fscrypt_symlink_data) and correctly calculating
    the size of the encrypted symlink.  Introduce helper functions to make
    things a bit easier:
    
    - fscrypt_prepare_symlink() computes and validates the size the symlink
      target will require on-disk.
    - fscrypt_encrypt_symlink() creates the encrypted target if needed.
    
    The new helpers actually fix some subtle bugs.  First, when checking
    whether the symlink target was too long, filesystems didn't account for
    the fact that the NUL padding is meant to be truncated if it would cause
    the maximum length to be exceeded, as is done for filenames in
    directories.  Consequently users would receive ENAMETOOLONG when
    creating symlinks close to what is supposed to be the maximum length.
    For example, with EXT4 with a 4K block size, the maximum symlink target
    length in an encrypted directory is supposed to be 4093 bytes (in
    comparison to 4095 in an unencrypted directory), but in
    FS_POLICY_FLAGS_PAD_32-mode only up to 4064 bytes were accepted.
    
    Second, symlink targets of "." and ".." were not being encrypted, even
    though they should be, as these names are special in *directory entries*
    but not in symlink targets.  Fortunately, we can fix this simply by
    starting to encrypt them, as old kernels already accept them in
    encrypted form.
    
    Third, the output string length the filesystems were providing when
    doing the actual encryption was incorrect, as it was forgotten to
    exclude 'sizeof(struct fscrypt_symlink_data)'.  Fortunately though, this
    bug didn't make a difference.
    Signed-off-by: default avatarEric Biggers <ebiggers@google.com>
    Signed-off-by: default avatarTheodore Ts'o <tytso@mit.edu>
    76e81d6d
fname.c 11.8 KB