Wireshark-commits: [Wireshark-commits] master-2.4 d24db8d: S7comm: fix range check to prevent infin
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Sun, 18 Feb 2018 12:01:02 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=d24db8ddcdd4cdd5c1ea859da93681b4cc98b399
Submitter: Pascal Quantin (pascal.quantin@xxxxxxxxx)
Changed: branch: master-2.4
Repository: wireshark

Commits:

d24db8d by Pascal Quantin (pascal.quantin@xxxxxxxxx):

    S7comm: fix range check to prevent infinite loop when upper bound is 255
    
    While we are at it, fix identification of not last element in a few
    places
    
    Bug: 14423
    Change-Id: I568530949d09ddfd8c5c58d24050dfed32ce10f5
    Reviewed-on: https://code.wireshark.org/review/25851
    Reviewed-by: Pascal Quantin <pascal.quantin@xxxxxxxxx>
    Petri-Dish: Pascal Quantin <pascal.quantin@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Michael Mann <mmann78@xxxxxxxxxxxx>
    (cherry picked from commit 5d99febe66e96b55a1defa58a906be254bad3a51)
    Conflicts:
    	epan/dissectors/packet-s7comm.c
    Reviewed-on: https://code.wireshark.org/review/25867
    

Actions performed:

    from  a509768   [Automatic update for 2018-02-18]
    adds  d24db8d   S7comm: fix range check to prevent infinite loop when upper bound is 255


Summary of changes:
 epan/dissectors/packet-s7comm.c | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)