Module Name:    src
Committed By:   maxv
Date:           Fri Aug 16 10:41:35 UTC 2019

Modified Files:
        src/sys/kern: subr_pool.c

Log Message:
Initialize pp->pr_redzone to false. For some reason with KUBSAN GCC does
not eliminate the unused branch in pr_item_linkedlist_put(), and this
leads to a unused uninitialized access which triggers KUBSAN messages.


To generate a diff of this commit:
cvs rdiff -u -r1.254 -r1.255 src/sys/kern/subr_pool.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/kern/subr_pool.c
diff -u src/sys/kern/subr_pool.c:1.254 src/sys/kern/subr_pool.c:1.255
--- src/sys/kern/subr_pool.c:1.254	Sat Aug  3 09:31:07 2019
+++ src/sys/kern/subr_pool.c	Fri Aug 16 10:41:35 2019
@@ -1,4 +1,4 @@
-/*	$NetBSD: subr_pool.c,v 1.254 2019/08/03 09:31:07 maxv Exp $	*/
+/*	$NetBSD: subr_pool.c,v 1.255 2019/08/16 10:41:35 maxv Exp $	*/
 
 /*
  * Copyright (c) 1997, 1999, 2000, 2002, 2007, 2008, 2010, 2014, 2015, 2018
@@ -33,7 +33,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: subr_pool.c,v 1.254 2019/08/03 09:31:07 maxv Exp $");
+__KERNEL_RCSID(0, "$NetBSD: subr_pool.c,v 1.255 2019/08/16 10:41:35 maxv Exp $");
 
 #ifdef _KERNEL_OPT
 #include "opt_ddb.h"
@@ -767,6 +767,7 @@ pool_init(struct pool *pp, size_t size, 
 	pp->pr_drain_hook = NULL;
 	pp->pr_drain_hook_arg = NULL;
 	pp->pr_freecheck = NULL;
+	pp->pr_redzone = false;
 	pool_redzone_init(pp, size);
 	pool_quarantine_init(pp);
 

Reply via email to