On 0526T0226, Marcelo Araujo wrote: > 2018-05-26 2:21 GMT+08:00 Brooks Davis <bro...@freebsd.org>:
[..] > > The correct code here would be one of: > > > > str = strdup(opt); > > if (str == NULL) > > goto out; > > > > No, it is not the correct code! If we go out and free(str) we have nothing > to free, because we even didn't allocated memory for str. FWIW, calling free(3) on a NULL pointer is perfectly fine. _______________________________________________ svn-src-head@freebsd.org mailing list https://lists.freebsd.org/mailman/listinfo/svn-src-head To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"