Author: shurd
Date: Fri Dec  8 19:50:06 2017
New Revision: 326706
URL: https://svnweb.freebsd.org/changeset/base/326706

Log:
  Free mbuf chain when m_dup fails
  
  Fix memory leak where mbuf chain wasn't free()d if iflib_ether_pad()
  has a failure in m_dup().
  
  Reported by:  "Ryan Stone" <ryst...@gmail.com>
  Sponsored by: Limelight Networks

Modified:
  head/sys/net/iflib.c

Modified: head/sys/net/iflib.c
==============================================================================
--- head/sys/net/iflib.c        Fri Dec  8 19:26:25 2017        (r326705)
+++ head/sys/net/iflib.c        Fri Dec  8 19:50:06 2017        (r326706)
@@ -3125,6 +3125,7 @@ iflib_ether_pad(device_t dev, struct mbuf **m_head, ui
        if (!M_WRITABLE(*m_head)) {
                new_head = m_dup(*m_head, M_NOWAIT);
                if (new_head == NULL) {
+                       m_freem(*m_head);
                        device_printf(dev, "cannot pad short frame, m_dup() 
failed");
                        return ENOMEM;
                }
_______________________________________________
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Reply via email to