This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

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


---------------------------- Original Message ----------------------------
Subject: Re: PATCH: readline/histexpand.c, resource leak
From:    msnyder@sonic.net
Date:    Tue, August 14, 2007 5:37 pm
To:      "Jim Blandy" <jimb@codesourcery.com>
Cc:      msnyder@sonic.net
         gdb-patches@sourceware.org
         bug-readline@gnu.org
--------------------------------------------------------------------------

>
> It seems to me that 'words' is a malloc'ed array of malloc'ed strings,
> so 'free (words)' still leaks storage.

Yeaaaahh... maybe the attached is better?  Or is this use
of goto a little opaque?

By the way, are we maintaining our own readline fork?
Should I check these in locally?

Attachment: 130b.txt
Description: Text document


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]