On 11/06/2017 08:43 AM, Borislav Petkov wrote:
...

+
+       region->uaddr = range->addr;
+       region->size = range->size;
+       list_add_tail(&region->list, &sev->regions_list);
+       return ret;

Nothing's protecting that list from concurrent modifications of adding
and removal of regions.


Ah good catch. I will fix this in next rev. Similar to the mem_enc_op(), we need to acquire the kvm->lock when adding or removing the regions.

-Brijesh

Reply via email to