Fixup alloc in wrong region.

git-svn-id: file:///svn/unbound/trunk@628 be551aaa-1e26-0410-a405-d3ace91eadb9

diff --git a/doc/Changelog b/doc/Changelog
index 02668134516403bd219f25b4121f8805841fbcb3..c2f62cfb1b961732690e79d55a460e89f3ce5dee 100644 (file)
--- a/doc/Changelog
+++ b/doc/Changelog
@@ -1,3 +1,6 @@
+21 September 2007: Wouter
+       - fixup empty_DS_name allocated in wrong region.
+
 20 September 2007: Wouter
        - fixup and test for NSEC wildcard with empty nonterminals. 
        - makedist.sh fixup for svn info.

diff --git a/doc/TODO b/doc/TODO
index 19e6115781e467dc782a3b84679199ca07658919..09534df602dd62ee32ef15306077c7b4f74db240 100644 (file)
--- a/doc/TODO
+++ b/doc/TODO
@@ -52,3 +52,4 @@ o grab ports nonconsequtive and change the set after a while (change within
 o workaround for nxdomain responses for ENT DS queries. Not look at rcode and
   look at valid empty nonterminal proof that is inside the packet.
 o make timeout backoffs randomized (a couple percent random) to spread traffic.
+o inspect date on executable, then warn user in log if its more than 1 year.

diff --git a/doc/plan b/doc/plan
index e66c720974971627adc58b792de74c3d7663f7e3..57d533473781467bedb01ea8211ae79cb53436ee 100644 (file)
--- a/doc/plan
+++ b/doc/plan
@@ -178,6 +178,8 @@ Styleguide:
   if phase 1, start servicing, phase is 0 again. Make robust against delays.
   readme: max about 1 second worth of incoming queries, 10k perhaps,
        or 1/number of seconds it takes start up of 10k.
+* should the source include a copy of the ldns lib for ease of building by 
+  new users.
 
 *** Local zones feature.
 * Build in local zone features. First the total stop for1912.

diff --git a/validator/validator.c b/validator/validator.c
index 6caf80e63fff3d0bc7b14c375ffc9a7ed5e1b39b..e9a69fd77860ee352249179bc32b150bf0c3176b 100644 (file)
--- a/validator/validator.c
+++ b/validator/validator.c
@@ -1789,7 +1789,14 @@ process_ds_response(struct module_qstate* qstate, struct val_qstate* vq,
                        return;
        }
        if(dske == NULL) {
-               vq->empty_DS_name = qinfo->qname;
+               vq->empty_DS_name = region_alloc_init(qstate->region,
+                       qinfo->qname, qinfo->qname_len);
+               if(!vq->empty_DS_name) {
+                       log_err("malloc failure in empty_DS_name");
+                       vq->key_entry = NULL; /* make it error */
+                       vq->state = VAL_VALIDATE_STATE;
+                       return;
+               }
                vq->empty_DS_len = qinfo->qname_len;
                /* ds response indicated that we aren't on a delegation point.
                 * Keep the forState.state on FINDKEY. */