On Mon, 4 Nov 2019, Hoyer, David wrote:

>               if (p->statname) free(p->statname);
> -             free(p->pathname);
> +             if (p->pathname) free(p->pathname);

I realise it’s the preexisting style, but free(NULL) is defined and a nop.

bye,
//mirabilos
-- 
15:41⎜<Lo-lan-do:#fusionforge> Somebody write a testsuite for helloworld :-)

Reply via email to