diff options
| author | Marko Hrastovec <marko.hrastovec@gmail.com> | 2014-12-01 09:43:26 +0100 |
|---|---|---|
| committer | Alexis La Goutte <alexis.lagoutte@gmail.com> | 2014-12-01 08:54:35 +0000 |
| commit | 0415df8951cf9c9a1a73ab912aad7125e56af327 (patch) | |
| tree | 5bd3d35b5f18c17d40709e370de8d9ceb1684d87 /epan/dissectors/packet-asterix.c | |
| parent | 4f9af8608cc4279b3f559db3d2f3bc6224ad58a5 (diff) | |
| download | wireshark-0415df8951cf9c9a1a73ab912aad7125e56af327.tar.gz | |
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@gmail.com>
Diffstat (limited to 'epan/dissectors/packet-asterix.c')
| -rw-r--r-- | epan/dissectors/packet-asterix.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/epan/dissectors/packet-asterix.c b/epan/dissectors/packet-asterix.c index c99a57f49f..ca6c78896b 100644 --- a/epan/dissectors/packet-asterix.c +++ b/epan/dissectors/packet-asterix.c @@ -7120,6 +7120,7 @@ static void asterix_build_subtree (tvbuff_t *tvb, guint offset, proto_tree *pare switch (field->part[i]->type) { case FIELD_PART_FX: if (!value) go_on = 0; + /* Fall through */ case FIELD_PART_INT: case FIELD_PART_UINT: case FIELD_PART_HEX: @@ -7129,6 +7130,7 @@ static void asterix_build_subtree (tvbuff_t *tvb, guint offset, proto_tree *pare break; case FIELD_PART_FLOAT: twos_complement (&value, field->part[i]->bit_length); + /* Fall through */ case FIELD_PART_UFLOAT: if (field->part[i]->format_string != NULL) proto_tree_add_double_format_value (parent, *field->part[i]->hf, tvb, offset + inner_offset / 8, byte_length (field->part[i]->bit_length), value * field->part[i]->scaling_factor, field->part[i]->format_string, value * field->part[i]->scaling_factor); |