|
 |
9ae3a8 |
From c62e0877b191e5fba9b678bbd518a57c8fdf7099 Mon Sep 17 00:00:00 2001
|
|
|
9ae3a8 |
From: Fam Zheng <famz@redhat.com>
|
|
|
9ae3a8 |
Date: Thu, 10 Mar 2016 04:00:53 +0100
|
|
|
9ae3a8 |
Subject: [PATCH 4/5] nbd: client_close on error in nbd_co_client_start
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
RH-Author: Fam Zheng <famz@redhat.com>
|
|
|
9ae3a8 |
Message-id: <1457582453-13835-4-git-send-email-famz@redhat.com>
|
|
|
9ae3a8 |
Patchwork-id: 69759
|
|
|
9ae3a8 |
O-Subject: [RHEL-7.3 qemu-kvm PATCH v2 3/3] nbd: client_close on error in nbd_co_client_start
|
|
|
9ae3a8 |
Bugzilla: 1285453
|
|
|
9ae3a8 |
RH-Acked-by: Stefan Hajnoczi <stefanha@redhat.com>
|
|
|
9ae3a8 |
RH-Acked-by: Paolo Bonzini <pbonzini@redhat.com>
|
|
|
9ae3a8 |
RH-Acked-by: Laurent Vivier <lvivier@redhat.com>
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
From: Max Reitz <mreitz@redhat.com>
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
Use client_close() if an error in nbd_co_client_start() occurs instead
|
|
|
9ae3a8 |
of manually inlining parts of it. This fixes an assertion error on the
|
|
|
9ae3a8 |
server side if nbd_negotiate() fails.
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
Signed-off-by: Max Reitz <mreitz@redhat.com>
|
|
|
9ae3a8 |
Acked-by: Paolo Bonzini <pbonzini@redhat.com>
|
|
|
9ae3a8 |
Reviewed-by: Eric Blake <eblake@redhat.com>
|
|
|
9ae3a8 |
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
|
|
|
9ae3a8 |
(cherry picked from commit d3780c2dce2c452759ee9d94f9d824cf14cc3ab8)
|
|
|
9ae3a8 |
Signed-off-by: Fam Zheng <famz@redhat.com>
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
Downstream: client_close -> nbd_client_close.
|
|
|
9ae3a8 |
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
|
9ae3a8 |
---
|
|
|
9ae3a8 |
nbd.c | 3 +--
|
|
|
9ae3a8 |
1 file changed, 1 insertion(+), 2 deletions(-)
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
diff --git a/nbd.c b/nbd.c
|
|
|
9ae3a8 |
index 97aeecb..c20e57e 100644
|
|
|
9ae3a8 |
--- a/nbd.c
|
|
|
9ae3a8 |
+++ b/nbd.c
|
|
|
9ae3a8 |
@@ -1282,8 +1282,7 @@ static coroutine_fn void nbd_co_client_start(void *opaque)
|
|
|
9ae3a8 |
nbd_export_get(exp);
|
|
|
9ae3a8 |
}
|
|
|
9ae3a8 |
if (nbd_negotiate(data)) {
|
|
|
9ae3a8 |
- shutdown(client->sock, 2);
|
|
|
9ae3a8 |
- client->close(client);
|
|
|
9ae3a8 |
+ nbd_client_close(client);
|
|
|
9ae3a8 |
goto out;
|
|
|
9ae3a8 |
}
|
|
|
9ae3a8 |
qemu_co_mutex_init(&client->send_lock);
|
|
|
9ae3a8 |
--
|
|
|
9ae3a8 |
1.8.3.1
|
|
|
9ae3a8 |
|