Re: [Bug-readline] Re: PATCH: readline/histexpand.c, resource leak

Chet Ramey
2007-08-22T02:02:23Z

Jim Blandy wrote:

>>> It seems to me that 'words' is a malloc'ed array of malloc'ed strings,
>>> so 'free (words)' still leaks storage.
>> I don't believe there's a case where wind == -1 and any malloced strings
>> exist in words[]. There's no leak unless that can happen.
>
> Not even if history_tokenize_internal's 'windp' argument falls in
> whitespace between words?

Yes, I think you're right. Thanks.

Chet
--
``The lyf so short, the craft so long to lerne.'' - Chaucer
Live Strong. No day but today.
Chet Ramey, ITS, CWRU chet@case.edu http://cnswww.cns.cwru.edu/~chet/