Wireshark · Wireshark-commits: [Wireshark-commits] master eb3cdb9: Zero out pinfo->src and dst before adding a frag.

[Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>]

URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=eb3cdb925c29e7e226f3d3a67b42126302bc0d38
Submitter: Anders Broman (a.broman58@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

eb3cdb9 by Guy Martin (gmsoft@xxxxxxxxxxxx):

    Zero out pinfo->src and dst before adding a frag.
    
    If an mp2t packet contains one full subpacket and the fragment of
    another one, it happens that the first subpacket will set src or dst to
    an ethernet or IP address. Adding the fragment of the second subpacket
    will then use this information for calculating the hash in the fragment
    table. However, later fragments in other mp2t packets will not have
    these info and reassembly will fail.
    
    Change-Id: Ic52763017cb854851b6686654c2d8a1624305d65
    Reviewed-on: https://code.wireshark.org/review/1692
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  a344c97   Revert "Allow wtap_read() and wtap_seek_read() to return non-packet records."
    adds  eb3cdb9   Zero out pinfo->src and dst before adding a frag.


Summary of changes:
 epan/dissectors/packet-mp2t.c |    6 ++++++
 1 file changed, 6 insertions(+)