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.