nwrap: fix nwrap_module_getgrgid_r
authorRalph Wuerthner <ralph.wuerthner@de.ibm.com>
Fri, 19 Oct 2018 13:22:23 +0000 (15:22 +0200)
committerAndreas Schneider <asn@samba.org>
Tue, 30 Oct 2018 19:49:03 +0000 (20:49 +0100)
On success *grdstp must point to grdst, on error return NULL instead.

Signed-off-by: Ralph Wuerthner <ralph.wuerthner@de.ibm.com>
Reviewed-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
src/nss_wrapper.c

index e58f482af4674964af1b6d277746cdd1cafa3afe..26d91066392c15b87646680190eb57d847669b36 100644 (file)
@@ -4132,7 +4132,7 @@ static int nwrap_module_getgrgid_r(struct nwrap_backend *b,
 {
        int ret;
 
-       (void) grdstp; /* unused */
+       *grdstp = NULL;
 
        if (!b->fns->_nss_getgrgid_r) {
                return ENOENT;
@@ -4141,6 +4141,7 @@ static int nwrap_module_getgrgid_r(struct nwrap_backend *b,
        ret = b->fns->_nss_getgrgid_r(gid, grdst, buf, buflen, &errno);
        switch (ret) {
        case NSS_STATUS_SUCCESS:
+               *grdstp = grdst;
                return 0;
        case NSS_STATUS_NOTFOUND:
                if (errno != 0) {