qemu: only return two values in virQEMUDriverConfigLoadMemoryEntry

The intention of these split Load*Entry functions is to prevent
virQEMUDriverConfigLoadFile from getting too large.

There's no need to signal to the caller whether an entry was found
or not, only whether there was an error.

Remove the non-standard return 1.

Signed-off-by: Ján Tomko <jtomko@redhat.com>
Reviewed-by: Daniel Henrique Barboza <danielhb413@gmail.com>
This commit is contained in:
Ján Tomko 2020-05-18 15:23:46 +02:00
parent 8da8cafdcb
commit d265171b57

View File

@ -977,7 +977,7 @@ virQEMUDriverConfigLoadMemoryEntry(virQEMUDriverConfigPtr cfg,
} else if (rc > 0) { } else if (rc > 0) {
VIR_FREE(cfg->memoryBackingDir); VIR_FREE(cfg->memoryBackingDir);
cfg->memoryBackingDir = g_strdup_printf("%s/libvirt/qemu", dir); cfg->memoryBackingDir = g_strdup_printf("%s/libvirt/qemu", dir);
return 1; return 0;
} }
return 0; return 0;