diff options
| author | Osama Abdelkader <osama.abdelkader@gmail.com> | 2026-03-16 16:16:11 +0100 |
|---|---|---|
| committer | Anup Patel <anup@brainfault.org> | 2026-03-27 18:40:29 +0530 |
| commit | b7c958d7c1eb1cb9b2be7b5ee4129fcd66cec978 (patch) | |
| tree | 1d78befa49c56a80981429b9aed540135d88e9a3 /arch/riscv/kvm | |
| parent | aa35bcf2e76234fef7bbca9bf364039692a27661 (diff) | |
| download | lwn-b7c958d7c1eb1cb9b2be7b5ee4129fcd66cec978.tar.gz lwn-b7c958d7c1eb1cb9b2be7b5ee4129fcd66cec978.zip | |
riscv: kvm: fix vector context allocation leak
When the second kzalloc (host_context.vector.datap) fails in
kvm_riscv_vcpu_alloc_vector_context, the first allocation
(guest_context.vector.datap) is leaked. Free it before returning.
Fixes: 0f4b82579716 ("riscv: KVM: Add vector lazy save/restore support")
Cc: stable@vger.kernel.org
Signed-off-by: Osama Abdelkader <osama.abdelkader@gmail.com>
Reviewed-by: Andy Chiu <andybnac@gmail.com>
Link: https://lore.kernel.org/r/20260316151612.13305-1-osama.abdelkader@gmail.com
Signed-off-by: Anup Patel <anup@brainfault.org>
Diffstat (limited to 'arch/riscv/kvm')
| -rw-r--r-- | arch/riscv/kvm/vcpu_vector.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/arch/riscv/kvm/vcpu_vector.c b/arch/riscv/kvm/vcpu_vector.c index 05f3cc2d8e31..5b6ad82d47be 100644 --- a/arch/riscv/kvm/vcpu_vector.c +++ b/arch/riscv/kvm/vcpu_vector.c @@ -80,8 +80,11 @@ int kvm_riscv_vcpu_alloc_vector_context(struct kvm_vcpu *vcpu) return -ENOMEM; vcpu->arch.host_context.vector.datap = kzalloc(riscv_v_vsize, GFP_KERNEL); - if (!vcpu->arch.host_context.vector.datap) + if (!vcpu->arch.host_context.vector.datap) { + kfree(vcpu->arch.guest_context.vector.datap); + vcpu->arch.guest_context.vector.datap = NULL; return -ENOMEM; + } return 0; } |