BIND 10 #1760: implement NSEC3-fetch interface in SQLite3 accessor

BIND 10 Development do-not-reply at isc.org
Mon Mar 19 16:36:08 UTC 2012


#1760: implement NSEC3-fetch interface in SQLite3 accessor
-------------------------------------+-------------------------------------
                   Reporter:         |                 Owner:  vorner
  jinmei                             |                Status:  reviewing
                       Type:  task   |             Milestone:
                   Priority:         |  Sprint-20120320
  medium                             |            Resolution:
                  Component:  data   |             Sensitive:  0
  source                             |           Sub-Project:  DNS
                   Keywords:         |  Estimated Difficulty:  5
            Defect Severity:  N/A    |           Total Hours:  0
Feature Depending on Ticket:  NSEC3  |
        Add Hours to Ticket:  0      |
                  Internal?:  0      |
-------------------------------------+-------------------------------------
Changes (by jelte):

 * owner:  jelte => vorner


Comment:

 The code looks ok, and can be merged.

 The finding of the last hash in findPreviousNSEC3Hash could potentially be
 moved into a local function to, make findprevious shorter, but no strong
 opinion there. And maybe it would be a bit cleaner to use a second stmt
 pointer object for the second select (NSEC3_LAST).

 There is one thing I would like to start doing here, but it's not exactly
 in scope. We've been using a pattern in some of the query/renderer reuse
 code; a RAII style cleanup object (in this case, one that would call
 sqlite3_stmt_reset() and sqlite3_clear_bindings() when it is destroyed,
 which would make those intermediate cleanup calls unnecessary (and the
 code more safe in this regard). But since it's not really in scope i think
 that can be deferred.

-- 
Ticket URL: <http://bind10.isc.org/ticket/1760#comment:5>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development


More information about the bind10-tickets mailing list