Wireshark-commits: [Wireshark-commits] master c19860d: lbmr: fix possible infinite loop

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: Wed, 11 Mar 2015 17:57:15 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=c19860dba8d158c42865c3d9051b6f318cb41169
Submitter: Evan Huus (eapache@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

c19860d by Evan Huus (eapache@xxxxxxxxx):

    lbmr: fix possible infinite loop
    
    Ensure that we handle when option_len is zero so we don't go into an infinite
    loop. Reported by Vlad Tsyrklevich and found by the "joern" tool. Also fix what
    appears to be two misplaced "curr_offset" values which would have resulted in a
    bad loop anyways.
    
    Bug: 11036
    Change-Id: I79e70fcf79015cb0add1744aff695143e11312aa
    Reviewed-on: https://code.wireshark.org/review/7593
    Reviewed-by: David Ameiss <netshark@xxxxxxxxxxxxx>
    Reviewed-by: Evan Huus <eapache@xxxxxxxxx>
    

Actions performed:

    from  9b9aa9a   RADIUS: Sync with FreeRADIUS Master
    adds  c19860d   lbmr: fix possible infinite loop


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