Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 13 Feb 2024 21:34:01 GMT
From:      Kristof Provost <kp@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org
Subject:   git: 17167f757e0a - stable/13 - pf: uncomment counter asserts after mem leak fix
Message-ID:  <202402132134.41DLY1cE026039@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch stable/13 has been updated by kp:

URL: https://cgit.FreeBSD.org/src/commit/?id=17167f757e0a14ac487c608459aaba6d10f43f12

commit 17167f757e0a14ac487c608459aaba6d10f43f12
Author:     Igor Ostapenko <pm@igoro.pro>
AuthorDate: 2024-01-30 21:04:57 +0000
Commit:     Kristof Provost <kp@FreeBSD.org>
CommitDate: 2024-02-13 21:31:04 +0000

    pf: uncomment counter asserts after mem leak fix
    
    Reviewed by:    kp
    Differential Revision:  https://reviews.freebsd.org/D43657
    
    (cherry picked from commit 9d784da3a7af9b9b04536c2e97459a7d9f92e364)
---
 sys/netpfil/pf/pf_ioctl.c | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/sys/netpfil/pf/pf_ioctl.c b/sys/netpfil/pf/pf_ioctl.c
index 81b766cb7f6a..98b5ce318596 100644
--- a/sys/netpfil/pf/pf_ioctl.c
+++ b/sys/netpfil/pf/pf_ioctl.c
@@ -5863,13 +5863,8 @@ pf_unload_vnet(void)
 	V_pf_allrulecount--;
 	LIST_REMOVE(V_pf_rulemarker, allrulelist);
 
-	/*
-	 * There are known pf rule leaks when running the test suite.
-	 */
-#ifdef notyet
 	MPASS(LIST_EMPTY(&V_pf_allrulelist));
 	MPASS(V_pf_allrulecount == 0);
-#endif
 
 	PF_RULES_WUNLOCK();
 



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