fs/sysv: dereferencing ERR_PTR()
authorDan Carpenter <error27@gmail.com>
Wed, 21 Apr 2010 10:30:32 +0000 (12:30 +0200)
committerAl Viro <viro@zeniv.linux.org.uk>
Sat, 15 May 2010 11:16:33 +0000 (07:16 -0400)
I moved the dir_put_page() inside the if condition so we don't dereference
"page", if it's an ERR_PTR().

Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
fs/sysv/dir.c

index 4e50286a4cc32425c1061c92aba680b265ecaad1..1dabed286b4cf061a34c8521f5b1e44b81147089 100644 (file)
@@ -164,8 +164,8 @@ struct sysv_dir_entry *sysv_find_entry(struct dentry *dentry, struct page **res_
                                                        name, de->name))
                                        goto found;
                        }
+                       dir_put_page(page);
                }
-               dir_put_page(page);
 
                if (++n >= npages)
                        n = 0;