diff options
| author | Alexis La Goutte <alexis.lagoutte@gmail.com> | 2015-01-07 18:19:58 +0100 |
|---|---|---|
| committer | Pascal Quantin <pascal.quantin@gmail.com> | 2015-01-07 18:32:22 +0000 |
| commit | d3f67c16d4ac25bc1b70d2d65da82690b370e64e (patch) | |
| tree | 0ad02ac188ada14a96e0f821c9f448bdbed1414b /epan/dissectors/packet-s5066dts.c | |
| parent | a6ba632ec31fbccd76f8d7ddab39759669fe7c76 (diff) | |
S5066DTS: Nesting level does not match indendation (CID 1159211 & 1159212)
Change-Id: I8145696de995e821793e7bc1576d5a5673c7b503
Reviewed-on: https://code.wireshark.org/review/6377
Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
Diffstat (limited to 'epan/dissectors/packet-s5066dts.c')
| -rw-r--r-- | epan/dissectors/packet-s5066dts.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/epan/dissectors/packet-s5066dts.c b/epan/dissectors/packet-s5066dts.c index 0d559e7cf1..62b1cf64af 100644 --- a/epan/dissectors/packet-s5066dts.c +++ b/epan/dissectors/packet-s5066dts.c @@ -566,8 +566,9 @@ static guint dissect_s5066dts_ack_only(tvbuff_t *tvb, guint offset, proto_tree * ack_size = header_size - 7; proto_tree_add_item(tree, hf_s5066dts_ack_only_rx_lwe, tvb, offset, 1, ENC_BIG_ENDIAN); offset++; - if (ack_size > 0) + if (ack_size > 0){ proto_tree_add_item(tree, hf_s5066dts_ack_only_acks, tvb, offset, ack_size, ENC_NA); offset += ack_size; + } return offset; } @@ -588,8 +589,9 @@ static guint dissect_s5066dts_data_ack(tvbuff_t *tvb, guint offset, proto_tree * proto_tree_add_item(tree, hf_s5066dts_data_ack_segmented_cpdu_size, tvb, offset, 2, ENC_BIG_ENDIAN); offset +=2; proto_tree_add_item(tree, hf_s5066dts_data_ack_transmit_sequence_number, tvb, offset, 1, ENC_BIG_ENDIAN); offset++; proto_tree_add_item(tree, hf_s5066dts_data_ack_rx_lwe, tvb, offset, 1, ENC_BIG_ENDIAN); offset++; - if (ack_size > 0) + if (ack_size > 0){ proto_tree_add_item(tree, hf_s5066dts_data_ack_acks, tvb, offset, ack_size, ENC_NA); offset += ack_size; + } return offset; } |