diff options
| author | Paolo Bonzini <pbonzini@redhat.com> | 2012-09-27 13:30:15 +0200 |
|---|---|---|
| committer | Paolo Bonzini <pbonzini@redhat.com> | 2012-11-02 18:35:07 +0100 |
| commit | 6c3601361ff22cb8bda3f483ea11c4f7bd095094 (patch) | |
| tree | 4294f16cf1572fa5118ca15f4dcfb618e39f4279 /migration-tcp.c | |
| parent | 09bac73c13b57acd304efb54a361c244d60d375c (diff) | |
| download | qemu-6c3601361ff22cb8bda3f483ea11c4f7bd095094.tar.gz qemu-6c3601361ff22cb8bda3f483ea11c4f7bd095094.tar.bz2 qemu-6c3601361ff22cb8bda3f483ea11c4f7bd095094.zip | |
migration: xxx_close will only be called once
No need to test s->fd again, it is tested in the caller.
Reviewed-by: Orit Wasserman <owasserm@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'migration-tcp.c')
| -rw-r--r-- | migration-tcp.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/migration-tcp.c b/migration-tcp.c index 1a12f17886..bb27ce832a 100644 --- a/migration-tcp.c +++ b/migration-tcp.c @@ -44,11 +44,8 @@ static int tcp_close(MigrationState *s) { int r = 0; DPRINTF("tcp_close\n"); - if (s->fd != -1) { - if (closesocket(s->fd) < 0) { - r = -errno; - } - s->fd = -1; + if (closesocket(s->fd) < 0) { + r = -socket_error(); } return r; } |