diff options
author | martinm <martinm@f5534014-38df-0310-8fa8-9805f1628bb7> | 2010-11-15 23:32:34 +0000 |
---|---|---|
committer | martinm <martinm@f5534014-38df-0310-8fa8-9805f1628bb7> | 2010-11-15 23:32:34 +0000 |
commit | 43b6d7043c5c3ffbd19578f726407e35f700984b (patch) | |
tree | 5e20552bfd2f94a4a2eaf787f32168c51ae9b083 | |
parent | f10b2641f96c7ccb52ea5c4c63bbe4feb74c8513 (diff) |
Assert was firing while working out sequence length for user plane frames.
Treat signalling plane case separately.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@34893 f5534014-38df-0310-8fa8-9805f1628bb7
-rw-r--r-- | epan/dissectors/packet-rlc-lte.c | 26 |
1 files changed, 14 insertions, 12 deletions
diff --git a/epan/dissectors/packet-rlc-lte.c b/epan/dissectors/packet-rlc-lte.c index 550534aa88..ae84c8f3c2 100644 --- a/epan/dissectors/packet-rlc-lte.c +++ b/epan/dissectors/packet-rlc-lte.c @@ -514,24 +514,26 @@ static void show_AM_PDU_in_tree(packet_info *pinfo, proto_tree *tree, tvbuff_t * p_pdcp_lte_info->channelType = Channel_DCCH; p_pdcp_lte_info->channelId = rlc_info->channelId; p_pdcp_lte_info->direction = rlc_info->direction; + + /* Set plane and sequnce number length */ p_pdcp_lte_info->no_header_pdu = FALSE; if (rlc_info->channelType == CHANNEL_TYPE_SRB) { p_pdcp_lte_info->plane = SIGNALING_PLANE; + p_pdcp_lte_info->seqnum_length = 5; } else { p_pdcp_lte_info->plane = USER_PLANE; - } - /* Set sequence number field length */ - switch (global_rlc_lte_call_pdcp_for_drb) { - case PDCP_drb_SN_7: - p_pdcp_lte_info->seqnum_length = 7; - break; - case PDCP_drb_SN_12: - p_pdcp_lte_info->seqnum_length = 12; - break; - default: - DISSECTOR_ASSERT(FALSE); - break; + switch (global_rlc_lte_call_pdcp_for_drb) { + case PDCP_drb_SN_7: + p_pdcp_lte_info->seqnum_length = 7; + break; + case PDCP_drb_SN_12: + p_pdcp_lte_info->seqnum_length = 12; + break; + default: + DISSECTOR_ASSERT(FALSE); + break; + } } p_pdcp_lte_info->rohc_compression = FALSE; |