Author: bz
Date: Sat Jun  9 22:44:49 2012
New Revision: 236827
URL: http://svn.freebsd.org/changeset/base/236827

Log:
  MFC r236615:
  
   Plug two interface address refcount leaks in early error return cases
   in the ioctl path.
  
   Reported by: rpaulo
   Reviewed by: emax

Modified:
  stable/8/sys/netinet6/in6.c
Directory Properties:
  stable/8/sys/   (props changed)

Modified: stable/8/sys/netinet6/in6.c
==============================================================================
--- stable/8/sys/netinet6/in6.c Sat Jun  9 22:44:24 2012        (r236826)
+++ stable/8/sys/netinet6/in6.c Sat Jun  9 22:44:49 2012        (r236827)
@@ -1556,14 +1556,19 @@ in6_lifaddr_ioctl(struct socket *so, u_l
                        hostid = IFA_IN6(ifa);
 
                        /* prefixlen must be <= 64. */
-                       if (64 < iflr->prefixlen)
+                       if (64 < iflr->prefixlen) {
+                               if (ifa != NULL)
+                                       ifa_free(ifa);
                                return EINVAL;
+                       }
                        prefixlen = iflr->prefixlen;
 
                        /* hostid part must be zero. */
                        sin6 = (struct sockaddr_in6 *)&iflr->addr;
                        if (sin6->sin6_addr.s6_addr32[2] != 0 ||
                            sin6->sin6_addr.s6_addr32[3] != 0) {
+                               if (ifa != NULL)
+                                       ifa_free(ifa);
                                return EINVAL;
                        }
                } else
_______________________________________________
[email protected] mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "[email protected]"

Reply via email to