apparmor: cleanup rename XXX_file_context() to XXX_file_ctx()
authorJohn Johansen <john.johansen@canonical.com>
Fri, 9 Jun 2017 18:43:45 +0000 (11:43 -0700)
committerJohn Johansen <john.johansen@canonical.com>
Sun, 11 Jun 2017 00:11:37 +0000 (17:11 -0700)
Signed-off-by: John Johansen <john.johansen@canonical.com>
security/apparmor/include/file.h
security/apparmor/lsm.c

index 365ca7ead133605fbd091a8c2c35cc58ae3154b0..19c483850770e21359aaf6d5d7dfe7439d56cb54 100644 (file)
@@ -30,6 +30,8 @@ struct path;
                                 AA_MAY_CHMOD | AA_MAY_CHOWN | AA_MAY_LOCK | \
                                 AA_EXEC_MMAP | AA_MAY_LINK)
 
+#define file_ctx(X) ((struct aa_file_ctx *)(X)->f_security)
+
 /* struct aa_file_ctx - the AppArmor context the file was opened in
  * @perms: the permission the file was opened with
  *
@@ -42,21 +44,26 @@ struct aa_file_ctx {
 };
 
 /**
- * aa_alloc_file_context - allocate file_ctx
+ * aa_alloc_file_ctx - allocate file_ctx
+ * @label: initial label of task creating the file
  * @gfp: gfp flags for allocation
  *
  * Returns: file_ctx or NULL on failure
  */
-static inline struct aa_file_ctx *aa_alloc_file_context(gfp_t gfp)
+static inline struct aa_file_ctx *aa_alloc_file_ctx(gfp_t gfp)
 {
-       return kzalloc(sizeof(struct aa_file_ctx), gfp);
+       struct aa_file_ctx *ctx;
+
+       ctx = kzalloc(sizeof(struct aa_file_ctx), gfp);
+
+       return ctx;
 }
 
 /**
- * aa_free_file_context - free a file_ctx
+ * aa_free_file_ctx - free a file_ctx
  * @ctx: file_ctx to free  (MAYBE_NULL)
  */
-static inline void aa_free_file_context(struct aa_file_ctx *ctx)
+static inline void aa_free_file_ctx(struct aa_file_ctx *ctx)
 {
        if (ctx)
                kzfree(ctx);
index e07dd5a204d76b25d2c9272d156137f8d33f2f5a..3c6fa9753675d4927bd870e115e4368803d51762 100644 (file)
@@ -398,9 +398,9 @@ static int apparmor_file_alloc_security(struct file *file)
 
        /* freed by apparmor_file_free_security */
        struct aa_profile *profile = begin_current_profile_crit_section();
-       file->f_security = aa_alloc_file_context(GFP_KERNEL);
-       if (!file->f_security)
-               return -ENOMEM;
+       file->f_security = aa_alloc_file_ctx(GFP_KERNEL);
+       if (!file_ctx(file))
+               error = -ENOMEM;
        end_current_profile_crit_section(profile);
 
        return error;
@@ -408,9 +408,7 @@ static int apparmor_file_alloc_security(struct file *file)
 
 static void apparmor_file_free_security(struct file *file)
 {
-       struct aa_file_ctx *ctx = file->f_security;
-
-       aa_free_file_context(ctx);
+       aa_free_file_ctx(file_ctx(file));
 }
 
 static int common_file_perm(const char *op, struct file *file, u32 mask)