[GIT PULL] KVM/arm64 fixes for 6.9, part #2

[GIT PULL] KVM/arm64 fixes for 6.9, part #2

[Oliver Upton]

Hi Paolo,

Single fix this time around for a rather straightforward NULL
dereference in one of the vgic ioctls, along with a reproducer I've
added as a testcase in selftests.

Please pull.

-- 
Thanks,
Oliver

The following changes since commit fec50db7033ea478773b159e0e2efb135270e3b7:

  Linux 6.9-rc3 (2024-04-07 13:22:46 -0700)

are available in the Git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/kvmarm/kvmarm.git tags/kvmarm-fixes-6.9-2

for you to fetch changes up to 160933e330f4c5a13931d725a4d952a4b9aefa71:

  KVM: selftests: Add test for uaccesses to non-existent vgic-v2 CPUIF (2024-04-24 19:09:36 +0000)

----------------------------------------------------------------
KVM/arm64 fixes for 6.9, part #2

- Fix + test for a NULL dereference resulting from unsanitised user
  input in the vgic-v2 device attribute accessors

----------------------------------------------------------------
Oliver Upton (2):
      KVM: arm64: vgic-v2: Check for non-NULL vCPU in vgic_v2_parse_attr()
      KVM: selftests: Add test for uaccesses to non-existent vgic-v2 CPUIF

 arch/arm64/kvm/vgic/vgic-kvm-device.c           |  8 ++--
 tools/testing/selftests/kvm/aarch64/vgic_init.c | 49 +++++++++++++++++++++++++
 2 files changed, 53 insertions(+), 4 deletions(-)

Re: [GIT PULL] KVM/arm64 fixes for 6.9, part #2

[Oliver Upton]

On Wed, Apr 24, 2024 at 12:39:46PM -0700, Oliver Upton wrote:
> Hi Paolo,
> 
> Single fix this time around for a rather straightforward NULL
> dereference in one of the vgic ioctls, along with a reproducer I've
> added as a testcase in selftests.
> 
> Please pull.

Nudging this, Paolo do you plan to pick this up or shall I make other
arrangements for getting this in?

> -- 
> Thanks,
> Oliver
> 
> The following changes since commit fec50db7033ea478773b159e0e2efb135270e3b7:
> 
>   Linux 6.9-rc3 (2024-04-07 13:22:46 -0700)
> 
> are available in the Git repository at:
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/kvmarm/kvmarm.git tags/kvmarm-fixes-6.9-2
> 
> for you to fetch changes up to 160933e330f4c5a13931d725a4d952a4b9aefa71:
> 
>   KVM: selftests: Add test for uaccesses to non-existent vgic-v2 CPUIF (2024-04-24 19:09:36 +0000)
> 
> ----------------------------------------------------------------
> KVM/arm64 fixes for 6.9, part #2
> 
> - Fix + test for a NULL dereference resulting from unsanitised user
>   input in the vgic-v2 device attribute accessors
> 
> ----------------------------------------------------------------
> Oliver Upton (2):
>       KVM: arm64: vgic-v2: Check for non-NULL vCPU in vgic_v2_parse_attr()
>       KVM: selftests: Add test for uaccesses to non-existent vgic-v2 CPUIF
> 
>  arch/arm64/kvm/vgic/vgic-kvm-device.c           |  8 ++--
>  tools/testing/selftests/kvm/aarch64/vgic_init.c | 49 +++++++++++++++++++++++++
>  2 files changed, 53 insertions(+), 4 deletions(-)
> 

-- 
Thanks,
Oliver

Re: [GIT PULL] KVM/arm64 fixes for 6.9, part #2

[Paolo Bonzini]

Hi Oliver,

I was on vacation, and unfortunately I saw it pretty much the day I left home. I will get it to Linus this afternoon.

Paolo

