powerpc/powernv: Avoid crashing if rng is NULL
authorMichael Ellerman <mpe@ellerman.id.au>
Wed, 27 Jul 2022 14:32:17 +0000 (00:32 +1000)
committerMichael Ellerman <mpe@ellerman.id.au>
Thu, 28 Jul 2022 06:22:15 +0000 (16:22 +1000)
On a bare-metal Power8 system that doesn't have an "ibm,power-rng", a
malicious QEMU and guest that ignore the absence of the
KVM_CAP_PPC_HWRNG flag, and calls H_RANDOM anyway, will dereference a
NULL pointer.

In practice all Power8 machines have an "ibm,power-rng", but let's not
rely on that, add a NULL check and early return in
powernv_get_random_real_mode().

Fixes: e928e9cb3601 ("KVM: PPC: Book3S HV: Add fast real-mode H_RANDOM implementation.")
Cc: stable@vger.kernel.org # v4.1+
Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Link: https://lore.kernel.org/r/20220727143219.2684192-1-mpe@ellerman.id.au
arch/powerpc/platforms/powernv/rng.c

index 3805ad1..2287c9c 100644 (file)
@@ -63,6 +63,8 @@ int powernv_get_random_real_mode(unsigned long *v)
        struct powernv_rng *rng;
 
        rng = raw_cpu_read(powernv_rng);
+       if (!rng)
+               return 0;
 
        *v = rng_whiten(rng, __raw_rm_readq(rng->regs_real));