BIND 10 #80: review: auth server incorrectly returns SERVFAIL to queries of class ANY

[BIND 10 Development]

#80: review: auth server incorrectly returns SERVFAIL to queries of class ANY
-------------------------------------+-------------------------------------
                 Reporter:  jinmei   |                Owner:  jinmei
                     Type:  defect   |               Status:  closed
                 Priority:           |            Milestone:  A-Team-
  critical                           |  Sprint-20110316
                Component:           |           Resolution:  fixed
  b10-auth                           |            Sensitive:  0
                 Keywords:           |  Add Hours to Ticket:
Estimated Number of Hours:  5.0      |          Total Hours:
                Billable?:  1        |
                Internal?:  0        |
-------------------------------------+-------------------------------------
Changes (by jinmei):

 * status:  reviewing => closed
 * resolution:   => fixed


Comment:

 Replying to [comment:24 vorner]:

 > I wasn't accusing you of it ;-). I just saw the code and worried little
 bit if it can happen or not and if it was considered. So I just asked. I
 usually write something like assert(pointer); to both make sure the bug is
 caught soon enough if it happens and to mark I considered the situation
 and I think it can't happen.

 Don't worry, I didn't interpret it as if I was accused:-) Double check
 is always good and appreciated.

 > Anyway, I agree that this bugfix is not a place for making the code
 nicer, let's worry about that during the refactoring. So, go ahead and
 merge it.

 Thanks, merged, and now closing this ticket.  I'll open a new ticket
 for the TODO (remaining minor NSEC3 cases).

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