If none of the bank types match, the function would return an
uninitialized value.
Signed-off-by: Fuad Tabba <tabba@xxxxxxxxxx>
---
kvm.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/kvm.c b/kvm.c
index 42b8812..765dc71 100644
--- a/kvm.c
+++ b/kvm.c
@@ -381,14 +381,15 @@ u64 host_to_guest_flat(struct kvm *kvm, void *ptr)
* their type.
*
* If one call to @fun returns a non-zero value, stop iterating and return the
- * value. Otherwise, return zero.
+ * value. If none of the bank types match, return -ENODEV. Otherwise, return
+ * zero.
*/
int kvm__for_each_mem_bank(struct kvm *kvm, enum kvm_mem_type type,
int (*fun)(struct kvm *kvm, struct kvm_mem_bank *bank, void *data),
void *data)
{
- int ret;
struct kvm_mem_bank *bank;
+ int ret = -ENODEV;
list_for_each_entry(bank, &kvm->mem_banks, list) {
if (type != KVM_MEM_TYPE_ALL && !(bank->type & type))
--
2.39.0.rc0.267.gcb52ba06e7-goog
