diff options
author | Evan Huus <eapache@gmail.com> | 2013-05-17 21:50:27 +0000 |
---|---|---|
committer | Evan Huus <eapache@gmail.com> | 2013-05-17 21:50:27 +0000 |
commit | 48285bb16b0a5655a47a8a59c34bc98e6bf4cb75 (patch) | |
tree | 1cef655e56b7009a24126bfa8a6f5689daad464e /wiretap/i4b_trace.h | |
parent | 0091c984df28b5e9c4d33e64dff85349f9c3e4e3 (diff) | |
download | wireshark-48285bb16b0a5655a47a8a59c34bc98e6bf4cb75.tar.gz |
From Robert Bullen via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8643
When a TCP segment contains the end of two or more SSL PDUs, the TCP reassembly
code passes that segment up to the SSL dissector multiple times--one for each
SSL PDU. The SSL dissector queues the packet for SSL tap listeners each time it
is invoked. Therefore a single packet can be processed by SSL tap listeners
multiple times. But the tap data that the SSL dissector sends to its tap
listeners is a linked list of all PDUs in the packet.
The SSL tap listener responsible for populating the Follow SSL Stream dialog
did not account for the possibility of seeing a packet multiple times. As a
result, it would process the entire linked list of PDUs each time it received a
packet, and that would result in some SSL PDUs showing up two or more times in
the dialog.
This patch fixes the described bug. It also implements a few slight
improvements in closely related code. See bugzilla for details.
svn path=/trunk/; revision=49387
Diffstat (limited to 'wiretap/i4b_trace.h')
0 files changed, 0 insertions, 0 deletions