diff options
author | Birol Capa <birol.capa@siemens.com> | 2016-09-28 07:56:14 +0300 |
---|---|---|
committer | Michael Mann <mmann78@netscape.net> | 2016-09-29 00:51:15 +0000 |
commit | a88b7bc1a6f4033a0b98c9847d259fb3d687fcf4 (patch) | |
tree | 3c75ba5b14df0c2f5225c4cd9ac0f6ddf86f1c4b /plugins | |
parent | 814477c32f9ac7838cf7364fcc83f36378ea2159 (diff) |
PROFINET: Fix switch statement has no default
A switch statement has no section labeled default.
To make the code more readable add the default statement with a good comment.
Change-Id: Iacd9b1520628e229baedc89277d07ffac2caf1b6
Reviewed-on: https://code.wireshark.org/review/17960
Reviewed-by: Jaap Keuter <jaap.keuter@xs4all.nl>
Petri-Dish: Jaap Keuter <jaap.keuter@xs4all.nl>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Michael Mann <mmann78@netscape.net>
Diffstat (limited to 'plugins')
-rw-r--r-- | plugins/profinet/packet-dcerpc-pn-io.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/plugins/profinet/packet-dcerpc-pn-io.c b/plugins/profinet/packet-dcerpc-pn-io.c index 8b7948405e..3f22db8d00 100644 --- a/plugins/profinet/packet-dcerpc-pn-io.c +++ b/plugins/profinet/packet-dcerpc-pn-io.c @@ -8464,6 +8464,8 @@ dissect_ExpectedSubmoduleBlockReq_block(tvbuff_t *tvb, int offset, offset = dissect_DataDescription(tvb, offset, pinfo, sub_tree, drep, io_data_object); offset = dissect_DataDescription(tvb, offset, pinfo, sub_tree, drep, io_data_object); break; + default: /* will not execute because of the line preceding the switch */ + break; } proto_item_append_text(sub_item, ": Subslot:0x%x, Ident:0x%x Properties:0x%x", @@ -10034,6 +10036,8 @@ dissect_IODWriteReq(tvbuff_t *tvb, int offset, case(1): offset += 3; break; + default: /* will not execute because of the line preceding the switch */ + break; } } |