KVM: PPC: Book3S: Fix H_RTAS rets buffer overflow
authorNicholas Piggin <npiggin@gmail.com>
Tue, 20 Jul 2021 10:43:09 +0000 (20:43 +1000)
committerMichael Ellerman <mpe@ellerman.id.au>
Fri, 23 Jul 2021 06:14:31 +0000 (16:14 +1000)
commitf62f3c20647ebd5fb6ecb8f0b477b9281c44c10a
treea08a7c167eca712838258b1d48c08f0661dccb7c
parentbc4188a2f56e821ea057aca6bf444e138d06c252
KVM: PPC: Book3S: Fix H_RTAS rets buffer overflow

The kvmppc_rtas_hcall() sets the host rtas_args.rets pointer based on
the rtas_args.nargs that was provided by the guest. That guest nargs
value is not range checked, so the guest can cause the host rets pointer
to be pointed outside the args array. The individual rtas function
handlers check the nargs and nrets values to ensure they are correct,
but if they are not, the handlers store a -3 (0xfffffffd) failure
indication in rets[0] which corrupts host memory.

Fix this by testing up front whether the guest supplied nargs and nret
would exceed the array size, and fail the hcall directly without storing
a failure indication to rets[0].

Also expand on a comment about why we kill the guest and try not to
return errors directly if we have a valid rets[0] pointer.

Fixes: 8e591cb72047 ("KVM: PPC: Book3S: Add infrastructure to implement kernel-side RTAS calls")
Cc: stable@vger.kernel.org # v3.10+
Reported-by: Alexey Kardashevskiy <aik@ozlabs.ru>
Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
arch/powerpc/kvm/book3s_rtas.c