aboutsummaryrefslogtreecommitdiff
path: root/dump.c
diff options
context:
space:
mode:
authorDaniel P. Berrange <berrange@redhat.com>2016-02-10 18:41:09 +0000
committerPaolo Bonzini <pbonzini@redhat.com>2016-02-16 17:16:11 +0100
commit9344e5f554690d5e379b5426daebadef7c87baf5 (patch)
tree8de641d57f171aa6dcaddb056d3feb43e88f848f /dump.c
parent3d4b2f9c9449ee1acdb1a488177204ebfdaccd0d (diff)
downloadqemu-9344e5f554690d5e379b5426daebadef7c87baf5.zip
qemu-9344e5f554690d5e379b5426daebadef7c87baf5.tar.gz
qemu-9344e5f554690d5e379b5426daebadef7c87baf5.tar.bz2
nbd: always query export list in fixed new style protocol
With the new style protocol, the NBD client will currenetly send NBD_OPT_EXPORT_NAME as the first (and indeed only) option it wants. The problem is that the NBD protocol spec does not allow for returning an error message with the NBD_OPT_EXPORT_NAME option. So if the server mandates use of TLS, the client will simply see an immediate connection close after issuing NBD_OPT_EXPORT_NAME which is not user friendly. To improve this situation, if we have the fixed new style protocol, we can sent NBD_OPT_LIST as the first option to query the list of server exports. We can check for our named export in this list and raise an error if it is not found, instead of going ahead and sending NBD_OPT_EXPORT_NAME with a name that we know will be rejected. This improves the error reporting both in the case that the server required TLS, and in the case that the client requested export name does not exist on the server. If the server does not support NBD_OPT_LIST, we just ignore that and carry on with NBD_OPT_EXPORT_NAME as before. Signed-off-by: Daniel P. Berrange <berrange@redhat.com> Message-Id: <1455129674-17255-12-git-send-email-berrange@redhat.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'dump.c')
0 files changed, 0 insertions, 0 deletions