[patch] clean up potential memory leak in thread.c

Kris Warkentin kewarken@qnx.com
Mon Oct 3 15:15:00 GMT 2005


Actually, that's a pretty good idea.  We do that in a number of other 
places and I suppose it's no harder to realloc the whole structure than 
to just re-create the thread name string.

Consider the patch withdrawn.

cheers,

Kris

Daniel Jacobowitz wrote:

> On Mon, Oct 03, 2005 at 10:30:12AM -0400, Kris Warkentin wrote:
> > We're throwing named thread support into the kernel so I'm keeping an
> > arbitrary length string in there.  Would never be a huge memory leak 
> but
> > I'm fussy that way.
>
> You don't actually need this to add an arbitrary lenth string; the
> target gets to malloc this, you can use a trailing char[1] or simply
> malloc sizeof(struct) + strlen (char) + 1 and point the char* right
> after the struct.
>
> If it's clearer to do it this way, though, we can go with your patch.
> If you'd rather do that let me know; IIRC there were some formatting
> glitches.
>
> -- 
> Daniel Jacobowitz
> CodeSourcery, LLC
>

-- 
Stay up-to-date on all the QNX news!  Register at
http://www.qnx.com/news/forms/newsletter.html to
receive our newsletter.



More information about the Gdb-patches mailing list