diff options
author | John Thacker <johnthacker@gmail.com> | 2022-06-09 22:49:03 -0400 |
---|---|---|
committer | A Wireshark GitLab Utility <gerald+gitlab-utility@wireshark.org> | 2022-06-14 00:59:34 +0000 |
commit | f1cbc6b6623074ad6f403993e9f0cfee9bee4300 (patch) | |
tree | a74405dc8a2be44b16f8925e06089ca1f58688fb /packaging/debian | |
parent | 47c418d4195327d9b506a39ba078838dc4a45204 (diff) |
epan: Remove fragment_get_reassembled()
Because completed reassemblies are hashed in the reassembled_table for
all the frame numbers that contributed fragments,
fragment_get_reassembled_id() works wherever fragment_get_reassembled()
does, and also works where the fragment id is not the frame number.
However, since the reassembled_table hash key only depends on the
fragment id and the frame number, it only allows a frame to have
one reassembly with a given fragment id. Some protocols can have
more than one reassembly with a given fragment id (that differ on
addresses or other keys), such as GSM SMS, and the wrong reassembly
is retrieved on the second pass in those cases.
For this reason, we might want to add additional key elements to
reassembled_table, such as layer number. fragment_get_reassembled_id
already takes packet_info as a parameter and can accommodate that
without further changes, but fragment_get_reassembled cannot, so
remove the latter in favor of the former.
Diffstat (limited to 'packaging/debian')
-rw-r--r-- | packaging/debian/libwireshark0.symbols | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/packaging/debian/libwireshark0.symbols b/packaging/debian/libwireshark0.symbols index 6f42e613a6..38c802d14e 100644 --- a/packaging/debian/libwireshark0.symbols +++ b/packaging/debian/libwireshark0.symbols @@ -710,7 +710,6 @@ libwireshark.so.0 libwireshark0 #MINVER# fragment_delete@Base 1.9.1 fragment_end_seq_next@Base 1.9.1 fragment_get@Base 1.9.1 - fragment_get_reassembled@Base 1.9.1 fragment_get_reassembled_id@Base 1.9.1 fragment_get_tot_len@Base 1.9.1 fragment_set_partial_reassembly@Base 1.9.1 |