Wireshark · Wireshark-commits: [Wireshark-commits] lts-1.12.1 e062961: ASTERIX: Avoid an integer overflow.

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

URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=e062961f0ac149069ee4beb807c7e5bda53c3b17
Submitter: Balint Reczey (balint@xxxxxxxxxxxxxxx)
Changed: branch: lts-1.12.1
Repository: wireshark

Commits:

e062961 by Gerald Combs (gerald@xxxxxxxxxxxxx):

    ASTERIX: Avoid an integer overflow.
    
    Count using a guint instead of a guint8.
    
    Bug: 13344
    Change-Id: Ic7d337dbb29b92ebb8332f50fd47b5ba2aa3f41e
    Reviewed-on: https://code.wireshark.org/review/19746
    Petri-Dish: Gerald Combs <gerald@xxxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Gerald Combs <gerald@xxxxxxxxxxxxx>
    (cherry picked from commit 781f03580c81339513bb1238b202b72469a1240b)
    Reviewed-on: https://code.wireshark.org/review/19749
    (cherry picked from commit efdff1fde0c2f2593327b42bea8c50fad92d89a6)
    Reviewed-on: https://code.wireshark.org/review/19842
    Reviewed-by: Balint Reczey <balint@xxxxxxxxxxxxxxx>
    

Actions performed:

    from  0bde1d7   packet-dtn.c break loop if evaluate_sdnv doesn't succeed.
    adds  e062961   ASTERIX: Avoid an integer overflow.


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