UBIFS: fix oops on error path in read_pnode
authorArtem Bityutskiy <Artem.Bityutskiy@nokia.com>
Fri, 25 Mar 2011 17:09:54 +0000 (19:09 +0200)
committerArtem Bityutskiy <Artem.Bityutskiy@nokia.com>
Tue, 5 Apr 2011 07:40:31 +0000 (10:40 +0300)
Thanks to coverity which spotted that UBIFS will oops if 'kmalloc()'
in 'read_pnode()' fails and we dereference a NULL 'pnode' pointer
when we 'goto out'.

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>
Cc: stable@kernel.org
fs/ubifs/lpt.c

index 72775d35b99e55308b1ad963cc9c13487b5412fc..ef5155e109a2744dc0c493945adb41511e632680 100644 (file)
@@ -1270,10 +1270,9 @@ static int read_pnode(struct ubifs_info *c, struct ubifs_nnode *parent, int iip)
        lnum = branch->lnum;
        offs = branch->offs;
        pnode = kzalloc(sizeof(struct ubifs_pnode), GFP_NOFS);
-       if (!pnode) {
-               err = -ENOMEM;
-               goto out;
-       }
+       if (!pnode)
+               return -ENOMEM;
+
        if (lnum == 0) {
                /*
                 * This pnode was not written which just means that the LEB