git: 7a5de1d4cc6a - main - fib_dxr: KASSERTs for chasing NULL ptr and runaway refcount suspects

From: Marko Zec <zec_at_FreeBSD.org>
Date: Tue, 07 May 2024 15:26:59 UTC
The branch main has been updated by zec:

URL: https://cgit.FreeBSD.org/src/commit/?id=7a5de1d4cc6af84456cef03323f041652e84326b

commit 7a5de1d4cc6af84456cef03323f041652e84326b
Author:     Marko Zec <zec@FreeBSD.org>
AuthorDate: 2024-05-07 15:22:00 +0000
Commit:     Marko Zec <zec@FreeBSD.org>
CommitDate: 2024-05-07 15:22:00 +0000

    fib_dxr: KASSERTs for chasing NULL ptr and runaway refcount suspects
    
    MFC after:      1 week
---
 sys/netinet/in_fib_dxr.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/sys/netinet/in_fib_dxr.c b/sys/netinet/in_fib_dxr.c
index 3b2dc837d50b..5c5311068169 100644
--- a/sys/netinet/in_fib_dxr.c
+++ b/sys/netinet/in_fib_dxr.c
@@ -1280,6 +1280,9 @@ dxr_change_rib_batch(struct rib_head *rnh, struct fib_change_queue *q,
 
 	da = dxr->aux;
 	KASSERT(da != NULL, ("%s: NULL dxr->aux", __FUNCTION__));
+	KASSERT(da->fd != NULL, ("%s: da->fd %p", __FUNCTION__, da->fd));
+	KASSERT(da->refcnt > 0, ("%s: da->refcnt %d", __FUNCTION__,
+	    da->refcnt));
 
 	FIB_PRINTF(LOG_INFO, da->fd, "processing %d update(s)", q->count);
 	for (ui = 0; ui < q->count; ui++) {