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 /AUTHORS | |
| parent | 0091c984df28b5e9c4d33e64dff85349f9c3e4e3 (diff) | |
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 'AUTHORS')
| -rw-r--r-- | AUTHORS | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -3681,7 +3681,7 @@ Johanna Sochos <johanna.sochos[AT]swissqual.com> Felix Obenhuber <felix[AT]obenhuber.de> Hilko Bengen <bengen--wireshark[AT]hilluzination.de> Hadar Shoham <hadar[AT]ti.com> -Robert Bullen <robert[AT]bitcricket.com> +Robert Bullen <robert[AT]robertbullen.com> Chuck Kristofek <chuck.kristofek[AT]ngc.com> Markus Renz <Markus.Renz[AT]hirschmann.de> Toshihiro Kataoka <kataoka.toshihiro[AT]gmail.com> |