fscrypt: use FSCRYPT_* definitions, not FS_*

Update fs/crypto/ to use the new names for the UAPI constants rather
than the old names, then make the old definitions conditional on
!__KERNEL__.

Reviewed-by: Theodore Ts'o <tytso@mit.edu>
Signed-off-by: Eric Biggers <ebiggers@google.com>

diff --git a/fs/crypto/crypto.c b/fs/crypto/crypto.c
index 3e4624cfe4b54db6d8bd824cfa8a74b479a25ac1..7502c1f0ede9e98c8437475b228754824ef0de6a 100644 (file)
--- a/fs/crypto/crypto.c
+++ b/fs/crypto/crypto.c
@@ -141,7 +141,7 @@ void fscrypt_generate_iv(union fscrypt_iv *iv, u64 lblk_num,
        memset(iv, 0, ci->ci_mode->ivsize);
        iv->lblk_num = cpu_to_le64(lblk_num);
 
-       if (ci->ci_flags & FS_POLICY_FLAG_DIRECT_KEY)
+       if (ci->ci_flags & FSCRYPT_POLICY_FLAG_DIRECT_KEY)
                memcpy(iv->nonce, ci->ci_nonce, FS_KEY_DERIVATION_NONCE_SIZE);
 
        if (ci->ci_essiv_tfm != NULL)

diff --git a/fs/crypto/fname.c b/fs/crypto/fname.c
index 5cab3bb2d1fc00860133ef81397728918b50cdb5..f4977d44d69b8111941df38ada1c7865b7adc5c1 100644 (file)
--- a/fs/crypto/fname.c
+++ b/fs/crypto/fname.c
@@ -182,7 +182,7 @@ bool fscrypt_fname_encrypted_size(const struct inode *inode, u32 orig_len,
                                  u32 max_len, u32 *encrypted_len_ret)
 {
        int padding = 4 << (inode->i_crypt_info->ci_flags &
-                           FS_POLICY_FLAGS_PAD_MASK);
+                           FSCRYPT_POLICY_FLAGS_PAD_MASK);
        u32 encrypted_len;
 
        if (orig_len > max_len)

diff --git a/fs/crypto/fscrypt_private.h b/fs/crypto/fscrypt_private.h
index 4d715708c6e1f7e12c61b8bd28a53d2b1022f5c7..fae411b2f78dcbe27b977aa29c8632a0d2c262fa 100644 (file)
--- a/fs/crypto/fscrypt_private.h
+++ b/fs/crypto/fscrypt_private.h
@@ -34,7 +34,7 @@ struct fscrypt_context {
        u8 contents_encryption_mode;
        u8 filenames_encryption_mode;
        u8 flags;
-       u8 master_key_descriptor[FS_KEY_DESCRIPTOR_SIZE];
+       u8 master_key_descriptor[FSCRYPT_KEY_DESCRIPTOR_SIZE];
        u8 nonce[FS_KEY_DERIVATION_NONCE_SIZE];
 } __packed;
 
@@ -84,7 +84,7 @@ struct fscrypt_info {
        u8 ci_data_mode;
        u8 ci_filename_mode;
        u8 ci_flags;
-       u8 ci_master_key_descriptor[FS_KEY_DESCRIPTOR_SIZE];
+       u8 ci_master_key_descriptor[FSCRYPT_KEY_DESCRIPTOR_SIZE];
        u8 ci_nonce[FS_KEY_DERIVATION_NONCE_SIZE];
 };
 
@@ -98,16 +98,16 @@ typedef enum {
 static inline bool fscrypt_valid_enc_modes(u32 contents_mode,
                                           u32 filenames_mode)
 {
-       if (contents_mode == FS_ENCRYPTION_MODE_AES_128_CBC &&
-           filenames_mode == FS_ENCRYPTION_MODE_AES_128_CTS)
+       if (contents_mode == FSCRYPT_MODE_AES_128_CBC &&
+           filenames_mode == FSCRYPT_MODE_AES_128_CTS)
                return true;
 
-       if (contents_mode == FS_ENCRYPTION_MODE_AES_256_XTS &&
-           filenames_mode == FS_ENCRYPTION_MODE_AES_256_CTS)
+       if (contents_mode == FSCRYPT_MODE_AES_256_XTS &&
+           filenames_mode == FSCRYPT_MODE_AES_256_CTS)
                return true;
 
-       if (contents_mode == FS_ENCRYPTION_MODE_ADIANTUM &&
-           filenames_mode == FS_ENCRYPTION_MODE_ADIANTUM)
+       if (contents_mode == FSCRYPT_MODE_ADIANTUM &&
+           filenames_mode == FSCRYPT_MODE_ADIANTUM)
                return true;
 
        return false;

diff --git a/fs/crypto/keyinfo.c b/fs/crypto/keyinfo.c
index 2129943002335cafa7ce495c22973de6b6fc35e2..22345ddede1199ffddaa461495af2558eeef1ed6 100644 (file)
--- a/fs/crypto/keyinfo.c
+++ b/fs/crypto/keyinfo.c
@@ -20,7 +20,7 @@
 
 static struct crypto_shash *essiv_hash_tfm;
 
-/* Table of keys referenced by FS_POLICY_FLAG_DIRECT_KEY policies */
+/* Table of keys referenced by DIRECT_KEY policies */
 static DEFINE_HASHTABLE(fscrypt_master_keys, 6); /* 6 bits = 64 buckets */
 static DEFINE_SPINLOCK(fscrypt_master_keys_lock);
 
@@ -77,7 +77,7 @@ static int derive_key_aes(const u8 *master_key,
  */
 static struct key *
 find_and_lock_process_key(const char *prefix,
-                         const u8 descriptor[FS_KEY_DESCRIPTOR_SIZE],
+                         const u8 descriptor[FSCRYPT_KEY_DESCRIPTOR_SIZE],
                          unsigned int min_keysize,
                          const struct fscrypt_key **payload_ret)
 {
@@ -87,7 +87,7 @@ find_and_lock_process_key(const char *prefix,
        const struct fscrypt_key *payload;
 
        description = kasprintf(GFP_NOFS, "%s%*phN", prefix,
-                               FS_KEY_DESCRIPTOR_SIZE, descriptor);
+                               FSCRYPT_KEY_DESCRIPTOR_SIZE, descriptor);
        if (!description)
                return ERR_PTR(-ENOMEM);
 
@@ -105,7 +105,7 @@ find_and_lock_process_key(const char *prefix,
        payload = (const struct fscrypt_key *)ukp->data;
 
        if (ukp->datalen != sizeof(struct fscrypt_key) ||
-           payload->size < 1 || payload->size > FS_MAX_KEY_SIZE) {
+           payload->size < 1 || payload->size > FSCRYPT_MAX_KEY_SIZE) {
                fscrypt_warn(NULL,
                             "key with description '%s' has invalid payload",
                             key->description);
@@ -129,32 +129,32 @@ find_and_lock_process_key(const char *prefix,
 }
 
 static struct fscrypt_mode available_modes[] = {
-       [FS_ENCRYPTION_MODE_AES_256_XTS] = {
+       [FSCRYPT_MODE_AES_256_XTS] = {
                .friendly_name = "AES-256-XTS",
                .cipher_str = "xts(aes)",
                .keysize = 64,
                .ivsize = 16,
        },
-       [FS_ENCRYPTION_MODE_AES_256_CTS] = {
+       [FSCRYPT_MODE_AES_256_CTS] = {
                .friendly_name = "AES-256-CTS-CBC",
                .cipher_str = "cts(cbc(aes))",
                .keysize = 32,
                .ivsize = 16,
        },
-       [FS_ENCRYPTION_MODE_AES_128_CBC] = {
+       [FSCRYPT_MODE_AES_128_CBC] = {
                .friendly_name = "AES-128-CBC",
                .cipher_str = "cbc(aes)",
                .keysize = 16,
                .ivsize = 16,
                .needs_essiv = true,
        },
-       [FS_ENCRYPTION_MODE_AES_128_CTS] = {
+       [FSCRYPT_MODE_AES_128_CTS] = {
                .friendly_name = "AES-128-CTS-CBC",
                .cipher_str = "cts(cbc(aes))",
                .keysize = 16,
                .ivsize = 16,
        },
-       [FS_ENCRYPTION_MODE_ADIANTUM] = {
+       [FSCRYPT_MODE_ADIANTUM] = {
                .friendly_name = "Adiantum",
                .cipher_str = "adiantum(xchacha12,aes)",
                .keysize = 32,
@@ -192,7 +192,7 @@ static int find_and_derive_key(const struct inode *inode,
        const struct fscrypt_key *payload;
        int err;
 
-       key = find_and_lock_process_key(FS_KEY_DESC_PREFIX,
+       key = find_and_lock_process_key(FSCRYPT_KEY_DESC_PREFIX,
                                        ctx->master_key_descriptor,
                                        mode->keysize, &payload);
        if (key == ERR_PTR(-ENOKEY) && inode->i_sb->s_cop->key_prefix) {
@@ -203,7 +203,7 @@ static int find_and_derive_key(const struct inode *inode,
        if (IS_ERR(key))
                return PTR_ERR(key);
 
-       if (ctx->flags & FS_POLICY_FLAG_DIRECT_KEY) {
+       if (ctx->flags & FSCRYPT_POLICY_FLAG_DIRECT_KEY) {
                if (mode->ivsize < offsetofend(union fscrypt_iv, nonce)) {
                        fscrypt_warn(inode,
                                     "Direct key mode not allowed with %s",
@@ -272,14 +272,14 @@ allocate_skcipher_for_mode(struct fscrypt_mode *mode, const u8 *raw_key,
        return ERR_PTR(err);
 }
 
-/* Master key referenced by FS_POLICY_FLAG_DIRECT_KEY policy */
+/* Master key referenced by DIRECT_KEY policy */
 struct fscrypt_master_key {
        struct hlist_node mk_node;
        refcount_t mk_refcount;
        const struct fscrypt_mode *mk_mode;
        struct crypto_skcipher *mk_ctfm;
-       u8 mk_descriptor[FS_KEY_DESCRIPTOR_SIZE];
-       u8 mk_raw[FS_MAX_KEY_SIZE];
+       u8 mk_descriptor[FSCRYPT_KEY_DESCRIPTOR_SIZE];
+       u8 mk_raw[FSCRYPT_MAX_KEY_SIZE];
 };
 
 static void free_master_key(struct fscrypt_master_key *mk)
@@ -320,13 +320,13 @@ find_or_insert_master_key(struct fscrypt_master_key *to_insert,
         * raw key, and use crypto_memneq() when comparing raw keys.
         */
 
-       BUILD_BUG_ON(sizeof(hash_key) > FS_KEY_DESCRIPTOR_SIZE);
+       BUILD_BUG_ON(sizeof(hash_key) > FSCRYPT_KEY_DESCRIPTOR_SIZE);
        memcpy(&hash_key, ci->ci_master_key_descriptor, sizeof(hash_key));
 
        spin_lock(&fscrypt_master_keys_lock);
        hash_for_each_possible(fscrypt_master_keys, mk, mk_node, hash_key) {
                if (memcmp(ci->ci_master_key_descriptor, mk->mk_descriptor,
-                          FS_KEY_DESCRIPTOR_SIZE) != 0)
+                          FSCRYPT_KEY_DESCRIPTOR_SIZE) != 0)
                        continue;
                if (mode != mk->mk_mode)
                        continue;
@@ -370,7 +370,7 @@ fscrypt_get_master_key(const struct fscrypt_info *ci, struct fscrypt_mode *mode,
                goto err_free_mk;
        }
        memcpy(mk->mk_descriptor, ci->ci_master_key_descriptor,
-              FS_KEY_DESCRIPTOR_SIZE);
+              FSCRYPT_KEY_DESCRIPTOR_SIZE);
        memcpy(mk->mk_raw, raw_key, mode->keysize);
 
        return find_or_insert_master_key(mk, raw_key, mode, ci);
@@ -448,8 +448,8 @@ static int init_essiv_generator(struct fscrypt_info *ci, const u8 *raw_key,
 
 /*
  * Given the encryption mode and key (normally the derived key, but for
- * FS_POLICY_FLAG_DIRECT_KEY mode it's the master key), set up the inode's
- * symmetric cipher transform object(s).
+ * DIRECT_KEY mode it's the master key), set up the inode's symmetric cipher
+ * transform object(s).
  */
 static int setup_crypto_transform(struct fscrypt_info *ci,
                                  struct fscrypt_mode *mode,
@@ -459,7 +459,7 @@ static int setup_crypto_transform(struct fscrypt_info *ci,
        struct crypto_skcipher *ctfm;
        int err;
 
-       if (ci->ci_flags & FS_POLICY_FLAG_DIRECT_KEY) {
+       if (ci->ci_flags & FSCRYPT_POLICY_FLAG_DIRECT_KEY) {
                mk = fscrypt_get_master_key(ci, mode, raw_key, inode);
                if (IS_ERR(mk))
                        return PTR_ERR(mk);
@@ -476,7 +476,7 @@ static int setup_crypto_transform(struct fscrypt_info *ci,
        if (mode->needs_essiv) {
                /* ESSIV implies 16-byte IVs which implies !DIRECT_KEY */
                WARN_ON(mode->ivsize != AES_BLOCK_SIZE);
-               WARN_ON(ci->ci_flags & FS_POLICY_FLAG_DIRECT_KEY);
+               WARN_ON(ci->ci_flags & FSCRYPT_POLICY_FLAG_DIRECT_KEY);
 
                err = init_essiv_generator(ci, raw_key, mode->keysize);
                if (err) {
@@ -530,9 +530,10 @@ int fscrypt_get_encryption_info(struct inode *inode)
                /* Fake up a context for an unencrypted directory */
                memset(&ctx, 0, sizeof(ctx));
                ctx.format = FS_ENCRYPTION_CONTEXT_FORMAT_V1;
-               ctx.contents_encryption_mode = FS_ENCRYPTION_MODE_AES_256_XTS;
-               ctx.filenames_encryption_mode = FS_ENCRYPTION_MODE_AES_256_CTS;
-               memset(ctx.master_key_descriptor, 0x42, FS_KEY_DESCRIPTOR_SIZE);
+               ctx.contents_encryption_mode = FSCRYPT_MODE_AES_256_XTS;
+               ctx.filenames_encryption_mode = FSCRYPT_MODE_AES_256_CTS;
+               memset(ctx.master_key_descriptor, 0x42,
+                      FSCRYPT_KEY_DESCRIPTOR_SIZE);
        } else if (res != sizeof(ctx)) {
                fscrypt_warn(inode,
                             "Unknown encryption context size (%d bytes)", res);
@@ -545,7 +546,7 @@ int fscrypt_get_encryption_info(struct inode *inode)
                return -EINVAL;
        }
 
-       if (ctx.flags & ~FS_POLICY_FLAGS_VALID) {
+       if (ctx.flags & ~FSCRYPT_POLICY_FLAGS_VALID) {
                fscrypt_warn(inode, "Unknown encryption context flags (0x%02x)",
                             ctx.flags);
                return -EINVAL;
@@ -559,7 +560,7 @@ int fscrypt_get_encryption_info(struct inode *inode)
        crypt_info->ci_data_mode = ctx.contents_encryption_mode;
        crypt_info->ci_filename_mode = ctx.filenames_encryption_mode;
        memcpy(crypt_info->ci_master_key_descriptor, ctx.master_key_descriptor,
-              FS_KEY_DESCRIPTOR_SIZE);
+              FSCRYPT_KEY_DESCRIPTOR_SIZE);
        memcpy(crypt_info->ci_nonce, ctx.nonce, FS_KEY_DERIVATION_NONCE_SIZE);
 
        mode = select_encryption_mode(crypt_info, inode);

diff --git a/fs/crypto/policy.c b/fs/crypto/policy.c
index 4941fe8471ceff03cea2032890b0e16318eff8fb..da7ae9c8b4ad0ba322b1197ee06fcb4446e27d81 100644 (file)
--- a/fs/crypto/policy.c
+++ b/fs/crypto/policy.c
@@ -22,7 +22,7 @@ static bool is_encryption_context_consistent_with_policy(
                                const struct fscrypt_policy *policy)
 {
        return memcmp(ctx->master_key_descriptor, policy->master_key_descriptor,
-                     FS_KEY_DESCRIPTOR_SIZE) == 0 &&
+                     FSCRYPT_KEY_DESCRIPTOR_SIZE) == 0 &&
                (ctx->flags == policy->flags) &&
                (ctx->contents_encryption_mode ==
                 policy->contents_encryption_mode) &&
@@ -37,13 +37,13 @@ static int create_encryption_context_from_policy(struct inode *inode,
 
        ctx.format = FS_ENCRYPTION_CONTEXT_FORMAT_V1;
        memcpy(ctx.master_key_descriptor, policy->master_key_descriptor,
-                                       FS_KEY_DESCRIPTOR_SIZE);
+                                       FSCRYPT_KEY_DESCRIPTOR_SIZE);
 
        if (!fscrypt_valid_enc_modes(policy->contents_encryption_mode,
                                     policy->filenames_encryption_mode))
                return -EINVAL;
 
-       if (policy->flags & ~FS_POLICY_FLAGS_VALID)
+       if (policy->flags & ~FSCRYPT_POLICY_FLAGS_VALID)
                return -EINVAL;
 
        ctx.contents_encryption_mode = policy->contents_encryption_mode;
@@ -128,7 +128,7 @@ int fscrypt_ioctl_get_policy(struct file *filp, void __user *arg)
        policy.filenames_encryption_mode = ctx.filenames_encryption_mode;
        policy.flags = ctx.flags;
        memcpy(policy.master_key_descriptor, ctx.master_key_descriptor,
-                               FS_KEY_DESCRIPTOR_SIZE);
+                               FSCRYPT_KEY_DESCRIPTOR_SIZE);
 
        if (copy_to_user(arg, &policy, sizeof(policy)))
                return -EFAULT;
@@ -202,7 +202,7 @@ int fscrypt_has_permitted_context(struct inode *parent, struct inode *child)
        if (parent_ci && child_ci) {
                return memcmp(parent_ci->ci_master_key_descriptor,
                              child_ci->ci_master_key_descriptor,
-                             FS_KEY_DESCRIPTOR_SIZE) == 0 &&
+                             FSCRYPT_KEY_DESCRIPTOR_SIZE) == 0 &&
                        (parent_ci->ci_data_mode == child_ci->ci_data_mode) &&
                        (parent_ci->ci_filename_mode ==
                         child_ci->ci_filename_mode) &&
@@ -219,7 +219,7 @@ int fscrypt_has_permitted_context(struct inode *parent, struct inode *child)
 
        return memcmp(parent_ctx.master_key_descriptor,
                      child_ctx.master_key_descriptor,
-                     FS_KEY_DESCRIPTOR_SIZE) == 0 &&
+                     FSCRYPT_KEY_DESCRIPTOR_SIZE) == 0 &&
                (parent_ctx.contents_encryption_mode ==
                 child_ctx.contents_encryption_mode) &&
                (parent_ctx.filenames_encryption_mode ==
@@ -257,7 +257,7 @@ int fscrypt_inherit_context(struct inode *parent, struct inode *child,
        ctx.filenames_encryption_mode = ci->ci_filename_mode;
        ctx.flags = ci->ci_flags;
        memcpy(ctx.master_key_descriptor, ci->ci_master_key_descriptor,
-              FS_KEY_DESCRIPTOR_SIZE);
+              FSCRYPT_KEY_DESCRIPTOR_SIZE);
        get_random_bytes(ctx.nonce, FS_KEY_DERIVATION_NONCE_SIZE);
        BUILD_BUG_ON(sizeof(ctx) != FSCRYPT_SET_CONTEXT_MAX_SIZE);
        res = parent->i_sb->s_cop->set_context(child, &ctx,

diff --git a/include/uapi/linux/fscrypt.h b/include/uapi/linux/fscrypt.h
index 674b0452ef5759358b13b986842dbee811d45269..29a945d165def3379ec21352429f84359cf73edd 100644 (file)
--- a/include/uapi/linux/fscrypt.h
+++ b/include/uapi/linux/fscrypt.h
@@ -55,6 +55,7 @@ struct fscrypt_key {
 /**********************************************************************/
 
 /* old names; don't add anything new here! */
+#ifndef __KERNEL__
 #define FS_KEY_DESCRIPTOR_SIZE         FSCRYPT_KEY_DESCRIPTOR_SIZE
 #define FS_POLICY_FLAGS_PAD_4          FSCRYPT_POLICY_FLAGS_PAD_4
 #define FS_POLICY_FLAGS_PAD_8          FSCRYPT_POLICY_FLAGS_PAD_8
@@ -76,5 +77,6 @@ struct fscrypt_key {
 #define FS_KEY_DESC_PREFIX             FSCRYPT_KEY_DESC_PREFIX
 #define FS_KEY_DESC_PREFIX_SIZE                FSCRYPT_KEY_DESC_PREFIX_SIZE
 #define FS_MAX_KEY_SIZE                        FSCRYPT_MAX_KEY_SIZE
+#endif /* !__KERNEL__ */
 
 #endif /* _UAPI_LINUX_FSCRYPT_H */