Re: [PATCH] KVM: x86: Fix the NULL pointer parameter in check_cr_write()

From: Paolo Bonzini
Date: Wed Sep 20 2017 - 04:15:58 EST


On 20/09/2017 08:35, Yu Zhang wrote:
>
> 2 reasons I did not choose to change kvm_cpuid(): 1> like Jim's
> comments, kvm_cpuid() will eventually write the *eax - *edx no
> matter a cpuid entry is found or not; 2> currently, return value of
> kvm_cpuid() is either true when an entry is found or false otherwise.
> We can change kvm_cpuid() to check the pointers of GPRs against NULL
> and return false immediately. Then the false value would have 2
> different meanings - entry not found, or invalid params.
>
> Paolo, any suggestion? :-)

Radim, has already sent this version to Linus. :)

Paolo