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.

[...]

Reply via email to