Module Name: src Committed By: knakahara Date: Thu Mar 9 09:57:36 UTC 2017
Modified Files: src/sys/net: if.c Log Message: ifp->if_transmit() must free mbuf even if error occurred. Add missing m_freem(m) to if_nulltransmit(). Below ifp->if_transmit() implementations are already added m_freem(m) properly. - wm(4) - ixg(4) - ixv(4) - pppoe(4) - gif(4) - l2tp(4) pointed out by ozaki-r@n.o, thanks. To generate a diff of this commit: cvs rdiff -u -r1.382 -r1.383 src/sys/net/if.c Please note that diffs are not public domain; they are subject to the copyright notices on the relevant files.
Modified files: Index: src/sys/net/if.c diff -u src/sys/net/if.c:1.382 src/sys/net/if.c:1.383 --- src/sys/net/if.c:1.382 Tue Mar 7 01:32:03 2017 +++ src/sys/net/if.c Thu Mar 9 09:57:36 2017 @@ -1,4 +1,4 @@ -/* $NetBSD: if.c,v 1.382 2017/03/07 01:32:03 ozaki-r Exp $ */ +/* $NetBSD: if.c,v 1.383 2017/03/09 09:57:36 knakahara Exp $ */ /*- * Copyright (c) 1999, 2000, 2001, 2008 The NetBSD Foundation, Inc. @@ -90,7 +90,7 @@ */ #include <sys/cdefs.h> -__KERNEL_RCSID(0, "$NetBSD: if.c,v 1.382 2017/03/07 01:32:03 ozaki-r Exp $"); +__KERNEL_RCSID(0, "$NetBSD: if.c,v 1.383 2017/03/09 09:57:36 knakahara Exp $"); #if defined(_KERNEL_OPT) #include "opt_inet.h" @@ -379,6 +379,7 @@ int if_nulltransmit(struct ifnet *ifp, struct mbuf *m) { + m_freem(m); return ENXIO; } @@ -3260,9 +3261,11 @@ if_transmit_lock(struct ifnet *ifp, stru } else { KERNEL_UNLOCK_ONE(NULL); error = (*ifp->if_transmit)(ifp, m); + /* mbuf is alredy freed */ } #else /* !ALTQ */ error = (*ifp->if_transmit)(ifp, m); + /* mbuf is alredy freed */ #endif /* !ALTQ */ return error;