make _is_selinux_enabled() return bool explicitly

diff --git a/snapper/Selinux.cc b/snapper/Selinux.cc
index ed059b850bfb89dd6e54edcce7656cbc333d3ae1..36a8b23797285cc8766624a9879dad29fe98ca4a 100644 (file)
--- a/snapper/Selinux.cc
+++ b/snapper/Selinux.cc
@@ -125,15 +125,15 @@ namespace snapper
     }
 
 
-    int
+    bool
     _is_selinux_enabled()
     {
-       static int selinux_checked = 0, selinux_enabled = 0;
+       static bool selinux_enabled, selinux_checked = false;
 
        if (!selinux_checked)
        {
-           selinux_enabled = is_selinux_enabled();
-           selinux_checked = 1;
+           selinux_enabled = (is_selinux_enabled() == 1); // may return -1 on error
+           selinux_checked = true;
            y2mil("Selinux support " << (selinux_enabled ? "en" : "dis") << "abled");
        }
 

diff --git a/snapper/Selinux.h b/snapper/Selinux.h
index 8698cd58fbba52b6ca66b5a6c524f211277a0f8e..e4e3549ff644fef10436ca234047c8913697df32 100644 (file)
--- a/snapper/Selinux.h
+++ b/snapper/Selinux.h
@@ -45,7 +45,7 @@ namespace snapper {
 
     const static string selinux_snapperd_data = "snapperd_data";
 
-    int _is_selinux_enabled();
+    bool _is_selinux_enabled();
 
     class SnapperContexts
     {