Skip to content

fscrypt preparations for encryption+casefolding

Eric Biggers linux-ext4 linux-f2fs-devel linux-fscrypt linux-fsdevel linux-mtd lkml 2 versions In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v5 Latest6 patches 5 repliesIn review
  2. v44 patches 0 repliesSuperseded

Changes in v5 from the cover letter

- Fixed UBIFS encryption to work with the new no-key name format.

Patches in v5 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/6fscrypt: don't allow v1 policies with casefoldingDocumentation/filesystems/fscrypt.rst fs/crypto/hooks.c fs/crypto/policy.c +2 more+49 -20 replies
2/6fscrypt: derive dirhash key for casefolded directoriesDocumentation/filesystems/fscrypt.rst fs/crypto/fname.c fs/crypto/fscrypt_private.h +3 more+109 -140 replies
3/6fscrypt: clarify what is meant by a per-file keyDocumentation/filesystems/fscrypt.rst fs/crypto/fscrypt_private.h fs/crypto/keysetup.c +1 more+37 -361 reply
4/6ubifs: don't trigger assertion on invalid no-key filenamefs/ubifs/dir.c+2 -02 replies
5/6ubifs: allow both hash and disk name to be provided in no-key namesfs/ubifs/dir.c fs/ubifs/journal.c fs/ubifs/key.h+3 -60 replies
6/6fscrypt: improve format of no-key namesDocumentation/filesystems/fscrypt.rst fs/crypto/Kconfig fs/crypto/fname.c +1 more+171 -1260 replies

Reviews trailers per patch

1 of 6 patches has a Reviewed-by or Acked-by.

PatchDaniel Rosenberg
1/6
2/6
3/6R-b
4/6
5/6
6/6

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v5 thread

Cover letter [PATCH v5 0/6] fscrypt preparations for encryption+casefolding

This is a cleaned up and fixed version of the fscrypt patches to prepare for directories that are both encrypted and casefolded.

Patches 1-3 start deriving a SipHash key for the new dirhash method that will be used by encrypted+casefolded directories. To avoid unnecessary overhead, we only do this if the directory is actually casefolded.

Patch 4 fixes a bug in UBIFS where it didn't gracefully handle invalid hash values in fscrypt no-key names. This is an existing bug, but the new fscrypt no-key name format (patch 6) made it much easier to trigger; it started being hit by 'kvm-xfstests -c ubifs -g encrypt'.

Patch 5 updates UBIFS to make it ready for the new fscrypt no-key name format that always includes the dirhash.

Patch 6 modifies the fscrypt no-key names to always include the dirhash, since with the new dirhash method the dirhash will no longer be computable from the ciphertext filename without the key. It also fixes a longstanding issue where there could be collisions in the no-key names, due to not using a proper cryptographic hash to abbreviate names.

For more information see the main patch series, which includes the filesystem-specific changes:
https://lkml.kernel.org/linux-fscrypt/[email protected]/T/#u

This applies to fscrypt.git#master.

Changed v4 => v5:
- Fixed UBIFS encryption to work with the new no-key name format.

Daniel Rosenberg (3):
fscrypt: don't allow v1 policies with casefolding
fscrypt: derive dirhash key for casefolded directories
fscrypt: improve format of no-key names

Eric Biggers (3):
fscrypt: clarify what is meant by a per-file key
ubifs: don't trigger assertion on invalid no-key filename
ubifs: allow both hash and disk name to be provided in no-key names

 Documentation/filesystems/fscrypt.rst |  40 +++--
 fs/crypto/Kconfig                     |   1 +
 fs/crypto/fname.c                     | 239 ++++++++++++++++++++------
 fs/crypto/fscrypt_private.h           |  19 +-
 fs/crypto/hooks.c                     |  44 +++++
 fs/crypto/keysetup.c                  |  81 ++++++---
 fs/crypto/keysetup_v1.c               |   4 +-
 fs/crypto/policy.c                    |   7 +
 fs/inode.c                            |   3 +-
 fs/ubifs/dir.c                        |   6 +-
 fs/ubifs/journal.c                    |   4 +-
 fs/ubifs/key.h                        |   1 -
 include/linux/fscrypt.h               |  94 +++-------
 13 files changed, 365 insertions(+), 178 deletions(-)
Signature · 1 line
2.25.0