nfsd: Drop reference in expkey_parse error cases
authorKrishna Kumar <krkumar2@in.ibm.com>
Mon, 20 Oct 2008 06:14:40 +0000 (11:44 +0530)
committerJ. Bruce Fields <bfields@citi.umich.edu>
Wed, 22 Oct 2008 18:04:34 +0000 (14:04 -0400)
Drop reference to export key on error. Compile tested.

Signed-off-by: Krishna Kumar <krkumar2@in.ibm.com>
Signed-off-by: J. Bruce Fields <bfields@citi.umich.edu>
fs/nfsd/export.c

index 9dc036f1835614e3c2505abad50496071cf8b0b2..7ce2c6e4e23ee11c5b2cfc0a2b056dbf6d57e6a1 100644 (file)
@@ -151,8 +151,10 @@ static int expkey_parse(struct cache_detail *cd, char *mesg, int mlen)
 
        /* now we want a pathname, or empty meaning NEGATIVE  */
        err = -EINVAL;
-       if ((len=qword_get(&mesg, buf, PAGE_SIZE)) < 0)
+       if ((len=qword_get(&mesg, buf, PAGE_SIZE)) < 0) {
+               cache_put(&ek->h, &svc_expkey_cache);
                goto out;
+       }
        dprintk("Path seems to be <%s>\n", buf);
        err = 0;
        if (len == 0) {
@@ -164,8 +166,10 @@ static int expkey_parse(struct cache_detail *cd, char *mesg, int mlen)
        } else {
                struct nameidata nd;
                err = path_lookup(buf, 0, &nd);
-               if (err)
+               if (err) {
+                       cache_put(&ek->h, &svc_expkey_cache);
                        goto out;
+               }
 
                dprintk("Found the path %s\n", buf);
                key.ek_path = nd.path;