From 4ffa687bceed12b4c7163310fb0390320ac48e3b Mon Sep 17 00:00:00 2001 From: sashan Date: Tue, 7 Jun 2022 22:18:34 +0000 Subject: [PATCH] fixes potential memory leak. if_vinput() should always consume packet by either passing it further or releasing it. OK mvs@ --- sys/net/if.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/sys/net/if.c b/sys/net/if.c index f354c9d8a6c..58af5730cde 100644 --- a/sys/net/if.c +++ b/sys/net/if.c @@ -1,4 +1,4 @@ -/* $OpenBSD: if.c,v 1.652 2022/05/03 11:47:03 bluhm Exp $ */ +/* $OpenBSD: if.c,v 1.653 2022/06/07 22:18:34 sashan Exp $ */ /* $NetBSD: if.c,v 1.35 1996/05/07 05:26:04 thorpej Exp $ */ /* @@ -869,6 +869,8 @@ if_vinput(struct ifnet *ifp, struct mbuf *m) if (__predict_true(!ISSET(ifp->if_xflags, IFXF_MONITOR))) (*ifp->if_input)(ifp, m); + else + m_freem(m); } void -- 2.20.1