KVM: PPC: Book3S: Fix race and leak in kvm_vm_ioctl_create_spapr_tce()
authorPaul Mackerras <paulus@ozlabs.org>
Thu, 24 Aug 2017 09:14:47 +0000 (19:14 +1000)
committerPaolo Bonzini <pbonzini@redhat.com>
Fri, 25 Aug 2017 09:08:57 +0000 (11:08 +0200)
commit47c5310a8dbe7c2cb9f0083daa43ceed76c257fa
tree0ad1a020325eec0a20e943b78ad8a87459fb5395
parent38cfd5e3df9c4f88e76b547eee2087ee5c042ae2
KVM: PPC: Book3S: Fix race and leak in kvm_vm_ioctl_create_spapr_tce()

Nixiaoming pointed out that there is a memory leak in
kvm_vm_ioctl_create_spapr_tce() if the call to anon_inode_getfd()
fails; the memory allocated for the kvmppc_spapr_tce_table struct
is not freed, and nor are the pages allocated for the iommu
tables.  In addition, we have already incremented the process's
count of locked memory pages, and this doesn't get restored on
error.

David Hildenbrand pointed out that there is a race in that the
function checks early on that there is not already an entry in the
stt->iommu_tables list with the same LIOBN, but an entry with the
same LIOBN could get added between then and when the new entry is
added to the list.

This fixes all three problems.  To simplify things, we now call
anon_inode_getfd() before placing the new entry in the list.  The
check for an existing entry is done while holding the kvm->lock
mutex, immediately before adding the new entry to the list.
Finally, on failure we now call kvmppc_account_memlimit to
decrement the process's count of locked memory pages.

Reported-by: Nixiaoming <nixiaoming@huawei.com>
Reported-by: David Hildenbrand <david@redhat.com>
Signed-off-by: Paul Mackerras <paulus@ozlabs.org>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
arch/powerpc/kvm/book3s_64_vio.c