bug-hurd
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH 1/3] Add the ``--mount'' command line option


From: Sergiu Ivanov
Subject: Re: [PATCH 1/3] Add the ``--mount'' command line option
Date: Mon, 3 Aug 2009 19:54:02 +0300
User-agent: Mutt/1.5.18 (2008-05-17)

Hello,

On Fri, Jul 17, 2009 at 04:05:42AM +0200, olafBuddenhagen@gmx.net wrote:
> On Tue, Jul 14, 2009 at 12:27:02PM +0300, Sergiu Ivanov wrote:
> > +
> > +      /* The mountee command line converted to a 0-terminated string
> > +    form.  */
> > +      char * mountee_cl = malloc (mountee_argz_len);
> 
> [...]
> BTW, does unionfs use this style (malloc right in the variable
> definition) in other places? It's discouraged by GCS...

No, it does not.  I'd suppose I was being lazy when I used such style :-(
 
> > +
> > +      err = argz_add (argz, argz_len, opt);
> > +    }
> > +
> >    ulfs_iterate
> >      {
> >        if (! err)
> 
> You forgot to free the temporary strings...

Great :-( I'll try to avoid such silly mistakes in the future.

Regards,
scolobb




reply via email to

[Prev in Thread] Current Thread [Next in Thread]