[PATCH] drop some obsolete (char*) casts

Mike Frysinger vapier at gentoo.org
Fri Oct 1 06:39:22 AEST 2021


On 30 Sep 2021 20:26, Thorsten Glaser wrote:
> On Thu, 30 Sep 2021, Mike Frysinger wrote:
> > -			execl(shell, shell, "-c", args, (char *)NULL);
> > +			execl(shell, shell, "-c", args, NULL);
> 
> This just introduced a bug on LP64 platforms on some OSes ;-)
> 
> tl;dr: there are OSes that #define NULL 0 (and dalias even
> defends doing so), and this is all the standards guarantee
> as well, so use of a nil pointer constant as sentinel must
> be explicitly cast, otherwise it’s just an int.
> 
> The style(9) manpage of various BSDs has a paragraph on that
> (regarding variadic functions).

thanks ... i'm aware of the nuance, but in my hurry i was thinking these
were execv style funcs, not variadic ones.  i'll respin.
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.mindrot.org/pipermail/openssh-unix-dev/attachments/20210930/bf3eac70/attachment-0001.asc>


More information about the openssh-unix-dev mailing list