-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Fixes: #261
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)
Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/

iEYEARECAAYFAkr4DYkACgkQHsardTLnvCXsowCfT+Y397TDtbHi/URDiQ9QGTJp
YFMAnjM94R+flCjOxwmjJAw4snG7U4FO
=inC1
-----END PGP SIGNATURE-----
>From 5df9ae8f08ad5aff52693cb60ade3ed7ec9724c8 Mon Sep 17 00:00:00 2001
From: Jakub Hrozek <[email protected]>
Date: Mon, 9 Nov 2009 13:37:29 +0100
Subject: [PATCH] Fix regression in error message when deleting groups

---
 server/tools/sss_groupdel.c |   10 ++++------
 1 files changed, 4 insertions(+), 6 deletions(-)

diff --git a/server/tools/sss_groupdel.c b/server/tools/sss_groupdel.c
index d2b4edf..84ea0be 100644
--- a/server/tools/sss_groupdel.c
+++ b/server/tools/sss_groupdel.c
@@ -94,10 +94,8 @@ int main(int argc, const char **argv)
                               tctx->octx->name, tctx->local,
                               &tctx->octx);
     if (ret != EOK) {
-        ERROR("Cannot find group in local domain, "
-              "modifying groups is allowed only in local domain\n");
-        ret = EXIT_FAILURE;
-        goto fini;
+        /* Error message will be printed in the switch */
+        goto done;
     }
 
     if ((tctx->octx->gid < tctx->local->id_min) ||
@@ -125,9 +123,9 @@ int main(int argc, const char **argv)
 
     end_transaction(tctx);
 
+    ret = tctx->error;
 done:
-    if (tctx->error) {
-        ret = tctx->error;
+    if (ret) {
         DEBUG(1, ("sysdb operation failed (%d)[%s]\n", ret, strerror(ret)));
         switch (ret) {
             case ENOENT:
-- 
1.6.2.5

_______________________________________________
sssd-devel mailing list
[email protected]
https://fedorahosted.org/mailman/listinfo/sssd-devel

Reply via email to