[src/netbsd-7-1]: src/sys/arch/newsmips/apbus Pull up following revision(s) (...

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

details:   https://anonhg.NetBSD.org/src/rev/8602e7a4dbe9
branches:  netbsd-7-1
changeset: 800762:8602e7a4dbe9
user:      snj <snj%NetBSD.org@localhost>
date:      Fri Aug 11 15:20:52 2017 +0000

description:
Pull up following revision(s) (requested by mrg in ticket #1467):
        sys/arch/newsmips/apbus/if_sn.c: revision 1.39
Avoid memory leak in sonic_get.
If this is the first time around, top is null and nothing else will
free m.
>From Ilja Van Sprundel.

diffstat:

 sys/arch/newsmips/apbus/if_sn.c |  9 ++++++---
 1 files changed, 6 insertions(+), 3 deletions(-)

diffs (30 lines):

diff -r 1294ff011ac3 -r 8602e7a4dbe9 sys/arch/newsmips/apbus/if_sn.c
--- a/sys/arch/newsmips/apbus/if_sn.c   Wed Aug 09 07:27:01 2017 +0000
+++ b/sys/arch/newsmips/apbus/if_sn.c   Fri Aug 11 15:20:52 2017 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: if_sn.c,v 1.34 2012/10/13 06:24:15 tsutsui Exp $       */
+/*     $NetBSD: if_sn.c,v 1.34.20.1 2017/08/11 15:20:52 snj Exp $      */
 
 /*
  * National Semiconductor  DP8393X SONIC Driver
@@ -16,7 +16,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: if_sn.c,v 1.34 2012/10/13 06:24:15 tsutsui Exp $");
+__KERNEL_RCSID(0, "$NetBSD: if_sn.c,v 1.34.20.1 2017/08/11 15:20:52 snj Exp $");
 
 #include "opt_inet.h"
 
@@ -1093,7 +1093,10 @@
                if (datalen >= MINCLSIZE) {
                        MCLGET(m, M_DONTWAIT);
                        if ((m->m_flags & M_EXT) == 0) {
-                               if (top) m_freem(top);
+                               if (top)
+                                       m_freem(top);
+                               else
+                                       m_freem(m);
                                return 0;
                        }
                        len = MCLBYTES;