Wireshark-commits: [Wireshark-commits] master-2.0 a111aad: 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:16:59 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=a111aadafba06ded239288b377f84d68671bb305
Submitter: Pascal Quantin (pascal.quantin@xxxxxxxxx)
Changed: branch: master-2.0
Repository: wireshark

Commits:

a111aad 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/20997
    

Actions performed:

    from  805c02b   SSH: display protocol without carriage return / newline
    adds  a111aad   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(+)