[Opendnssec-commits] [svn.opendnssec.org/svn/dnssec] r5983 - in branches/OpenDNSSEC-1.3: . signer/src/signer

matthijs at nlnetlabs.nl matthijs at nlnetlabs.nl
Mon Jan 2 15:03:59 CET 2012


Author: matthijs
Date: 2012-01-02 15:03:59 +0100 (Mon, 02 Jan 2012)
New Revision: 5983

Modified:
   branches/OpenDNSSEC-1.3/NEWS
   branches/OpenDNSSEC-1.3/signer/src/signer/zonedata.c
Log:
OPENDNSSEC-192: Signer crashes on deleting NSEC3 for a domain that becomes opt-out

Nullify pointers to denial of existence that has been deleted



Modified: branches/OpenDNSSEC-1.3/NEWS
===================================================================
--- branches/OpenDNSSEC-1.3/NEWS	2012-01-02 09:55:34 UTC (rev 5982)
+++ branches/OpenDNSSEC-1.3/NEWS	2012-01-02 14:03:59 UTC (rev 5983)
@@ -2,16 +2,18 @@
 
 OpenDNSSEC 1.3
 
-* ods-ksmutil: Suppress database connection information when no -v flag is given.
-* ods-enforcerd: Stop multiple instances of the enforcer running by checking for
-  the pidfile at startup. If you want to run multiple instances then a different 
-  pidfile will need to be specified with the -P flag.
+* ods-ksmutil: Suppress database connection information when no -v flag is
+  given.
+* ods-enforcerd: Stop multiple instances of the enforcer running by checking
+  for the pidfile at startup. If you want to run multiple instances then a
+  different pidfile will need to be specified with the -P flag.
 
 Bugfixes:
 * Signer Engine: A file descriptor for sockets with value zero is allowed.
 * Signer Engine: Only log messages about a full signing queue in debug mode.
+* Bugfix OPENDNSSEC-192: Signer crashes on deleting NSEC3 for a domain that
+  becomes opt-out.
 
-
 OpenDNSSEC 1.3.4 - 2011-12-09
 
 Bugfixes:

Modified: branches/OpenDNSSEC-1.3/signer/src/signer/zonedata.c
===================================================================
--- branches/OpenDNSSEC-1.3/signer/src/signer/zonedata.c	2012-01-02 09:55:34 UTC (rev 5982)
+++ branches/OpenDNSSEC-1.3/signer/src/signer/zonedata.c	2012-01-02 14:03:59 UTC (rev 5983)
@@ -404,6 +404,7 @@
             "denial of existence data point", 1);
         return domain;
     }
+    domain->denial = NULL;
     log_rdf(domain->dname, "-DD", 6);
     return zonedata_del_domain_fixup(zd->domains, domain);
 }
@@ -991,6 +992,7 @@
                         "delete denial of existence data point", zd_str);
                     return ODS_STATUS_ERR;
                 }
+                domain->denial = NULL;
             }
             node = ldns_rbtree_next(node);
             continue;
@@ -1085,6 +1087,7 @@
                         "delete denial of existence data point", zd_str);
                     return ODS_STATUS_ERR;
                 }
+                domain->denial = NULL;
             }
             node = ldns_rbtree_next(node);
             continue;
@@ -1108,6 +1111,7 @@
                             "delete denial of existence data point", zd_str);
                         return ODS_STATUS_ERR;
                     }
+                    domain->denial = NULL;
                 }
                 node = ldns_rbtree_next(node);
                 continue;




More information about the Opendnssec-commits mailing list