chiark / gitweb /
chiark-named-conf: Do not apply end-of-file foreign zone defaults
authorianmdlvl <ianmdlvl>
Wed, 27 Aug 2008 22:12:54 +0000 (22:12 +0000)
committerianmdlvl <ianmdlvl>
Wed, 27 Aug 2008 22:12:54 +0000 (22:12 +0000)
to explicitly named zones specified in a command line invocation.
(Fixes bug introduced in 4.1.26.)

debian/changelog
scripts/named-conf

index 9da4db5..b11c86b 100644 (file)
@@ -2,9 +2,12 @@ chiark-utils (4.1.27) high; urgency=low
 
   * chiark-named-conf: Remove obsolete +nodebug option from calls
     to dig; required for BIND9 compatibility.  Thanks to Ross Younger.
+  * chiark-named-conf: Do not apply end-of-file foreign zone defaults
+    to explicitly named zones specified in a command line invocation.
+    (Fixes bug introduced in 4.1.26.)
   * Declare and document the dependency of hexterm on tcl8.4.
 
- -- Ian Jackson <ian@davenant.greenend.org.uk>  Wed, 27 Aug 2008 22:53:52 +0100
+ --
 
 chiark-utils (4.1.26) unstable; urgency=low
 
index fc55b24..0108a74 100755 (executable)
@@ -425,12 +425,15 @@ sub process_zones (@) {
     local ($zone,$cfg);
 
     foreach $zone (@zones) {
-       $cfg= $zone_cfg{$zone} || {
-           'style_p' => 'foreign',
-           's' => "f $group2modcmd{'foreign'}",
-           'servers' => [ ],
+       $cfg= $zone_cfg{$zone};
+       if (!$cfg) {
+           $cfg= {
+               'style_p' => 'foreign',
+               's' => "f $group2modcmd{'foreign'}",
+               'servers' => [ ],
            };
-       zone_conf_settings($cfg, $zone);
+           zone_conf_settings($cfg, $zone);
+       }
 
        mail_zone_before() or next
            if $domail;