Il 30 aprile 2024 10:16:21 CEST, Oliver Upton <oliver.upton@linux.dev> ha scritto:
>On Wed, Apr 24, 2024 at 12:39:46PM -0700, Oliver Upton wrote:
>> Hi Paolo,
>> 
>> Single fix this time around for a rather straightforward NULL
>> dereference in one of the vgic ioctls, along with a reproducer I've
>> added as a testcase in selftests.
>> 
>> Please pull.
>
>Nudging this, Paolo do you plan to pick this up or shall I make other
>arrangements for getting this in?
>
>> -- 
>> Thanks,
>> Oliver
>> 
>> The following changes since commit fec50db7033ea478773b159e0e2efb135270e3b7:
>> 
>>   Linux 6.9-rc3 (2024-04-07 13:22:46 -0700)
>> 
>> are available in the Git repository at:
>> 
>>   git://git.kernel.org/pub/scm/linux/kernel/git/kvmarm/kvmarm.git tags/kvmarm-fixes-6.9-2
>> 
>> for you to fetch changes up to 160933e330f4c5a13931d725a4d952a4b9aefa71:
>> 
>>   KVM: selftests: Add test for uaccesses to non-existent vgic-v2 CPUIF (2024-04-24 19:09:36 +0000)
>> 
>> ----------------------------------------------------------------
>> KVM/arm64 fixes for 6.9, part #2
>> 
>> - Fix + test for a NULL dereference resulting from unsanitised user
>>   input in the vgic-v2 device attribute accessors
>> 
>> ----------------------------------------------------------------
>> Oliver Upton (2):
>>       KVM: arm64: vgic-v2: Check for non-NULL vCPU in vgic_v2_parse_attr()
>>       KVM: selftests: Add test for uaccesses to non-existent vgic-v2 CPUIF
>> 
>>  arch/arm64/kvm/vgic/vgic-kvm-device.c           |  8 ++--
>>  tools/testing/selftests/kvm/aarch64/vgic_init.c | 49 +++++++++++++++++++++++++
>>  2 files changed, 53 insertions(+), 4 deletions(-)
>> 
>

Paolo

Re: [GIT PULL] KVM/arm64 fixes for 6.9, part #2

[Oliver Upton]

On Tue, Apr 30, 2024 at 10:28:32AM +0200, Paolo Bonzini wrote:
> Hi Oliver,
> 
> I was on vacation, and unfortunately I saw it pretty much the day I left home. I will get it to Linus this afternoon.

No worries, thanks. Hope you had a good vacation!

> Il 30 aprile 2024 10:16:21 CEST, Oliver Upton <oliver.upton@linux.dev> ha scritto:
> >On Wed, Apr 24, 2024 at 12:39:46PM -0700, Oliver Upton wrote:
> >> Hi Paolo,
> >> 
> >> Single fix this time around for a rather straightforward NULL
> >> dereference in one of the vgic ioctls, along with a reproducer I've
> >> added as a testcase in selftests.
> >> 
> >> Please pull.
> >
> >Nudging this, Paolo do you plan to pick this up or shall I make other
> >arrangements for getting this in?
> >
> >> -- 
> >> Thanks,
> >> Oliver
> >> 
> >> The following changes since commit fec50db7033ea478773b159e0e2efb135270e3b7:
> >> 
> >>   Linux 6.9-rc3 (2024-04-07 13:22:46 -0700)
> >> 
> >> are available in the Git repository at:
> >> 
> >>   git://git.kernel.org/pub/scm/linux/kernel/git/kvmarm/kvmarm.git tags/kvmarm-fixes-6.9-2
> >> 
> >> for you to fetch changes up to 160933e330f4c5a13931d725a4d952a4b9aefa71:
> >> 
> >>   KVM: selftests: Add test for uaccesses to non-existent vgic-v2 CPUIF (2024-04-24 19:09:36 +0000)
> >> 
> >> ----------------------------------------------------------------
> >> KVM/arm64 fixes for 6.9, part #2
> >> 
> >> - Fix + test for a NULL dereference resulting from unsanitised user
> >>   input in the vgic-v2 device attribute accessors
> >> 
> >> ----------------------------------------------------------------
> >> Oliver Upton (2):
> >>       KVM: arm64: vgic-v2: Check for non-NULL vCPU in vgic_v2_parse_attr()
> >>       KVM: selftests: Add test for uaccesses to non-existent vgic-v2 CPUIF
> >> 
> >>  arch/arm64/kvm/vgic/vgic-kvm-device.c           |  8 ++--
> >>  tools/testing/selftests/kvm/aarch64/vgic_init.c | 49 +++++++++++++++++++++++++
> >>  2 files changed, 53 insertions(+), 4 deletions(-)
> >> 
> >
> 
> Paolo
> 

-- 
Thanks,
Oliver