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

From: Yu Zhang
Date: Wed Sep 20 2017 - 04:50:54 EST




On 9/20/2017 4:13 PM, Paolo Bonzini wrote:
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. :)

Got it. Thanks. :)

Yu
Paolo