diff options
| author | Max Reitz | 2016-01-25 19:41:08 +0100 |
|---|---|---|
| committer | Max Reitz | 2016-02-02 17:49:42 +0100 |
| commit | d3780c2dce2c452759ee9d94f9d824cf14cc3ab8 (patch) | |
| tree | fe03e55c49a40a5309ecdc8f0a8da73a7aec81e3 | |
| parent | iotests: Limit supported formats for 118 (diff) | |
| download | qemu-d3780c2dce2c452759ee9d94f9d824cf14cc3ab8.tar.gz qemu-d3780c2dce2c452759ee9d94f9d824cf14cc3ab8.tar.xz qemu-d3780c2dce2c452759ee9d94f9d824cf14cc3ab8.zip | |
nbd: client_close on error in nbd_co_client_start
Use client_close() if an error in nbd_co_client_start() occurs instead
of manually inlining parts of it. This fixes an assertion error on the
server side if nbd_negotiate() fails.
Signed-off-by: Max Reitz <mreitz@redhat.com>
Acked-by: Paolo Bonzini <pbonzini@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
| -rw-r--r-- | nbd/server.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/nbd/server.c b/nbd/server.c index 256feafcec..7fa7f53768 100644 --- a/nbd/server.c +++ b/nbd/server.c @@ -1082,8 +1082,7 @@ static coroutine_fn void nbd_co_client_start(void *opaque) nbd_export_get(exp); } if (nbd_negotiate(data)) { - shutdown(client->sock, 2); - client->close(client); + client_close(client); goto out; } qemu_co_mutex_init(&client->send_lock); |