diff options
| author | Linus Torvalds <torvalds@linux-foundation.org> | 2012-08-17 12:17:11 -0400 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2012-08-17 12:17:11 -0400 |
| commit | 0b0402d45326bc167d1652169c7ce5669f973730 (patch) | |
| tree | 1f36e2898527eec7b4585eae236cb0a06d65b2bd | |
| parent | 9d0f8140fc8a723af4eb1ec4d757847515649574 (diff) | |
| parent | 7612bfeecc197bbb6629842b5c6ff1967f0a9b70 (diff) | |
Merge branch 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/jmorris/linux-security
Pull a Yama bugfix from James Morris.
* 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/jmorris/linux-security:
Yama: access task_struct->comm directly
| -rw-r--r-- | security/yama/yama_lsm.c | 10 |
1 files changed, 2 insertions, 8 deletions
diff --git a/security/yama/yama_lsm.c b/security/yama/yama_lsm.c index d51b7c76c37d..0cc99a3ea42d 100644 --- a/security/yama/yama_lsm.c +++ b/security/yama/yama_lsm.c | |||
| @@ -279,12 +279,9 @@ static int yama_ptrace_access_check(struct task_struct *child, | |||
| 279 | } | 279 | } |
| 280 | 280 | ||
| 281 | if (rc) { | 281 | if (rc) { |
| 282 | char name[sizeof(current->comm)]; | ||
| 283 | printk_ratelimited(KERN_NOTICE | 282 | printk_ratelimited(KERN_NOTICE |
| 284 | "ptrace of pid %d was attempted by: %s (pid %d)\n", | 283 | "ptrace of pid %d was attempted by: %s (pid %d)\n", |
| 285 | child->pid, | 284 | child->pid, current->comm, current->pid); |
| 286 | get_task_comm(name, current), | ||
| 287 | current->pid); | ||
| 288 | } | 285 | } |
| 289 | 286 | ||
| 290 | return rc; | 287 | return rc; |
| @@ -319,12 +316,9 @@ static int yama_ptrace_traceme(struct task_struct *parent) | |||
| 319 | } | 316 | } |
| 320 | 317 | ||
| 321 | if (rc) { | 318 | if (rc) { |
| 322 | char name[sizeof(current->comm)]; | ||
| 323 | printk_ratelimited(KERN_NOTICE | 319 | printk_ratelimited(KERN_NOTICE |
| 324 | "ptraceme of pid %d was attempted by: %s (pid %d)\n", | 320 | "ptraceme of pid %d was attempted by: %s (pid %d)\n", |
| 325 | current->pid, | 321 | current->pid, parent->comm, parent->pid); |
| 326 | get_task_comm(name, parent), | ||
| 327 | parent->pid); | ||
| 328 | } | 322 | } |
| 329 | 323 | ||
| 330 | return rc; | 324 | return rc; |
