summaryrefslogtreecommitdiff
path: root/libc/inet/ifaddrs.c
diff options
context:
space:
mode:
authorDenys Vlasenko <vda.linux@googlemail.com>2009-09-05 23:37:40 +0200
committerDenys Vlasenko <vda.linux@googlemail.com>2009-09-05 23:37:40 +0200
commitfdf14ae2e653f275c085329b183373e1fb062693 (patch)
treeaf64aedb51e30bda270721bf37729bcb1f4cb567 /libc/inet/ifaddrs.c
parenta76558a92e21643a628c3c6a8cd22816634b1749 (diff)
do not save/restore errno around free() calls
In any non-buggy program free() does not fail. And when it fails in a buggy program, the failure is usually fatal (heap corruption and segfault). Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'libc/inet/ifaddrs.c')
-rw-r--r--libc/inet/ifaddrs.c5
1 files changed, 1 insertions, 4 deletions
diff --git a/libc/inet/ifaddrs.c b/libc/inet/ifaddrs.c
index 1d54d5123..77ca7ce02 100644
--- a/libc/inet/ifaddrs.c
+++ b/libc/inet/ifaddrs.c
@@ -81,7 +81,6 @@ void
__netlink_free_handle (struct netlink_handle *h)
{
struct netlink_res *ptr;
- int saved_errno = errno;
ptr = h->nlm_list;
while (ptr != NULL)
@@ -89,11 +88,9 @@ __netlink_free_handle (struct netlink_handle *h)
struct netlink_res *tmpptr;
tmpptr = ptr->next;
- free (ptr);
+ free (ptr); /* doesn't affect errno */
ptr = tmpptr;
}
-
- __set_errno (saved_errno);
}