Markus Armbruster <arm...@redhat.com> writes: > Eric Blake <ebl...@redhat.com> writes: [...] >> diff --git a/hw/virtio/virtio-balloon.c b/hw/virtio/virtio-balloon.c >> index c419b17..48867c4 100644 >> --- a/hw/virtio/virtio-balloon.c >> +++ b/hw/virtio/virtio-balloon.c >> @@ -132,14 +132,16 @@ static void balloon_stats_get_all(Object *obj, struct >> Visitor *v, > visit_start_struct(v, NULL, NULL, "stats", 0, &err); > if (err) { > goto out_end; > } > for (i = 0; !err && i < VIRTIO_BALLOON_S_NR; i++) { >> visit_type_int64(v, (int64_t *) &s->stats[i], balloon_stat_names[i], >> &err); > > Preexisting: if visit_type_int64() fails, we need to break the loop. > Separate fix.
Nonsense. [...]