Wireshark-bugs: [Wireshark-bugs] [Bug 8962] Enhancement: Add STANAG 4607 file format
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: bugzilla-daemon@xxxxxxxxxxxxx
Date: Mon, 29 Jul 2013 00:00:54 +0000

changed bug 8962

What Removed Added
Status IN_PROGRESS RESOLVED
Resolution --- FIXED

Comment # 16 on bug 8962 from 
(In reply to comment #15)
> Comment on attachment 11292 [details]
> patch to add 4607 file format support
> 
> Thanks for the updated patch, committed in r50996. I don't think arpa/inet.h
> is available on all the platforms for which Wireshark builds, but it's
> available on all the ones I've got, so we'll have to wait for the buildbots
> to tell us where it doesn't work.

Fixed the header in r50997.

You read a 64-bit seconds value from the packet, but there are some platforms
on which the time_t we store it in is only 32-bits. I've added casts for now,
but there may be explicit handling you want to do in case that field is less
than 8 bytes.

You are receiving this mail because:
  • You are watching all bug changes.