Re: [PATCH 1/1] libctrl: only free() msgb if it was alloc()ed

[Alexander Huemer]

On Tue, Oct 11, 2011 at 11:07:35PM +0200, Holger Hans Peter Freyther wrote:
> On 10/11/2011 10:25 PM, Alexander Huemer wrote:
> 
> >     struct ipaccess_head_ext *iph_ext;
> > -   struct msgb *msg;
> > +   struct msgb *msg = NULL;
> 
> yes
> 
> con);
> > -   msgb_free(msg);
> > +   if (msg)
> > +           msgb_free(msg);
> 
> no, i think we should assume that msgb_free(NULL) will work.
> 
> thanks
> 
You are right. I'll resend the patch.