diff options
| author | Michael Adam <obnox@samba.org> | 2008-02-29 02:23:29 +0100 |
|---|---|---|
| committer | Michael Adam <obnox@samba.org> | 2008-02-29 09:31:57 +0100 |
| commit | 34d906ef44d1a5c3452097a9cac438d721fe35c1 (patch) | |
| tree | 5960d5cc6a2db7ce28eb32d1e794e69273ed8843 | |
| parent | 0906120a6e4fe0eb66f3c3fd99c625b30672cb2b (diff) | |
| download | samba-34d906ef44d1a5c3452097a9cac438d721fe35c1.tar.gz samba-34d906ef44d1a5c3452097a9cac438d721fe35c1.tar.xz samba-34d906ef44d1a5c3452097a9cac438d721fe35c1.zip | |
libreplace: fix rep_freeifaddrs to not segfault on NULL input.
Michael
(cherry picked from commit 0cbb87453beb52c6b0bc3a48791f49678f4030c5)
| -rw-r--r-- | source/lib/replace/getifaddrs.c | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/source/lib/replace/getifaddrs.c b/source/lib/replace/getifaddrs.c index adc2517e5c1..f6f0ec080c2 100644 --- a/source/lib/replace/getifaddrs.c +++ b/source/lib/replace/getifaddrs.c @@ -44,13 +44,14 @@ void rep_freeifaddrs(struct ifaddrs *ifp) { - free(ifp->ifa_name); - free(ifp->ifa_addr); - free(ifp->ifa_netmask); - free(ifp->ifa_dstaddr); - if (ifp->ifa_next != NULL) + if (ifp != NULL) { + free(ifp->ifa_name); + free(ifp->ifa_addr); + free(ifp->ifa_netmask); + free(ifp->ifa_dstaddr); freeifaddrs(ifp->ifa_next); - free(ifp); + free(ifp); + } } static struct sockaddr *sockaddr_dup(struct sockaddr *sa) |