[src/trunk]: src/sys/net fix a potential leak on tap device close, purging th...

[plunky <plunky%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/dc16956be65d
branches:  trunk
changeset: 749424:dc16956be65d
user:      plunky <plunky%NetBSD.org@localhost>
date:      Sun Nov 29 10:44:23 2009 +0000

description:
fix a potential leak on tap device close, purging the send queue
did not actually release the dequeued mbufs.

pointed out by Paul Forgey on tech-net

diffstat:

 sys/net/if_tap.c |  5 +++--
 1 files changed, 3 insertions(+), 2 deletions(-)

diffs (26 lines):

diff -r 246e3d7c2987 -r dc16956be65d sys/net/if_tap.c
--- a/sys/net/if_tap.c  Sun Nov 29 10:33:56 2009 +0000
+++ b/sys/net/if_tap.c  Sun Nov 29 10:44:23 2009 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: if_tap.c,v 1.59 2009/09/15 19:38:15 drochner Exp $     */
+/*     $NetBSD: if_tap.c,v 1.60 2009/11/29 10:44:23 plunky Exp $       */
 
 /*
  *  Copyright (c) 2003, 2004, 2008, 2009 The NetBSD Foundation.
@@ -33,7 +33,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: if_tap.c,v 1.59 2009/09/15 19:38:15 drochner Exp $");
+__KERNEL_RCSID(0, "$NetBSD: if_tap.c,v 1.60 2009/11/29 10:44:23 plunky Exp $");
 
 #if defined(_KERNEL_OPT)
 #include "bpfilter.h"
@@ -859,6 +859,7 @@
                        if (ifp->if_bpf)
                                bpf_mtap(ifp->if_bpf, m);
 #endif
+                       m_freem(m);
                }
        }
        splx(s);