Wireshark-commits: [Wireshark-commits] master-2.2 ca28587: dns: fix this statement may fall through

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: Mon, 10 Apr 2017 11:15:50 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=ca285878294c959d386d83ce9f3acea65b0b3194
Submitter: Pascal Quantin (pascal.quantin@xxxxxxxxx)
Changed: branch: master-2.2
Repository: wireshark

Commits:

ca28587 by Alexis La Goutte (alexis.lagoutte@xxxxxxxxx):

    dns: fix this statement may fall through [-Werror=implicit-fallthrough] found by gcc7
    
    and add also a missing (optional) break after default
    
    Change-Id: I03d594db4702ccb4ebab3fca6b071d323e7f8d39
    Reviewed-on: https://code.wireshark.org/review/20994
    Reviewed-by: Pascal Quantin <pascal.quantin@xxxxxxxxx>
    (cherry picked from commit 8344f5a6919d04376fe06db8db36fe430e327371)
    Reviewed-on: https://code.wireshark.org/review/20996
    

Actions performed:

    from  ac56833   SSH: display protocol without carriage return / newline
    adds  ca28587   dns: fix this statement may fall through [-Werror=implicit-fallthrough] found by gcc7


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