<div dir="auto"><div>POSIX says free(NULL) is valid and no action.</div><div dir="auto"><br></div><div dir="auto"><a href="https://pubs.opengroup.org/onlinepubs/009695399/functions/free.html">https://pubs.opengroup.org/onlinepubs/009695399/functions/free.html</a></div><div dir="auto"><br></div><div dir="auto">--joel<br><br><div class="gmail_quote gmail_quote_container" dir="auto"><div dir="ltr" class="gmail_attr">On Sun, Jan 12, 2025, 12:46 AM Hal Murray via devel <<a href="mailto:devel@ntpsec.org">devel@ntpsec.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
The Linux man page says <br>
       If ptr is NULL, no operation is performed.<br>
<br>
I'd expect a wrapper if there really was a problem.<br>
<br>
This code is in ntpd/ntp_config.c<br>
        if (NULL == s) {                        /* free() hates NULL */<br>
                s = estrdup("");<br>
        }<br>
<br>
Any reason not to nuke that chunk of code?<br>
<br>
<br>
-- <br>
These are my opinions.  I hate spam.<br>
<br>
<br>
<br>
_______________________________________________<br>
devel mailing list<br>
<a href="mailto:devel@ntpsec.org" target="_blank" rel="noreferrer">devel@ntpsec.org</a><br>
<a href="https://lists.ntpsec.org/mailman/listinfo/devel" rel="noreferrer noreferrer" target="_blank">https://lists.ntpsec.org/mailman/listinfo/devel</a><br>
</blockquote></div></div></div>