Coverity has just pointed out (CID 1412229) that the function kvm_set_user_memory_region() in accel/kvm/kvm-all.c calls kvm_vm_ioctl(s, KVM_SET_USER_MEMORY_REGION, ...) twice, but it only checks the ioctl return value for failure in the second case. Shouldn't it be checking both calls?
(This code seems to have been in place for years so I guess Coverity has only just decided that kvm_vm_ioctl() passes its 'needs return value checking' heuristic...) thanks -- PMM
