projects/linux·commit 2dc958fa

ipc: set msg back to -EAGAIN if copy wasn't performed

2dc958fa · Stanislav Kinsbursky <[email protected]> · 2013-04-01 11:40:51 +04:00

Make sure that msg pointer is set back to error value in case of
MSG_COPY flag is set and desired message to copy wasn't found.  This
garantees that msg is either a error pointer or a copy address.

Otherwise the last message in queue will be freed without unlinking from
the queue (which leads to memory corruption) and the dummy allocated
copy won't be released.

Signed-off-by: Stanislav Kinsbursky <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>
Commit
2dc958fa2fe6987e7ab106bd97029a09a82fcd8d
Parents
118c9a45

Changes

1 files changed · +1 −0

FileStatusAddDelContent
ipc/msg.cmodified+1−0View file

ipc/msg.c

View file
diff --git a/ipc/msg.c b/ipc/msg.c
index 31cd1bf..fede1d0 100644
--- a/ipc/msg.c
+++ b/ipc/msg.c
@@ -872,6 +872,7 @@ long do_msgrcv(int msqid, void __user *buf, size_t bufsz, long msgtyp,
goto out_unlock;
break;
}
+ msg = ERR_PTR(-EAGAIN);
} else
break;
msg_counter++;