diff options
| author | Jakub Hrozek <jhrozek@redhat.com> | 2010-08-31 18:12:25 +0200 |
|---|---|---|
| committer | Stephen Gallagher <sgallagh@redhat.com> | 2010-09-08 09:36:22 -0400 |
| commit | 8443d24c0584f45151e0c80506d7a572b8a38ed7 (patch) | |
| tree | 3df77db1747c04ed38095bb6b78087c6534adc50 /src/tools/sss_groupmod.c | |
| parent | 4f6a396fcf16f97b2abc3d0cba10e9aa9bc38619 (diff) | |
| download | sssd_unused-8443d24c0584f45151e0c80506d7a572b8a38ed7.tar.gz sssd_unused-8443d24c0584f45151e0c80506d7a572b8a38ed7.tar.xz sssd_unused-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.c | 14 |
1 files changed, 6 insertions, 8 deletions
diff --git a/src/tools/sss_groupmod.c b/src/tools/sss_groupmod.c index bd684ba0..59ec10ba 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; |