Re: [PATCH v2 2/4] fix a realloc-related memory leak in localhost()

[Thomas Schwinge]

Hi!

On Wed, 07 Sep 2011 17:19:12 +0200, Thomas Schwinge <thomas@schwinge.name> 
wrote:
> On Mon, 15 Aug 2011 22:10:07 +0200, Jonathan Neuschäfer 
> <j.neuschaefer@gmx.net> wrote:
> >  * libshouldbeinlibc/localhost.c (localhost): assign the return value
> >    of realloc to a temporary variable to avoid losing the old value of
> >    buf in the case of realloc failing.
> > ---
> >  libshouldbeinlibc/localhost.c |   21 ++++++++++++++-------
> >  1 files changed, 14 insertions(+), 7 deletions(-)
> > 
> > diff --git a/libshouldbeinlibc/localhost.c b/libshouldbeinlibc/localhost.c
> > index f0c6754..f21f970 100644
> > --- a/libshouldbeinlibc/localhost.c
> > +++ b/libshouldbeinlibc/localhost.c
> > @@ -39,18 +39,25 @@ localhost ()
> >     errno = 0;
> >  
> >     if (buf) {
> > +     char *new;
> >       buf_len += buf_len;
> > -     buf = realloc (buf, buf_len);
> > +     new = realloc (buf, buf_len);
> > +     if (! new)
> > +       {
> 
> Am I being dense (which is entirely possible), or did you mean to ``free
> (buf)'' here, or what is the intent here?
> 
> > +         errno = ENOMEM;
> > +         return 0;
> > +       }

Whoops/indeed, Samuel had already added the missing ``free (buf)''.


Grüße,
 Thomas

pgpAdOysoDPos.pgp
Description: PGP signature