From 0415df8951cf9c9a1a73ab912aad7125e56af327 Mon Sep 17 00:00:00 2001 From: Marko Hrastovec Date: Mon, 1 Dec 2014 09:43:26 +0100 Subject: 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 --- epan/dissectors/packet-asterix.c | 2 ++ 1 file changed, 2 insertions(+) (limited to 'epan/dissectors/packet-asterix.c') 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); -- cgit v1.2.1