x86/mmiotrace: Lock down the testmmiotrace module
authorDavid Howells <dhowells@redhat.com>
Wed, 8 Nov 2017 15:11:36 +0000 (15:11 +0000)
committerBastian Blank <waldi@debian.org>
Fri, 22 Jun 2018 09:50:22 +0000 (10:50 +0100)
The testmmiotrace module shouldn't be permitted when the kernel is locked
down as it can be used to arbitrarily read and write MMIO space.

Suggested-by: Thomas Gleixner <tglx@linutronix.de>
Signed-off-by: David Howells <dhowells@redhat.com
cc: Thomas Gleixner <tglx@linutronix.de>
cc: Steven Rostedt <rostedt@goodmis.org>
cc: Ingo Molnar <mingo@kernel.org>
cc: "H. Peter Anvin" <hpa@zytor.com>
cc: x86@kernel.org

Gbp-Pq: Topic features/all/lockdown
Gbp-Pq: Name 0023-x86-mmiotrace-Lock-down-the-testmmiotrace-module.patch

arch/x86/mm/testmmiotrace.c

index f6ae6830b341ba4fb98f8efdb81bc03c11408773..bbaad357f5d738a9ecef0c31963fcd4b4cc3e1f7 100644 (file)
@@ -115,6 +115,9 @@ static int __init init(void)
 {
        unsigned long size = (read_far) ? (8 << 20) : (16 << 10);
 
+       if (kernel_is_locked_down("MMIO trace testing"))
+               return -EPERM;
+
        if (mmio_address == 0) {
                pr_err("you have to use the module argument mmio_address.\n");
                pr_err("DO NOT LOAD THIS MODULE UNLESS YOU REALLY KNOW WHAT YOU ARE DOING!\n");