Re: [PATCH] KVM: SVM: no need to call access_ok() in LAUNCH_MEASURE command

2018-02-23 Thread Paolo Bonzini
On 23/02/2018 19:36, Brijesh Singh wrote: > Using the access_ok() to validate the input before issuing the SEV > command does not buy us anything in this case. If userland is > giving us a garbage pointer then copy_to_user() will catch it when we try > to return the measurement. > > Suggested-by:

[PATCH] KVM: SVM: no need to call access_ok() in LAUNCH_MEASURE command

2018-02-23 Thread Brijesh Singh
Using the access_ok() to validate the input before issuing the SEV command does not buy us anything in this case. If userland is giving us a garbage pointer then copy_to_user() will catch it when we try to return the measurement. Suggested-by: Al Viro Fixes: 0d0736f76347 (KVM: SVM: Add support fo