Re: [PATCH] getlogin.3: Reformat thread-safety information

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

 



On 10/17/2014 02:05 PM, Michael Kerrisk (man-pages) wrote:
> Hello Haitao,
> 
> Just double checking one point. See below.
> 
> On 10/17/2014 05:59 AM, Peng Haitao wrote:
>> Signed-off-by: Peng Haitao <penght@xxxxxxxxxxxxxx>
>> ---
>>  man3/getlogin.3 | 24 ++++++++++++++----------
>>  1 file changed, 14 insertions(+), 10 deletions(-)
>>
>> diff --git a/man3/getlogin.3 b/man3/getlogin.3
>> index fcbd0fb..828df2b 100644
>> --- a/man3/getlogin.3
>> +++ b/man3/getlogin.3
>> @@ -138,19 +138,23 @@ password database file
>>  (traditionally \fI/etc/utmp\fP;
>>  some libc versions used \fI/var/adm/utmp\fP)
>>  .SH ATTRIBUTES
>> -.SS Multithreading (see pthreads(7))
>> -The
>> +For an explanation of the terms used in this section, see
>> +.BR attributes (7).
>> +.TS
>> +allbox;
>> +lb lb lb
>> +l l l.
>> +Interface	Attribute	Value
>> +T{
>>  .BR getlogin ()
>> -function is not thread-safe.
>> -.LP
>> -The
>> +T}	Thread safety	MT-Unsafe locale
> 
> Should this be "MT-Safe locale"?
> 

getlogin() is MT-Unsafe and is influenced by setlocale(),
So "MT-Unsafe locale" is right.


-- 
Best Regards,
Peng

> Cheers,
> 
> Michael
> 
> 
>> +T{
>>  .BR getlogin_r ()
>> -function is thread-safe.
>> -.LP
>> -The
>> +T}	Thread safety	MT-Safe locale
>> +T{
>>  .BR cuserid ()
>> -function is thread-safe with exceptions.
>> -It is not thread-safe if called with a NULL parameter.
>> +T}	Thread safety	MT-Unsafe race:cuserid/!string locale
>> +.TE
>>  .SH CONFORMING TO
>>  .BR getlogin ()
>>  and
>>
> 
> 

--
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