ipc: simplify msg list search

Signed-off-by: Peter Hurley <peter@hurleysoftware.com>
Acked-by: Stanislav Kinsbursky <skinsbursky@parallels.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>

diff --git a/ipc/msg.c b/ipc/msg.c
index 80324fc50cbfa2c32f942aad9ae17ca589b54628..860f217f3d9ead2fef9cc292584c70e28a9c400a 100644 (file)
--- a/ipc/msg.c
+++ b/ipc/msg.c
@@ -835,7 +835,7 @@ long do_msgrcv(int msqid, void __user *buf, size_t bufsz, long msgtyp,
 
        for (;;) {
                struct msg_receiver msr_d;
-               struct list_head *tmp;
+               struct msg_msg *walk_msg;
                long msg_counter = 0;
 
                msg = ERR_PTR(-EACCES);
@@ -843,11 +843,8 @@ long do_msgrcv(int msqid, void __user *buf, size_t bufsz, long msgtyp,
                        goto out_unlock;
 
                msg = ERR_PTR(-EAGAIN);
-               tmp = msq->q_messages.next;
-               while (tmp != &msq->q_messages) {
-                       struct msg_msg *walk_msg;
+               list_for_each_entry(walk_msg, &msq->q_messages, m_list) {
 
-                       walk_msg = list_entry(tmp, struct msg_msg, m_list);
                        if (testmsg(walk_msg, msgtyp, mode) &&
                            !security_msg_queue_msgrcv(msq, walk_msg, current,
                                                       msgtyp, mode)) {
@@ -864,7 +861,6 @@ long do_msgrcv(int msqid, void __user *buf, size_t bufsz, long msgtyp,
                                        break;
                                msg_counter++;
                        }
-                       tmp = tmp->next;
                }
                if (!IS_ERR(msg)) {
                        /*