diff options
| author | NeilBrown <neilb@suse.com> | 2016-12-06 11:27:21 -0500 |
|---|---|---|
| committer | Steve Dickson <steved@redhat.com> | 2016-12-06 11:27:21 -0500 |
| commit | 29aced680a423e6d97cab4faaae81c3e77eb5fc3 (patch) | |
| tree | 4ec845d0a536302f9a94a88e1bb421d97d181814 /utils/nfsidmap/nfsidmap.c | |
| parent | fc1127d754578cd1dc3b52aebce1d5ae09f2d347 (diff) | |
| download | nfs-utils-29aced680a423e6d97cab4faaae81c3e77eb5fc3.tar.gz nfs-utils-29aced680a423e6d97cab4faaae81c3e77eb5fc3.tar.xz nfs-utils-29aced680a423e6d97cab4faaae81c3e77eb5fc3.zip | |
Remove error messages on xstrdup failure.
xstrdup() prints a messages and exits, except in statd where
is prints a message and fails. So there is no point printing
an extra message when xstrdup() fails, and except in statd,
no point calling exit() as well.
So remove some pointless code.
Signed-off-by: NeilBrown <neilb@suse.com>
Signed-off-by: Steve Dickson <steved@redhat.com>
Diffstat (limited to 'utils/nfsidmap/nfsidmap.c')
| -rw-r--r-- | utils/nfsidmap/nfsidmap.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/utils/nfsidmap/nfsidmap.c b/utils/nfsidmap/nfsidmap.c index 63545fc..374bc5d 100644 --- a/utils/nfsidmap/nfsidmap.c +++ b/utils/nfsidmap/nfsidmap.c @@ -15,6 +15,7 @@ #include <unistd.h> #include "xlog.h" #include "conffile.h" +#include "xcommon.h" int verbose = 0; char *usage = "Usage: %s [-vh] [-c || [-u|-g|-r key] || -d || -l || [-t timeout] key desc]"; @@ -441,11 +442,7 @@ int main(int argc, char **argv) key = strtol(argv[optind++], NULL, 10); - arg = strdup(argv[optind]); - if (arg == NULL) { - xlog_err("strdup failed: %m"); - return EXIT_FAILURE; - } + arg = xstrdup(argv[optind]); type = strtok(arg, ":"); value = strtok(NULL, ":"); if (value == NULL) { |