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

[Pádraig Brady]

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.