Wireshark · Wireshark-commits: [Wireshark-commits] lts-1.12.1 fb35776: use the captured length as payload length for zigbee decrypt

[Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>]

URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=fb35776294988941c0f304d6a9bc3a86a996201b
Submitter: Balint Reczey (balint@xxxxxxxxxxxxxxx)
Changed: branch: lts-1.12.1
Repository: wireshark

Commits:

fb35776 by Martin Kaiser (wireshark@xxxxxxxxx):

    use the captured length as payload length for zigbee decryption
    
    this length is used for allocating a buffer and for crypto calculations
    we should use the bytes that were actually captured, not the reported
    length
    
    the capture in https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=11389
    has one packet with a stupidly large reported length and a reasonable
    capture length (and one completely bogus packet) - this fix ensures that
    the decryption does not break for the former packet
    
    also, tvb_reported_length_remaining() does not return values < 0 any
    more, remove the check for this
    
    Bug: 11389
    Change-Id: I42cb4526483160416b51e3cb72442148b5fac4f3
    Reviewed-on: https://code.wireshark.org/review/9950
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    (cherry picked from commit 655b0dc623e29da212be3e205314624fe3182562)
    Reviewed-on: https://code.wireshark.org/review/9955
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    (cherry picked from commit 84a3493c074ba0d59927593e58175521fee07069)
    Reviewed-on: https://code.wireshark.org/review/10615
    Reviewed-by: Balint Reczey <balint@xxxxxxxxxxxxxxx>
    

Actions performed:

    from  fedf79a   Protect dissector_get_(default_)string_handle() against NULL input string
    adds  fb35776   use the captured length as payload length for zigbee decryption


Summary of changes:
 epan/dissectors/packet-zbee-security.c |    6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)