From Tom Brezinski - fix for bug 5869:
authorguy <guy@f5534014-38df-0310-8fa8-9805f1628bb7>
Tue, 3 May 2011 05:26:10 +0000 (05:26 +0000)
committerguy <guy@f5534014-38df-0310-8fa8-9805f1628bb7>
Tue, 3 May 2011 05:26:10 +0000 (05:26 +0000)
commit9f7e7a8748fa5d1df1e83437a83d3f9395a8a94b
tree1c94612101f678e7ca6f248b7e18c20787385bb9
parentad7667f91d3a37eb0ea3fdea474e9047b05390d3
From Tom Brezinski - fix for bug 5869:

This patch incorporates the following fixes from the patch attached to
bug 5671 with changes as noted below:

1.) Files where the packet header and packet data are noncontiguous are
handled improperly, resulting in read misalignment and ultimately the
error message, "Observer: bad record: Invalid magic number 0xXXXXXXXX."
This bug is caused by not obeying the packet_entry_header.offset_to_frame
field.

2.) Daylight savings time is not properly accounted for in files using
local time encoding.

3.) As of Observer/GigaStor v13.10 (bug 5671 incorrectly stated v14),
timestamps in the file format changed from local time encoding to GMT
encoding.  Wiretap has been changed to support reading both formats.
Patch submitted with bug 5671 added a separate file type to allow
writing local format.  This patch does not add the separate file type
and always writes GMT.

4.) The wtap_dumper.bytes_dumped field is not being properly incremented
as data is written to files.

This patch also incorporates the following additional enhancements /
fixes not in bug 5671:

1.) Support for reading BFR files which contain Fibre Channel captures.
Test file Fibre_Channel_Capture.bfr attached.

2.) Support for modified file header used in upcoming v15.  New header
file format takes an unused byte from the version string to allow for a
larger offset to the first packet to be specified.  Test file
V15_Lrg_Hdr_Test.bfr is attached, it is also a fuzz test as the number
of TLV items given in the header is less then the actual.

3.) It was found that if the number of TLV items given in the header was
larger then present it would fail to open the file.  Test file
V9_Num_TLVs_Too_Big.bfr is attached.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@36970 f5534014-38df-0310-8fa8-9805f1628bb7
AUTHORS
wiretap/file_access.c
wiretap/network_instruments.c
wiretap/network_instruments.h
wiretap/wtap.h