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

[Chet Ramey]

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    address@hidden    http://cnswww.cns.cwru.edu/~chet/