Fix initialization of static.

Return error instead of failing silently when page faults are disabled.
draios · Jan 12, 2018 · 7e09185 · 7e09185
1 parent 8766a50
commit 7e09185
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/driver/main.c b/driver/main.c
@@ -201,7 +201,7 @@ static struct tracepoint *tp_signal_deliver;
 static struct tracepoint *tp_page_fault_user;
 static struct tracepoint *tp_page_fault_kernel;
 static bool g_fault_tracepoint_registered;
-static bool g_fault_tracepoint_disabled = false;
+static bool g_fault_tracepoint_disabled;
 #endif
 
 #ifdef _DEBUG
@@ -1031,8 +1031,8 @@ static long ppm_ioctl(struct file *filp, unsigned int cmd, unsigned long arg)
 		ASSERT(g_tracepoint_registered);
 
 		if (g_fault_tracepoint_disabled) {
-			ret = 0;
-			pr_info("kernel page fault tracepoints are disabled\n");
+			pr_err("kernel page fault tracepoints are disabled\n");
+			ret = -EPERM;
 			goto cleanup_ioctl;
 		}