Re: [PATCH] mkstemp: fix _POSIX_C_SOURCE value for mkstemp()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi, thanks for the response!

> On 08 Jul 2016 02:06, Quentin Rameau wrote:
> > Here are more informations to save you the time searching for it.
> > Introduced in f095bb7 (2010-01-09) in stdlib/stdlib.h line 609:
> > 
> > +#if defined __USE_MISC || defined __USE_XOPEN_EXTENDED \
> > +    || defined __USE_XOPEN2K8
> > [..]
> > extern int mkstemp (char *__template) __nonnull ((1)) __wur;
> > 
> > And in include/features.h (from glibc 2.12 e28c887):
> > 
> > #if (_POSIX_C_SOURCE - 0) >= 200112L
> > # define __USE_XOPEN2K		1
> > # undef __USE_ISOC95
> > # define __USE_ISOC95		1
> > # undef __USE_ISOC99
> > # define __USE_ISOC99		1
> > #endif
> > 
> > #if (_POSIX_C_SOURCE - 0) >= 200809L
> > # define __USE_XOPEN2K8		1
> > # undef  _ATFILE_SOURCE
> > # define _ATFILE_SOURCE	1
> > #endif
> 
> what is it you're trying to report ?  the man page already reads:
>        mkstemp():
>            _XOPEN_SOURCE >= 500
>                || /* Since glibc 2.12: */ _POSIX_C_SOURCE >= 200112L
>                || /* Glibc versions <= 2.19: */ _SVID_SOURCE || _BSD_SOURCE

Yes, that's the whole point, it is incorrect!
mkstemp is declared with __USE_XOPEN2K8 defined, which occurs when
_POSIX_C_SOURCE is defined to 200809L, not 200112L as it is incorrectly
stated in the current manpage version.

> can you attach a patch ?
> -mike

Of course, already did that in fact, that's the first mail of the
thread.
I can link the marc.info archive to you if that can help:
http://marc.info/?l=linux-man&m=146737548209893

--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Kernel Documentation]     [Netdev]     [Linux Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]

  Powered by Linux