4359. [bug] Inherited 'also-notify' lists were not being checked

by named-checkconf. [RT #42174]

CHANGES

@@ -1,3 +1,6 @@
+4359.	[bug]		Inherited 'also-notify' lists were not being checked
+			by named-checkconf. [RT #42174]
+
 4358.	[test]		Added American Fuzzy Lop harness that allows
 			feeding fuzzed packets into BIND.
 			[RT #41723]

bin/tests/system/checkconf/bad-options-also-notify.conf

@@ -0,0 +1,8 @@
+options {
+	also-notify { missing; };
+};
+
+zone "example.net" {
+	type slave;
+	masters { 192.168.1.1; };
+};

bin/tests/system/checkconf/bad-view-also-notify.conf

@@ -0,0 +1,7 @@
+view example {
+	also-notify { missing; };
+	zone "example.net" {
+		type slave;
+		masters { 192.168.1.1; };
+	};
+};

bin/tests/system/checkconf/good-options-also-notify.conf

@@ -0,0 +1,9 @@
+options {
+	also-notify { missing; };
+};
+
+zone "example.net" {
+	type slave;
+	notify no;
+	masters { 192.168.1.1; };
+};

bin/tests/system/checkconf/good-view-also-notify.conf

@@ -0,0 +1,8 @@
+view example {
+	also-notify { missing; };
+	zone "example.net" {
+		type slave;
+		notify no;
+		masters { 192.168.1.1; };
+	};
+};

lib/bind9/check.c

@@ -1780,7 +1780,12 @@ check_zoneconf(const cfg_obj_t *zconfig, const cfg_obj_t *voptions,
 			cfg_obj_log(zoptions, logctx, ISC_LOG_WARNING,
 				    "zone '%s': 'also-notify' set but "
 				    "'notify' is disabled", znamestr);
-		} else if (tresult == ISC_R_SUCCESS) {
+		}
+		if (tresult != ISC_R_SUCCESS && voptions != NULL)
+			tresult = cfg_map_get(voptions, "also-notify", &obj);
+		if (tresult != ISC_R_SUCCESS && goptions != NULL)
+			tresult = cfg_map_get(goptions, "also-notify", &obj);
+	        if (tresult == ISC_R_SUCCESS && donotify) {
 			isc_uint32_t count;
 			tresult = validate_masters(obj, config, &count,
 						   logctx, mctx);