apparmor: replace remaining BUG_ON() asserts with AA_BUG()

AA_BUG() uses WARN and won't break the kernel like BUG_ON().

Signed-off-by: John Johansen <john.johansen@canonical.com>

diff --git a/security/apparmor/apparmorfs.c b/security/apparmor/apparmorfs.c
index 6834000640d70944baf0f4375ba71b46ef0b6831..41073f70eb41db135b8b68defa7523c161624991 100644 (file)
--- a/security/apparmor/apparmorfs.c
+++ b/security/apparmor/apparmorfs.c
@@ -91,7 +91,7 @@ static struct aa_loaddata *aa_simple_write_to_buffer(const char __user *userbuf,
 {
        struct aa_loaddata *data;
 
-       BUG_ON(copy_size > alloc_size);
+       AA_BUG(copy_size > alloc_size);
 
        if (*pos != 0)
                /* only writes from pos 0, that is complete writes */

diff --git a/security/apparmor/audit.c b/security/apparmor/audit.c
index 0c81ff64993bfa59b9c908305c473bf3c4a4237b..87f40fa8c431c93fb63b61025d263fa9e11b86fd 100644 (file)
--- a/security/apparmor/audit.c
+++ b/security/apparmor/audit.c
@@ -119,7 +119,7 @@ void aa_audit_msg(int type, struct common_audit_data *sa,
 int aa_audit(int type, struct aa_profile *profile, struct common_audit_data *sa,
             void (*cb) (struct audit_buffer *, void *))
 {
-       BUG_ON(!profile);
+       AA_BUG(!profile);
 
        if (type == AUDIT_APPARMOR_AUTO) {
                if (likely(!aad(sa)->error)) {

diff --git a/security/apparmor/context.c b/security/apparmor/context.c
index 71e9910cca7bf7e6f32efb6abe87c59428aa346a..1fc16b88efbf7543b424effd19231b08bf7ab824 100644 (file)
--- a/security/apparmor/context.c
+++ b/security/apparmor/context.c
@@ -95,7 +95,7 @@ int aa_replace_current_profile(struct aa_profile *profile)
 {
        struct aa_task_ctx *ctx = current_ctx();
        struct cred *new;
-       BUG_ON(!profile);
+       AA_BUG(!profile);
 
        if (ctx->profile == profile)
                return 0;
@@ -166,7 +166,7 @@ int aa_set_current_hat(struct aa_profile *profile, u64 token)
        struct cred *new = prepare_creds();
        if (!new)
                return -ENOMEM;
-       BUG_ON(!profile);
+       AA_BUG(!profile);
 
        ctx = cred_ctx(new);
        if (!ctx->previous) {

diff --git a/security/apparmor/lsm.c b/security/apparmor/lsm.c
index fc82ac0247dc39166491863ab8057cc8800e61e9..0dd48184946aa34ca4180111edba9c951a324c7d 100644 (file)
--- a/security/apparmor/lsm.c
+++ b/security/apparmor/lsm.c
@@ -406,7 +406,7 @@ static int common_file_perm(const char *op, struct file *file, u32 mask)
        struct aa_profile *profile, *fprofile = aa_cred_profile(file->f_cred);
        int error = 0;
 
-       BUG_ON(!fprofile);
+       AA_BUG(!fprofile);
 
        if (!file->f_path.mnt ||
            !path_mediated_fs(file->f_path.dentry))