Wireshark-commits: [Wireshark-commits] master 2874334: ISAKMP: The modecfg attribute type INTERNAL_

From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Sat, 25 Jul 2015 17:32:50 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=2874334b5993ce6ba9e40854e40f20f419ca2385
Submitter: Alexis La Goutte (alexis.lagoutte@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

2874334 by Alexis La Goutte (alexis.lagoutte@xxxxxxxxx):

    ISAKMP: The modecfg attribute type INTERNAL_IP6_ADDRESS is not decoded correctly
    
    According to RFC7296, section 3.15.1:
    
    "The INTERNAL_IP6_ADDRESS is made up of two fields: the first is a 16-octet IPv6 address, and the second is a one-octet prefix-length as defined in [ADDRIPV6]."
    
    In the code, the type is declared as FT_IPv4 and the field is decoded only if it has 16 bytes. Instead, it should be declared as FT_IPv6 and the first 16 bytes should be added to the tree.
    
    Issue reported by Andrei Cipu
    
    Bug: 11393
    Ping-Bug: 11392
    Change-Id: I8dbc268e71fd6239dffa5469652345a68e0adc86
    Reviewed-on: https://code.wireshark.org/review/9761
    Petri-Dish: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  38a2385   MAC LTE: fix dissection of Long DRX Command Control Element
    adds  2874334   ISAKMP: The modecfg attribute type INTERNAL_IP6_ADDRESS is not decoded correctly


Summary of changes:
 epan/dissectors/packet-isakmp.c |   17 ++++++++++++-----
 1 file changed, 12 insertions(+), 5 deletions(-)