Wireshark-commits: [Wireshark-commits] master-2.0 345034e: h261: fix "hf->bitmask != 0" assertion
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Wed, 21 Oct 2015 17:20:19 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=345034e76b6a25d90333fcfde4aea2c710f9d271
Submitter: Michael Mann (mmann78@xxxxxxxxxxxx)
Changed: branch: master-2.0
Repository: wireshark

Commits:

345034e by Peter Wu (peter@xxxxxxxxxxxxx):

    h261: fix "hf->bitmask != 0" assertion
    
    fuzz-2014-03-13-20306.pcap from bug 9887 triggered:
    
        Dissector bug, protocol H.261, in packet 978: epan/proto.c:8418:
        failed assertion "hf->bitmask != 0" (h261.sbit)
    
    Fixes regression from v1.99.6rc0-311-g8d6324f.
    
    Change-Id: I69f537576743347cad41873222b4905d48c2fb8d
    Reviewed-on: https://code.wireshark.org/review/11194
    Petri-Dish: Peter Wu <peter@xxxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Michael Mann <mmann78@xxxxxxxxxxxx>
    (cherry picked from commit ed27dad41eb2085fa2afb51c6a66a89bd1c64ad4)
    Reviewed-on: https://code.wireshark.org/review/11196
    

Actions performed:

    from  2f58181   GSM SMS: do not call GSM SMS port IE subdissectors until message is reassembled
    adds  345034e   h261: fix "hf->bitmask != 0" assertion


Summary of changes:
 epan/dissectors/packet-h261.c |   12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)