nwrap: Use SAFE_FREE() in nwrap_files_gethostbyname2_r()
authorAndreas Schneider <asn@samba.org>
Thu, 19 Mar 2020 08:39:16 +0000 (09:39 +0100)
committerAndreas Schneider <asn@samba.org>
Thu, 19 Mar 2020 08:49:39 +0000 (09:49 +0100)
Signed-off-by: Andreas Schneider <asn@samba.org>
Reviewed-by: Alexander Bokovoy <ab@samba.org>
Reviewed-by: Samuel Cabrero <scabrero@suse.de>
src/nss_wrapper.c

index e7b52bbab659b1fe9641696bd6c8920f325d2222..6ece2668a99754377442232e10471326b1629d3e 100644 (file)
@@ -3732,9 +3732,7 @@ static int nwrap_files_gethostbyname2_r(struct nwrap_backend *b,
        rc = nwrap_files_internal_gethostbyname(name, AF_UNSPEC, hedst,
                                                addr_list);
        if (rc == -1) {
-               if (addr_list->items != NULL) {
-                       free(addr_list->items);
-               }
+               SAFE_FREE(addr_list->items);
                SAFE_FREE(addr_list);
                errno = ENOENT;
                return -1;
@@ -3751,8 +3749,8 @@ static int nwrap_files_gethostbyname2_r(struct nwrap_backend *b,
         * +1 is for ending NULL pointer. */
        memcpy(buf, addr_list->items, (addr_list->count + 1) * sizeof(void *));
 
-       free(addr_list->items);
-       free(addr_list);
+       SAFE_FREE(addr_list->items);
+       SAFE_FREE(addr_list);
 
        g.ptr = buf;
        hedst->h_addr_list = g.list;