[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH V2] x86/mm: Fix Coverity issues 1373105 and 1373106



>>> On 21.09.16 at 14:55, <george.dunlap@xxxxxxxxxx> wrote:
> On 21/09/16 13:52, Jan Beulich wrote:
>>>>> On 21.09.16 at 14:41, <rcojocaru@xxxxxxxxxxxxxxx> wrote:
>>> Added missing error checks in p2m_set_mem_access_multi().
>> 
>> I think the patch subject should say what is being changed/fixed,
>> and the two Coverity IDs should be listed here instead. Otherwise
>> someone looking over just the patch titles will have no idea what
>> this is actually about. If I end up committing this, I'll take the liberty
>> to do adjustments.
> 
> I said I'd commit it, so how about:
> 
> x86/mm: Add missing copy_from_user error checks in p2m_set_access_multi

Sound good.

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.