Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 24 Mar 2009 04:20:18 +0000 (UTC)
From:      Weongyo Jeong <weongyo@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r190367 - head/sys/dev/if_ndis
Message-ID:  <200903240420.n2O4KIwS032588@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: weongyo
Date: Tue Mar 24 04:20:17 2009
New Revision: 190367
URL: http://svn.freebsd.org/changeset/base/190367

Log:
  set NULL after free to avoid duplicate free.
  
  Tested by:	Ganbold <ganbold_at_micom.mng.net>

Modified:
  head/sys/dev/if_ndis/if_ndis.c

Modified: head/sys/dev/if_ndis/if_ndis.c
==============================================================================
--- head/sys/dev/if_ndis/if_ndis.c	Tue Mar 24 03:08:09 2009	(r190366)
+++ head/sys/dev/if_ndis/if_ndis.c	Tue Mar 24 04:20:17 2009	(r190367)
@@ -3250,8 +3250,10 @@ ndis_stop(sc)
 
 	NDIS_LOCK(sc);
 	for (i = 0; i < NDIS_EVENTS; i++) {
-		if (sc->ndis_evt[i].ne_sts && sc->ndis_evt[i].ne_buf != NULL)
+		if (sc->ndis_evt[i].ne_sts && sc->ndis_evt[i].ne_buf != NULL) {
 			free(sc->ndis_evt[i].ne_buf, M_TEMP);
+			sc->ndis_evt[i].ne_buf = NULL;
+		}
 		sc->ndis_evt[i].ne_sts = 0;
 		sc->ndis_evt[i].ne_len = 0;
 	}



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200903240420.n2O4KIwS032588>