Skip to content

Commit

Permalink
fix: usr: Fix response policy zones and catalog zones with an $INCLUD…
Browse files Browse the repository at this point in the history
…E statement defined

Response policy zones (RPZ) and catalog zones were not working correctly if they had an $INCLUDE statement defined. This has been fixed.

Closes #5111

Merge branch '5111-includes-disable-rpz-and-catz-fix' into 'main'

See merge request isc-projects/bind9!9930
  • Loading branch information
Arаm Sаrgsyаn committed Jan 8, 2025
2 parents e71549e + d75bdab commit 19a2aab
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion lib/dns/include/dns/master.h
Original file line number Diff line number Diff line change
Expand Up @@ -175,7 +175,7 @@ dns_master_loadfileasync(const char *master_file, dns_name_t *top,
*
* Returns:
*\li ISC_R_SUCCESS upon successfully loading the master file.
*\li ISC_R_SEENINCLUDE upon successfully loading the master file with
*\li DNS_R_SEENINCLUDE upon successfully loading the master file with
* a $INCLUDE statement.
*\li ISC_R_NOMEMORY out of memory.
*\li ISC_R_UNEXPECTEDEND expected to be able to read a input token and
Expand Down
4 changes: 2 additions & 2 deletions lib/dns/zone.c
Original file line number Diff line number Diff line change
Expand Up @@ -2657,13 +2657,13 @@ zone_startload(dns_db_t *db, dns_zone_t *zone, isc_time_t loadtime) {
}

cleanup:
if (result != ISC_R_SUCCESS) {
if (result != ISC_R_SUCCESS && result != DNS_R_SEENINCLUDE) {
dns_zone_rpz_disable_db(zone, load->db);
dns_zone_catz_disable_db(zone, load->db);
}

tresult = dns_db_endload(db, &load->callbacks);
if (result == ISC_R_SUCCESS) {
if (result == ISC_R_SUCCESS || result == DNS_R_SEENINCLUDE) {
result = tresult;
}

Expand Down

0 comments on commit 19a2aab

Please sign in to comment.