Source-Changes-HG archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

[src/netbsd-7]: src/sys Pull up following revision(s) (requested by maxv in t...



details:   https://anonhg.NetBSD.org/src/rev/ea2d3eac88f5
branches:  netbsd-7
changeset: 799238:ea2d3eac88f5
user:      snj <snj%NetBSD.org@localhost>
date:      Tue Apr 21 04:55:15 2015 +0000

description:
Pull up following revision(s) (requested by maxv in ticket #713):
        sys/dev/ic/bwi.c: revision 1.26
        sys/nfs/krpc_subr.c: revision 1.38
Fix uninitialized variable.
Found by The Brainy Code Scanner in FreeBSD.
--
Fix a double free. "Suggested" by Brainy.
ok rjs@ riastradh@

diffstat:

 sys/dev/ic/bwi.c    |  6 ++----
 sys/nfs/krpc_subr.c |  6 +++---
 2 files changed, 5 insertions(+), 7 deletions(-)

diffs (61 lines):

diff -r cac185e4dd23 -r ea2d3eac88f5 sys/dev/ic/bwi.c
--- a/sys/dev/ic/bwi.c  Tue Apr 21 04:50:17 2015 +0000
+++ b/sys/dev/ic/bwi.c  Tue Apr 21 04:55:15 2015 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: bwi.c,v 1.24 2014/02/25 18:30:09 pooka Exp $   */
+/*     $NetBSD: bwi.c,v 1.24.4.1 2015/04/21 04:55:15 snj Exp $ */
 /*     $OpenBSD: bwi.c,v 1.74 2008/02/25 21:13:30 mglocker Exp $       */
 
 /*
@@ -48,7 +48,7 @@
 
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: bwi.c,v 1.24 2014/02/25 18:30:09 pooka Exp $");
+__KERNEL_RCSID(0, "$NetBSD: bwi.c,v 1.24.4.1 2015/04/21 04:55:15 snj Exp $");
 
 #include <sys/param.h>
 #include <sys/callout.h>
@@ -9140,7 +9140,6 @@
 
                MGETHDR(m_new, M_DONTWAIT, MT_DATA);
                if (m_new == NULL) {
-                       m_freem(m);
                        error = ENOBUFS;
                        aprint_error_dev(sc->sc_dev,
                            "can't defrag TX buffer (1)\n");
@@ -9151,7 +9150,6 @@
                if (m->m_pkthdr.len > MHLEN) {
                        MCLGET(m_new, M_DONTWAIT);
                        if (!(m_new->m_flags & M_EXT)) {
-                               m_freem(m);
                                m_freem(m_new);
                                error = ENOBUFS;
                        }
diff -r cac185e4dd23 -r ea2d3eac88f5 sys/nfs/krpc_subr.c
--- a/sys/nfs/krpc_subr.c       Tue Apr 21 04:50:17 2015 +0000
+++ b/sys/nfs/krpc_subr.c       Tue Apr 21 04:55:15 2015 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: krpc_subr.c,v 1.37.38.1 2015/04/06 01:37:29 snj Exp $  */
+/*     $NetBSD: krpc_subr.c,v 1.37.38.2 2015/04/21 04:55:15 snj Exp $  */
 
 /*
  * Copyright (c) 1995 Gordon Ross, Adam Glass
@@ -43,7 +43,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: krpc_subr.c,v 1.37.38.1 2015/04/06 01:37:29 snj Exp $");
+__KERNEL_RCSID(0, "$NetBSD: krpc_subr.c,v 1.37.38.2 2015/04/21 04:55:15 snj Exp $");
 
 #include <sys/param.h>
 #include <sys/systm.h>
@@ -242,7 +242,7 @@
         * Create socket and set its receive timeout.
         */
        if ((error = socreate(AF_INET, &so, SOCK_DGRAM, 0, l, NULL)))
-               goto out;
+               return error;
 
        if ((error = nfs_boot_setrecvtimo(so)))
                goto out;



Home | Main Index | Thread Index | Old Index