diff options
author | martinm <martinm@f5534014-38df-0310-8fa8-9805f1628bb7> | 2011-03-22 17:37:47 +0000 |
---|---|---|
committer | martinm <martinm@f5534014-38df-0310-8fa8-9805f1628bb7> | 2011-03-22 17:37:47 +0000 |
commit | 7a8f9fc8b4efc840b0a4e26ce1482f0dd79018bf (patch) | |
tree | c72e5f20ee3bb5be7d24d5a44ab1488e2f7da19b /epan/dissectors | |
parent | 6adf5370a8cd7fe2ad56e83f2b8ea6a1ee9874f6 (diff) |
Don't assign top_ti - delete it for now.
Coverity CID 862
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@36254 f5534014-38df-0310-8fa8-9805f1628bb7
Diffstat (limited to 'epan/dissectors')
-rw-r--r-- | epan/dissectors/packet-catapult-dct2000.c | 14 |
1 files changed, 1 insertions, 13 deletions
diff --git a/epan/dissectors/packet-catapult-dct2000.c b/epan/dissectors/packet-catapult-dct2000.c index cf0146db8f..845b685a2e 100644 --- a/epan/dissectors/packet-catapult-dct2000.c +++ b/epan/dissectors/packet-catapult-dct2000.c @@ -811,11 +811,9 @@ static void dissect_ccpri_lte(tvbuff_t *tvb, gint offset, tvbuff_t *ccpri_tvb; dissector_handle_t protocol_handle = 0; guint16 length; - proto_item *top_ti; /* TODO: pass one in */ /* Top-level opcode */ - top_ti = proto_tree_add_item(tree, hf_catapult_dct2000_lte_ccpri_opcode, - tvb, offset, 1, FALSE); + proto_tree_add_item(tree, hf_catapult_dct2000_lte_ccpri_opcode, tvb, offset, 1, FALSE); opcode = tvb_get_guint8(tvb, offset++); /* Skip 2-byte length field */ @@ -863,16 +861,6 @@ static void dissect_ccpri_lte(tvbuff_t *tvb, gint offset, ccpri_tvb = tvb_new_subset(tvb, offset, length, length); call_dissector_only(protocol_handle, ccpri_tvb, pinfo, tree); } - -#if 0 - /* Grumble if there are bytes left after indicated payload */ - offset += length; - if (tvb_length_remaining(tvb, offset) > 0) { - expert_add_info_format(pinfo, top_ti, PI_MALFORMED, PI_ERROR, - "CCPRI frame has %u bytes remaining after payload", - tvb_length_remaining(tvb, offset)); - } -#endif } |