Module Name:    src
Committed By:   maxv
Date:           Mon Jun 22 16:39:56 UTC 2020

Modified Files:
        src/sys/net: pfil.c

Log Message:
pfil_psz gets dropped by the compiler because it is unused if !NET_MPSAFE,
so add an #ifdef around it, not to leak memory. Found by kLSan.


To generate a diff of this commit:
cvs rdiff -u -r1.38 -r1.39 src/sys/net/pfil.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/sys/net/pfil.c
diff -u src/sys/net/pfil.c:1.38 src/sys/net/pfil.c:1.39
--- src/sys/net/pfil.c:1.38	Mon Apr 27 23:35:40 2020
+++ src/sys/net/pfil.c	Mon Jun 22 16:39:56 2020
@@ -1,4 +1,4 @@
-/*	$NetBSD: pfil.c,v 1.38 2020/04/27 23:35:40 nat Exp $	*/
+/*	$NetBSD: pfil.c,v 1.39 2020/06/22 16:39:56 maxv Exp $	*/
 
 /*
  * Copyright (c) 2013 Mindaugas Rasiukevicius <rmind at NetBSD org>
@@ -28,7 +28,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: pfil.c,v 1.38 2020/04/27 23:35:40 nat Exp $");
+__KERNEL_RCSID(0, "$NetBSD: pfil.c,v 1.39 2020/06/22 16:39:56 maxv Exp $");
 
 #if defined(_KERNEL_OPT)
 #include "opt_net_mpsafe.h"
@@ -86,13 +86,17 @@ static LIST_HEAD(, pfil_head) pfil_head_
 
 static kmutex_t pfil_mtx __cacheline_aligned;
 static struct psref_class *pfil_psref_class __read_mostly;
+#ifdef NET_MPSAFE
 static pserialize_t pfil_psz;
+#endif
 
 void
 pfil_init(void)
 {
 	mutex_init(&pfil_mtx, MUTEX_DEFAULT, IPL_NONE);
+#ifdef NET_MPSAFE
 	pfil_psz = pserialize_create();
+#endif
 	pfil_psref_class = psref_class_create("pfil", IPL_SOFTNET);
 }
 

Reply via email to