diff options
| author | Dave Jones <davej@redhat.com> | 2012-09-06 12:01:00 -0400 |
|---|---|---|
| committer | Ben Hutchings <ben@decadent.org.uk> | 2012-09-19 15:04:44 +0100 |
| commit | 9600be784136611dcab44490998f8d7420ac7a79 (patch) | |
| tree | 4a446e5618e2bc6006214564219f1bfab082cddf /mm/mempolicy.c | |
| parent | 7c013154e1cb6ffd5f2a0b6b17b4b70df9e16ebd (diff) | |
| download | lwn-9600be784136611dcab44490998f8d7420ac7a79.tar.gz lwn-9600be784136611dcab44490998f8d7420ac7a79.zip | |
Remove user-triggerable BUG from mpol_to_str
commit 80de7c3138ee9fd86a98696fd2cf7ad89b995d0a upstream.
Trivially triggerable, found by trinity:
kernel BUG at mm/mempolicy.c:2546!
Process trinity-child2 (pid: 23988, threadinfo ffff88010197e000, task ffff88007821a670)
Call Trace:
show_numa_map+0xd5/0x450
show_pid_numa_map+0x13/0x20
traverse+0xf2/0x230
seq_read+0x34b/0x3e0
vfs_read+0xac/0x180
sys_pread64+0xa2/0xc0
system_call_fastpath+0x1a/0x1f
RIP: mpol_to_str+0x156/0x360
Signed-off-by: Dave Jones <davej@redhat.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
Diffstat (limited to 'mm/mempolicy.c')
| -rw-r--r-- | mm/mempolicy.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/mm/mempolicy.c b/mm/mempolicy.c index c0007f916170..11b8d47dc6c1 100644 --- a/mm/mempolicy.c +++ b/mm/mempolicy.c @@ -2533,7 +2533,7 @@ int mpol_to_str(char *buffer, int maxlen, struct mempolicy *pol, int no_context) break; default: - BUG(); + return -EINVAL; } l = strlen(policy_modes[mode]); |