]> git.decadent.org.uk Git - nfs-utils.git/commitdiff
mountd: mountlist_del_all cleanup
authorJ. Bruce Fields <bfields@citi.umich.edu>
Thu, 8 Oct 2009 23:05:22 +0000 (18:05 -0500)
committerSteve Dickson <steved@redhat.com>
Wed, 13 Jan 2010 20:39:14 +0000 (15:39 -0500)
Common exit code.

Signed-off-by: J. Bruce Fields <bfields@citi.umich.edu>
utils/mountd/rmtab.c

index c371f8db8bef27bd8d9dd5e98807d026e754122d..b028529f4b274e9f2fcfa5f61b70110934483680 100644 (file)
@@ -143,23 +143,16 @@ mountlist_del_all(struct sockaddr_in *sin)
                return;
        if (!(hp = gethostbyaddr((char *)&addr, sizeof(addr), AF_INET))) {
                xlog(L_ERROR, "can't get hostname of %s", inet_ntoa(addr));
-               xfunlock(lockid);
-               return;
+               goto out_unlock;
        }
-       else
-               hp = hostent_dup (hp);
+       hp = hostent_dup (hp);
+
+       if (!setrmtabent("r"))
+               goto out_free;
+
+       if (!(fp = fsetrmtabent(_PATH_RMTABTMP, "w")))
+               goto out_close;
 
-       if (!setrmtabent("r")) {
-               xfunlock(lockid);
-               free (hp);
-               return;
-       }
-       if (!(fp = fsetrmtabent(_PATH_RMTABTMP, "w"))) {
-               endrmtabent();
-               xfunlock(lockid);
-               free (hp);
-               return;
-       }
        while ((rep = getrmtabent(1, NULL)) != NULL) {
                if (strcmp(rep->r_client, hp->h_name) == 0 &&
                    (exp = auth_authenticate("umountall", sin, rep->r_path)))
@@ -170,10 +163,13 @@ mountlist_del_all(struct sockaddr_in *sin)
                xlog(L_ERROR, "couldn't rename %s to %s",
                                _PATH_RMTABTMP, _PATH_RMTAB);
        }
-       endrmtabent();  /* close & unlink */
        fendrmtabent(fp);
-       xfunlock(lockid);
+out_close:
+       endrmtabent();  /* close & unlink */
+out_free:
        free (hp);
+out_unlock:
+       xfunlock(lockid);
 }
 
 mountlist