diff options
author | Mike Frysinger <vapier@gentoo.org> | 2007-09-28 20:42:24 +0000 |
---|---|---|
committer | Mike Frysinger <vapier@gentoo.org> | 2007-09-28 20:42:24 +0000 |
commit | 9e5250dae08a1601c3c7c82d9128613d7c22dd84 (patch) | |
tree | 0b109afe567e37a93efd994976a63656be17cd1f /libc/inet/resolv.c | |
parent | 283dc2c2d0a89fae1f1ac39c51df2e2e48dd616a (diff) |
do not check the pointer before freeing it since free(NULL) works just fine
Diffstat (limited to 'libc/inet/resolv.c')
-rw-r--r-- | libc/inet/resolv.c | 12 |
1 files changed, 4 insertions, 8 deletions
diff --git a/libc/inet/resolv.c b/libc/inet/resolv.c index 6634d594b..749c647fc 100644 --- a/libc/inet/resolv.c +++ b/libc/inet/resolv.c @@ -989,10 +989,8 @@ int attribute_hidden __dns_lookup(const char *name, int type, int nscount, char fail: if (fd != -1) close(fd); - if (lookup) - free(lookup); - if (packet) - free(packet); + free(lookup); + free(packet); h_errno = NETDB_INTERNAL; /* Mess with globals while under lock */ if (local_ns != -1) { @@ -1252,12 +1250,10 @@ int res_query(const char *dname, int class, int type, if (a.atype == type) { /* CNAME*/ int len = MIN(anslen, i); memcpy(answer, packet, len); - if (packet) - free(packet); + free(packet); return(len); } - if (packet) - free(packet); + free(packet); return i; } libc_hidden_def(res_query) |