diff options
author | Alexis La Goutte <alexis.lagoutte@gmail.com> | 2015-11-30 17:26:30 +0100 |
---|---|---|
committer | Anders Broman <a.broman58@gmail.com> | 2015-11-30 20:59:22 +0000 |
commit | d59335b9061fecfae59ffe54b86d9ee6ebfe9317 (patch) | |
tree | c9b4015c44f987b763b68b0608a4c456ab81928a /epan/dissectors/packet-adb_service.c | |
parent | 83516dce1aced2841dfd64b0311889c901296c67 (diff) |
adb service/logcat : fix Consider inspecting this expression. The expression is excessive or contains a misprint found by PVS Studio (V590)
Change-Id: I59cf9c8b2aab347023dea323602dd6ff85ac7ab6
Reviewed-on: https://code.wireshark.org/review/12309
Reviewed-by: Anders Broman <a.broman58@gmail.com>
Diffstat (limited to 'epan/dissectors/packet-adb_service.c')
-rw-r--r-- | epan/dissectors/packet-adb_service.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/epan/dissectors/packet-adb_service.c b/epan/dissectors/packet-adb_service.c index 5d2db14c5f..14df595976 100644 --- a/epan/dissectors/packet-adb_service.c +++ b/epan/dissectors/packet-adb_service.c @@ -494,7 +494,7 @@ dissect_adb_service(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void *d payload_length = tvb_get_letohs(next_tvb, i_offset); try_header_size = tvb_get_letohs(next_tvb, i_offset + 2); - if (try_header_size == 0 || try_header_size != 24) + if (try_header_size != 24) logcat_length = payload_length + 20; else logcat_length = payload_length + 24; |