diff options
| author | Eduardo Habkost <ehabkost@redhat.com> | 2011-11-10 10:41:41 -0200 |
|---|---|---|
| committer | Anthony Liguori <aliguori@us.ibm.com> | 2011-12-12 11:47:19 -0600 |
| commit | ce812673dc11de6c067e87a13f5ac0341df7772d (patch) | |
| tree | 1497b64ec388d2b696a0c69f60d00bee381e43ae /migration-exec.c | |
| parent | 90d8454ed0ffdec3871053b696e4a44f349e561c (diff) | |
| download | qemu-ce812673dc11de6c067e87a13f5ac0341df7772d.zip qemu-ce812673dc11de6c067e87a13f5ac0341df7772d.tar.gz qemu-ce812673dc11de6c067e87a13f5ac0341df7772d.tar.bz2 | |
exec_close(): accept any negative value as qemu_fclose() error
Note that we don't return the unchanged return value back yet, because
we need to change all qemu_fclose() callers to accept any positive value
as success.
Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
Diffstat (limited to 'migration-exec.c')
| -rw-r--r-- | migration-exec.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/migration-exec.c b/migration-exec.c index b7b1055..626b648 100644 --- a/migration-exec.c +++ b/migration-exec.c @@ -50,7 +50,7 @@ static int exec_close(MigrationState *s) ret = qemu_fclose(s->opaque); s->opaque = NULL; s->fd = -1; - if (ret != -1 && + if (ret >= 0 && WIFEXITED(ret) && WEXITSTATUS(ret) == 0) { ret = 0; |