By removing a non-migratable feature in a for loop we would fail to drop
every second non-migratable feature if the features array contained
several of them in a row.
Signed-off-by: Jiri Denemark <jdene...@redhat.com>
---
src/cpu/cpu_x86.c | 14 ++++++++++----
1 file changed, 10 insertions(+), 4 deletions(-)
diff --git a/src/cpu/cpu_x86.c b/src/cpu/cpu_x86.c
index 7bb2bb6..24ef76b 100644
--- a/src/cpu/cpu_x86.c
+++ b/src/cpu/cpu_x86.c
@@ -1816,8 +1816,11 @@ x86Decode(virCPUDefPtr cpu,
* Note: this only works as long as no CPU model contains non-migratable
* features directly */
if (flags & VIR_CONNECT_BASELINE_CPU_MIGRATABLE) {
- for (i = 0; i < cpuModel->nfeatures; i++) {
- if (!x86FeatureIsMigratable(cpuModel->features[i].name, map)) {
+ i = 0;
+ while (i < cpuModel->nfeatures) {
+ if (x86FeatureIsMigratable(cpuModel->features[i].name, map)) {
+ i++;
+ } else {
VIR_FREE(cpuModel->features[i].name);
VIR_DELETE_ELEMENT_INPLACE(cpuModel->features, i,
cpuModel->nfeatures);
@@ -2542,8 +2545,11 @@ x86UpdateHostModel(virCPUDefPtr guest,
/* Remove non-migratable features by default
* Note: this only works as long as no CPU model contains non-migratable
* features directly */
- for (i = 0; i < guest->nfeatures; i++) {
- if (!x86FeatureIsMigratable(guest->features[i].name, map)) {
+ i = 0;
+ while (i < guest->nfeatures) {
+ if (x86FeatureIsMigratable(guest->features[i].name, map)) {
+ i++;
+ } else {
VIR_FREE(guest->features[i].name);
VIR_DELETE_ELEMENT_INPLACE(guest->features, i, guest->nfeatures);
}
--
2.9.2
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
