Re: [monit-dev] monit sendmail.c

[Martin Pala]

I think i saw crash on some OS in the past, when free(0) is called ... 
if you are 100% sure that it will work on all systems, no problem :)

Martin


Jan-Henrik Haukeland wrote:
> FYI its completely valid to call free with a NULL pointer. From the C 
> (99) spec
>
> "The free function causes the space pointed to by ptr to be deallocated, 
> that is, made
> available for further allocation. If ptr is a null pointer, no action 
> occurs."
>
> On 18. feb.. 2008, at 21.15, Martin Pala wrote:
>
> > CVSROOT:    /sources/monit
> > Module name:    monit
> > Changes by:    Martin Pala <martinp>    08/02/18 20:15:52
> >
> > Modified files:
> >     .              : sendmail.c
> >
> > Log message:
> >     revert the recent change - monit's FREE macro doesn't check for 
> > NULL ... the b64 may be NULL and if it is used as argument for free it 
> > can crash
> >
> > CVSWeb URLs:
> > http://cvs.savannah.gnu.org/viewcvs/monit/sendmail.c?cvsroot=monit&r1=1.56&r2=1.57 
>
>
>
> _______________________________________________
> monit-dev mailing list
> address@hidden
> http://lists.nongnu.org/mailman/listinfo/monit-dev