From: visa <visa@openbsd.org>
Date: Mon, 5 Dec 2022 08:58:49 +0000 (+0000)
Subject: Avoid event counter re-insert in evcount_percpu()
X-Git-Url: http://artulab.com/gitweb/?a=commitdiff_plain;h=1e5f4b12257797b7c33c6668fcbd970c836e9c4b;p=openbsd

Avoid event counter re-insert in evcount_percpu()

This fixes evcount_list corruption that happened when evcount_percpu()
was called after evcount_init_percpu().

OK jca@ cheloha@ jmatthew@
---

diff --git a/sys/kern/subr_evcount.c b/sys/kern/subr_evcount.c
index cc9fc58890c..c77c953630d 100644
--- a/sys/kern/subr_evcount.c
+++ b/sys/kern/subr_evcount.c
@@ -1,4 +1,4 @@
-/*	$OpenBSD: subr_evcount.c,v 1.14 2022/11/10 07:05:41 jmatthew Exp $ */
+/*	$OpenBSD: subr_evcount.c,v 1.15 2022/12/05 08:58:49 visa Exp $ */
 /*
  * Copyright (c) 2004 Artur Grabowski <art@openbsd.org>
  * Copyright (c) 2004 Aaron Campbell <aaron@openbsd.org>
@@ -56,7 +56,6 @@ evcount_percpu(struct evcount *ec)
 		TAILQ_INSERT_TAIL(&evcount_percpu_init_list, ec, next);
 	} else {
 		ec->ec_percpu = counters_alloc(1);
-		TAILQ_INSERT_TAIL(&evcount_list, ec, next);
 	}
 }