coreutils
[Top][All Lists]
Advanced

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

Re: [coreutils] [PATCHv2] stat: print SELinux context when available


From: Pádraig Brady
Subject: Re: [coreutils] [PATCHv2] stat: print SELinux context when available
Date: Tue, 05 Oct 2010 11:02:27 +0100
User-agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.1.8) Gecko/20100227 Thunderbird/3.0.3

On 05/10/10 10:03, Jim Meyering wrote:
>> -          if (0 < is_selinux_enabled ())
>> +          if (is_selinux_enabled ())
> ...
>> -          if (0 < is_selinux_enabled ())
>> +          if (is_selinux_enabled ())
> 
> However, the changes to the use of is_selinux_enabled look wrong,
> since that function returns -1 upon failure.
> That's why we test 0 < is_selinux_enabled ().
> 
>>From the man page:
> 
>   DESCRIPTION
>          is_selinux_enabled returns 1 if SELinux is running or 0 if it  is
>          not.  On error, -1 is returned.

Eep. That man page has changed since Fedora 12.
Fixed, and pushed.

thanks,
Pádraig.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]