upstream/bind9
1
0
Fork 0
mirror of https://github.com/isc-projects/bind9.git synced 2026-05-28 04:34:54 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions 4

Remove qpzone.c:first_existing_header_indirect() which had bugs

Browse source 
The first_existing_header_indirect() static function is used only
in one place and it has bugs:

1. It doesn't advance the pointer and can cause an infinite loop
   if it doesn't break out from the loop on the first iteration.
2. It doesn't check if the header EXISTS, though its name indicates
   that it should.
3. Even if the infinite loop bug is fixed, it would eventually
   return the last checked header's pointer even if all the
   candidates do not match the criteria of the selection.

Instead of fixing it, remove the function and use simpler code in
the place where it was being called.
This commit is contained in:
Aram Sargsyan 2026-01-16 14:07:39 +00:00 committed by Arаm Sаrgsyаn
parent 9c646e5213
commit d48df6b084
1 changed files with 9 additions and 20 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

29
lib/dns/qpzone.c
View file

@ -820,22 +820,6 @@ first_existing_header(dns_vectop_t *top, uint32_t serial) {
return NULL;
}
static dns_vecheader_t **
first_existing_header_indirect(dns_vectop_t *top, uint32_t serial) {
REQUIRE(top != NULL);
dns_vecheader_t **result = &ISC_SLIST_HEAD(top->headers);
ISC_SLIST_FOREACH_PTR(p, &ISC_SLIST_HEAD(top->headers)) {
dns_vecheader_t *header = *p;
result = p;
if (header->serial <= serial && !IGNORE(header)) {
break;
}
}
return result;
}
static void
clean_multiple_headers(dns_vectop_t *top) {
uint32_t parent_serial = UINT32_MAX;
@ -1814,9 +1798,14 @@ add(qpzonedb_t *qpdb, qpznode_t *node, const dns_name_t *nodename,
dns_vecheader_t **header_p = NULL;
dns_vecheader_t *header = NULL;
if (foundtop != NULL) {
header_p = first_existing_header_indirect(foundtop, UINT32_MAX);
INSIST(header_p != NULL);
header = EXISTS(*header_p) ? *header_p : NULL;
ISC_SLIST_FOREACH_PTR(p, &ISC_SLIST_HEAD(foundtop->headers)) {
if (!IGNORE(*p)) {
header_p = p;
header = *p;
break;
}
ISC_SLIST_PTR_ADVANCE(p, next_header);
}
}
if (header != NULL) {
@ -1901,7 +1890,7 @@ add(qpzonedb_t *qpdb, qpznode_t *node, const dns_name_t *nodename,
* Since we don't generate changed records when
* loading, we MUST clean up 'header' now.
*/
*header_p = ISC_SLIST_NEXT(header, next_header);
ISC_SLIST_PTR_REMOVE(header_p, header, next_header);
ISC_SLIST_PREPEND(foundtop->headers, newheader,
next_header);
maybe_update_recordsandsize(false, version, header,