diff options
author | etxrab <etxrab@f5534014-38df-0310-8fa8-9805f1628bb7> | 2008-07-30 14:50:17 +0000 |
---|---|---|
committer | etxrab <etxrab@f5534014-38df-0310-8fa8-9805f1628bb7> | 2008-07-30 14:50:17 +0000 |
commit | d695b8d623804b8fc827e23846cf134a6f48c4f9 (patch) | |
tree | 6b1bef0664c65aa31873850467960aac8df5b0c1 /epan/dissectors/packet-x25.c | |
parent | f5c7a2ddb4d79679b6439e9d71fdc7bc9166e5a7 (diff) |
From Gerhard Olsson:
X25 reassembly fragment table and dissected proto presented for all X25 packets
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@25874 f5534014-38df-0310-8fa8-9805f1628bb7
Diffstat (limited to 'epan/dissectors/packet-x25.c')
-rw-r--r-- | epan/dissectors/packet-x25.c | 20 |
1 files changed, 14 insertions, 6 deletions
diff --git a/epan/dissectors/packet-x25.c b/epan/dissectors/packet-x25.c index 197dbc2ac1..6eb705877e 100644 --- a/epan/dissectors/packet-x25.c +++ b/epan/dissectors/packet-x25.c @@ -2357,8 +2357,14 @@ dissect_x25_common(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, x25_reassembled_table, payload_len, m_bit_set); pinfo->fragmented = m_bit_set; - - if (fd_head) { + + /* Fragment handling is not adapted to handle several x25 + * packets in the same frame. This is common with XOT and + * shorter packet sizes. + * Therefore, fragment_add_seq_next seem to always return fd_head + * A fix to use m_bit_set to only show fragments for last pkt + */ + if (!m_bit_set && fd_head) { if (fd_head->next) { proto_item *frag_tree_item; @@ -2368,10 +2374,12 @@ dissect_x25_common(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, fd_head->len); tvb_set_child_real_data_tvbuff(tvb, next_tvb); add_new_data_source(pinfo, next_tvb, "Reassembled X.25"); - show_fragment_seq_tree(fd_head, - &x25_frag_items, - x25_tree, - pinfo, next_tvb, &frag_tree_item); + if (x25_tree) { + show_fragment_seq_tree(fd_head, + &x25_frag_items, + x25_tree, + pinfo, next_tvb, &frag_tree_item); + } } } |