[PATCH 05/10] ipc: Set EFAULT as default error in load_msg()

[Peter Hurley]

Signed-off-by: Peter Hurley <pe...@hurleysoftware.com>
---
 ipc/msgutil.c | 10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

diff --git a/ipc/msgutil.c b/ipc/msgutil.c
index b79582d..d33fbb2 100644
--- a/ipc/msgutil.c
+++ b/ipc/msgutil.c
@@ -84,7 +84,7 @@ struct msg_msg *load_msg(const void __user *src, int len)
 {
        struct msg_msg *msg;
        struct msg_msgseg *seg;
-       int err;
+       int err = -EFAULT;
        int alen;
 
        msg = alloc_msg(len);
@@ -92,19 +92,15 @@ struct msg_msg *load_msg(const void __user *src, int len)
                return ERR_PTR(-ENOMEM);
 
        alen = min(len, DATALEN_MSG);
-       if (copy_from_user(msg + 1, src, alen)) {
-               err = -EFAULT;
+       if (copy_from_user(msg + 1, src, alen))
                goto out_err;
-       }
 
        for (seg = msg->next; seg != NULL; seg = seg->next) {
                len -= alen;
                src = (char __user *)src + alen;
                alen = min(len, DATALEN_SEG);
-               if (copy_from_user(seg + 1, src, alen)) {
-                       err = -EFAULT;
+               if (copy_from_user(seg + 1, src, alen))
                        goto out_err;
-               }
        }
 
        err = security_msg_msg_alloc(msg);
-- 
1.8.1.2

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/