Wireshark-commits: [Wireshark-commits] master-2.2 e553366: ISAKMP: Fix handling of cert requests wi
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: Mon, 5 Sep 2016 22:58:19 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=e553366562bd04fd9a2aa7937c49b9291e84a77e
Submitter: Michael Mann (mmann78@xxxxxxxxxxxx)
Changed: branch: master-2.2
Repository: wireshark

Commits:

e553366 by Mirko Parthey (mirko.parthey@xxxxxx):

    ISAKMP: Fix handling of cert requests without CA
    
    Check IKEv1 Certificate Request Payloads for an empty
    Certificate Authority field, which is allowed by RFC 2408.
    Suppress dissection of this field if it is indeed empty.
    
    Change-Id: Ifb997e460a4c12003215fde86c374cfc769c5d72
    Reviewed-on: https://code.wireshark.org/review/17501
    Reviewed-by: Michael Mann <mmann78@xxxxxxxxxxxx>
    Petri-Dish: Michael Mann <mmann78@xxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
    (cherry picked from commit 70f3737c3e4d9402cb2bb67cdd892e0e7e0ee991)
    Reviewed-on: https://code.wireshark.org/review/17504
    

Actions performed:

    from  ae6ae02   ERF: Fix dissector abort on short meta tags and typos
    adds  e553366   ISAKMP: Fix handling of cert requests without CA


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