Wireshark-commits: [Wireshark-commits] master ba3cfb4: WCCP: Prevent very long loop in dissect_wccp

Date Prev · Date Next · Thread Prev · Thread Next
Date Index · Thread Index · Other Months · All Mailing Lists
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Mon, 12 Feb 2018 06:34:41 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=ba3cfb403a403647241062b933ff82808ac62a59
Submitter: Anders Broman (a.broman58@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

ba3cfb4 by Michael Mann (mmann78@xxxxxxxxxxxx):

    WCCP: Prevent very long loop in dissect_wccp2_alternate_mask_value_set_element
    
    Sanity check the total_length to make sure it doesn't go negative.
    
    Bug: 14412
    Change-Id: I87e38f6c792fa81184e4c412d6433fbbf2060f28
    Reviewed-on: https://code.wireshark.org/review/25744
    Reviewed-by: Michael Mann <mmann78@xxxxxxxxxxxx>
    Petri-Dish: Michael Mann <mmann78@xxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  620647a   SCCP: Use int datatype for offset to prevent infinite loops
    adds  ba3cfb4   WCCP: Prevent very long loop in dissect_wccp2_alternate_mask_value_set_element


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