From 136fddf5387f56ce5c954d068cfff141090f50ae Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ond=C5=99ej=20Sur=C3=BD?= Date: Sat, 13 Sep 2025 06:10:29 +0200 Subject: [PATCH] Use result of first_*_header() calls instead of direct value Fix places where we got the header by calling first_*_header() function, but then worked with top->header instead of the result. --- lib/dns/qpcache.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/dns/qpcache.c b/lib/dns/qpcache.c index 0d0e9b1bd2..06d9a91b04 100644 --- a/lib/dns/qpcache.c +++ b/lib/dns/qpcache.c @@ -474,7 +474,7 @@ first_header(dns_slabtop_t *top) { static dns_slabheader_t * first_existing_header(dns_slabtop_t *top) { dns_slabheader_t *header = first_header(top); - if (EXISTS(top->header)) { + if (EXISTS(header)) { return header; } return NULL; @@ -2933,7 +2933,7 @@ add(qpcache_t *qpdb, qpcnode_t *qpnode, dns_slabheader_t *newheader, dns_slabheader_t *expireheader = first_header(expiretop); if (expireheader != NULL) { - mark_ancient(expiretop->header); + mark_ancient(expireheader); } /* * FIXME: In theory, we should mark the RRSIG