diff options
author | James Morris <jmorris@namei.org> | 2011-06-30 04:44:55 -0400 |
---|---|---|
committer | James Morris <jmorris@namei.org> | 2011-06-30 04:44:55 -0400 |
commit | da3f1c7b7d34e66cf571a613525ca0d23f92d42f (patch) | |
tree | 0233826259cbe4cd44755ccc893b77a3e8ac5213 | |
parent | 5b944a71a192977c1c018bbcfa0c52dca48e2368 (diff) | |
parent | 25e75dff519bcce2cb35023105e7df51d7b9e691 (diff) |
Merge branch 'for-security' of git://git.kernel.org/pub/scm/linux/kernel/git/jj/apparmor-dev into next
-rw-r--r-- | security/apparmor/domain.c | 2 | ||||
-rw-r--r-- | security/apparmor/lsm.c | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/security/apparmor/domain.c b/security/apparmor/domain.c index c825c6e0b636..78adc4303efa 100644 --- a/security/apparmor/domain.c +++ b/security/apparmor/domain.c | |||
@@ -73,7 +73,6 @@ static int may_change_ptraced_domain(struct task_struct *task, | |||
73 | cred = get_task_cred(tracer); | 73 | cred = get_task_cred(tracer); |
74 | tracerp = aa_cred_profile(cred); | 74 | tracerp = aa_cred_profile(cred); |
75 | } | 75 | } |
76 | rcu_read_unlock(); | ||
77 | 76 | ||
78 | /* not ptraced */ | 77 | /* not ptraced */ |
79 | if (!tracer || unconfined(tracerp)) | 78 | if (!tracer || unconfined(tracerp)) |
@@ -82,6 +81,7 @@ static int may_change_ptraced_domain(struct task_struct *task, | |||
82 | error = aa_may_ptrace(tracer, tracerp, to_profile, PTRACE_MODE_ATTACH); | 81 | error = aa_may_ptrace(tracer, tracerp, to_profile, PTRACE_MODE_ATTACH); |
83 | 82 | ||
84 | out: | 83 | out: |
84 | rcu_read_unlock(); | ||
85 | if (cred) | 85 | if (cred) |
86 | put_cred(cred); | 86 | put_cred(cred); |
87 | 87 | ||
diff --git a/security/apparmor/lsm.c b/security/apparmor/lsm.c index 3d2fd141dff7..37832026e58a 100644 --- a/security/apparmor/lsm.c +++ b/security/apparmor/lsm.c | |||
@@ -127,7 +127,7 @@ static int apparmor_capget(struct task_struct *target, kernel_cap_t *effective, | |||
127 | *inheritable = cred->cap_inheritable; | 127 | *inheritable = cred->cap_inheritable; |
128 | *permitted = cred->cap_permitted; | 128 | *permitted = cred->cap_permitted; |
129 | 129 | ||
130 | if (!unconfined(profile)) { | 130 | if (!unconfined(profile) && !COMPLAIN_MODE(profile)) { |
131 | *effective = cap_intersect(*effective, profile->caps.allow); | 131 | *effective = cap_intersect(*effective, profile->caps.allow); |
132 | *permitted = cap_intersect(*permitted, profile->caps.allow); | 132 | *permitted = cap_intersect(*permitted, profile->caps.allow); |
133 | } | 133 | } |