summaryrefslogtreecommitdiffstats
path: root/src/tools/sss_groupmod.c
diff options
context:
space:
mode:
authorJakub Hrozek <jhrozek@redhat.com>2010-08-31 18:12:25 +0200
committerStephen Gallagher <sgallagh@redhat.com>2010-09-08 09:36:22 -0400
commit8443d24c0584f45151e0c80506d7a572b8a38ed7 (patch)
tree3df77db1747c04ed38095bb6b78087c6534adc50 /src/tools/sss_groupmod.c
parent4f6a396fcf16f97b2abc3d0cba10e9aa9bc38619 (diff)
downloadsssd-8443d24c0584f45151e0c80506d7a572b8a38ed7.tar.gz
sssd-8443d24c0584f45151e0c80506d7a572b8a38ed7.tar.xz
sssd-8443d24c0584f45151e0c80506d7a572b8a38ed7.zip
Fix assorted minor bugs in sss_ tools
Fixes: #585
Diffstat (limited to 'src/tools/sss_groupmod.c')
-rw-r--r--src/tools/sss_groupmod.c14
1 files changed, 6 insertions, 8 deletions
diff --git a/src/tools/sss_groupmod.c b/src/tools/sss_groupmod.c
index bd684ba0c..59ec10bab 100644
--- a/src/tools/sss_groupmod.c
+++ b/src/tools/sss_groupmod.c
@@ -73,31 +73,29 @@ int main(int argc, const char **argv)
case 'a':
addgroups = poptGetOptArg(pc);
if (addgroups == NULL) {
- ret = -1;
+ BAD_POPT_PARAMS(pc, _("Specify group to add to\n"),
+ ret, fini);
}
break;
case 'r':
rmgroups = poptGetOptArg(pc);
if (rmgroups == NULL) {
- ret = -1;
+ BAD_POPT_PARAMS(pc, _("Specify group to remove from\n"),
+ ret, fini);
}
break;
}
}
if (ret != -1) {
- usage(pc, poptStrerror(ret));
- ret = EXIT_FAILURE;
- goto fini;
+ BAD_POPT_PARAMS(pc, poptStrerror(ret), ret, fini);
}
/* groupname is an argument without --option */
pc_groupname = poptGetArg(pc);
if (pc_groupname == NULL) {
- usage(pc, _("Specify group to modify\n"));
- ret = EXIT_FAILURE;
- goto fini;
+ BAD_POPT_PARAMS(pc, _("Specify group to modify\n"), ret, fini);
}
debug_level = pc_debug;