diff mbox

[33/41] savevm: make qemu_fill_buffer() be consistent

Message ID 1348217255-22441-34-git-send-email-quintela@redhat.com
State New
Headers show

Commit Message

Juan Quintela Sept. 21, 2012, 8:47 a.m. UTC 
It was setting last_error directly once, and with the helper the other time.

Signed-off-by: Juan Quintela <quintela@redhat.com>
---
 savevm.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Paolo Bonzini Sept. 21, 2012, 12:48 p.m. UTC | #1 
Il 21/09/2012 10:47, Juan Quintela ha scritto:
> It was setting last_error directly once, and with the helper the other time.
> 
> Signed-off-by: Juan Quintela <quintela@redhat.com>
> ---
>  savevm.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/savevm.c b/savevm.c
> index 8ddb9d5..4e4aa3c 100644
> --- a/savevm.c
> +++ b/savevm.c
> @@ -501,7 +501,7 @@ static void qemu_fill_buffer(QEMUFile *f)
>          f->buf_size += len;
>          f->buf_offset += len;
>      } else if (len == 0) {
> -        f->last_error = -EIO;
> +        qemu_file_set_error(f, -EIO);
>      } else if (len != -EAGAIN)
>          qemu_file_set_error(f, len);
>  }
> 

Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
diff mbox

Patch

diff --git a/savevm.c b/savevm.c
index 8ddb9d5..4e4aa3c 100644
--- a/savevm.c
+++ b/savevm.c
@@ -501,7 +501,7 @@  static void qemu_fill_buffer(QEMUFile *f)
         f->buf_size += len;
         f->buf_offset += len;
     } else if (len == 0) {
-        f->last_error = -EIO;
+        qemu_file_set_error(f, -EIO);
     } else if (len != -EAGAIN)
         qemu_file_set_error(f, len);
 }