BIND 10 #2264: Pkt4::pack() and Pkt6::pack() should throw an exception on failure

[BIND 10 Development]

#2264: Pkt4::pack() and Pkt6::pack() should throw an exception on failure
-------------------------------------+-------------------------------------
            Reporter:  stephen       |                        Owner:  tmark
                Type:  enhancement   |                       Status:
            Priority:  medium        |  reviewing
           Component:  dhcp          |                    Milestone:
            Keywords:                |  Sprint-DHCP-20130731
           Sensitive:  0             |                   Resolution:
         Sub-Project:  DHCP          |                 CVSS Scoring:
Estimated Difficulty:  0             |              Defect Severity:
         Total Hours:  0             |  Medium
                                     |  Feature Depending on Ticket:
                                     |          Add Hours to Ticket:  0
                                     |                    Internal?:  0
-------------------------------------+-------------------------------------
Changes (by stephen):

 * owner:  stephen => tmark


Comment:

 Reviewed commit f395f315a51a23ac058456ec15580231f145bd75

 I think change is sufficiently small enough not to need a !ChangeLog
 entry.

 '''src/bin/dhcp4/dhcp4_srv.cc'''[[BR]]
 The change has removed the DHCP4_PACK_FAIL error; this should be removed
 from the message file as well.

 '''src/bin/dhcp6/dhcp6_srv.cc'''[[BR]]
 The change has removed the DHCP6_PACK_FAIL error; this should be removed
 from the message file as well.

 '''src/lib/dhcp/pkt6.cc'''[[BR]]
 '''src/lib/dhcp/pkt6.{cc,h}'''[[BR]]
 packUDP():  Method header needs to be altered.  The method has been
 implemented and throws an !InvalidOperation exception on an error.

 packTCP(): Method header needs to be altered to indicate the exception
 thrown.  Currently the code throws the "Unexpected" exception: I suggest
 that "!NotImplemented" would be a better choice. (If this is changed, the
 header for pack() needs to be altered.)

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