svn commit: r326702 - head/sys/net

shurd at sasktel.net shurd at sasktel.net
Fri Dec 8 20:10:10 UTC 2017



On Fri, 8 Dec 2017 13:57:13 -0500, Ryan Stone  wrote:

       On Fri, Dec 8, 2017 at 1:43 PM, Stephen Hurd  wrote:
> Author: shurd
> Date: Fri Dec 8 18:43:31 2017
> New Revision: 326702
> URL: https://svnweb.freebsd.org/changeset/base/326702
> + if (!M_WRITABLE(*m_head)) {
> + new_head = m_dup(*m_head, M_NOWAIT);
> + if (new_head == NULL) {
> + device_printf(dev, "cannot pad short frame, m_dup() failed");
> + return ENOMEM;

Don't we leak *m_head here? As far as I can tell the caller doesn't
deal with it. 

Thanks for catching that, fixed in r326706. 



More information about the svn-src-all mailing list