Wireshark-commits: [Wireshark-commits] master 0415df8: Asterix: fix Coverity scan (1025624 & 102562
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: Mon, 1 Dec 2014 08:54:38 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=0415df8951cf9c9a1a73ab912aad7125e56af327
Submitter: Alexis La Goutte (alexis.lagoutte@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

0415df8 by Marko Hrastovec (marko.hrastovec@xxxxxxxxx):

    Asterix: fix Coverity scan (1025624 & 1025625: Missing break in switch)
    
    
    Two comments are added to the code to notify Coverity scan that breaks are not put in switch statement intentionally.
    
    Change-Id: Ie391790ee7365da56ddf0bf7b19042c9a11efddd
    Reviewed-on: https://code.wireshark.org/review/5554
    Reviewed-by: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
    

Actions performed:

    from  4f9af86   Use proto_tree_add_bitmask for groups of proto_tree_add_boolean.
    adds  0415df8   Asterix: fix Coverity scan (1025624 & 1025625: Missing break in switch)


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