summaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorSumit Bose <sbose@redhat.com>2012-10-19 18:10:43 +0200
committerJakub Hrozek <jhrozek@redhat.com>2012-11-05 00:14:04 +0100
commitc99b23a9d67eec293480cd3d9ab1bd53ebf62566 (patch)
tree8bde61e0d819f0cde77f763accf045349bf315aa /src
parent9a51f72efd8bc36d7ece97b692f409799d4aac03 (diff)
downloadsssd-c99b23a9d67eec293480cd3d9ab1bd53ebf62566.tar.gz
sssd-c99b23a9d67eec293480cd3d9ab1bd53ebf62566.tar.xz
sssd-c99b23a9d67eec293480cd3d9ab1bd53ebf62566.zip
Fix two errors in the nss responder
One is a copy-and-paste error which was introduce by 1774ee9a61b9d691dadd1a0538f32bcdcc84f72f. The second fixes a missing explicit setting of the return value. In the case where we want fully qualified names ret contains the number of characters from the last snprintf() which is almost ever not 0.
Diffstat (limited to 'src')
-rw-r--r--src/responder/nss/nsssrv_cmd.c4
1 files changed, 3 insertions, 1 deletions
diff --git a/src/responder/nss/nsssrv_cmd.c b/src/responder/nss/nsssrv_cmd.c
index 42d32e9db..33b3fd104 100644
--- a/src/responder/nss/nsssrv_cmd.c
+++ b/src/responder/nss/nsssrv_cmd.c
@@ -1867,6 +1867,8 @@ static int fill_members(struct sss_packet *packet,
memnum++;
}
+ ret = 0;
+
done:
*_memnum = memnum;
*_rzero = rzero;
@@ -2687,7 +2689,7 @@ static void nss_cmd_getgrgid_cb(struct tevent_req *req)
ret = nss_cmd_getgrgid_search(dctx);
if (ret == EOK) {
/* we have results to return */
- ret = nss_cmd_getpw_send_reply(dctx, true);
+ ret = nss_cmd_getgr_send_reply(dctx, true);
}
done: