diff options
author | Pascal Quantin <pascal.quantin@gmail.com> | 2013-03-24 21:53:53 +0000 |
---|---|---|
committer | Pascal Quantin <pascal.quantin@gmail.com> | 2013-03-24 21:53:53 +0000 |
commit | 88c0e8e873ec945e4bf2b6c5831c16540691af52 (patch) | |
tree | e0bef2aee4c565e0d530b197b78e1e6f0326a779 /epan/dissectors/packet-gsm_map.c | |
parent | bda771b86bd18f8f5cc5ad9c22df427c17898e7c (diff) |
Fix https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8517 :
Do not tap an unknown Opration Code
svn path=/trunk/; revision=48539
Diffstat (limited to 'epan/dissectors/packet-gsm_map.c')
-rw-r--r-- | epan/dissectors/packet-gsm_map.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/epan/dissectors/packet-gsm_map.c b/epan/dissectors/packet-gsm_map.c index 9ce27a09cf..b7d909baa5 100644 --- a/epan/dissectors/packet-gsm_map.c +++ b/epan/dissectors/packet-gsm_map.c @@ -19761,13 +19761,13 @@ dissect_gsm_map(tvbuff_t *tvb, packet_info *pinfo, proto_tree *parent_tree) dissect_gsm_map_GSMMAPPDU(FALSE, tvb, 0, &asn1_ctx, tree, -1); match_strval_idx(opcode, gsm_map_opr_code_strings, &op_idx); - tap_rec.invoke = FALSE; - if ( gsmmap_pdu_type == 1 ) - tap_rec.invoke = TRUE; - tap_rec.opr_code_idx = op_idx; - tap_rec.size = gsm_map_pdu_size; + if (op_idx != -1) { + tap_rec.invoke = (gsmmap_pdu_type == 1) ? TRUE : FALSE; + tap_rec.opr_code_idx = op_idx; + tap_rec.size = gsm_map_pdu_size; - tap_queue_packet(gsm_map_tap, pinfo, &tap_rec); + tap_queue_packet(gsm_map_tap, pinfo, &tap_rec); + } } const value_string ssCode_vals[] = { |