Wireshark · Wireshark-commits: [Wireshark-commits] master-2.2 d8a0cbc: S7comm: fix range check to prevent infinite loop when upper

[Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>]

URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=d8a0cbc4f2979e0b1cadbe79f0b8b4ecb92477be
Submitter: Pascal Quantin (pascal.quantin@xxxxxxxxx)
Changed: branch: master-2.2
Repository: wireshark

Commits:

d8a0cbc 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
    (cherry picked from commit d24db8ddcdd4cdd5c1ea859da93681b4cc98b399)
    Reviewed-on: https://code.wireshark.org/review/25868
    

Actions performed:

    from  0b13a23   [Automatic update for 2018-02-18]
    adds  d8a0cbc   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(-)