From: Corinna Vinschen <corinna@vinschen.de>
Date: Wed, 24 Jan 2007 15:55:51 +0000 (+0000)
Subject: 	* net.cc (if_nametoindex): This time, really free IP_ADAPTER_ADDRESSES
X-Git-Tag: drop_9x_support_start~78
X-Git-Url: https://sourceware.org/git/?a=commitdiff_plain;h=e7fc99384e42754a610f66ae72ca201095d83595;p=newlib-cygwin.git

	* net.cc (if_nametoindex): This time, really free IP_ADAPTER_ADDRESSES
	memory.
	(if_indextoname): Ditto.
---

diff --git a/winsup/cygwin/ChangeLog b/winsup/cygwin/ChangeLog
index fe75a1b81..addb06f6a 100644
--- a/winsup/cygwin/ChangeLog
+++ b/winsup/cygwin/ChangeLog
@@ -1,3 +1,9 @@
+2007-01-24  Corinna Vinschen  <corinna@vinschen.de>
+
+	* net.cc (if_nametoindex): This time, really free IP_ADAPTER_ADDRESSES
+	memory.
+	(if_indextoname): Ditto.
+
 2007-01-24  Corinna Vinschen  <corinna@vinschen.de>
 
 	* security.h (pop_thread_privilege): Revert to previous state, not
diff --git a/winsup/cygwin/net.cc b/winsup/cygwin/net.cc
index 338b504f9..a28a414f7 100644
--- a/winsup/cygwin/net.cc
+++ b/winsup/cygwin/net.cc
@@ -2091,7 +2091,7 @@ get_ifconf (SOCKET s, struct ifconf *ifc, int what)
 extern "C" unsigned
 if_nametoindex (const char *name)
 {
-  PIP_ADAPTER_ADDRESSES pap = NULL;
+  PIP_ADAPTER_ADDRESSES pa0 = NULL, pap;
   unsigned index = 0;
 
   myfault efault;
@@ -2107,13 +2107,13 @@ if_nametoindex (const char *name)
       strncat (lname, name, IF_NAMESIZE - 1);
       if (lname[0] == '{' && (c = strchr (lname, ':')))
 	*c = '\0';
-      for (; pap; pap = pap->Next)
+      for (pap = pa0; pap; pap = pap->Next)
 	if (strcasematch (lname, pap->AdapterName))
 	  {
 	    index = pap->IfIndex;
 	    break;
 	  }
-      free (pap);
+      free (pa0);
     }
   return index;
 }
@@ -2121,7 +2121,7 @@ if_nametoindex (const char *name)
 extern "C" char *
 if_indextoname (unsigned ifindex, char *ifname)
 {
-  PIP_ADAPTER_ADDRESSES pap = NULL;
+  PIP_ADAPTER_ADDRESSES pa0 = NULL, pap;
   char *name = NULL;
 
   myfault efault;
@@ -2129,15 +2129,15 @@ if_indextoname (unsigned ifindex, char *ifname)
     return NULL;
 
   if (wincap.has_gaa_prefixes ()
-      && get_adapters_addresses (&pap, AF_UNSPEC))
+      && get_adapters_addresses (&pa0, AF_UNSPEC))
     {
-      for (; pap; pap = pap->Next)
+      for (pap = pa0; pap; pap = pap->Next)
         if (ifindex == pap->IfIndex)
 	  {
 	    name = strcpy (ifname, pap->AdapterName);
 	    break;
 	  }
-      free (pap);
+      free (pa0);
     }
   else
     set_errno (ENXIO);