Evan Huus
changed
bug 8962
| What |
Removed |
Added |
| Status |
IN_PROGRESS
|
RESOLVED
|
| Resolution |
---
|
FIXED
|
Comment # 16
on bug 8962
from Evan Huus
(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.
