Wireshark-commits: [Wireshark-commits] master-3.0 5c82699: ISAKMP: Critical flag should not be in p
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Wed, 05 Feb 2020 04:36:33 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=5c82699596f3dbf5a981aca75a2e2cae6181bc3f
Submitter: "Anders Broman <a.broman58@xxxxxxxxx>"
Changed: branch: master-3.0
Repository: wireshark

Commits:

5c82699 by Dr. Lars Völker (lars.voelker@xxxxxxxxxxxxxxxxxxxxxxx):

    ISAKMP: Critical flag should not be in proposals and transforms
    
    IKEv2 added the critical flag to all payloads except payloads and
    transforms. The dissection code just checked that it was looking at
    IKEv1 but missed a check for proposals and transforms.
    
    Bug: 16364
    Change-Id: Ia9297af039fddf2da81f9712fdf7ac165fb2d86d
    Signed-off-by: Dr. Lars Völker <lars.voelker@xxxxxxxxxxxxxxxxxxxxxxx>
    Reviewed-on: https://code.wireshark.org/review/36009
    Petri-Dish: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
    (cherry picked from commit 8c9efa5f1bc1f57b4293ea39b76c91548dc45121)
    Reviewed-on: https://code.wireshark.org/review/36014
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  231d498   [Automatic update for 2020-02-02]
     add  5c82699   ISAKMP: Critical flag should not be in proposals and transforms


Summary of changes:
 epan/dissectors/packet-isakmp.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)