Author: bms
Date: Tue Mar 24 01:22:12 2009
New Revision: 190354
URL: http://svn.freebsd.org/changeset/base/190354
Log:
  Don't call m_freem() after ip_output(), as it always consumes
  the mbuf chain provided to it.
  
  Found by:     Pierre Guinoiseau

Modified:
  head/sys/netinet/igmp.c

Modified: head/sys/netinet/igmp.c
==============================================================================
--- head/sys/netinet/igmp.c     Tue Mar 24 01:18:35 2009        (r190353)
+++ head/sys/netinet/igmp.c     Tue Mar 24 01:22:12 2009        (r190354)
@@ -3451,7 +3451,6 @@ igmp_intr(struct mbuf *m)
        error = ip_output(m0, ipopts, NULL, 0, &imo, NULL);
        if (error) {
                CTR3(KTR_IGMPV3, "%s: ip_output(%p) = %d", __func__, m0, error);
-               m_freem(m0);
                goto out;
        }
 
_______________________________________________
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Reply via email to