summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPeter Hurley <peter@hurleysoftware.com>2013-03-08 12:43:26 -0800
committerLinus Torvalds <torvalds@linux-foundation.org>2013-03-08 15:05:33 -0800
commite1082f45f1e2bbf6e25f6b614fc6616ebf709d19 (patch)
treef9d3e4326eed088acfe4cd1e9a8caf9c2138da55
parent47b3bc907328db968bc9b43c41f48f8d1e140750 (diff)
downloadop-kernel-dev-e1082f45f1e2bbf6e25f6b614fc6616ebf709d19.zip
op-kernel-dev-e1082f45f1e2bbf6e25f6b614fc6616ebf709d19.tar.gz
ipc: fix potential oops when src msg > 4k w/ MSG_COPY
If the src msg is > 4k, then dest->next points to the next allocated segment; resetting it just prior to dereferencing is bad. Signed-off-by: Peter Hurley <peter@hurleysoftware.com> Acked-by: Stanislav Kinsbursky <skinsbursky@parallels.com> Cc: <stable@vger.kernel.org> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r--ipc/msgutil.c3
1 files changed, 0 insertions, 3 deletions
diff --git a/ipc/msgutil.c b/ipc/msgutil.c
index ebfcbfa..5df8e4b 100644
--- a/ipc/msgutil.c
+++ b/ipc/msgutil.c
@@ -117,9 +117,6 @@ struct msg_msg *copy_msg(struct msg_msg *src, struct msg_msg *dst)
if (alen > DATALEN_MSG)
alen = DATALEN_MSG;
- dst->next = NULL;
- dst->security = NULL;
-
memcpy(dst + 1, src + 1, alen);
len -= alen;
OpenPOWER on IntegriCloud