Wireshark-commits: [Wireshark-commits] master 8ff55a9: forces: make redirecttlv robust to invalid l
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: Tue, 10 Mar 2015 15:53:45 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=8ff55a910ee0408910cbbe6d1823987f47f8f521
Submitter: Evan Huus (eapache@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

8ff55a9 by Evan Huus (eapache@xxxxxxxxx):

    forces: make redirecttlv robust to invalid lengths
    
    Fixes a potential infinite loop reported by Vlad Tsyrklevich found via the
    "joern" tool. I'm pretty sure the semantics of proto_tree_add_item would have
    prevented this, but not 100% and making it explicit doesn't hurt.
    
    Bug: 11037
    Change-Id: I92049a95d23ca9c233b3fd830637e6bca19a7434
    Reviewed-on: https://code.wireshark.org/review/7592
    Petri-Dish: Evan Huus <eapache@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Evan Huus <eapache@xxxxxxxxx>
    

Actions performed:

    from  5a1b32b   packet-spnego: fix krb5_cfx_wrap without encryption
    adds  8ff55a9   forces: make redirecttlv robust to invalid lengths


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