Things are supposed to look like:
<machine canonical='pc-0.12'>pc</machine>
But are currently swapped. This can cause many VMs to revert to having
machine type='pc' which will affect save/restore across qemu upgrades.
for (i = 0 ; i < nmachines ; i++) {
if (machines[i]->alias) {
- if (!(caps->machineAliases[i] = strdup(machines[i]->name))) {
- virReportOOMError();
- goto cleanup;
- }
- if (!(caps->machineTypes[i] = strdup(machines[i]->alias))) {
- virReportOOMError();
- goto cleanup;
- }
- } else {
- if (!(caps->machineTypes[i] = strdup(machines[i]->name))) {
+ if (!(caps->machineAliases[i] = strdup(machines[i]->alias))) {
virReportOOMError();
goto cleanup;
}
}
+ if (!(caps->machineTypes[i] = strdup(machines[i]->name))) {
+ virReportOOMError();
+ goto cleanup;
+ }
if (machines[i]->isDefault)
defIdx = i;
}