diff options
author | NeilBrown <neilb@suse.com> | 2017-01-11 11:08:36 -0500 |
---|---|---|
committer | Steve Dickson <steved@redhat.com> | 2017-01-11 11:08:36 -0500 |
commit | 9e12e221ead6d869977b43a99e4a7d13a33d82ee (patch) | |
tree | 94b2999e90844f6783b8dcd0e31f9e5b33548f4e | |
parent | b975792a0ae15bb742cee2e18582009d22857069 (diff) | |
download | nfs-utils-9e12e221ead6d869977b43a99e4a7d13a33d82ee.tar.gz nfs-utils-9e12e221ead6d869977b43a99e4a7d13a33d82ee.tar.xz nfs-utils-9e12e221ead6d869977b43a99e4a7d13a33d82ee.zip |
nfsstats: fix some exit codes.
"nfsstat -m" always exits with "1" (unless there is an error opening
/proc/mounts). It should exist "0".
Also, a few usage errors cause an exit of "255" when it should probably
be "1".
Signed-off-by: NeilBrown <neilb@suse.com>
Signed-off-by: Steve Dickson <steved@redhat.com>
-rw-r--r-- | utils/nfsstat/nfsstat.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/utils/nfsstat/nfsstat.c b/utils/nfsstat/nfsstat.c index 8376347..eddbe9a 100644 --- a/utils/nfsstat/nfsstat.c +++ b/utils/nfsstat/nfsstat.c @@ -389,7 +389,7 @@ main(int argc, char **argv) switch (c) { case 'a': fprintf(stderr, "nfsstat: nfs acls are not yet supported.\n"); - return -1; + return 1; case 'c': opt_clt = 1; break; @@ -455,7 +455,7 @@ main(int argc, char **argv) "not yet supported\n"); return 2; case 'm': - return mounts(MOUNTSFILE); + return ! mounts(MOUNTSFILE); case '\1': usage(progname); return 0; @@ -464,7 +464,7 @@ main(int argc, char **argv) return 0; default: printf("Try `%s --help' for more information.\n", progname); - return -1; + return 1; } } |