diff options
| author | Sumit Bose <sbose@redhat.com> | 2012-10-19 18:10:43 +0200 |
|---|---|---|
| committer | Jakub Hrozek <jhrozek@redhat.com> | 2012-11-05 00:14:04 +0100 |
| commit | c99b23a9d67eec293480cd3d9ab1bd53ebf62566 (patch) | |
| tree | 8bde61e0d819f0cde77f763accf045349bf315aa | |
| parent | 9a51f72efd8bc36d7ece97b692f409799d4aac03 (diff) | |
| download | sssd-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.
| -rw-r--r-- | src/responder/nss/nsssrv_cmd.c | 4 |
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: |