Hi Michael again,
Thanks. I hadn't hit that one yet, but I found a similar issue a
little while back and modified tsk_malloc() to do a memset for all
memory allocated.
I haven't run valgrind in a while, but I should since I have made so
many changes recently for the new APIs. I'll make sure I do.
thanks,
brian
On Jun 23, 2008, at 8:41 AM, Michael Cohen wrote:
> Hi All,
> I was tracking down a crash in pyflag's sk support and ran valgrind
> on fls with an iso image. It turns out that in iso9660.c at line 406
> the structure in_inode is allocated, but not initialised. This may
> cause a crash later in line 571:
>
> if (in_node->inode.rr)
> free(in_node->inode.rr);
>
> (because there could be any rubbish in there).
>
> I found that a memset(in_node, 0 , sizeof
> (iso9660_inode_node));
> right after the allocation at line 412 fixed things.
>
> Also added a memset(&t, 0, sizeof(t));
> at line 869 to stop other valgrind complaints.
>
> Probably not a bad idea to run valgrind over all the sk tools now and
> again. The ntfs drivers are certainly very quiet with valgrind which
> is great.
>
> Thanks,
> Michael.
>
> ----------------------------------------------------------------------
> ---
> Check out the new SourceForge.net Marketplace.
> It's the best place to buy or sell services for
> just about anything Open Source.
> http://sourceforge.net/services/buy/index.php
> _______________________________________________
> sleuthkit-developers mailing list
> sle...@li...
> https://lists.sourceforge.net/lists/listinfo/sleuthkit-developers
|
