Wireshark-commits: [Wireshark-commits] master ed27dad: h261: fix "hf->bitmask != 0" assertion

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, 21 Oct 2015 17:19:59 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=ed27dad41eb2085fa2afb51c6a66a89bd1c64ad4
Submitter: Michael Mann (mmann78@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

ed27dad 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>
    

Actions performed:

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


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