diff options
author | Alexis La Goutte <alexis.lagoutte@gmail.com> | 2018-05-13 18:54:03 +0200 |
---|---|---|
committer | Anders Broman <a.broman58@gmail.com> | 2018-05-16 04:21:08 +0000 |
commit | d3427b8a65d926ae4c85d0ec730eaf1a574041c3 (patch) | |
tree | 3d8f88060ea4149f1fdd5aa1df1ed18a10e495b7 /epan/dissectors/packet-megaco.c | |
parent | 4dc27d9b7db7b6ca93b31abe7fd3c9123bb562de (diff) |
megaco: fix issue found PVS Studio (V517)
The use of 'if (A) {...} else if (A) {...}' pattern was detected. There is a probability of logical error presence.
Change-Id: Id3e4584d1cafebd6643dd97c21916ef6b1818d9d
Reviewed-on: https://code.wireshark.org/review/27510
Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com>
Tested-by: Petri Dish Buildbot
Reviewed-by: Anders Broman <a.broman58@gmail.com>
Diffstat (limited to 'epan/dissectors/packet-megaco.c')
-rw-r--r-- | epan/dissectors/packet-megaco.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/epan/dissectors/packet-megaco.c b/epan/dissectors/packet-megaco.c index c0dc55ae1d..d27506e185 100644 --- a/epan/dissectors/packet-megaco.c +++ b/epan/dissectors/packet-megaco.c @@ -1422,12 +1422,6 @@ nextcontext: case GCP_TRX_REPLY: cmd_type = GCP_CMD_SVCCHG_REPLY; break; default: cmd_type = GCP_CMD_NONE; break; } - } else if ( g_str_equal(command,"Subtract") ) { - switch(trx_type) { - case GCP_TRX_REQUEST: cmd_type = GCP_CMD_SUB_REQ; break; - case GCP_TRX_REPLY: cmd_type = GCP_CMD_SUB_REPLY; break; - default: cmd_type = GCP_CMD_NONE; break; - } } else { switch(trx_type) { case GCP_TRX_REQUEST: cmd_type = GCP_CMD_OTHER_REQ; break; |