On Monday 11,October,2010 04:17 AM, Nico Golde wrote:
> Hi,
> * Chow Loong Jin <[email protected]> [2010-10-10 21:33]:
>> When operating on certain pcap files, tcpxtract segfaults after dumping out 
>> the
>> first file. Running it through valgrind shows that there are some 
>> uninitialized
>> values of a struct in add_session(), which the patch zeroes out, thereby 
>> fixing
>> the segfaults.
> 
> Thanks for the patch! I'm not sure if this program is developed upstream 
> anymore and maybe it should get deleted from the archive. I'll look into it 
> in 
> the next days.

That's a pity. Besides the code being in a horrible state, this seems to be the
only program of its kind (combining foremost's functionality with pcap files).
Either way, it's fine if it still works, is it not?

I have another patch here that removes the multitude of sync() calls that drive
btrfs up the wall and another one that changes the way tcpxtract handles its
open files in order to avoid hitting the max number of open files, which I will
tidy up and file if this package isn't getting removed.

-- 
Kind regards,
Chow Loong Jin

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to