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.
This commit is contained in:
Ondřej Surý 2025-09-13 06:10:29 +02:00
parent fa12ba28ce
commit 136fddf538
No known key found for this signature in database
GPG key ID: 2820F37E873DEA41

View file

@ -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