BIND 10 #2000: update libdns++ OPT RDATA so it can handle EDNS options

BIND 10 Development do-not-reply at isc.org
Wed Feb 19 10:39:24 UTC 2014


#2000: update libdns++ OPT RDATA so it can handle EDNS options
-------------------------------------+-------------------------------------
            Reporter:  jinmei        |                        Owner:  muks
                Type:  enhancement   |                       Status:
            Priority:  medium        |  reviewing
           Component:  libdns++      |                    Milestone:
            Keywords:                |  bind10-1.2-release-freeze
           Sensitive:  0             |                   Resolution:
         Sub-Project:  DNS           |                 CVSS Scoring:
Estimated Difficulty:  4             |              Defect Severity:  N/A
         Total Hours:  0             |  Feature Depending on Ticket:  NSID
                                     |          Add Hours to Ticket:  0
                                     |                    Internal?:  0
-------------------------------------+-------------------------------------
Changes (by stephen):

 * owner:  stephen => muks


Comment:

 > The pseudo-RRs are actually stored as a boost::shared_ptr to a
 std::vector of PseudoRR objects.
 Ack.  Should really be looking at the code while I type the comments and
 not rely on memory.

 > that it would first address the copy-construction issue, and also be
 simple to understand and change if API changes are required.
 Fair point, although as the number of options is generally small, I doubt
 that the copy construction overhead would be high. However, what is there
 is a valid solution to the problem, and there is no need to change it.

 All changes OK, please merge.

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


More information about the bind10-tickets mailing list