Wireshark-commits: [Wireshark-commits] master e915db7: Do not iterate over reported length when usi
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Tue, 10 Feb 2015 14:02:25 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=e915db7cc8c779368afef51c12241337b4d7beda
Submitter: Pascal Quantin (pascal.quantin@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

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

    Do not iterate over reported length when using tvb_find_line_end
    
    if captured length < reported length, this will trigger an infinite loop
    
    Change-Id: I6557b455e7bbff12658a934e5bb13a42c023e133
    Reviewed-on: https://code.wireshark.org/review/7053
    Petri-Dish: Pascal Quantin <pascal.quantin@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Michael Mann <mmann78@xxxxxxxxxxxx>
    Reviewed-by: Pascal Quantin <pascal.quantin@xxxxxxxxx>
    

Actions performed:

    from  7369c25   IMAP: do not iterate over reported length when using tvb_find_line_end
    adds  e915db7   Do not iterate over reported length when using tvb_find_line_end


Summary of changes:
 epan/dissectors/packet-catapult-dct2000.c |    2 +-
 epan/dissectors/packet-http.c             |    4 ++--
 epan/dissectors/packet-irc.c              |    2 +-
 epan/dissectors/packet-l1-events.c        |    2 +-
 epan/dissectors/packet-ldss.c             |    2 +-
 epan/dissectors/packet-megaco.c           |    5 ++---
 epan/dissectors/packet-memcache.c         |    6 +++---
 epan/dissectors/packet-mgcp.c             |   11 +++++------
 epan/dissectors/packet-mrcpv2.c           |    2 +-
 epan/dissectors/packet-msrp.c             |    4 ++--
 epan/dissectors/packet-multipart.c        |    4 ++--
 epan/dissectors/packet-rtsp.c             |    2 +-
 epan/dissectors/packet-sdp.c              |    4 ++--
 epan/dissectors/packet-sip.c              |    2 +-
 epan/dissectors/packet-sipfrag.c          |    2 +-
 epan/dissectors/packet-text-media.c       |    2 +-
 16 files changed, 27 insertions(+), 29 deletions(-)