2008-07-03 13:41:03 +00:00
|
|
|
/*
|
2016-10-14 18:33:04 +00:00
|
|
|
* Copyright (C) 2016 Red Hat, Inc.
|
2008-05-27 21:13:40 +00:00
|
|
|
* Copyright (C) 2005 Anthony Liguori <anthony@codemonkey.ws>
|
|
|
|
*
|
2016-01-14 08:41:02 +00:00
|
|
|
* Network Block Device Server Side
|
2008-05-27 21:13:40 +00:00
|
|
|
*
|
|
|
|
* This program is free software; you can redistribute it and/or modify
|
|
|
|
* it under the terms of the GNU General Public License as published by
|
|
|
|
* the Free Software Foundation; under version 2 of the License.
|
|
|
|
*
|
|
|
|
* This program is distributed in the hope that it will be useful,
|
|
|
|
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
|
|
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
|
|
* GNU General Public License for more details.
|
|
|
|
*
|
|
|
|
* You should have received a copy of the GNU General Public License
|
2009-07-16 20:47:01 +00:00
|
|
|
* along with this program; if not, see <http://www.gnu.org/licenses/>.
|
2008-07-03 13:41:03 +00:00
|
|
|
*/
|
2008-05-27 21:13:40 +00:00
|
|
|
|
2016-01-29 17:50:05 +00:00
|
|
|
#include "qemu/osdep.h"
|
include/qemu/osdep.h: Don't include qapi/error.h
Commit 57cb38b included qapi/error.h into qemu/osdep.h to get the
Error typedef. Since then, we've moved to include qemu/osdep.h
everywhere. Its file comment explains: "To avoid getting into
possible circular include dependencies, this file should not include
any other QEMU headers, with the exceptions of config-host.h,
compiler.h, os-posix.h and os-win32.h, all of which are doing a
similar job to this file and are under similar constraints."
qapi/error.h doesn't do a similar job, and it doesn't adhere to
similar constraints: it includes qapi-types.h. That's in excess of
100KiB of crap most .c files don't actually need.
Add the typedef to qemu/typedefs.h, and include that instead of
qapi/error.h. Include qapi/error.h in .c files that need it and don't
get it now. Include qapi-types.h in qom/object.h for uint16List.
Update scripts/clean-includes accordingly. Update it further to match
reality: replace config.h by config-target.h, add sysemu/os-posix.h,
sysemu/os-win32.h. Update the list of includes in the qemu/osdep.h
comment quoted above similarly.
This reduces the number of objects depending on qapi/error.h from "all
of them" to less than a third. Unfortunately, the number depending on
qapi-types.h shrinks only a little. More work is needed for that one.
Signed-off-by: Markus Armbruster <armbru@redhat.com>
[Fix compilation without the spice devel packages. - Paolo]
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
2016-03-14 08:01:28 +00:00
|
|
|
#include "qapi/error.h"
|
2016-01-14 08:41:02 +00:00
|
|
|
#include "nbd-internal.h"
|
qemu-nbd: only send a limited number of errno codes on the wire
Right now, NBD includes potentially platform-specific error values in
the wire protocol.
Luckily, most common error values are more or less universal: in
particular, of all errno values <= 34 (up to ERANGE), they are all the
same on supported platforms except for 11 (which is EAGAIN on Windows and
Linux, but EDEADLK on Darwin and the *BSDs). So, in order to guarantee
some portability, only keep a handful of possible error codes and squash
everything else to EINVAL.
This patch defines a limited set of errno values that are valid for the
NBD protocol, and specifies recommendations for what error to return
in specific corner cases. The set of errno values is roughly based on
the errors listed in the read(2) and write(2) man pages, with some
exceptions:
- ENOMEM is added for servers that implement copy-on-write or other
formats that require dynamic allocation.
- EDQUOT is not part of the universal set of errors; it can be changed
to ENOSPC on the wire format.
- EFBIG is part of the universal set of errors, but it is also changed
to ENOSPC because it is pretty similar to ENOSPC or EDQUOT.
Incoming values will in general match system errno values, but not
on the Hurd which has different errno values (they have a "subsystem
code" equal to 0x10 in bits 24-31). The Hurd is probably not something
to which QEMU has been ported, but still do the right thing and
reverse-map the NBD errno values to the system errno values.
The corresponding patch to the NBD protocol description can be found at
http://article.gmane.org/gmane.linux.drivers.nbd.general/3154.
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
2015-05-07 15:25:10 +00:00
|
|
|
|
|
|
|
static int system_errno_to_nbd_errno(int err)
|
|
|
|
{
|
|
|
|
switch (err) {
|
|
|
|
case 0:
|
|
|
|
return NBD_SUCCESS;
|
|
|
|
case EPERM:
|
2016-04-06 03:35:02 +00:00
|
|
|
case EROFS:
|
qemu-nbd: only send a limited number of errno codes on the wire
Right now, NBD includes potentially platform-specific error values in
the wire protocol.
Luckily, most common error values are more or less universal: in
particular, of all errno values <= 34 (up to ERANGE), they are all the
same on supported platforms except for 11 (which is EAGAIN on Windows and
Linux, but EDEADLK on Darwin and the *BSDs). So, in order to guarantee
some portability, only keep a handful of possible error codes and squash
everything else to EINVAL.
This patch defines a limited set of errno values that are valid for the
NBD protocol, and specifies recommendations for what error to return
in specific corner cases. The set of errno values is roughly based on
the errors listed in the read(2) and write(2) man pages, with some
exceptions:
- ENOMEM is added for servers that implement copy-on-write or other
formats that require dynamic allocation.
- EDQUOT is not part of the universal set of errors; it can be changed
to ENOSPC on the wire format.
- EFBIG is part of the universal set of errors, but it is also changed
to ENOSPC because it is pretty similar to ENOSPC or EDQUOT.
Incoming values will in general match system errno values, but not
on the Hurd which has different errno values (they have a "subsystem
code" equal to 0x10 in bits 24-31). The Hurd is probably not something
to which QEMU has been ported, but still do the right thing and
reverse-map the NBD errno values to the system errno values.
The corresponding patch to the NBD protocol description can be found at
http://article.gmane.org/gmane.linux.drivers.nbd.general/3154.
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
2015-05-07 15:25:10 +00:00
|
|
|
return NBD_EPERM;
|
|
|
|
case EIO:
|
|
|
|
return NBD_EIO;
|
|
|
|
case ENOMEM:
|
|
|
|
return NBD_ENOMEM;
|
|
|
|
#ifdef EDQUOT
|
|
|
|
case EDQUOT:
|
|
|
|
#endif
|
|
|
|
case EFBIG:
|
|
|
|
case ENOSPC:
|
|
|
|
return NBD_ENOSPC;
|
2016-10-14 18:33:16 +00:00
|
|
|
case ESHUTDOWN:
|
|
|
|
return NBD_ESHUTDOWN;
|
qemu-nbd: only send a limited number of errno codes on the wire
Right now, NBD includes potentially platform-specific error values in
the wire protocol.
Luckily, most common error values are more or less universal: in
particular, of all errno values <= 34 (up to ERANGE), they are all the
same on supported platforms except for 11 (which is EAGAIN on Windows and
Linux, but EDEADLK on Darwin and the *BSDs). So, in order to guarantee
some portability, only keep a handful of possible error codes and squash
everything else to EINVAL.
This patch defines a limited set of errno values that are valid for the
NBD protocol, and specifies recommendations for what error to return
in specific corner cases. The set of errno values is roughly based on
the errors listed in the read(2) and write(2) man pages, with some
exceptions:
- ENOMEM is added for servers that implement copy-on-write or other
formats that require dynamic allocation.
- EDQUOT is not part of the universal set of errors; it can be changed
to ENOSPC on the wire format.
- EFBIG is part of the universal set of errors, but it is also changed
to ENOSPC because it is pretty similar to ENOSPC or EDQUOT.
Incoming values will in general match system errno values, but not
on the Hurd which has different errno values (they have a "subsystem
code" equal to 0x10 in bits 24-31). The Hurd is probably not something
to which QEMU has been ported, but still do the right thing and
reverse-map the NBD errno values to the system errno values.
The corresponding patch to the NBD protocol description can be found at
http://article.gmane.org/gmane.linux.drivers.nbd.general/3154.
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
2015-05-07 15:25:10 +00:00
|
|
|
case EINVAL:
|
|
|
|
default:
|
|
|
|
return NBD_EINVAL;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2012-08-22 13:30:31 +00:00
|
|
|
/* Definitions for opaque data types */
|
|
|
|
|
2016-10-14 18:33:05 +00:00
|
|
|
typedef struct NBDRequestData NBDRequestData;
|
2012-08-22 13:30:31 +00:00
|
|
|
|
2016-10-14 18:33:05 +00:00
|
|
|
struct NBDRequestData {
|
|
|
|
QSIMPLEQ_ENTRY(NBDRequestData) entry;
|
2012-08-22 13:30:31 +00:00
|
|
|
NBDClient *client;
|
|
|
|
uint8_t *data;
|
2016-05-11 22:39:37 +00:00
|
|
|
bool complete;
|
2012-08-22 13:30:31 +00:00
|
|
|
};
|
|
|
|
|
|
|
|
struct NBDExport {
|
2012-09-18 11:26:25 +00:00
|
|
|
int refcount;
|
2012-09-18 11:59:03 +00:00
|
|
|
void (*close)(NBDExport *exp);
|
|
|
|
|
2014-11-18 11:21:18 +00:00
|
|
|
BlockBackend *blk;
|
2012-08-22 13:59:23 +00:00
|
|
|
char *name;
|
2016-10-14 18:33:03 +00:00
|
|
|
char *description;
|
2012-08-22 13:30:31 +00:00
|
|
|
off_t dev_offset;
|
|
|
|
off_t size;
|
2016-07-21 19:34:46 +00:00
|
|
|
uint16_t nbdflags;
|
2012-09-18 11:58:25 +00:00
|
|
|
QTAILQ_HEAD(, NBDClient) clients;
|
2012-08-22 13:59:23 +00:00
|
|
|
QTAILQ_ENTRY(NBDExport) next;
|
2014-06-20 19:57:32 +00:00
|
|
|
|
|
|
|
AioContext *ctx;
|
2016-01-29 15:36:06 +00:00
|
|
|
|
2016-07-06 09:22:39 +00:00
|
|
|
BlockBackend *eject_notifier_blk;
|
2016-01-29 15:36:06 +00:00
|
|
|
Notifier eject_notifier;
|
2012-08-22 13:30:31 +00:00
|
|
|
};
|
|
|
|
|
2012-08-22 13:59:23 +00:00
|
|
|
static QTAILQ_HEAD(, NBDExport) exports = QTAILQ_HEAD_INITIALIZER(exports);
|
|
|
|
|
2012-08-22 13:30:31 +00:00
|
|
|
struct NBDClient {
|
|
|
|
int refcount;
|
nbd: Fix regression on resiliency to port scan
Back in qemu 2.5, qemu-nbd was immune to port probes (a transient
server would not quit, regardless of how many probe connections
came and went, until a connection actually negotiated). But we
broke that in commit ee7d7aa when removing the return value to
nbd_client_new(), although that patch also introduced a bug causing
an assertion failure on a client that fails negotiation. We then
made it worse during refactoring in commit 1a6245a (a segfault
before we could even assert); the (masked) assertion was cleaned
up in d3780c2 (still in 2.6), and just recently we finally fixed
the segfault ("nbd: Fully intialize client in case of failed
negotiation"). But that still means that ever since we added
TLS support to qemu-nbd, we have been vulnerable to an ill-timed
port-scan being able to cause a denial of service by taking down
qemu-nbd before a real client has a chance to connect.
Since negotiation is now handled asynchronously via coroutines,
we no longer have a synchronous point of return by re-adding a
return value to nbd_client_new(). So this patch instead wires
things up to pass the negotiation status through the close_fn
callback function.
Simple test across two terminals:
$ qemu-nbd -f raw -p 30001 file
$ nmap 127.0.0.1 -p 30001 && \
qemu-io -c 'r 0 512' -f raw nbd://localhost:30001
Note that this patch does not change what constitutes successful
negotiation (thus, a client must enter transmission phase before
that client can be considered as a reason to terminate the server
when the connection ends). Perhaps we may want to tweak things
in a later patch to also treat a client that uses NBD_OPT_ABORT
as being a 'successful' negotiation (the client correctly talked
the NBD protocol, and informed us it was not going to use our
export after all), but that's a discussion for another day.
Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1451614
Signed-off-by: Eric Blake <eblake@redhat.com>
Message-Id: <20170608222617.20376-1-eblake@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
2017-06-08 22:26:17 +00:00
|
|
|
void (*close_fn)(NBDClient *client, bool negotiated);
|
2012-08-22 13:30:31 +00:00
|
|
|
|
2016-10-14 18:33:14 +00:00
|
|
|
bool no_zeroes;
|
2012-08-22 13:30:31 +00:00
|
|
|
NBDExport *exp;
|
2016-02-10 18:41:11 +00:00
|
|
|
QCryptoTLSCreds *tlscreds;
|
|
|
|
char *tlsaclname;
|
2016-02-10 18:41:04 +00:00
|
|
|
QIOChannelSocket *sioc; /* The underlying data channel */
|
|
|
|
QIOChannel *ioc; /* The current I/O channel which may differ (eg TLS) */
|
2012-08-22 13:30:31 +00:00
|
|
|
|
|
|
|
Coroutine *recv_coroutine;
|
|
|
|
|
|
|
|
CoMutex send_lock;
|
|
|
|
Coroutine *send_coroutine;
|
|
|
|
|
2012-09-18 11:58:25 +00:00
|
|
|
QTAILQ_ENTRY(NBDClient) next;
|
2012-08-22 13:30:31 +00:00
|
|
|
int nb_requests;
|
2012-08-22 16:45:12 +00:00
|
|
|
bool closing;
|
2012-08-22 13:30:31 +00:00
|
|
|
};
|
|
|
|
|
2008-05-27 21:13:40 +00:00
|
|
|
/* That's all folks */
|
|
|
|
|
2017-02-13 13:52:24 +00:00
|
|
|
static void nbd_client_receive_next_request(NBDClient *client);
|
2014-06-20 19:57:32 +00:00
|
|
|
|
2012-08-23 12:57:11 +00:00
|
|
|
/* Basic flow for negotiation
|
2008-05-27 21:13:40 +00:00
|
|
|
|
|
|
|
Server Client
|
|
|
|
Negotiate
|
2012-08-23 12:57:11 +00:00
|
|
|
|
|
|
|
or
|
|
|
|
|
|
|
|
Server Client
|
|
|
|
Negotiate #1
|
|
|
|
Option
|
|
|
|
Negotiate #2
|
|
|
|
|
|
|
|
----
|
|
|
|
|
|
|
|
followed by
|
|
|
|
|
|
|
|
Server Client
|
2008-05-27 21:13:40 +00:00
|
|
|
Request
|
|
|
|
Response
|
|
|
|
Request
|
|
|
|
Response
|
|
|
|
...
|
|
|
|
...
|
|
|
|
Request (type == 2)
|
2012-08-23 12:57:11 +00:00
|
|
|
|
2008-05-27 21:13:40 +00:00
|
|
|
*/
|
|
|
|
|
2016-10-14 18:33:08 +00:00
|
|
|
/* Send a reply header, including length, but no payload.
|
|
|
|
* Return -errno on error, 0 on success. */
|
|
|
|
static int nbd_negotiate_send_rep_len(QIOChannel *ioc, uint32_t type,
|
|
|
|
uint32_t opt, uint32_t len)
|
2012-08-23 12:57:11 +00:00
|
|
|
{
|
|
|
|
uint64_t magic;
|
|
|
|
|
2016-10-14 18:33:08 +00:00
|
|
|
TRACE("Reply opt=%" PRIx32 " type=%" PRIx32 " len=%" PRIu32,
|
|
|
|
type, opt, len);
|
2016-02-10 18:41:11 +00:00
|
|
|
|
2014-06-07 00:32:31 +00:00
|
|
|
magic = cpu_to_be64(NBD_REP_MAGIC);
|
nbd/server: get rid of nbd_negotiate_read and friends
Functions nbd_negotiate_{read,write,drop_sync} were introduced in
1a6245a5b, when nbd_rwv (was nbd_wr_sync) was working through
qemu_co_sendv_recvv (the path is nbd_wr_sync -> qemu_co_{recv/send} ->
qemu_co_send_recv -> qemu_co_sendv_recvv), which just yields, without
setting any handlers. But starting from ff82911cd nbd_rwv (was
nbd_wr_syncv) works through qio_channel_yield() which sets handlers, so
watchers are redundant in nbd_negotiate_{read,write,drop_sync}, then,
let's just use nbd_{read,write,drop} functions.
Functions nbd_{read,write,drop} has errp parameter, which is unused in
this patch. This will be fixed later.
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <20170602150150.258222-4-vsementsov@virtuozzo.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
2017-06-02 15:01:41 +00:00
|
|
|
if (nbd_write(ioc, &magic, sizeof(magic), NULL) < 0) {
|
2014-06-07 00:32:31 +00:00
|
|
|
LOG("write failed (rep magic)");
|
|
|
|
return -EINVAL;
|
2012-08-23 12:57:11 +00:00
|
|
|
}
|
2014-06-07 00:32:31 +00:00
|
|
|
opt = cpu_to_be32(opt);
|
nbd/server: get rid of nbd_negotiate_read and friends
Functions nbd_negotiate_{read,write,drop_sync} were introduced in
1a6245a5b, when nbd_rwv (was nbd_wr_sync) was working through
qemu_co_sendv_recvv (the path is nbd_wr_sync -> qemu_co_{recv/send} ->
qemu_co_send_recv -> qemu_co_sendv_recvv), which just yields, without
setting any handlers. But starting from ff82911cd nbd_rwv (was
nbd_wr_syncv) works through qio_channel_yield() which sets handlers, so
watchers are redundant in nbd_negotiate_{read,write,drop_sync}, then,
let's just use nbd_{read,write,drop} functions.
Functions nbd_{read,write,drop} has errp parameter, which is unused in
this patch. This will be fixed later.
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <20170602150150.258222-4-vsementsov@virtuozzo.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
2017-06-02 15:01:41 +00:00
|
|
|
if (nbd_write(ioc, &opt, sizeof(opt), NULL) < 0) {
|
2014-06-07 00:32:31 +00:00
|
|
|
LOG("write failed (rep opt)");
|
|
|
|
return -EINVAL;
|
2012-08-23 12:57:11 +00:00
|
|
|
}
|
2014-06-07 00:32:31 +00:00
|
|
|
type = cpu_to_be32(type);
|
nbd/server: get rid of nbd_negotiate_read and friends
Functions nbd_negotiate_{read,write,drop_sync} were introduced in
1a6245a5b, when nbd_rwv (was nbd_wr_sync) was working through
qemu_co_sendv_recvv (the path is nbd_wr_sync -> qemu_co_{recv/send} ->
qemu_co_send_recv -> qemu_co_sendv_recvv), which just yields, without
setting any handlers. But starting from ff82911cd nbd_rwv (was
nbd_wr_syncv) works through qio_channel_yield() which sets handlers, so
watchers are redundant in nbd_negotiate_{read,write,drop_sync}, then,
let's just use nbd_{read,write,drop} functions.
Functions nbd_{read,write,drop} has errp parameter, which is unused in
this patch. This will be fixed later.
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <20170602150150.258222-4-vsementsov@virtuozzo.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
2017-06-02 15:01:41 +00:00
|
|
|
if (nbd_write(ioc, &type, sizeof(type), NULL) < 0) {
|
2014-06-07 00:32:31 +00:00
|
|
|
LOG("write failed (rep type)");
|
|
|
|
return -EINVAL;
|
2012-08-23 12:57:11 +00:00
|
|
|
}
|
2016-10-14 18:33:08 +00:00
|
|
|
len = cpu_to_be32(len);
|
nbd/server: get rid of nbd_negotiate_read and friends
Functions nbd_negotiate_{read,write,drop_sync} were introduced in
1a6245a5b, when nbd_rwv (was nbd_wr_sync) was working through
qemu_co_sendv_recvv (the path is nbd_wr_sync -> qemu_co_{recv/send} ->
qemu_co_send_recv -> qemu_co_sendv_recvv), which just yields, without
setting any handlers. But starting from ff82911cd nbd_rwv (was
nbd_wr_syncv) works through qio_channel_yield() which sets handlers, so
watchers are redundant in nbd_negotiate_{read,write,drop_sync}, then,
let's just use nbd_{read,write,drop} functions.
Functions nbd_{read,write,drop} has errp parameter, which is unused in
this patch. This will be fixed later.
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <20170602150150.258222-4-vsementsov@virtuozzo.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
2017-06-02 15:01:41 +00:00
|
|
|
if (nbd_write(ioc, &len, sizeof(len), NULL) < 0) {
|
2014-06-07 00:32:31 +00:00
|
|
|
LOG("write failed (rep data length)");
|
|
|
|
return -EINVAL;
|
2012-08-23 12:57:11 +00:00
|
|
|
}
|
2014-06-07 00:32:31 +00:00
|
|
|
return 0;
|
|
|
|
}
|
2012-08-23 12:57:11 +00:00
|
|
|
|
2016-10-14 18:33:08 +00:00
|
|
|
/* Send a reply header with default 0 length.
|
|
|
|
* Return -errno on error, 0 on success. */
|
|
|
|
static int nbd_negotiate_send_rep(QIOChannel *ioc, uint32_t type, uint32_t opt)
|
|
|
|
{
|
|
|
|
return nbd_negotiate_send_rep_len(ioc, type, opt, 0);
|
|
|
|
}
|
|
|
|
|
2016-10-14 18:33:09 +00:00
|
|
|
/* Send an error reply.
|
|
|
|
* Return -errno on error, 0 on success. */
|
|
|
|
static int GCC_FMT_ATTR(4, 5)
|
|
|
|
nbd_negotiate_send_rep_err(QIOChannel *ioc, uint32_t type,
|
|
|
|
uint32_t opt, const char *fmt, ...)
|
|
|
|
{
|
|
|
|
va_list va;
|
|
|
|
char *msg;
|
|
|
|
int ret;
|
|
|
|
size_t len;
|
|
|
|
|
|
|
|
va_start(va, fmt);
|
|
|
|
msg = g_strdup_vprintf(fmt, va);
|
|
|
|
va_end(va);
|
|
|
|
len = strlen(msg);
|
|
|
|
assert(len < 4096);
|
|
|
|
TRACE("sending error message \"%s\"", msg);
|
|
|
|
ret = nbd_negotiate_send_rep_len(ioc, type, opt, len);
|
|
|
|
if (ret < 0) {
|
|
|
|
goto out;
|
|
|
|
}
|
nbd/server: get rid of nbd_negotiate_read and friends
Functions nbd_negotiate_{read,write,drop_sync} were introduced in
1a6245a5b, when nbd_rwv (was nbd_wr_sync) was working through
qemu_co_sendv_recvv (the path is nbd_wr_sync -> qemu_co_{recv/send} ->
qemu_co_send_recv -> qemu_co_sendv_recvv), which just yields, without
setting any handlers. But starting from ff82911cd nbd_rwv (was
nbd_wr_syncv) works through qio_channel_yield() which sets handlers, so
watchers are redundant in nbd_negotiate_{read,write,drop_sync}, then,
let's just use nbd_{read,write,drop} functions.
Functions nbd_{read,write,drop} has errp parameter, which is unused in
this patch. This will be fixed later.
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <20170602150150.258222-4-vsementsov@virtuozzo.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
2017-06-02 15:01:41 +00:00
|
|
|
if (nbd_write(ioc, msg, len, NULL) < 0) {
|
2016-10-14 18:33:09 +00:00
|
|
|
LOG("write failed (error message)");
|
|
|
|
ret = -EIO;
|
|
|
|
} else {
|
|
|
|
ret = 0;
|
|
|
|
}
|
|
|
|
out:
|
|
|
|
g_free(msg);
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
2016-10-14 18:33:08 +00:00
|
|
|
/* Send a single NBD_REP_SERVER reply to NBD_OPT_LIST, including payload.
|
|
|
|
* Return -errno on error, 0 on success. */
|
2016-02-10 18:41:04 +00:00
|
|
|
static int nbd_negotiate_send_rep_list(QIOChannel *ioc, NBDExport *exp)
|
2014-06-07 00:32:32 +00:00
|
|
|
{
|
2016-10-14 18:33:03 +00:00
|
|
|
size_t name_len, desc_len;
|
2016-10-14 18:33:08 +00:00
|
|
|
uint32_t len;
|
2016-10-14 18:33:03 +00:00
|
|
|
const char *name = exp->name ? exp->name : "";
|
|
|
|
const char *desc = exp->description ? exp->description : "";
|
2016-10-14 18:33:08 +00:00
|
|
|
int rc;
|
2014-06-07 00:32:32 +00:00
|
|
|
|
2016-10-14 18:33:03 +00:00
|
|
|
TRACE("Advertising export name '%s' description '%s'", name, desc);
|
|
|
|
name_len = strlen(name);
|
|
|
|
desc_len = strlen(desc);
|
2016-10-14 18:33:08 +00:00
|
|
|
len = name_len + desc_len + sizeof(len);
|
|
|
|
rc = nbd_negotiate_send_rep_len(ioc, NBD_REP_SERVER, NBD_OPT_LIST, len);
|
|
|
|
if (rc < 0) {
|
|
|
|
return rc;
|
2014-06-07 00:32:32 +00:00
|
|
|
}
|
2016-10-14 18:33:08 +00:00
|
|
|
|
2014-06-07 00:32:32 +00:00
|
|
|
len = cpu_to_be32(name_len);
|
nbd/server: get rid of nbd_negotiate_read and friends
Functions nbd_negotiate_{read,write,drop_sync} were introduced in
1a6245a5b, when nbd_rwv (was nbd_wr_sync) was working through
qemu_co_sendv_recvv (the path is nbd_wr_sync -> qemu_co_{recv/send} ->
qemu_co_send_recv -> qemu_co_sendv_recvv), which just yields, without
setting any handlers. But starting from ff82911cd nbd_rwv (was
nbd_wr_syncv) works through qio_channel_yield() which sets handlers, so
watchers are redundant in nbd_negotiate_{read,write,drop_sync}, then,
let's just use nbd_{read,write,drop} functions.
Functions nbd_{read,write,drop} has errp parameter, which is unused in
this patch. This will be fixed later.
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <20170602150150.258222-4-vsementsov@virtuozzo.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
2017-06-02 15:01:41 +00:00
|
|
|
if (nbd_write(ioc, &len, sizeof(len), NULL) < 0) {
|
2016-10-14 18:33:03 +00:00
|
|
|
LOG("write failed (name length)");
|
|
|
|
return -EINVAL;
|
|
|
|
}
|
nbd/server: get rid of nbd_negotiate_read and friends
Functions nbd_negotiate_{read,write,drop_sync} were introduced in
1a6245a5b, when nbd_rwv (was nbd_wr_sync) was working through
qemu_co_sendv_recvv (the path is nbd_wr_sync -> qemu_co_{recv/send} ->
qemu_co_send_recv -> qemu_co_sendv_recvv), which just yields, without
setting any handlers. But starting from ff82911cd nbd_rwv (was
nbd_wr_syncv) works through qio_channel_yield() which sets handlers, so
watchers are redundant in nbd_negotiate_{read,write,drop_sync}, then,
let's just use nbd_{read,write,drop} functions.
Functions nbd_{read,write,drop} has errp parameter, which is unused in
this patch. This will be fixed later.
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <20170602150150.258222-4-vsementsov@virtuozzo.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
2017-06-02 15:01:41 +00:00
|
|
|
if (nbd_write(ioc, name, name_len, NULL) < 0) {
|
2016-10-14 18:33:03 +00:00
|
|
|
LOG("write failed (name buffer)");
|
2014-06-07 00:32:32 +00:00
|
|
|
return -EINVAL;
|
|
|
|
}
|
nbd/server: get rid of nbd_negotiate_read and friends
Functions nbd_negotiate_{read,write,drop_sync} were introduced in
1a6245a5b, when nbd_rwv (was nbd_wr_sync) was working through
qemu_co_sendv_recvv (the path is nbd_wr_sync -> qemu_co_{recv/send} ->
qemu_co_send_recv -> qemu_co_sendv_recvv), which just yields, without
setting any handlers. But starting from ff82911cd nbd_rwv (was
nbd_wr_syncv) works through qio_channel_yield() which sets handlers, so
watchers are redundant in nbd_negotiate_{read,write,drop_sync}, then,
let's just use nbd_{read,write,drop} functions.
Functions nbd_{read,write,drop} has errp parameter, which is unused in
this patch. This will be fixed later.
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <20170602150150.258222-4-vsementsov@virtuozzo.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
2017-06-02 15:01:41 +00:00
|
|
|
if (nbd_write(ioc, desc, desc_len, NULL) < 0) {
|
2016-10-14 18:33:03 +00:00
|
|
|
LOG("write failed (description buffer)");
|
2014-06-07 00:32:32 +00:00
|
|
|
return -EINVAL;
|
|
|
|
}
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2016-10-14 18:33:08 +00:00
|
|
|
/* Process the NBD_OPT_LIST command, with a potential series of replies.
|
|
|
|
* Return -errno on error, 0 on success. */
|
2016-01-14 08:41:03 +00:00
|
|
|
static int nbd_negotiate_handle_list(NBDClient *client, uint32_t length)
|
2014-06-07 00:32:32 +00:00
|
|
|
{
|
|
|
|
NBDExport *exp;
|
|
|
|
|
|
|
|
if (length) {
|
nbd/server: get rid of nbd_negotiate_read and friends
Functions nbd_negotiate_{read,write,drop_sync} were introduced in
1a6245a5b, when nbd_rwv (was nbd_wr_sync) was working through
qemu_co_sendv_recvv (the path is nbd_wr_sync -> qemu_co_{recv/send} ->
qemu_co_send_recv -> qemu_co_sendv_recvv), which just yields, without
setting any handlers. But starting from ff82911cd nbd_rwv (was
nbd_wr_syncv) works through qio_channel_yield() which sets handlers, so
watchers are redundant in nbd_negotiate_{read,write,drop_sync}, then,
let's just use nbd_{read,write,drop} functions.
Functions nbd_{read,write,drop} has errp parameter, which is unused in
this patch. This will be fixed later.
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <20170602150150.258222-4-vsementsov@virtuozzo.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
2017-06-02 15:01:41 +00:00
|
|
|
if (nbd_drop(client->ioc, length, NULL) < 0) {
|
2015-02-25 18:08:34 +00:00
|
|
|
return -EIO;
|
|
|
|
}
|
2016-10-14 18:33:09 +00:00
|
|
|
return nbd_negotiate_send_rep_err(client->ioc,
|
|
|
|
NBD_REP_ERR_INVALID, NBD_OPT_LIST,
|
|
|
|
"OPT_LIST should not have length");
|
2014-06-07 00:32:32 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
/* For each export, send a NBD_REP_SERVER reply. */
|
|
|
|
QTAILQ_FOREACH(exp, &exports, next) {
|
2016-02-10 18:41:04 +00:00
|
|
|
if (nbd_negotiate_send_rep_list(client->ioc, exp)) {
|
2014-06-07 00:32:32 +00:00
|
|
|
return -EINVAL;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
/* Finish with a NBD_REP_ACK. */
|
2016-02-10 18:41:04 +00:00
|
|
|
return nbd_negotiate_send_rep(client->ioc, NBD_REP_ACK, NBD_OPT_LIST);
|
2014-06-07 00:32:32 +00:00
|
|
|
}
|
|
|
|
|
2016-01-14 08:41:03 +00:00
|
|
|
static int nbd_negotiate_handle_export_name(NBDClient *client, uint32_t length)
|
2014-06-07 00:32:31 +00:00
|
|
|
{
|
2016-02-10 18:41:04 +00:00
|
|
|
int rc = -EINVAL;
|
2016-05-11 22:39:44 +00:00
|
|
|
char name[NBD_MAX_NAME_SIZE + 1];
|
2012-08-23 12:57:11 +00:00
|
|
|
|
2014-06-07 00:32:31 +00:00
|
|
|
/* Client sends:
|
|
|
|
[20 .. xx] export name (length bytes)
|
|
|
|
*/
|
2012-08-23 12:57:11 +00:00
|
|
|
TRACE("Checking length");
|
2016-05-11 22:39:44 +00:00
|
|
|
if (length >= sizeof(name)) {
|
2012-08-23 12:57:11 +00:00
|
|
|
LOG("Bad length received");
|
|
|
|
goto fail;
|
|
|
|
}
|
nbd/server: get rid of nbd_negotiate_read and friends
Functions nbd_negotiate_{read,write,drop_sync} were introduced in
1a6245a5b, when nbd_rwv (was nbd_wr_sync) was working through
qemu_co_sendv_recvv (the path is nbd_wr_sync -> qemu_co_{recv/send} ->
qemu_co_send_recv -> qemu_co_sendv_recvv), which just yields, without
setting any handlers. But starting from ff82911cd nbd_rwv (was
nbd_wr_syncv) works through qio_channel_yield() which sets handlers, so
watchers are redundant in nbd_negotiate_{read,write,drop_sync}, then,
let's just use nbd_{read,write,drop} functions.
Functions nbd_{read,write,drop} has errp parameter, which is unused in
this patch. This will be fixed later.
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <20170602150150.258222-4-vsementsov@virtuozzo.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
2017-06-02 15:01:41 +00:00
|
|
|
if (nbd_read(client->ioc, name, length, NULL) < 0) {
|
2012-08-23 12:57:11 +00:00
|
|
|
LOG("read failed");
|
|
|
|
goto fail;
|
|
|
|
}
|
|
|
|
name[length] = '\0';
|
|
|
|
|
2016-02-10 18:41:09 +00:00
|
|
|
TRACE("Client requested export '%s'", name);
|
|
|
|
|
2012-08-23 12:57:11 +00:00
|
|
|
client->exp = nbd_export_find(name);
|
|
|
|
if (!client->exp) {
|
|
|
|
LOG("export not found");
|
|
|
|
goto fail;
|
|
|
|
}
|
|
|
|
|
|
|
|
QTAILQ_INSERT_TAIL(&client->exp->clients, client, next);
|
|
|
|
nbd_export_get(client->exp);
|
|
|
|
rc = 0;
|
|
|
|
fail:
|
|
|
|
return rc;
|
|
|
|
}
|
|
|
|
|
2016-10-14 18:33:09 +00:00
|
|
|
/* Handle NBD_OPT_STARTTLS. Return NULL to drop connection, or else the
|
|
|
|
* new channel for all further (now-encrypted) communication. */
|
2016-02-10 18:41:11 +00:00
|
|
|
static QIOChannel *nbd_negotiate_handle_starttls(NBDClient *client,
|
|
|
|
uint32_t length)
|
|
|
|
{
|
|
|
|
QIOChannel *ioc;
|
|
|
|
QIOChannelTLS *tioc;
|
|
|
|
struct NBDTLSHandshakeData data = { 0 };
|
|
|
|
|
|
|
|
TRACE("Setting up TLS");
|
|
|
|
ioc = client->ioc;
|
|
|
|
if (length) {
|
nbd/server: get rid of nbd_negotiate_read and friends
Functions nbd_negotiate_{read,write,drop_sync} were introduced in
1a6245a5b, when nbd_rwv (was nbd_wr_sync) was working through
qemu_co_sendv_recvv (the path is nbd_wr_sync -> qemu_co_{recv/send} ->
qemu_co_send_recv -> qemu_co_sendv_recvv), which just yields, without
setting any handlers. But starting from ff82911cd nbd_rwv (was
nbd_wr_syncv) works through qio_channel_yield() which sets handlers, so
watchers are redundant in nbd_negotiate_{read,write,drop_sync}, then,
let's just use nbd_{read,write,drop} functions.
Functions nbd_{read,write,drop} has errp parameter, which is unused in
this patch. This will be fixed later.
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <20170602150150.258222-4-vsementsov@virtuozzo.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
2017-06-02 15:01:41 +00:00
|
|
|
if (nbd_drop(ioc, length, NULL) < 0) {
|
2016-02-10 18:41:11 +00:00
|
|
|
return NULL;
|
|
|
|
}
|
2016-10-14 18:33:09 +00:00
|
|
|
nbd_negotiate_send_rep_err(ioc, NBD_REP_ERR_INVALID, NBD_OPT_STARTTLS,
|
|
|
|
"OPT_STARTTLS should not have length");
|
2016-02-10 18:41:11 +00:00
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
2016-05-11 22:39:36 +00:00
|
|
|
if (nbd_negotiate_send_rep(client->ioc, NBD_REP_ACK,
|
|
|
|
NBD_OPT_STARTTLS) < 0) {
|
|
|
|
return NULL;
|
|
|
|
}
|
2016-02-10 18:41:11 +00:00
|
|
|
|
|
|
|
tioc = qio_channel_tls_new_server(ioc,
|
|
|
|
client->tlscreds,
|
|
|
|
client->tlsaclname,
|
|
|
|
NULL);
|
|
|
|
if (!tioc) {
|
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
2016-09-30 10:57:14 +00:00
|
|
|
qio_channel_set_name(QIO_CHANNEL(tioc), "nbd-server-tls");
|
2016-02-10 18:41:11 +00:00
|
|
|
TRACE("Starting TLS handshake");
|
|
|
|
data.loop = g_main_loop_new(g_main_context_default(), FALSE);
|
|
|
|
qio_channel_tls_handshake(tioc,
|
|
|
|
nbd_tls_handshake,
|
|
|
|
&data,
|
|
|
|
NULL);
|
|
|
|
|
|
|
|
if (!data.complete) {
|
|
|
|
g_main_loop_run(data.loop);
|
|
|
|
}
|
|
|
|
g_main_loop_unref(data.loop);
|
|
|
|
if (data.error) {
|
|
|
|
object_unref(OBJECT(tioc));
|
|
|
|
error_free(data.error);
|
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
|
|
|
return QIO_CHANNEL(tioc);
|
|
|
|
}
|
|
|
|
|
|
|
|
|
2016-10-14 18:33:08 +00:00
|
|
|
/* Process all NBD_OPT_* client option commands.
|
|
|
|
* Return -errno on error, 0 on success. */
|
2016-01-14 08:41:03 +00:00
|
|
|
static int nbd_negotiate_options(NBDClient *client)
|
2014-06-07 00:32:31 +00:00
|
|
|
{
|
2015-02-25 18:08:31 +00:00
|
|
|
uint32_t flags;
|
2016-02-10 18:41:06 +00:00
|
|
|
bool fixedNewstyle = false;
|
2015-02-25 18:08:31 +00:00
|
|
|
|
|
|
|
/* Client sends:
|
|
|
|
[ 0 .. 3] client flags
|
|
|
|
|
|
|
|
[ 0 .. 7] NBD_OPTS_MAGIC
|
|
|
|
[ 8 .. 11] NBD option
|
|
|
|
[12 .. 15] Data length
|
|
|
|
... Rest of request
|
|
|
|
|
|
|
|
[ 0 .. 7] NBD_OPTS_MAGIC
|
|
|
|
[ 8 .. 11] Second NBD option
|
|
|
|
[12 .. 15] Data length
|
|
|
|
... Rest of request
|
|
|
|
*/
|
|
|
|
|
nbd/server: get rid of nbd_negotiate_read and friends
Functions nbd_negotiate_{read,write,drop_sync} were introduced in
1a6245a5b, when nbd_rwv (was nbd_wr_sync) was working through
qemu_co_sendv_recvv (the path is nbd_wr_sync -> qemu_co_{recv/send} ->
qemu_co_send_recv -> qemu_co_sendv_recvv), which just yields, without
setting any handlers. But starting from ff82911cd nbd_rwv (was
nbd_wr_syncv) works through qio_channel_yield() which sets handlers, so
watchers are redundant in nbd_negotiate_{read,write,drop_sync}, then,
let's just use nbd_{read,write,drop} functions.
Functions nbd_{read,write,drop} has errp parameter, which is unused in
this patch. This will be fixed later.
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <20170602150150.258222-4-vsementsov@virtuozzo.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
2017-06-02 15:01:41 +00:00
|
|
|
if (nbd_read(client->ioc, &flags, sizeof(flags), NULL) < 0) {
|
2015-02-25 18:08:31 +00:00
|
|
|
LOG("read failed");
|
|
|
|
return -EIO;
|
|
|
|
}
|
|
|
|
TRACE("Checking client flags");
|
|
|
|
be32_to_cpus(&flags);
|
2016-02-10 18:41:06 +00:00
|
|
|
if (flags & NBD_FLAG_C_FIXED_NEWSTYLE) {
|
2016-05-11 22:39:35 +00:00
|
|
|
TRACE("Client supports fixed newstyle handshake");
|
2016-02-10 18:41:06 +00:00
|
|
|
fixedNewstyle = true;
|
|
|
|
flags &= ~NBD_FLAG_C_FIXED_NEWSTYLE;
|
|
|
|
}
|
2016-10-14 18:33:14 +00:00
|
|
|
if (flags & NBD_FLAG_C_NO_ZEROES) {
|
|
|
|
TRACE("Client supports no zeroes at handshake end");
|
|
|
|
client->no_zeroes = true;
|
|
|
|
flags &= ~NBD_FLAG_C_NO_ZEROES;
|
|
|
|
}
|
2016-02-10 18:41:06 +00:00
|
|
|
if (flags != 0) {
|
2016-05-11 22:39:35 +00:00
|
|
|
TRACE("Unknown client flags 0x%" PRIx32 " received", flags);
|
2015-02-25 18:08:31 +00:00
|
|
|
return -EIO;
|
|
|
|
}
|
|
|
|
|
2014-06-07 00:32:31 +00:00
|
|
|
while (1) {
|
2015-02-25 18:08:31 +00:00
|
|
|
int ret;
|
2016-02-10 18:41:06 +00:00
|
|
|
uint32_t clientflags, length;
|
2014-06-07 00:32:31 +00:00
|
|
|
uint64_t magic;
|
|
|
|
|
nbd/server: get rid of nbd_negotiate_read and friends
Functions nbd_negotiate_{read,write,drop_sync} were introduced in
1a6245a5b, when nbd_rwv (was nbd_wr_sync) was working through
qemu_co_sendv_recvv (the path is nbd_wr_sync -> qemu_co_{recv/send} ->
qemu_co_send_recv -> qemu_co_sendv_recvv), which just yields, without
setting any handlers. But starting from ff82911cd nbd_rwv (was
nbd_wr_syncv) works through qio_channel_yield() which sets handlers, so
watchers are redundant in nbd_negotiate_{read,write,drop_sync}, then,
let's just use nbd_{read,write,drop} functions.
Functions nbd_{read,write,drop} has errp parameter, which is unused in
this patch. This will be fixed later.
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <20170602150150.258222-4-vsementsov@virtuozzo.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
2017-06-02 15:01:41 +00:00
|
|
|
if (nbd_read(client->ioc, &magic, sizeof(magic), NULL) < 0) {
|
2014-06-07 00:32:31 +00:00
|
|
|
LOG("read failed");
|
|
|
|
return -EINVAL;
|
|
|
|
}
|
|
|
|
TRACE("Checking opts magic");
|
|
|
|
if (magic != be64_to_cpu(NBD_OPTS_MAGIC)) {
|
|
|
|
LOG("Bad magic received");
|
|
|
|
return -EINVAL;
|
|
|
|
}
|
|
|
|
|
nbd/server: get rid of nbd_negotiate_read and friends
Functions nbd_negotiate_{read,write,drop_sync} were introduced in
1a6245a5b, when nbd_rwv (was nbd_wr_sync) was working through
qemu_co_sendv_recvv (the path is nbd_wr_sync -> qemu_co_{recv/send} ->
qemu_co_send_recv -> qemu_co_sendv_recvv), which just yields, without
setting any handlers. But starting from ff82911cd nbd_rwv (was
nbd_wr_syncv) works through qio_channel_yield() which sets handlers, so
watchers are redundant in nbd_negotiate_{read,write,drop_sync}, then,
let's just use nbd_{read,write,drop} functions.
Functions nbd_{read,write,drop} has errp parameter, which is unused in
this patch. This will be fixed later.
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <20170602150150.258222-4-vsementsov@virtuozzo.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
2017-06-02 15:01:41 +00:00
|
|
|
if (nbd_read(client->ioc, &clientflags,
|
|
|
|
sizeof(clientflags), NULL) < 0)
|
nbd: read_sync and friends: return 0 on success
functions read_sync, drop_sync, write_sync, and also
nbd_negotiate_write, nbd_negotiate_read, nbd_negotiate_drop_sync
returns number of processed bytes. But what this number can be,
except requested number of bytes?
Actually, underlying nbd_wr_syncv function returns a value >= 0 and
!= requested_bytes only on eof on read operation. So, firstly, it is
impossible on write (let's add an assert) and on read it actually
means, that communication is broken (except nbd_receive_reply, see
below).
Most of callers operate like this:
if (func(..., size) != size) {
/* error path */
}
, i.e.:
1. They are not interested in partial success
2. Extra duplications in code (especially bad are duplications of
magic numbers)
3. User doesn't see actual error message, as return code is lost.
(this patch doesn't fix this point, but it makes fixing easier)
Several callers handles ret >= 0 and != requested-size separately, by
just returning EINVAL in this case. This patch makes read_sync and
friends return EINVAL in this case, so final behavior is the same.
And only one caller - nbd_receive_reply() does something not so
obvious. It returns EINVAL for ret > 0 and != requested-size, like
previous group, but for ret == 0 it returns 0. The only caller of
nbd_receive_reply() - nbd_read_reply_entry() handles ret == 0 in the
same way as ret < 0, so for now it doesn't matter. However, in
following commits error path handling will be improved and we'll need
to distinguish success from fail in this case too. So, this patch adds
separate helper for this case - read_sync_eof.
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Message-Id: <20170516094533.6160-3-vsementsov@virtuozzo.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
2017-05-16 09:45:30 +00:00
|
|
|
{
|
2014-06-07 00:32:31 +00:00
|
|
|
LOG("read failed");
|
|
|
|
return -EINVAL;
|
|
|
|
}
|
2016-02-10 18:41:06 +00:00
|
|
|
clientflags = be32_to_cpu(clientflags);
|
2014-06-07 00:32:31 +00:00
|
|
|
|
nbd/server: get rid of nbd_negotiate_read and friends
Functions nbd_negotiate_{read,write,drop_sync} were introduced in
1a6245a5b, when nbd_rwv (was nbd_wr_sync) was working through
qemu_co_sendv_recvv (the path is nbd_wr_sync -> qemu_co_{recv/send} ->
qemu_co_send_recv -> qemu_co_sendv_recvv), which just yields, without
setting any handlers. But starting from ff82911cd nbd_rwv (was
nbd_wr_syncv) works through qio_channel_yield() which sets handlers, so
watchers are redundant in nbd_negotiate_{read,write,drop_sync}, then,
let's just use nbd_{read,write,drop} functions.
Functions nbd_{read,write,drop} has errp parameter, which is unused in
this patch. This will be fixed later.
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <20170602150150.258222-4-vsementsov@virtuozzo.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
2017-06-02 15:01:41 +00:00
|
|
|
if (nbd_read(client->ioc, &length, sizeof(length), NULL) < 0) {
|
2014-06-07 00:32:31 +00:00
|
|
|
LOG("read failed");
|
|
|
|
return -EINVAL;
|
|
|
|
}
|
|
|
|
length = be32_to_cpu(length);
|
|
|
|
|
2016-05-11 22:39:35 +00:00
|
|
|
TRACE("Checking option 0x%" PRIx32, clientflags);
|
2016-02-10 18:41:11 +00:00
|
|
|
if (client->tlscreds &&
|
|
|
|
client->ioc == (QIOChannel *)client->sioc) {
|
|
|
|
QIOChannel *tioc;
|
|
|
|
if (!fixedNewstyle) {
|
2016-05-11 22:39:35 +00:00
|
|
|
TRACE("Unsupported option 0x%" PRIx32, clientflags);
|
2016-02-10 18:41:11 +00:00
|
|
|
return -EINVAL;
|
|
|
|
}
|
|
|
|
switch (clientflags) {
|
|
|
|
case NBD_OPT_STARTTLS:
|
|
|
|
tioc = nbd_negotiate_handle_starttls(client, length);
|
|
|
|
if (!tioc) {
|
|
|
|
return -EIO;
|
|
|
|
}
|
|
|
|
object_unref(OBJECT(client->ioc));
|
|
|
|
client->ioc = QIO_CHANNEL(tioc);
|
|
|
|
break;
|
|
|
|
|
nbd: Don't kill server on client that doesn't request TLS
Upstream NBD documents (as of commit 4feebc95) that servers MAY
choose to operate in a conditional mode, where it is up to the
client whether to use TLS. For qemu's case, we want to always be
in FORCEDTLS mode, because of the risk of man-in-the-middle
attacks, and since we never export more than one device; likewise,
the qemu client will ALWAYS send NBD_OPT_STARTTLS as its first
option. But now that SELECTIVETLS servers exist, it is feasible
to encounter a (non-qemu) client that is programmed to talk to
such a server, and does not do NBD_OPT_STARTTLS first, but rather
wants to probe if it can use a non-encrypted export.
The NBD protocol documents that we should let such a client
continue trying, on the grounds that maybe the client will get the
hint to send NBD_OPT_STARTTLS, rather than immediately dropping
the connection.
Note that NBD_OPT_EXPORT_NAME is a special case: since it is the
only option request that can't have an error return, we have to
(continue to) drop the connection on that one; rather, what we are
fixing here is that all other replies prior to TLS initiation tell
the client NBD_REP_ERR_TLS_REQD, but keep the connection alive.
Signed-off-by: Eric Blake <eblake@redhat.com>
Message-id: 1460671343-18485-1-git-send-email-eblake@redhat.com
Signed-off-by: Max Reitz <mreitz@redhat.com>
2016-04-14 22:02:23 +00:00
|
|
|
case NBD_OPT_EXPORT_NAME:
|
|
|
|
/* No way to return an error to client, so drop connection */
|
|
|
|
TRACE("Option 0x%x not permitted before TLS", clientflags);
|
|
|
|
return -EINVAL;
|
|
|
|
|
2016-02-10 18:41:11 +00:00
|
|
|
default:
|
nbd/server: get rid of nbd_negotiate_read and friends
Functions nbd_negotiate_{read,write,drop_sync} were introduced in
1a6245a5b, when nbd_rwv (was nbd_wr_sync) was working through
qemu_co_sendv_recvv (the path is nbd_wr_sync -> qemu_co_{recv/send} ->
qemu_co_send_recv -> qemu_co_sendv_recvv), which just yields, without
setting any handlers. But starting from ff82911cd nbd_rwv (was
nbd_wr_syncv) works through qio_channel_yield() which sets handlers, so
watchers are redundant in nbd_negotiate_{read,write,drop_sync}, then,
let's just use nbd_{read,write,drop} functions.
Functions nbd_{read,write,drop} has errp parameter, which is unused in
this patch. This will be fixed later.
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <20170602150150.258222-4-vsementsov@virtuozzo.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
2017-06-02 15:01:41 +00:00
|
|
|
if (nbd_drop(client->ioc, length, NULL) < 0) {
|
nbd: Don't kill server on client that doesn't request TLS
Upstream NBD documents (as of commit 4feebc95) that servers MAY
choose to operate in a conditional mode, where it is up to the
client whether to use TLS. For qemu's case, we want to always be
in FORCEDTLS mode, because of the risk of man-in-the-middle
attacks, and since we never export more than one device; likewise,
the qemu client will ALWAYS send NBD_OPT_STARTTLS as its first
option. But now that SELECTIVETLS servers exist, it is feasible
to encounter a (non-qemu) client that is programmed to talk to
such a server, and does not do NBD_OPT_STARTTLS first, but rather
wants to probe if it can use a non-encrypted export.
The NBD protocol documents that we should let such a client
continue trying, on the grounds that maybe the client will get the
hint to send NBD_OPT_STARTTLS, rather than immediately dropping
the connection.
Note that NBD_OPT_EXPORT_NAME is a special case: since it is the
only option request that can't have an error return, we have to
(continue to) drop the connection on that one; rather, what we are
fixing here is that all other replies prior to TLS initiation tell
the client NBD_REP_ERR_TLS_REQD, but keep the connection alive.
Signed-off-by: Eric Blake <eblake@redhat.com>
Message-id: 1460671343-18485-1-git-send-email-eblake@redhat.com
Signed-off-by: Max Reitz <mreitz@redhat.com>
2016-04-14 22:02:23 +00:00
|
|
|
return -EIO;
|
|
|
|
}
|
2016-10-14 18:33:09 +00:00
|
|
|
ret = nbd_negotiate_send_rep_err(client->ioc,
|
|
|
|
NBD_REP_ERR_TLS_REQD,
|
|
|
|
clientflags,
|
|
|
|
"Option 0x%" PRIx32
|
|
|
|
"not permitted before TLS",
|
|
|
|
clientflags);
|
2016-05-11 22:39:36 +00:00
|
|
|
if (ret < 0) {
|
|
|
|
return ret;
|
|
|
|
}
|
2016-10-14 18:33:16 +00:00
|
|
|
/* Let the client keep trying, unless they asked to quit */
|
|
|
|
if (clientflags == NBD_OPT_ABORT) {
|
|
|
|
return -EINVAL;
|
|
|
|
}
|
nbd: Don't kill server on client that doesn't request TLS
Upstream NBD documents (as of commit 4feebc95) that servers MAY
choose to operate in a conditional mode, where it is up to the
client whether to use TLS. For qemu's case, we want to always be
in FORCEDTLS mode, because of the risk of man-in-the-middle
attacks, and since we never export more than one device; likewise,
the qemu client will ALWAYS send NBD_OPT_STARTTLS as its first
option. But now that SELECTIVETLS servers exist, it is feasible
to encounter a (non-qemu) client that is programmed to talk to
such a server, and does not do NBD_OPT_STARTTLS first, but rather
wants to probe if it can use a non-encrypted export.
The NBD protocol documents that we should let such a client
continue trying, on the grounds that maybe the client will get the
hint to send NBD_OPT_STARTTLS, rather than immediately dropping
the connection.
Note that NBD_OPT_EXPORT_NAME is a special case: since it is the
only option request that can't have an error return, we have to
(continue to) drop the connection on that one; rather, what we are
fixing here is that all other replies prior to TLS initiation tell
the client NBD_REP_ERR_TLS_REQD, but keep the connection alive.
Signed-off-by: Eric Blake <eblake@redhat.com>
Message-id: 1460671343-18485-1-git-send-email-eblake@redhat.com
Signed-off-by: Max Reitz <mreitz@redhat.com>
2016-04-14 22:02:23 +00:00
|
|
|
break;
|
2016-02-10 18:41:11 +00:00
|
|
|
}
|
|
|
|
} else if (fixedNewstyle) {
|
2016-02-10 18:41:06 +00:00
|
|
|
switch (clientflags) {
|
|
|
|
case NBD_OPT_LIST:
|
|
|
|
ret = nbd_negotiate_handle_list(client, length);
|
|
|
|
if (ret < 0) {
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
break;
|
|
|
|
|
|
|
|
case NBD_OPT_ABORT:
|
2016-10-14 18:33:16 +00:00
|
|
|
/* NBD spec says we must try to reply before
|
|
|
|
* disconnecting, but that we must also tolerate
|
|
|
|
* guests that don't wait for our reply. */
|
|
|
|
nbd_negotiate_send_rep(client->ioc, NBD_REP_ACK, clientflags);
|
2016-02-10 18:41:06 +00:00
|
|
|
return -EINVAL;
|
|
|
|
|
|
|
|
case NBD_OPT_EXPORT_NAME:
|
|
|
|
return nbd_negotiate_handle_export_name(client, length);
|
|
|
|
|
2016-02-10 18:41:11 +00:00
|
|
|
case NBD_OPT_STARTTLS:
|
nbd/server: get rid of nbd_negotiate_read and friends
Functions nbd_negotiate_{read,write,drop_sync} were introduced in
1a6245a5b, when nbd_rwv (was nbd_wr_sync) was working through
qemu_co_sendv_recvv (the path is nbd_wr_sync -> qemu_co_{recv/send} ->
qemu_co_send_recv -> qemu_co_sendv_recvv), which just yields, without
setting any handlers. But starting from ff82911cd nbd_rwv (was
nbd_wr_syncv) works through qio_channel_yield() which sets handlers, so
watchers are redundant in nbd_negotiate_{read,write,drop_sync}, then,
let's just use nbd_{read,write,drop} functions.
Functions nbd_{read,write,drop} has errp parameter, which is unused in
this patch. This will be fixed later.
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <20170602150150.258222-4-vsementsov@virtuozzo.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
2017-06-02 15:01:41 +00:00
|
|
|
if (nbd_drop(client->ioc, length, NULL) < 0) {
|
nbd: Don't kill server on client that doesn't request TLS
Upstream NBD documents (as of commit 4feebc95) that servers MAY
choose to operate in a conditional mode, where it is up to the
client whether to use TLS. For qemu's case, we want to always be
in FORCEDTLS mode, because of the risk of man-in-the-middle
attacks, and since we never export more than one device; likewise,
the qemu client will ALWAYS send NBD_OPT_STARTTLS as its first
option. But now that SELECTIVETLS servers exist, it is feasible
to encounter a (non-qemu) client that is programmed to talk to
such a server, and does not do NBD_OPT_STARTTLS first, but rather
wants to probe if it can use a non-encrypted export.
The NBD protocol documents that we should let such a client
continue trying, on the grounds that maybe the client will get the
hint to send NBD_OPT_STARTTLS, rather than immediately dropping
the connection.
Note that NBD_OPT_EXPORT_NAME is a special case: since it is the
only option request that can't have an error return, we have to
(continue to) drop the connection on that one; rather, what we are
fixing here is that all other replies prior to TLS initiation tell
the client NBD_REP_ERR_TLS_REQD, but keep the connection alive.
Signed-off-by: Eric Blake <eblake@redhat.com>
Message-id: 1460671343-18485-1-git-send-email-eblake@redhat.com
Signed-off-by: Max Reitz <mreitz@redhat.com>
2016-04-14 22:02:23 +00:00
|
|
|
return -EIO;
|
|
|
|
}
|
2016-02-10 18:41:11 +00:00
|
|
|
if (client->tlscreds) {
|
2016-10-14 18:33:09 +00:00
|
|
|
ret = nbd_negotiate_send_rep_err(client->ioc,
|
|
|
|
NBD_REP_ERR_INVALID,
|
|
|
|
clientflags,
|
|
|
|
"TLS already enabled");
|
2016-02-10 18:41:11 +00:00
|
|
|
} else {
|
2016-10-14 18:33:09 +00:00
|
|
|
ret = nbd_negotiate_send_rep_err(client->ioc,
|
|
|
|
NBD_REP_ERR_POLICY,
|
|
|
|
clientflags,
|
|
|
|
"TLS not configured");
|
2016-05-11 22:39:36 +00:00
|
|
|
}
|
|
|
|
if (ret < 0) {
|
|
|
|
return ret;
|
2016-02-10 18:41:11 +00:00
|
|
|
}
|
nbd: Don't kill server on client that doesn't request TLS
Upstream NBD documents (as of commit 4feebc95) that servers MAY
choose to operate in a conditional mode, where it is up to the
client whether to use TLS. For qemu's case, we want to always be
in FORCEDTLS mode, because of the risk of man-in-the-middle
attacks, and since we never export more than one device; likewise,
the qemu client will ALWAYS send NBD_OPT_STARTTLS as its first
option. But now that SELECTIVETLS servers exist, it is feasible
to encounter a (non-qemu) client that is programmed to talk to
such a server, and does not do NBD_OPT_STARTTLS first, but rather
wants to probe if it can use a non-encrypted export.
The NBD protocol documents that we should let such a client
continue trying, on the grounds that maybe the client will get the
hint to send NBD_OPT_STARTTLS, rather than immediately dropping
the connection.
Note that NBD_OPT_EXPORT_NAME is a special case: since it is the
only option request that can't have an error return, we have to
(continue to) drop the connection on that one; rather, what we are
fixing here is that all other replies prior to TLS initiation tell
the client NBD_REP_ERR_TLS_REQD, but keep the connection alive.
Signed-off-by: Eric Blake <eblake@redhat.com>
Message-id: 1460671343-18485-1-git-send-email-eblake@redhat.com
Signed-off-by: Max Reitz <mreitz@redhat.com>
2016-04-14 22:02:23 +00:00
|
|
|
break;
|
2016-02-10 18:41:06 +00:00
|
|
|
default:
|
nbd/server: get rid of nbd_negotiate_read and friends
Functions nbd_negotiate_{read,write,drop_sync} were introduced in
1a6245a5b, when nbd_rwv (was nbd_wr_sync) was working through
qemu_co_sendv_recvv (the path is nbd_wr_sync -> qemu_co_{recv/send} ->
qemu_co_send_recv -> qemu_co_sendv_recvv), which just yields, without
setting any handlers. But starting from ff82911cd nbd_rwv (was
nbd_wr_syncv) works through qio_channel_yield() which sets handlers, so
watchers are redundant in nbd_negotiate_{read,write,drop_sync}, then,
let's just use nbd_{read,write,drop} functions.
Functions nbd_{read,write,drop} has errp parameter, which is unused in
this patch. This will be fixed later.
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <20170602150150.258222-4-vsementsov@virtuozzo.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
2017-06-02 15:01:41 +00:00
|
|
|
if (nbd_drop(client->ioc, length, NULL) < 0) {
|
2016-04-06 22:48:38 +00:00
|
|
|
return -EIO;
|
|
|
|
}
|
2016-10-14 18:33:09 +00:00
|
|
|
ret = nbd_negotiate_send_rep_err(client->ioc,
|
|
|
|
NBD_REP_ERR_UNSUP,
|
|
|
|
clientflags,
|
|
|
|
"Unsupported option 0x%"
|
|
|
|
PRIx32,
|
|
|
|
clientflags);
|
2016-05-11 22:39:36 +00:00
|
|
|
if (ret < 0) {
|
|
|
|
return ret;
|
|
|
|
}
|
2016-04-06 22:48:38 +00:00
|
|
|
break;
|
2016-02-10 18:41:06 +00:00
|
|
|
}
|
|
|
|
} else {
|
|
|
|
/*
|
|
|
|
* If broken new-style we should drop the connection
|
|
|
|
* for anything except NBD_OPT_EXPORT_NAME
|
|
|
|
*/
|
|
|
|
switch (clientflags) {
|
|
|
|
case NBD_OPT_EXPORT_NAME:
|
|
|
|
return nbd_negotiate_handle_export_name(client, length);
|
|
|
|
|
|
|
|
default:
|
2016-05-11 22:39:35 +00:00
|
|
|
TRACE("Unsupported option 0x%" PRIx32, clientflags);
|
2016-02-10 18:41:06 +00:00
|
|
|
return -EINVAL;
|
2014-06-07 00:32:32 +00:00
|
|
|
}
|
2014-06-07 00:32:31 +00:00
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2016-01-14 08:41:03 +00:00
|
|
|
typedef struct {
|
|
|
|
NBDClient *client;
|
|
|
|
Coroutine *co;
|
|
|
|
} NBDClientNewData;
|
|
|
|
|
|
|
|
static coroutine_fn int nbd_negotiate(NBDClientNewData *data)
|
2008-05-27 21:13:40 +00:00
|
|
|
{
|
2016-01-14 08:41:03 +00:00
|
|
|
NBDClient *client = data->client;
|
2011-02-22 15:44:51 +00:00
|
|
|
char buf[8 + 8 + 8 + 128];
|
2012-03-05 07:56:10 +00:00
|
|
|
int rc;
|
2016-07-21 19:34:46 +00:00
|
|
|
const uint16_t myflags = (NBD_FLAG_HAS_FLAGS | NBD_FLAG_SEND_TRIM |
|
2016-10-14 18:33:17 +00:00
|
|
|
NBD_FLAG_SEND_FLUSH | NBD_FLAG_SEND_FUA |
|
|
|
|
NBD_FLAG_SEND_WRITE_ZEROES);
|
2016-02-10 18:41:11 +00:00
|
|
|
bool oldStyle;
|
2016-10-14 18:33:14 +00:00
|
|
|
size_t len;
|
2011-02-22 15:44:51 +00:00
|
|
|
|
2016-02-10 18:41:11 +00:00
|
|
|
/* Old style negotiation header without options
|
2012-08-23 12:57:11 +00:00
|
|
|
[ 0 .. 7] passwd ("NBDMAGIC")
|
|
|
|
[ 8 .. 15] magic (NBD_CLIENT_MAGIC)
|
2011-02-22 15:44:51 +00:00
|
|
|
[16 .. 23] size
|
2012-08-23 12:57:11 +00:00
|
|
|
[24 .. 25] server flags (0)
|
2014-05-12 23:35:16 +00:00
|
|
|
[26 .. 27] export flags
|
2012-08-23 12:57:11 +00:00
|
|
|
[28 .. 151] reserved (0)
|
|
|
|
|
2016-02-10 18:41:11 +00:00
|
|
|
New style negotiation header with options
|
2012-08-23 12:57:11 +00:00
|
|
|
[ 0 .. 7] passwd ("NBDMAGIC")
|
|
|
|
[ 8 .. 15] magic (NBD_OPTS_MAGIC)
|
|
|
|
[16 .. 17] server flags (0)
|
2016-02-10 18:41:11 +00:00
|
|
|
....options sent....
|
2012-08-23 12:57:11 +00:00
|
|
|
[18 .. 25] size
|
|
|
|
[26 .. 27] export flags
|
2016-10-14 18:33:14 +00:00
|
|
|
[28 .. 151] reserved (0, omit if no_zeroes)
|
2011-02-22 15:44:51 +00:00
|
|
|
*/
|
|
|
|
|
2016-02-10 18:41:04 +00:00
|
|
|
qio_channel_set_blocking(client->ioc, false, NULL);
|
2012-03-05 07:56:10 +00:00
|
|
|
rc = -EINVAL;
|
|
|
|
|
2011-02-22 15:44:51 +00:00
|
|
|
TRACE("Beginning negotiation.");
|
2012-11-26 14:19:31 +00:00
|
|
|
memset(buf, 0, sizeof(buf));
|
2011-02-22 15:44:51 +00:00
|
|
|
memcpy(buf, "NBDMAGIC", 8);
|
2016-02-10 18:41:11 +00:00
|
|
|
|
|
|
|
oldStyle = client->exp != NULL && !client->tlscreds;
|
|
|
|
if (oldStyle) {
|
2016-05-11 22:39:35 +00:00
|
|
|
TRACE("advertising size %" PRIu64 " and flags %x",
|
|
|
|
client->exp->size, client->exp->nbdflags | myflags);
|
2016-02-04 10:27:55 +00:00
|
|
|
stq_be_p(buf + 8, NBD_CLIENT_MAGIC);
|
|
|
|
stq_be_p(buf + 16, client->exp->size);
|
|
|
|
stw_be_p(buf + 26, client->exp->nbdflags | myflags);
|
2012-08-23 12:57:11 +00:00
|
|
|
} else {
|
2016-02-04 10:27:55 +00:00
|
|
|
stq_be_p(buf + 8, NBD_OPTS_MAGIC);
|
2016-10-14 18:33:14 +00:00
|
|
|
stw_be_p(buf + 16, NBD_FLAG_FIXED_NEWSTYLE | NBD_FLAG_NO_ZEROES);
|
2012-08-23 12:57:11 +00:00
|
|
|
}
|
2011-02-22 15:44:51 +00:00
|
|
|
|
2016-02-10 18:41:11 +00:00
|
|
|
if (oldStyle) {
|
|
|
|
if (client->tlscreds) {
|
|
|
|
TRACE("TLS cannot be enabled with oldstyle protocol");
|
|
|
|
goto fail;
|
|
|
|
}
|
nbd/server: get rid of nbd_negotiate_read and friends
Functions nbd_negotiate_{read,write,drop_sync} were introduced in
1a6245a5b, when nbd_rwv (was nbd_wr_sync) was working through
qemu_co_sendv_recvv (the path is nbd_wr_sync -> qemu_co_{recv/send} ->
qemu_co_send_recv -> qemu_co_sendv_recvv), which just yields, without
setting any handlers. But starting from ff82911cd nbd_rwv (was
nbd_wr_syncv) works through qio_channel_yield() which sets handlers, so
watchers are redundant in nbd_negotiate_{read,write,drop_sync}, then,
let's just use nbd_{read,write,drop} functions.
Functions nbd_{read,write,drop} has errp parameter, which is unused in
this patch. This will be fixed later.
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <20170602150150.258222-4-vsementsov@virtuozzo.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
2017-06-02 15:01:41 +00:00
|
|
|
if (nbd_write(client->ioc, buf, sizeof(buf), NULL) < 0) {
|
2012-08-23 12:57:11 +00:00
|
|
|
LOG("write failed");
|
|
|
|
goto fail;
|
|
|
|
}
|
|
|
|
} else {
|
nbd/server: get rid of nbd_negotiate_read and friends
Functions nbd_negotiate_{read,write,drop_sync} were introduced in
1a6245a5b, when nbd_rwv (was nbd_wr_sync) was working through
qemu_co_sendv_recvv (the path is nbd_wr_sync -> qemu_co_{recv/send} ->
qemu_co_send_recv -> qemu_co_sendv_recvv), which just yields, without
setting any handlers. But starting from ff82911cd nbd_rwv (was
nbd_wr_syncv) works through qio_channel_yield() which sets handlers, so
watchers are redundant in nbd_negotiate_{read,write,drop_sync}, then,
let's just use nbd_{read,write,drop} functions.
Functions nbd_{read,write,drop} has errp parameter, which is unused in
this patch. This will be fixed later.
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <20170602150150.258222-4-vsementsov@virtuozzo.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
2017-06-02 15:01:41 +00:00
|
|
|
if (nbd_write(client->ioc, buf, 18, NULL) < 0) {
|
2012-08-23 12:57:11 +00:00
|
|
|
LOG("write failed");
|
|
|
|
goto fail;
|
|
|
|
}
|
2016-01-14 08:41:03 +00:00
|
|
|
rc = nbd_negotiate_options(client);
|
2014-06-07 00:32:31 +00:00
|
|
|
if (rc != 0) {
|
2012-08-23 12:57:11 +00:00
|
|
|
LOG("option negotiation failed");
|
|
|
|
goto fail;
|
|
|
|
}
|
|
|
|
|
2016-05-11 22:39:35 +00:00
|
|
|
TRACE("advertising size %" PRIu64 " and flags %x",
|
|
|
|
client->exp->size, client->exp->nbdflags | myflags);
|
2016-02-04 10:27:55 +00:00
|
|
|
stq_be_p(buf + 18, client->exp->size);
|
|
|
|
stw_be_p(buf + 26, client->exp->nbdflags | myflags);
|
2016-10-14 18:33:14 +00:00
|
|
|
len = client->no_zeroes ? 10 : sizeof(buf) - 18;
|
nbd/server: get rid of nbd_negotiate_read and friends
Functions nbd_negotiate_{read,write,drop_sync} were introduced in
1a6245a5b, when nbd_rwv (was nbd_wr_sync) was working through
qemu_co_sendv_recvv (the path is nbd_wr_sync -> qemu_co_{recv/send} ->
qemu_co_send_recv -> qemu_co_sendv_recvv), which just yields, without
setting any handlers. But starting from ff82911cd nbd_rwv (was
nbd_wr_syncv) works through qio_channel_yield() which sets handlers, so
watchers are redundant in nbd_negotiate_{read,write,drop_sync}, then,
let's just use nbd_{read,write,drop} functions.
Functions nbd_{read,write,drop} has errp parameter, which is unused in
this patch. This will be fixed later.
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <20170602150150.258222-4-vsementsov@virtuozzo.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
2017-06-02 15:01:41 +00:00
|
|
|
if (nbd_write(client->ioc, buf + 18, len, NULL) < 0) {
|
2012-08-23 12:57:11 +00:00
|
|
|
LOG("write failed");
|
|
|
|
goto fail;
|
|
|
|
}
|
2011-02-22 15:44:51 +00:00
|
|
|
}
|
|
|
|
|
2011-11-22 10:06:26 +00:00
|
|
|
TRACE("Negotiation succeeded.");
|
2012-03-05 07:56:10 +00:00
|
|
|
rc = 0;
|
|
|
|
fail:
|
|
|
|
return rc;
|
2008-05-27 21:13:40 +00:00
|
|
|
}
|
|
|
|
|
2017-06-02 15:01:42 +00:00
|
|
|
static int nbd_receive_request(QIOChannel *ioc, NBDRequest *request)
|
2008-07-03 13:41:03 +00:00
|
|
|
{
|
2012-08-22 13:13:30 +00:00
|
|
|
uint8_t buf[NBD_REQUEST_SIZE];
|
2011-02-22 15:44:51 +00:00
|
|
|
uint32_t magic;
|
2017-06-02 15:01:42 +00:00
|
|
|
int ret;
|
2011-02-22 15:44:51 +00:00
|
|
|
|
2017-06-02 15:01:39 +00:00
|
|
|
ret = nbd_read(ioc, buf, sizeof(buf), NULL);
|
2012-03-05 07:56:10 +00:00
|
|
|
if (ret < 0) {
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
2011-02-22 15:44:51 +00:00
|
|
|
/* Request
|
|
|
|
[ 0 .. 3] magic (NBD_REQUEST_MAGIC)
|
2016-10-14 18:33:04 +00:00
|
|
|
[ 4 .. 5] flags (NBD_CMD_FLAG_FUA, ...)
|
|
|
|
[ 6 .. 7] type (NBD_CMD_READ, ...)
|
2011-02-22 15:44:51 +00:00
|
|
|
[ 8 .. 15] handle
|
|
|
|
[16 .. 23] from
|
|
|
|
[24 .. 27] len
|
|
|
|
*/
|
|
|
|
|
2016-06-10 15:00:36 +00:00
|
|
|
magic = ldl_be_p(buf);
|
2016-10-14 18:33:04 +00:00
|
|
|
request->flags = lduw_be_p(buf + 4);
|
|
|
|
request->type = lduw_be_p(buf + 6);
|
2016-06-10 15:00:36 +00:00
|
|
|
request->handle = ldq_be_p(buf + 8);
|
|
|
|
request->from = ldq_be_p(buf + 16);
|
|
|
|
request->len = ldl_be_p(buf + 24);
|
2011-02-22 15:44:51 +00:00
|
|
|
|
2016-10-14 18:33:04 +00:00
|
|
|
TRACE("Got request: { magic = 0x%" PRIx32 ", .flags = %" PRIx16
|
|
|
|
", .type = %" PRIx16 ", from = %" PRIu64 ", len = %" PRIu32 " }",
|
|
|
|
magic, request->flags, request->type, request->from, request->len);
|
2011-02-22 15:44:51 +00:00
|
|
|
|
|
|
|
if (magic != NBD_REQUEST_MAGIC) {
|
2016-05-11 22:39:35 +00:00
|
|
|
LOG("invalid magic (got 0x%" PRIx32 ")", magic);
|
2012-03-05 07:56:10 +00:00
|
|
|
return -EINVAL;
|
2011-02-22 15:44:51 +00:00
|
|
|
}
|
|
|
|
return 0;
|
2008-07-03 13:41:03 +00:00
|
|
|
}
|
|
|
|
|
2017-06-02 15:01:42 +00:00
|
|
|
static int nbd_send_reply(QIOChannel *ioc, NBDReply *reply)
|
2008-07-03 13:41:03 +00:00
|
|
|
{
|
2012-08-22 13:13:30 +00:00
|
|
|
uint8_t buf[NBD_REPLY_SIZE];
|
2011-02-22 15:44:51 +00:00
|
|
|
|
qemu-nbd: only send a limited number of errno codes on the wire
Right now, NBD includes potentially platform-specific error values in
the wire protocol.
Luckily, most common error values are more or less universal: in
particular, of all errno values <= 34 (up to ERANGE), they are all the
same on supported platforms except for 11 (which is EAGAIN on Windows and
Linux, but EDEADLK on Darwin and the *BSDs). So, in order to guarantee
some portability, only keep a handful of possible error codes and squash
everything else to EINVAL.
This patch defines a limited set of errno values that are valid for the
NBD protocol, and specifies recommendations for what error to return
in specific corner cases. The set of errno values is roughly based on
the errors listed in the read(2) and write(2) man pages, with some
exceptions:
- ENOMEM is added for servers that implement copy-on-write or other
formats that require dynamic allocation.
- EDQUOT is not part of the universal set of errors; it can be changed
to ENOSPC on the wire format.
- EFBIG is part of the universal set of errors, but it is also changed
to ENOSPC because it is pretty similar to ENOSPC or EDQUOT.
Incoming values will in general match system errno values, but not
on the Hurd which has different errno values (they have a "subsystem
code" equal to 0x10 in bits 24-31). The Hurd is probably not something
to which QEMU has been ported, but still do the right thing and
reverse-map the NBD errno values to the system errno values.
The corresponding patch to the NBD protocol description can be found at
http://article.gmane.org/gmane.linux.drivers.nbd.general/3154.
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
2015-05-07 15:25:10 +00:00
|
|
|
reply->error = system_errno_to_nbd_errno(reply->error);
|
|
|
|
|
2016-05-11 22:39:35 +00:00
|
|
|
TRACE("Sending response to client: { .error = %" PRId32
|
|
|
|
", handle = %" PRIu64 " }",
|
2016-04-06 03:35:04 +00:00
|
|
|
reply->error, reply->handle);
|
|
|
|
|
2011-02-22 15:44:51 +00:00
|
|
|
/* Reply
|
|
|
|
[ 0 .. 3] magic (NBD_REPLY_MAGIC)
|
|
|
|
[ 4 .. 7] error (0 == no error)
|
|
|
|
[ 7 .. 15] handle
|
|
|
|
*/
|
2016-02-04 10:27:55 +00:00
|
|
|
stl_be_p(buf, NBD_REPLY_MAGIC);
|
|
|
|
stl_be_p(buf + 4, reply->error);
|
|
|
|
stq_be_p(buf + 8, reply->handle);
|
2011-02-22 15:44:51 +00:00
|
|
|
|
2017-06-02 15:01:39 +00:00
|
|
|
return nbd_write(ioc, buf, sizeof(buf), NULL);
|
2008-07-03 13:41:03 +00:00
|
|
|
}
|
2008-05-27 21:13:40 +00:00
|
|
|
|
2011-09-19 13:25:40 +00:00
|
|
|
#define MAX_NBD_REQUESTS 16
|
|
|
|
|
2012-09-18 11:17:52 +00:00
|
|
|
void nbd_client_get(NBDClient *client)
|
2011-09-19 12:33:23 +00:00
|
|
|
{
|
|
|
|
client->refcount++;
|
|
|
|
}
|
|
|
|
|
2012-09-18 11:17:52 +00:00
|
|
|
void nbd_client_put(NBDClient *client)
|
2011-09-19 12:33:23 +00:00
|
|
|
{
|
|
|
|
if (--client->refcount == 0) {
|
2012-08-22 16:45:12 +00:00
|
|
|
/* The last reference should be dropped by client->close,
|
2015-02-06 21:06:16 +00:00
|
|
|
* which is called by client_close.
|
2012-08-22 16:45:12 +00:00
|
|
|
*/
|
|
|
|
assert(client->closing);
|
|
|
|
|
2017-02-13 13:52:24 +00:00
|
|
|
qio_channel_detach_aio_context(client->ioc);
|
2016-02-10 18:41:04 +00:00
|
|
|
object_unref(OBJECT(client->sioc));
|
|
|
|
object_unref(OBJECT(client->ioc));
|
2016-02-10 18:41:11 +00:00
|
|
|
if (client->tlscreds) {
|
|
|
|
object_unref(OBJECT(client->tlscreds));
|
|
|
|
}
|
|
|
|
g_free(client->tlsaclname);
|
2012-08-23 12:57:11 +00:00
|
|
|
if (client->exp) {
|
|
|
|
QTAILQ_REMOVE(&client->exp->clients, client, next);
|
|
|
|
nbd_export_put(client->exp);
|
|
|
|
}
|
2011-09-19 12:33:23 +00:00
|
|
|
g_free(client);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
nbd: Fix regression on resiliency to port scan
Back in qemu 2.5, qemu-nbd was immune to port probes (a transient
server would not quit, regardless of how many probe connections
came and went, until a connection actually negotiated). But we
broke that in commit ee7d7aa when removing the return value to
nbd_client_new(), although that patch also introduced a bug causing
an assertion failure on a client that fails negotiation. We then
made it worse during refactoring in commit 1a6245a (a segfault
before we could even assert); the (masked) assertion was cleaned
up in d3780c2 (still in 2.6), and just recently we finally fixed
the segfault ("nbd: Fully intialize client in case of failed
negotiation"). But that still means that ever since we added
TLS support to qemu-nbd, we have been vulnerable to an ill-timed
port-scan being able to cause a denial of service by taking down
qemu-nbd before a real client has a chance to connect.
Since negotiation is now handled asynchronously via coroutines,
we no longer have a synchronous point of return by re-adding a
return value to nbd_client_new(). So this patch instead wires
things up to pass the negotiation status through the close_fn
callback function.
Simple test across two terminals:
$ qemu-nbd -f raw -p 30001 file
$ nmap 127.0.0.1 -p 30001 && \
qemu-io -c 'r 0 512' -f raw nbd://localhost:30001
Note that this patch does not change what constitutes successful
negotiation (thus, a client must enter transmission phase before
that client can be considered as a reason to terminate the server
when the connection ends). Perhaps we may want to tweak things
in a later patch to also treat a client that uses NBD_OPT_ABORT
as being a 'successful' negotiation (the client correctly talked
the NBD protocol, and informed us it was not going to use our
export after all), but that's a discussion for another day.
Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1451614
Signed-off-by: Eric Blake <eblake@redhat.com>
Message-Id: <20170608222617.20376-1-eblake@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
2017-06-08 22:26:17 +00:00
|
|
|
static void client_close(NBDClient *client, bool negotiated)
|
2011-09-19 12:33:23 +00:00
|
|
|
{
|
2012-08-22 16:45:12 +00:00
|
|
|
if (client->closing) {
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
|
|
|
client->closing = true;
|
|
|
|
|
|
|
|
/* Force requests to finish. They will drop their own references,
|
|
|
|
* then we'll close the socket and free the NBDClient.
|
|
|
|
*/
|
2016-02-10 18:41:04 +00:00
|
|
|
qio_channel_shutdown(client->ioc, QIO_CHANNEL_SHUTDOWN_BOTH,
|
|
|
|
NULL);
|
2012-08-22 16:45:12 +00:00
|
|
|
|
|
|
|
/* Also tell the client, so that they release their reference. */
|
nbd: Fix regression on resiliency to port scan
Back in qemu 2.5, qemu-nbd was immune to port probes (a transient
server would not quit, regardless of how many probe connections
came and went, until a connection actually negotiated). But we
broke that in commit ee7d7aa when removing the return value to
nbd_client_new(), although that patch also introduced a bug causing
an assertion failure on a client that fails negotiation. We then
made it worse during refactoring in commit 1a6245a (a segfault
before we could even assert); the (masked) assertion was cleaned
up in d3780c2 (still in 2.6), and just recently we finally fixed
the segfault ("nbd: Fully intialize client in case of failed
negotiation"). But that still means that ever since we added
TLS support to qemu-nbd, we have been vulnerable to an ill-timed
port-scan being able to cause a denial of service by taking down
qemu-nbd before a real client has a chance to connect.
Since negotiation is now handled asynchronously via coroutines,
we no longer have a synchronous point of return by re-adding a
return value to nbd_client_new(). So this patch instead wires
things up to pass the negotiation status through the close_fn
callback function.
Simple test across two terminals:
$ qemu-nbd -f raw -p 30001 file
$ nmap 127.0.0.1 -p 30001 && \
qemu-io -c 'r 0 512' -f raw nbd://localhost:30001
Note that this patch does not change what constitutes successful
negotiation (thus, a client must enter transmission phase before
that client can be considered as a reason to terminate the server
when the connection ends). Perhaps we may want to tweak things
in a later patch to also treat a client that uses NBD_OPT_ABORT
as being a 'successful' negotiation (the client correctly talked
the NBD protocol, and informed us it was not going to use our
export after all), but that's a discussion for another day.
Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1451614
Signed-off-by: Eric Blake <eblake@redhat.com>
Message-Id: <20170608222617.20376-1-eblake@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
2017-06-08 22:26:17 +00:00
|
|
|
if (client->close_fn) {
|
|
|
|
client->close_fn(client, negotiated);
|
2011-09-19 12:33:23 +00:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2016-10-14 18:33:05 +00:00
|
|
|
static NBDRequestData *nbd_request_get(NBDClient *client)
|
2011-09-19 12:18:33 +00:00
|
|
|
{
|
2016-10-14 18:33:05 +00:00
|
|
|
NBDRequestData *req;
|
2011-10-07 14:47:56 +00:00
|
|
|
|
2011-09-19 13:25:40 +00:00
|
|
|
assert(client->nb_requests <= MAX_NBD_REQUESTS - 1);
|
|
|
|
client->nb_requests++;
|
|
|
|
|
2016-10-14 18:33:05 +00:00
|
|
|
req = g_new0(NBDRequestData, 1);
|
2011-10-07 14:47:56 +00:00
|
|
|
nbd_client_get(client);
|
|
|
|
req->client = client;
|
2011-09-19 12:18:33 +00:00
|
|
|
return req;
|
|
|
|
}
|
|
|
|
|
2016-10-14 18:33:05 +00:00
|
|
|
static void nbd_request_put(NBDRequestData *req)
|
2011-09-19 12:18:33 +00:00
|
|
|
{
|
2011-10-07 14:47:56 +00:00
|
|
|
NBDClient *client = req->client;
|
2013-05-02 12:23:07 +00:00
|
|
|
|
2013-05-02 12:23:08 +00:00
|
|
|
if (req->data) {
|
|
|
|
qemu_vfree(req->data);
|
|
|
|
}
|
2015-10-01 10:59:08 +00:00
|
|
|
g_free(req);
|
2013-05-02 12:23:07 +00:00
|
|
|
|
2014-06-20 19:57:32 +00:00
|
|
|
client->nb_requests--;
|
2017-02-13 13:52:24 +00:00
|
|
|
nbd_client_receive_next_request(client);
|
|
|
|
|
2011-10-07 14:47:56 +00:00
|
|
|
nbd_client_put(client);
|
2011-09-19 12:18:33 +00:00
|
|
|
}
|
|
|
|
|
2014-11-18 11:21:18 +00:00
|
|
|
static void blk_aio_attached(AioContext *ctx, void *opaque)
|
2014-06-20 19:57:34 +00:00
|
|
|
{
|
|
|
|
NBDExport *exp = opaque;
|
|
|
|
NBDClient *client;
|
|
|
|
|
|
|
|
TRACE("Export %s: Attaching clients to AIO context %p\n", exp->name, ctx);
|
|
|
|
|
|
|
|
exp->ctx = ctx;
|
|
|
|
|
|
|
|
QTAILQ_FOREACH(client, &exp->clients, next) {
|
2017-02-13 13:52:24 +00:00
|
|
|
qio_channel_attach_aio_context(client->ioc, ctx);
|
|
|
|
if (client->recv_coroutine) {
|
|
|
|
aio_co_schedule(ctx, client->recv_coroutine);
|
|
|
|
}
|
|
|
|
if (client->send_coroutine) {
|
|
|
|
aio_co_schedule(ctx, client->send_coroutine);
|
|
|
|
}
|
2014-06-20 19:57:34 +00:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2014-11-18 11:21:18 +00:00
|
|
|
static void blk_aio_detach(void *opaque)
|
2014-06-20 19:57:34 +00:00
|
|
|
{
|
|
|
|
NBDExport *exp = opaque;
|
|
|
|
NBDClient *client;
|
|
|
|
|
|
|
|
TRACE("Export %s: Detaching clients from AIO context %p\n", exp->name, exp->ctx);
|
|
|
|
|
|
|
|
QTAILQ_FOREACH(client, &exp->clients, next) {
|
2017-02-13 13:52:24 +00:00
|
|
|
qio_channel_detach_aio_context(client->ioc);
|
2014-06-20 19:57:34 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
exp->ctx = NULL;
|
|
|
|
}
|
|
|
|
|
2016-01-29 15:36:06 +00:00
|
|
|
static void nbd_eject_notifier(Notifier *n, void *data)
|
|
|
|
{
|
|
|
|
NBDExport *exp = container_of(n, NBDExport, eject_notifier);
|
|
|
|
nbd_export_close(exp);
|
|
|
|
}
|
|
|
|
|
2016-07-06 09:22:39 +00:00
|
|
|
NBDExport *nbd_export_new(BlockDriverState *bs, off_t dev_offset, off_t size,
|
2016-07-21 19:34:46 +00:00
|
|
|
uint16_t nbdflags, void (*close)(NBDExport *),
|
2016-07-06 09:22:39 +00:00
|
|
|
bool writethrough, BlockBackend *on_eject_blk,
|
2015-02-25 18:08:21 +00:00
|
|
|
Error **errp)
|
2011-09-19 12:03:37 +00:00
|
|
|
{
|
2016-07-06 09:22:39 +00:00
|
|
|
BlockBackend *blk;
|
2011-09-19 12:03:37 +00:00
|
|
|
NBDExport *exp = g_malloc0(sizeof(NBDExport));
|
2017-02-09 14:43:38 +00:00
|
|
|
uint64_t perm;
|
2017-01-13 18:02:32 +00:00
|
|
|
int ret;
|
2016-07-06 09:22:39 +00:00
|
|
|
|
2017-02-09 14:43:38 +00:00
|
|
|
/* Don't allow resize while the NBD server is running, otherwise we don't
|
|
|
|
* care what happens with the node. */
|
|
|
|
perm = BLK_PERM_CONSISTENT_READ;
|
|
|
|
if ((nbdflags & NBD_FLAG_READ_ONLY) == 0) {
|
|
|
|
perm |= BLK_PERM_WRITE;
|
|
|
|
}
|
|
|
|
blk = blk_new(perm, BLK_PERM_CONSISTENT_READ | BLK_PERM_WRITE_UNCHANGED |
|
|
|
|
BLK_PERM_WRITE | BLK_PERM_GRAPH_MOD);
|
2017-01-13 18:02:32 +00:00
|
|
|
ret = blk_insert_bs(blk, bs, errp);
|
|
|
|
if (ret < 0) {
|
|
|
|
goto fail;
|
|
|
|
}
|
2016-07-06 09:22:39 +00:00
|
|
|
blk_set_enable_write_cache(blk, !writethrough);
|
|
|
|
|
2012-09-18 11:26:25 +00:00
|
|
|
exp->refcount = 1;
|
2012-09-18 11:58:25 +00:00
|
|
|
QTAILQ_INIT(&exp->clients);
|
2014-11-18 11:21:18 +00:00
|
|
|
exp->blk = blk;
|
2011-09-19 12:03:37 +00:00
|
|
|
exp->dev_offset = dev_offset;
|
|
|
|
exp->nbdflags = nbdflags;
|
2015-02-25 18:08:21 +00:00
|
|
|
exp->size = size < 0 ? blk_getlength(blk) : size;
|
|
|
|
if (exp->size < 0) {
|
|
|
|
error_setg_errno(errp, -exp->size,
|
|
|
|
"Failed to determine the NBD export's length");
|
|
|
|
goto fail;
|
|
|
|
}
|
|
|
|
exp->size -= exp->size % BDRV_SECTOR_SIZE;
|
|
|
|
|
2012-09-18 11:59:03 +00:00
|
|
|
exp->close = close;
|
2014-11-18 11:21:18 +00:00
|
|
|
exp->ctx = blk_get_aio_context(blk);
|
|
|
|
blk_add_aio_context_notifier(blk, blk_aio_attached, blk_aio_detach, exp);
|
2016-01-29 15:36:06 +00:00
|
|
|
|
2016-07-06 09:22:39 +00:00
|
|
|
if (on_eject_blk) {
|
|
|
|
blk_ref(on_eject_blk);
|
|
|
|
exp->eject_notifier_blk = on_eject_blk;
|
|
|
|
exp->eject_notifier.notify = nbd_eject_notifier;
|
|
|
|
blk_add_remove_bs_notifier(on_eject_blk, &exp->eject_notifier);
|
|
|
|
}
|
2016-01-29 15:36:06 +00:00
|
|
|
|
2014-10-09 02:50:46 +00:00
|
|
|
/*
|
|
|
|
* NBD exports are used for non-shared storage migration. Make sure
|
2016-01-13 14:56:06 +00:00
|
|
|
* that BDRV_O_INACTIVE is cleared and the image is ready for write
|
2014-10-09 02:50:46 +00:00
|
|
|
* access since the export could be available before migration handover.
|
|
|
|
*/
|
2016-01-20 07:12:20 +00:00
|
|
|
aio_context_acquire(exp->ctx);
|
2014-11-18 11:21:18 +00:00
|
|
|
blk_invalidate_cache(blk, NULL);
|
2016-01-20 07:12:20 +00:00
|
|
|
aio_context_release(exp->ctx);
|
2011-09-19 12:03:37 +00:00
|
|
|
return exp;
|
2015-02-25 18:08:21 +00:00
|
|
|
|
|
|
|
fail:
|
2016-07-06 09:22:39 +00:00
|
|
|
blk_unref(blk);
|
2015-02-25 18:08:21 +00:00
|
|
|
g_free(exp);
|
|
|
|
return NULL;
|
2011-09-19 12:03:37 +00:00
|
|
|
}
|
|
|
|
|
2012-08-22 13:59:23 +00:00
|
|
|
NBDExport *nbd_export_find(const char *name)
|
|
|
|
{
|
|
|
|
NBDExport *exp;
|
|
|
|
QTAILQ_FOREACH(exp, &exports, next) {
|
|
|
|
if (strcmp(name, exp->name) == 0) {
|
|
|
|
return exp;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
|
|
|
void nbd_export_set_name(NBDExport *exp, const char *name)
|
|
|
|
{
|
|
|
|
if (exp->name == name) {
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
|
|
|
nbd_export_get(exp);
|
|
|
|
if (exp->name != NULL) {
|
|
|
|
g_free(exp->name);
|
|
|
|
exp->name = NULL;
|
|
|
|
QTAILQ_REMOVE(&exports, exp, next);
|
|
|
|
nbd_export_put(exp);
|
|
|
|
}
|
|
|
|
if (name != NULL) {
|
|
|
|
nbd_export_get(exp);
|
|
|
|
exp->name = g_strdup(name);
|
|
|
|
QTAILQ_INSERT_TAIL(&exports, exp, next);
|
|
|
|
}
|
|
|
|
nbd_export_put(exp);
|
|
|
|
}
|
|
|
|
|
2016-10-14 18:33:03 +00:00
|
|
|
void nbd_export_set_description(NBDExport *exp, const char *description)
|
|
|
|
{
|
|
|
|
g_free(exp->description);
|
|
|
|
exp->description = g_strdup(description);
|
|
|
|
}
|
|
|
|
|
2011-09-19 12:03:37 +00:00
|
|
|
void nbd_export_close(NBDExport *exp)
|
|
|
|
{
|
2012-09-18 11:58:25 +00:00
|
|
|
NBDClient *client, *next;
|
2012-09-18 11:26:25 +00:00
|
|
|
|
2012-09-18 11:58:25 +00:00
|
|
|
nbd_export_get(exp);
|
|
|
|
QTAILQ_FOREACH_SAFE(client, &exp->clients, next, next) {
|
nbd: Fix regression on resiliency to port scan
Back in qemu 2.5, qemu-nbd was immune to port probes (a transient
server would not quit, regardless of how many probe connections
came and went, until a connection actually negotiated). But we
broke that in commit ee7d7aa when removing the return value to
nbd_client_new(), although that patch also introduced a bug causing
an assertion failure on a client that fails negotiation. We then
made it worse during refactoring in commit 1a6245a (a segfault
before we could even assert); the (masked) assertion was cleaned
up in d3780c2 (still in 2.6), and just recently we finally fixed
the segfault ("nbd: Fully intialize client in case of failed
negotiation"). But that still means that ever since we added
TLS support to qemu-nbd, we have been vulnerable to an ill-timed
port-scan being able to cause a denial of service by taking down
qemu-nbd before a real client has a chance to connect.
Since negotiation is now handled asynchronously via coroutines,
we no longer have a synchronous point of return by re-adding a
return value to nbd_client_new(). So this patch instead wires
things up to pass the negotiation status through the close_fn
callback function.
Simple test across two terminals:
$ qemu-nbd -f raw -p 30001 file
$ nmap 127.0.0.1 -p 30001 && \
qemu-io -c 'r 0 512' -f raw nbd://localhost:30001
Note that this patch does not change what constitutes successful
negotiation (thus, a client must enter transmission phase before
that client can be considered as a reason to terminate the server
when the connection ends). Perhaps we may want to tweak things
in a later patch to also treat a client that uses NBD_OPT_ABORT
as being a 'successful' negotiation (the client correctly talked
the NBD protocol, and informed us it was not going to use our
export after all), but that's a discussion for another day.
Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1451614
Signed-off-by: Eric Blake <eblake@redhat.com>
Message-Id: <20170608222617.20376-1-eblake@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
2017-06-08 22:26:17 +00:00
|
|
|
client_close(client, true);
|
2012-09-18 11:58:25 +00:00
|
|
|
}
|
2012-09-18 12:31:44 +00:00
|
|
|
nbd_export_set_name(exp, NULL);
|
2016-10-14 18:33:03 +00:00
|
|
|
nbd_export_set_description(exp, NULL);
|
2012-09-18 11:58:25 +00:00
|
|
|
nbd_export_put(exp);
|
2012-09-18 11:26:25 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
void nbd_export_get(NBDExport *exp)
|
|
|
|
{
|
|
|
|
assert(exp->refcount > 0);
|
|
|
|
exp->refcount++;
|
|
|
|
}
|
|
|
|
|
|
|
|
void nbd_export_put(NBDExport *exp)
|
|
|
|
{
|
|
|
|
assert(exp->refcount > 0);
|
|
|
|
if (exp->refcount == 1) {
|
|
|
|
nbd_export_close(exp);
|
2011-09-19 12:18:33 +00:00
|
|
|
}
|
|
|
|
|
2012-09-18 11:26:25 +00:00
|
|
|
if (--exp->refcount == 0) {
|
2012-08-22 13:59:23 +00:00
|
|
|
assert(exp->name == NULL);
|
2016-10-14 18:33:03 +00:00
|
|
|
assert(exp->description == NULL);
|
2012-08-22 13:59:23 +00:00
|
|
|
|
2012-09-18 11:59:03 +00:00
|
|
|
if (exp->close) {
|
|
|
|
exp->close(exp);
|
|
|
|
}
|
|
|
|
|
2015-09-16 08:35:46 +00:00
|
|
|
if (exp->blk) {
|
2016-07-06 09:22:39 +00:00
|
|
|
if (exp->eject_notifier_blk) {
|
|
|
|
notifier_remove(&exp->eject_notifier);
|
|
|
|
blk_unref(exp->eject_notifier_blk);
|
|
|
|
}
|
2015-09-16 08:35:46 +00:00
|
|
|
blk_remove_aio_context_notifier(exp->blk, blk_aio_attached,
|
|
|
|
blk_aio_detach, exp);
|
|
|
|
blk_unref(exp->blk);
|
|
|
|
exp->blk = NULL;
|
|
|
|
}
|
|
|
|
|
2012-09-18 11:26:25 +00:00
|
|
|
g_free(exp);
|
|
|
|
}
|
2011-09-19 12:03:37 +00:00
|
|
|
}
|
|
|
|
|
2014-11-18 11:21:17 +00:00
|
|
|
BlockBackend *nbd_export_get_blockdev(NBDExport *exp)
|
2012-09-18 12:31:44 +00:00
|
|
|
{
|
2014-11-18 11:21:18 +00:00
|
|
|
return exp->blk;
|
2012-09-18 12:31:44 +00:00
|
|
|
}
|
|
|
|
|
2012-08-22 13:59:23 +00:00
|
|
|
void nbd_export_close_all(void)
|
|
|
|
{
|
|
|
|
NBDExport *exp, *next;
|
|
|
|
|
|
|
|
QTAILQ_FOREACH_SAFE(exp, &exports, next, next) {
|
|
|
|
nbd_export_close(exp);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2017-06-02 15:01:42 +00:00
|
|
|
static int nbd_co_send_reply(NBDRequestData *req, NBDReply *reply, int len)
|
2011-09-19 12:25:30 +00:00
|
|
|
{
|
2011-10-07 14:47:56 +00:00
|
|
|
NBDClient *client = req->client;
|
2017-06-02 15:01:42 +00:00
|
|
|
int rc, ret;
|
2011-09-19 12:25:30 +00:00
|
|
|
|
2016-02-10 18:41:04 +00:00
|
|
|
g_assert(qemu_in_coroutine());
|
2011-09-19 13:19:27 +00:00
|
|
|
qemu_co_mutex_lock(&client->send_lock);
|
|
|
|
client->send_coroutine = qemu_coroutine_self();
|
|
|
|
|
2011-09-19 12:25:30 +00:00
|
|
|
if (!len) {
|
2016-02-10 18:41:04 +00:00
|
|
|
rc = nbd_send_reply(client->ioc, reply);
|
2011-09-19 12:25:30 +00:00
|
|
|
} else {
|
2016-02-10 18:41:04 +00:00
|
|
|
qio_channel_set_cork(client->ioc, true);
|
|
|
|
rc = nbd_send_reply(client->ioc, reply);
|
2012-03-07 10:05:34 +00:00
|
|
|
if (rc >= 0) {
|
2017-06-02 15:01:39 +00:00
|
|
|
ret = nbd_write(client->ioc, req->data, len, NULL);
|
nbd: read_sync and friends: return 0 on success
functions read_sync, drop_sync, write_sync, and also
nbd_negotiate_write, nbd_negotiate_read, nbd_negotiate_drop_sync
returns number of processed bytes. But what this number can be,
except requested number of bytes?
Actually, underlying nbd_wr_syncv function returns a value >= 0 and
!= requested_bytes only on eof on read operation. So, firstly, it is
impossible on write (let's add an assert) and on read it actually
means, that communication is broken (except nbd_receive_reply, see
below).
Most of callers operate like this:
if (func(..., size) != size) {
/* error path */
}
, i.e.:
1. They are not interested in partial success
2. Extra duplications in code (especially bad are duplications of
magic numbers)
3. User doesn't see actual error message, as return code is lost.
(this patch doesn't fix this point, but it makes fixing easier)
Several callers handles ret >= 0 and != requested-size separately, by
just returning EINVAL in this case. This patch makes read_sync and
friends return EINVAL in this case, so final behavior is the same.
And only one caller - nbd_receive_reply() does something not so
obvious. It returns EINVAL for ret > 0 and != requested-size, like
previous group, but for ret == 0 it returns 0. The only caller of
nbd_receive_reply() - nbd_read_reply_entry() handles ret == 0 in the
same way as ret < 0, so for now it doesn't matter. However, in
following commits error path handling will be improved and we'll need
to distinguish success from fail in this case too. So, this patch adds
separate helper for this case - read_sync_eof.
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Message-Id: <20170516094533.6160-3-vsementsov@virtuozzo.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
2017-05-16 09:45:30 +00:00
|
|
|
if (ret < 0) {
|
2012-03-05 07:56:10 +00:00
|
|
|
rc = -EIO;
|
2011-09-19 12:25:30 +00:00
|
|
|
}
|
|
|
|
}
|
2016-02-10 18:41:04 +00:00
|
|
|
qio_channel_set_cork(client->ioc, false);
|
2011-09-19 12:25:30 +00:00
|
|
|
}
|
2011-09-19 13:19:27 +00:00
|
|
|
|
|
|
|
client->send_coroutine = NULL;
|
|
|
|
qemu_co_mutex_unlock(&client->send_lock);
|
2011-09-19 12:25:30 +00:00
|
|
|
return rc;
|
|
|
|
}
|
|
|
|
|
2016-05-11 22:39:37 +00:00
|
|
|
/* Collect a client request. Return 0 if request looks valid, -EAGAIN
|
|
|
|
* to keep trying the collection, -EIO to drop connection right away,
|
|
|
|
* and any other negative value to report an error to the client
|
|
|
|
* (although the caller may still need to disconnect after reporting
|
|
|
|
* the error). */
|
2017-06-02 15:01:42 +00:00
|
|
|
static int nbd_co_receive_request(NBDRequestData *req, NBDRequest *request)
|
2011-09-19 13:07:54 +00:00
|
|
|
{
|
2011-10-07 14:47:56 +00:00
|
|
|
NBDClient *client = req->client;
|
2017-06-02 15:01:42 +00:00
|
|
|
int rc;
|
2011-09-19 13:07:54 +00:00
|
|
|
|
2016-02-10 18:41:04 +00:00
|
|
|
g_assert(qemu_in_coroutine());
|
2017-02-13 13:52:24 +00:00
|
|
|
assert(client->recv_coroutine == qemu_coroutine_self());
|
2016-02-10 18:41:04 +00:00
|
|
|
rc = nbd_receive_request(client->ioc, request);
|
2012-03-05 08:10:35 +00:00
|
|
|
if (rc < 0) {
|
|
|
|
if (rc != -EAGAIN) {
|
|
|
|
rc = -EIO;
|
|
|
|
}
|
2011-09-19 13:07:54 +00:00
|
|
|
goto out;
|
|
|
|
}
|
|
|
|
|
2016-05-11 22:39:37 +00:00
|
|
|
TRACE("Decoding type");
|
|
|
|
|
2016-10-14 18:33:04 +00:00
|
|
|
if (request->type != NBD_CMD_WRITE) {
|
2016-05-11 22:39:37 +00:00
|
|
|
/* No payload, we are ready to read the next request. */
|
|
|
|
req->complete = true;
|
|
|
|
}
|
|
|
|
|
2016-10-14 18:33:04 +00:00
|
|
|
if (request->type == NBD_CMD_DISC) {
|
2016-05-11 22:39:37 +00:00
|
|
|
/* Special case: we're going to disconnect without a reply,
|
|
|
|
* whether or not flags, from, or len are bogus */
|
|
|
|
TRACE("Request type is DISCONNECT");
|
|
|
|
rc = -EIO;
|
|
|
|
goto out;
|
|
|
|
}
|
|
|
|
|
|
|
|
/* Check for sanity in the parameters, part 1. Defer as many
|
|
|
|
* checks as possible until after reading any NBD_CMD_WRITE
|
|
|
|
* payload, so we can try and keep the connection alive. */
|
2011-09-19 13:07:54 +00:00
|
|
|
if ((request->from + request->len) < request->from) {
|
2016-05-11 22:39:37 +00:00
|
|
|
LOG("integer overflow detected, you're probably being attacked");
|
2011-09-19 13:07:54 +00:00
|
|
|
rc = -EINVAL;
|
|
|
|
goto out;
|
|
|
|
}
|
|
|
|
|
2016-10-14 18:33:04 +00:00
|
|
|
if (request->type == NBD_CMD_READ || request->type == NBD_CMD_WRITE) {
|
2016-01-07 13:32:42 +00:00
|
|
|
if (request->len > NBD_MAX_BUFFER_SIZE) {
|
2016-05-11 22:39:35 +00:00
|
|
|
LOG("len (%" PRIu32" ) is larger than max len (%u)",
|
2016-01-07 13:32:42 +00:00
|
|
|
request->len, NBD_MAX_BUFFER_SIZE);
|
|
|
|
rc = -EINVAL;
|
|
|
|
goto out;
|
|
|
|
}
|
|
|
|
|
2016-01-07 13:34:13 +00:00
|
|
|
req->data = blk_try_blockalign(client->exp->blk, request->len);
|
|
|
|
if (req->data == NULL) {
|
|
|
|
rc = -ENOMEM;
|
|
|
|
goto out;
|
|
|
|
}
|
2013-05-02 12:23:08 +00:00
|
|
|
}
|
2016-10-14 18:33:04 +00:00
|
|
|
if (request->type == NBD_CMD_WRITE) {
|
2016-05-11 22:39:35 +00:00
|
|
|
TRACE("Reading %" PRIu32 " byte(s)", request->len);
|
2011-09-19 13:07:54 +00:00
|
|
|
|
2017-06-02 15:01:39 +00:00
|
|
|
if (nbd_read(client->ioc, req->data, request->len, NULL) < 0) {
|
2011-09-19 13:07:54 +00:00
|
|
|
LOG("reading from socket failed");
|
|
|
|
rc = -EIO;
|
|
|
|
goto out;
|
|
|
|
}
|
2016-05-11 22:39:37 +00:00
|
|
|
req->complete = true;
|
2011-09-19 13:07:54 +00:00
|
|
|
}
|
2016-05-11 22:39:37 +00:00
|
|
|
|
|
|
|
/* Sanity checks, part 2. */
|
|
|
|
if (request->from + request->len > client->exp->size) {
|
|
|
|
LOG("operation past EOF; From: %" PRIu64 ", Len: %" PRIu32
|
|
|
|
", Size: %" PRIu64, request->from, request->len,
|
|
|
|
(uint64_t)client->exp->size);
|
2016-10-14 18:33:04 +00:00
|
|
|
rc = request->type == NBD_CMD_WRITE ? -ENOSPC : -EINVAL;
|
2016-05-11 22:39:37 +00:00
|
|
|
goto out;
|
|
|
|
}
|
2016-10-14 18:33:17 +00:00
|
|
|
if (request->flags & ~(NBD_CMD_FLAG_FUA | NBD_CMD_FLAG_NO_HOLE)) {
|
2016-10-14 18:33:04 +00:00
|
|
|
LOG("unsupported flags (got 0x%x)", request->flags);
|
2016-07-21 19:34:45 +00:00
|
|
|
rc = -EINVAL;
|
|
|
|
goto out;
|
2016-05-11 22:39:38 +00:00
|
|
|
}
|
2016-10-14 18:33:17 +00:00
|
|
|
if (request->type != NBD_CMD_WRITE_ZEROES &&
|
|
|
|
(request->flags & NBD_CMD_FLAG_NO_HOLE)) {
|
|
|
|
LOG("unexpected flags (got 0x%x)", request->flags);
|
|
|
|
rc = -EINVAL;
|
|
|
|
goto out;
|
|
|
|
}
|
2016-05-11 22:39:37 +00:00
|
|
|
|
2011-09-19 13:07:54 +00:00
|
|
|
rc = 0;
|
|
|
|
|
|
|
|
out:
|
2011-09-19 13:19:27 +00:00
|
|
|
client->recv_coroutine = NULL;
|
2017-02-13 13:52:24 +00:00
|
|
|
nbd_client_receive_next_request(client);
|
2014-06-20 19:57:32 +00:00
|
|
|
|
2011-09-19 13:07:54 +00:00
|
|
|
return rc;
|
|
|
|
}
|
|
|
|
|
2017-02-13 13:52:24 +00:00
|
|
|
/* Owns a reference to the NBDClient passed as opaque. */
|
|
|
|
static coroutine_fn void nbd_trip(void *opaque)
|
2008-07-03 13:41:03 +00:00
|
|
|
{
|
2011-09-19 13:19:27 +00:00
|
|
|
NBDClient *client = opaque;
|
2011-09-19 12:33:23 +00:00
|
|
|
NBDExport *exp = client->exp;
|
2016-10-14 18:33:05 +00:00
|
|
|
NBDRequestData *req;
|
2017-02-13 13:52:24 +00:00
|
|
|
NBDRequest request = { 0 }; /* GCC thinks it can be used uninitialized */
|
2016-10-14 18:33:07 +00:00
|
|
|
NBDReply reply;
|
2017-06-02 15:01:42 +00:00
|
|
|
int ret;
|
2016-05-11 22:39:34 +00:00
|
|
|
int flags;
|
2011-02-22 15:44:51 +00:00
|
|
|
|
|
|
|
TRACE("Reading request.");
|
2012-08-22 16:45:12 +00:00
|
|
|
if (client->closing) {
|
2017-02-13 13:52:24 +00:00
|
|
|
nbd_client_put(client);
|
2012-08-22 16:45:12 +00:00
|
|
|
return;
|
|
|
|
}
|
2011-02-22 15:44:51 +00:00
|
|
|
|
2012-08-22 16:45:12 +00:00
|
|
|
req = nbd_request_get(client);
|
2011-09-19 13:19:27 +00:00
|
|
|
ret = nbd_co_receive_request(req, &request);
|
2012-03-05 08:10:35 +00:00
|
|
|
if (ret == -EAGAIN) {
|
|
|
|
goto done;
|
|
|
|
}
|
2011-09-19 13:07:54 +00:00
|
|
|
if (ret == -EIO) {
|
2011-09-19 12:18:33 +00:00
|
|
|
goto out;
|
2011-09-19 13:07:54 +00:00
|
|
|
}
|
2011-02-22 15:44:51 +00:00
|
|
|
|
2011-09-19 14:04:36 +00:00
|
|
|
reply.handle = request.handle;
|
|
|
|
reply.error = 0;
|
|
|
|
|
2011-09-19 13:07:54 +00:00
|
|
|
if (ret < 0) {
|
|
|
|
reply.error = -ret;
|
|
|
|
goto error_reply;
|
2011-02-22 15:44:51 +00:00
|
|
|
}
|
|
|
|
|
2015-09-16 08:35:46 +00:00
|
|
|
if (client->closing) {
|
|
|
|
/*
|
|
|
|
* The client may be closed when we are blocked in
|
|
|
|
* nbd_co_receive_request()
|
|
|
|
*/
|
|
|
|
goto done;
|
|
|
|
}
|
|
|
|
|
2016-10-14 18:33:04 +00:00
|
|
|
switch (request.type) {
|
2011-02-22 15:44:51 +00:00
|
|
|
case NBD_CMD_READ:
|
|
|
|
TRACE("Request type is READ");
|
|
|
|
|
2016-10-14 18:33:04 +00:00
|
|
|
/* XXX: NBD Protocol only documents use of FUA with WRITE */
|
|
|
|
if (request.flags & NBD_CMD_FLAG_FUA) {
|
2014-11-18 11:21:18 +00:00
|
|
|
ret = blk_co_flush(exp->blk);
|
2012-04-19 09:59:11 +00:00
|
|
|
if (ret < 0) {
|
|
|
|
LOG("flush failed");
|
|
|
|
reply.error = -ret;
|
|
|
|
goto error_reply;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2016-04-21 14:42:30 +00:00
|
|
|
ret = blk_pread(exp->blk, request.from + exp->dev_offset,
|
|
|
|
req->data, request.len);
|
2011-09-13 15:27:45 +00:00
|
|
|
if (ret < 0) {
|
2011-02-22 15:44:51 +00:00
|
|
|
LOG("reading from file failed");
|
2011-09-13 15:27:45 +00:00
|
|
|
reply.error = -ret;
|
2011-09-19 14:04:36 +00:00
|
|
|
goto error_reply;
|
2011-02-22 15:44:51 +00:00
|
|
|
}
|
|
|
|
|
2016-05-11 22:39:35 +00:00
|
|
|
TRACE("Read %" PRIu32" byte(s)", request.len);
|
2011-09-19 13:19:27 +00:00
|
|
|
if (nbd_co_send_reply(req, &reply, request.len) < 0)
|
2011-09-19 12:18:33 +00:00
|
|
|
goto out;
|
2011-02-22 15:44:51 +00:00
|
|
|
break;
|
|
|
|
case NBD_CMD_WRITE:
|
|
|
|
TRACE("Request type is WRITE");
|
|
|
|
|
2011-09-19 12:03:37 +00:00
|
|
|
if (exp->nbdflags & NBD_FLAG_READ_ONLY) {
|
2011-02-22 15:44:51 +00:00
|
|
|
TRACE("Server is read-only, return error");
|
2011-09-19 14:04:36 +00:00
|
|
|
reply.error = EROFS;
|
|
|
|
goto error_reply;
|
|
|
|
}
|
|
|
|
|
|
|
|
TRACE("Writing to device");
|
|
|
|
|
2016-05-11 22:39:34 +00:00
|
|
|
flags = 0;
|
2016-10-14 18:33:04 +00:00
|
|
|
if (request.flags & NBD_CMD_FLAG_FUA) {
|
2016-05-11 22:39:34 +00:00
|
|
|
flags |= BDRV_REQ_FUA;
|
|
|
|
}
|
2016-04-21 14:42:30 +00:00
|
|
|
ret = blk_pwrite(exp->blk, request.from + exp->dev_offset,
|
2016-05-11 22:39:34 +00:00
|
|
|
req->data, request.len, flags);
|
2011-09-19 14:04:36 +00:00
|
|
|
if (ret < 0) {
|
|
|
|
LOG("writing to file failed");
|
|
|
|
reply.error = -ret;
|
|
|
|
goto error_reply;
|
|
|
|
}
|
2011-02-22 15:44:51 +00:00
|
|
|
|
2016-10-14 18:33:17 +00:00
|
|
|
if (nbd_co_send_reply(req, &reply, 0) < 0) {
|
|
|
|
goto out;
|
|
|
|
}
|
|
|
|
break;
|
|
|
|
|
|
|
|
case NBD_CMD_WRITE_ZEROES:
|
|
|
|
TRACE("Request type is WRITE_ZEROES");
|
|
|
|
|
|
|
|
if (exp->nbdflags & NBD_FLAG_READ_ONLY) {
|
|
|
|
TRACE("Server is read-only, return error");
|
|
|
|
reply.error = EROFS;
|
|
|
|
goto error_reply;
|
|
|
|
}
|
|
|
|
|
|
|
|
TRACE("Writing to device");
|
|
|
|
|
|
|
|
flags = 0;
|
|
|
|
if (request.flags & NBD_CMD_FLAG_FUA) {
|
|
|
|
flags |= BDRV_REQ_FUA;
|
|
|
|
}
|
|
|
|
if (!(request.flags & NBD_CMD_FLAG_NO_HOLE)) {
|
|
|
|
flags |= BDRV_REQ_MAY_UNMAP;
|
|
|
|
}
|
|
|
|
ret = blk_pwrite_zeroes(exp->blk, request.from + exp->dev_offset,
|
|
|
|
request.len, flags);
|
|
|
|
if (ret < 0) {
|
|
|
|
LOG("writing to file failed");
|
|
|
|
reply.error = -ret;
|
|
|
|
goto error_reply;
|
|
|
|
}
|
|
|
|
|
2012-03-07 10:05:34 +00:00
|
|
|
if (nbd_co_send_reply(req, &reply, 0) < 0) {
|
2011-09-19 12:18:33 +00:00
|
|
|
goto out;
|
2012-03-07 10:05:34 +00:00
|
|
|
}
|
2011-02-22 15:44:51 +00:00
|
|
|
break;
|
2016-05-11 22:39:37 +00:00
|
|
|
|
2011-02-22 15:44:51 +00:00
|
|
|
case NBD_CMD_DISC:
|
2016-05-11 22:39:37 +00:00
|
|
|
/* unreachable, thanks to special case in nbd_co_receive_request() */
|
|
|
|
abort();
|
|
|
|
|
2011-10-21 11:17:14 +00:00
|
|
|
case NBD_CMD_FLUSH:
|
|
|
|
TRACE("Request type is FLUSH");
|
|
|
|
|
2014-11-18 11:21:18 +00:00
|
|
|
ret = blk_co_flush(exp->blk);
|
2011-10-21 11:17:14 +00:00
|
|
|
if (ret < 0) {
|
|
|
|
LOG("flush failed");
|
|
|
|
reply.error = -ret;
|
|
|
|
}
|
2012-03-07 10:05:34 +00:00
|
|
|
if (nbd_co_send_reply(req, &reply, 0) < 0) {
|
2011-09-19 12:18:33 +00:00
|
|
|
goto out;
|
2012-03-07 10:05:34 +00:00
|
|
|
}
|
2011-10-21 11:17:14 +00:00
|
|
|
break;
|
|
|
|
case NBD_CMD_TRIM:
|
|
|
|
TRACE("Request type is TRIM");
|
2016-07-15 23:22:54 +00:00
|
|
|
ret = blk_co_pdiscard(exp->blk, request.from + exp->dev_offset,
|
|
|
|
request.len);
|
|
|
|
if (ret < 0) {
|
|
|
|
LOG("discard failed");
|
|
|
|
reply.error = -ret;
|
2011-10-21 11:17:14 +00:00
|
|
|
}
|
2012-03-07 10:05:34 +00:00
|
|
|
if (nbd_co_send_reply(req, &reply, 0) < 0) {
|
2011-09-19 12:18:33 +00:00
|
|
|
goto out;
|
2012-03-07 10:05:34 +00:00
|
|
|
}
|
2011-10-21 11:17:14 +00:00
|
|
|
break;
|
2011-02-22 15:44:51 +00:00
|
|
|
default:
|
2016-05-11 22:39:35 +00:00
|
|
|
LOG("invalid request type (%" PRIu32 ") received", request.type);
|
2015-02-12 06:21:51 +00:00
|
|
|
reply.error = EINVAL;
|
2011-09-19 14:04:36 +00:00
|
|
|
error_reply:
|
2016-05-11 22:39:37 +00:00
|
|
|
/* We must disconnect after NBD_CMD_WRITE if we did not
|
|
|
|
* read the payload.
|
|
|
|
*/
|
|
|
|
if (nbd_co_send_reply(req, &reply, 0) < 0 || !req->complete) {
|
2011-09-19 12:18:33 +00:00
|
|
|
goto out;
|
2012-03-07 10:05:34 +00:00
|
|
|
}
|
2011-09-19 14:04:36 +00:00
|
|
|
break;
|
2011-02-22 15:44:51 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
TRACE("Request/Reply complete");
|
|
|
|
|
2012-03-05 08:10:35 +00:00
|
|
|
done:
|
2011-09-19 13:19:27 +00:00
|
|
|
nbd_request_put(req);
|
2017-02-13 13:52:24 +00:00
|
|
|
nbd_client_put(client);
|
2011-09-19 13:19:27 +00:00
|
|
|
return;
|
|
|
|
|
2011-09-19 12:18:33 +00:00
|
|
|
out:
|
2011-10-07 14:47:56 +00:00
|
|
|
nbd_request_put(req);
|
nbd: Fix regression on resiliency to port scan
Back in qemu 2.5, qemu-nbd was immune to port probes (a transient
server would not quit, regardless of how many probe connections
came and went, until a connection actually negotiated). But we
broke that in commit ee7d7aa when removing the return value to
nbd_client_new(), although that patch also introduced a bug causing
an assertion failure on a client that fails negotiation. We then
made it worse during refactoring in commit 1a6245a (a segfault
before we could even assert); the (masked) assertion was cleaned
up in d3780c2 (still in 2.6), and just recently we finally fixed
the segfault ("nbd: Fully intialize client in case of failed
negotiation"). But that still means that ever since we added
TLS support to qemu-nbd, we have been vulnerable to an ill-timed
port-scan being able to cause a denial of service by taking down
qemu-nbd before a real client has a chance to connect.
Since negotiation is now handled asynchronously via coroutines,
we no longer have a synchronous point of return by re-adding a
return value to nbd_client_new(). So this patch instead wires
things up to pass the negotiation status through the close_fn
callback function.
Simple test across two terminals:
$ qemu-nbd -f raw -p 30001 file
$ nmap 127.0.0.1 -p 30001 && \
qemu-io -c 'r 0 512' -f raw nbd://localhost:30001
Note that this patch does not change what constitutes successful
negotiation (thus, a client must enter transmission phase before
that client can be considered as a reason to terminate the server
when the connection ends). Perhaps we may want to tweak things
in a later patch to also treat a client that uses NBD_OPT_ABORT
as being a 'successful' negotiation (the client correctly talked
the NBD protocol, and informed us it was not going to use our
export after all), but that's a discussion for another day.
Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1451614
Signed-off-by: Eric Blake <eblake@redhat.com>
Message-Id: <20170608222617.20376-1-eblake@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
2017-06-08 22:26:17 +00:00
|
|
|
client_close(client, true);
|
2017-02-13 13:52:24 +00:00
|
|
|
nbd_client_put(client);
|
2008-05-27 21:13:40 +00:00
|
|
|
}
|
2011-09-19 12:03:37 +00:00
|
|
|
|
2017-02-13 13:52:24 +00:00
|
|
|
static void nbd_client_receive_next_request(NBDClient *client)
|
2014-06-20 19:57:32 +00:00
|
|
|
{
|
2017-02-13 13:52:24 +00:00
|
|
|
if (!client->recv_coroutine && client->nb_requests < MAX_NBD_REQUESTS) {
|
|
|
|
nbd_client_get(client);
|
|
|
|
client->recv_coroutine = qemu_coroutine_create(nbd_trip, client);
|
|
|
|
aio_co_schedule(client->exp->ctx, client->recv_coroutine);
|
2014-06-20 19:57:32 +00:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2016-01-14 08:41:03 +00:00
|
|
|
static coroutine_fn void nbd_co_client_start(void *opaque)
|
|
|
|
{
|
|
|
|
NBDClientNewData *data = opaque;
|
|
|
|
NBDClient *client = data->client;
|
|
|
|
NBDExport *exp = client->exp;
|
|
|
|
|
|
|
|
if (exp) {
|
|
|
|
nbd_export_get(exp);
|
2017-05-27 03:04:21 +00:00
|
|
|
QTAILQ_INSERT_TAIL(&exp->clients, client, next);
|
2016-01-14 08:41:03 +00:00
|
|
|
}
|
2017-05-27 03:04:21 +00:00
|
|
|
qemu_co_mutex_init(&client->send_lock);
|
|
|
|
|
2016-01-14 08:41:03 +00:00
|
|
|
if (nbd_negotiate(data)) {
|
nbd: Fix regression on resiliency to port scan
Back in qemu 2.5, qemu-nbd was immune to port probes (a transient
server would not quit, regardless of how many probe connections
came and went, until a connection actually negotiated). But we
broke that in commit ee7d7aa when removing the return value to
nbd_client_new(), although that patch also introduced a bug causing
an assertion failure on a client that fails negotiation. We then
made it worse during refactoring in commit 1a6245a (a segfault
before we could even assert); the (masked) assertion was cleaned
up in d3780c2 (still in 2.6), and just recently we finally fixed
the segfault ("nbd: Fully intialize client in case of failed
negotiation"). But that still means that ever since we added
TLS support to qemu-nbd, we have been vulnerable to an ill-timed
port-scan being able to cause a denial of service by taking down
qemu-nbd before a real client has a chance to connect.
Since negotiation is now handled asynchronously via coroutines,
we no longer have a synchronous point of return by re-adding a
return value to nbd_client_new(). So this patch instead wires
things up to pass the negotiation status through the close_fn
callback function.
Simple test across two terminals:
$ qemu-nbd -f raw -p 30001 file
$ nmap 127.0.0.1 -p 30001 && \
qemu-io -c 'r 0 512' -f raw nbd://localhost:30001
Note that this patch does not change what constitutes successful
negotiation (thus, a client must enter transmission phase before
that client can be considered as a reason to terminate the server
when the connection ends). Perhaps we may want to tweak things
in a later patch to also treat a client that uses NBD_OPT_ABORT
as being a 'successful' negotiation (the client correctly talked
the NBD protocol, and informed us it was not going to use our
export after all), but that's a discussion for another day.
Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1451614
Signed-off-by: Eric Blake <eblake@redhat.com>
Message-Id: <20170608222617.20376-1-eblake@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
2017-06-08 22:26:17 +00:00
|
|
|
client_close(client, false);
|
2016-01-14 08:41:03 +00:00
|
|
|
goto out;
|
|
|
|
}
|
2017-02-13 13:52:24 +00:00
|
|
|
|
|
|
|
nbd_client_receive_next_request(client);
|
|
|
|
|
2016-01-14 08:41:03 +00:00
|
|
|
out:
|
|
|
|
g_free(data);
|
|
|
|
}
|
|
|
|
|
nbd: Fix regression on resiliency to port scan
Back in qemu 2.5, qemu-nbd was immune to port probes (a transient
server would not quit, regardless of how many probe connections
came and went, until a connection actually negotiated). But we
broke that in commit ee7d7aa when removing the return value to
nbd_client_new(), although that patch also introduced a bug causing
an assertion failure on a client that fails negotiation. We then
made it worse during refactoring in commit 1a6245a (a segfault
before we could even assert); the (masked) assertion was cleaned
up in d3780c2 (still in 2.6), and just recently we finally fixed
the segfault ("nbd: Fully intialize client in case of failed
negotiation"). But that still means that ever since we added
TLS support to qemu-nbd, we have been vulnerable to an ill-timed
port-scan being able to cause a denial of service by taking down
qemu-nbd before a real client has a chance to connect.
Since negotiation is now handled asynchronously via coroutines,
we no longer have a synchronous point of return by re-adding a
return value to nbd_client_new(). So this patch instead wires
things up to pass the negotiation status through the close_fn
callback function.
Simple test across two terminals:
$ qemu-nbd -f raw -p 30001 file
$ nmap 127.0.0.1 -p 30001 && \
qemu-io -c 'r 0 512' -f raw nbd://localhost:30001
Note that this patch does not change what constitutes successful
negotiation (thus, a client must enter transmission phase before
that client can be considered as a reason to terminate the server
when the connection ends). Perhaps we may want to tweak things
in a later patch to also treat a client that uses NBD_OPT_ABORT
as being a 'successful' negotiation (the client correctly talked
the NBD protocol, and informed us it was not going to use our
export after all), but that's a discussion for another day.
Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1451614
Signed-off-by: Eric Blake <eblake@redhat.com>
Message-Id: <20170608222617.20376-1-eblake@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
2017-06-08 22:26:17 +00:00
|
|
|
/*
|
|
|
|
* Create a new client listener on the given export @exp, using the
|
|
|
|
* given channel @sioc. Begin servicing it in a coroutine. When the
|
|
|
|
* connection closes, call @close_fn with an indication of whether the
|
|
|
|
* client completed negotiation.
|
|
|
|
*/
|
2016-02-10 18:41:04 +00:00
|
|
|
void nbd_client_new(NBDExport *exp,
|
|
|
|
QIOChannelSocket *sioc,
|
2016-02-10 18:41:11 +00:00
|
|
|
QCryptoTLSCreds *tlscreds,
|
|
|
|
const char *tlsaclname,
|
nbd: Fix regression on resiliency to port scan
Back in qemu 2.5, qemu-nbd was immune to port probes (a transient
server would not quit, regardless of how many probe connections
came and went, until a connection actually negotiated). But we
broke that in commit ee7d7aa when removing the return value to
nbd_client_new(), although that patch also introduced a bug causing
an assertion failure on a client that fails negotiation. We then
made it worse during refactoring in commit 1a6245a (a segfault
before we could even assert); the (masked) assertion was cleaned
up in d3780c2 (still in 2.6), and just recently we finally fixed
the segfault ("nbd: Fully intialize client in case of failed
negotiation"). But that still means that ever since we added
TLS support to qemu-nbd, we have been vulnerable to an ill-timed
port-scan being able to cause a denial of service by taking down
qemu-nbd before a real client has a chance to connect.
Since negotiation is now handled asynchronously via coroutines,
we no longer have a synchronous point of return by re-adding a
return value to nbd_client_new(). So this patch instead wires
things up to pass the negotiation status through the close_fn
callback function.
Simple test across two terminals:
$ qemu-nbd -f raw -p 30001 file
$ nmap 127.0.0.1 -p 30001 && \
qemu-io -c 'r 0 512' -f raw nbd://localhost:30001
Note that this patch does not change what constitutes successful
negotiation (thus, a client must enter transmission phase before
that client can be considered as a reason to terminate the server
when the connection ends). Perhaps we may want to tweak things
in a later patch to also treat a client that uses NBD_OPT_ABORT
as being a 'successful' negotiation (the client correctly talked
the NBD protocol, and informed us it was not going to use our
export after all), but that's a discussion for another day.
Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1451614
Signed-off-by: Eric Blake <eblake@redhat.com>
Message-Id: <20170608222617.20376-1-eblake@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
2017-06-08 22:26:17 +00:00
|
|
|
void (*close_fn)(NBDClient *, bool))
|
2011-09-19 12:03:37 +00:00
|
|
|
{
|
2011-09-19 12:33:23 +00:00
|
|
|
NBDClient *client;
|
2016-01-14 08:41:03 +00:00
|
|
|
NBDClientNewData *data = g_new(NBDClientNewData, 1);
|
|
|
|
|
2011-09-19 12:33:23 +00:00
|
|
|
client = g_malloc0(sizeof(NBDClient));
|
|
|
|
client->refcount = 1;
|
|
|
|
client->exp = exp;
|
2016-02-10 18:41:11 +00:00
|
|
|
client->tlscreds = tlscreds;
|
|
|
|
if (tlscreds) {
|
|
|
|
object_ref(OBJECT(client->tlscreds));
|
|
|
|
}
|
|
|
|
client->tlsaclname = g_strdup(tlsaclname);
|
2016-02-10 18:41:04 +00:00
|
|
|
client->sioc = sioc;
|
|
|
|
object_ref(OBJECT(client->sioc));
|
|
|
|
client->ioc = QIO_CHANNEL(sioc);
|
|
|
|
object_ref(OBJECT(client->ioc));
|
nbd: Fix regression on resiliency to port scan
Back in qemu 2.5, qemu-nbd was immune to port probes (a transient
server would not quit, regardless of how many probe connections
came and went, until a connection actually negotiated). But we
broke that in commit ee7d7aa when removing the return value to
nbd_client_new(), although that patch also introduced a bug causing
an assertion failure on a client that fails negotiation. We then
made it worse during refactoring in commit 1a6245a (a segfault
before we could even assert); the (masked) assertion was cleaned
up in d3780c2 (still in 2.6), and just recently we finally fixed
the segfault ("nbd: Fully intialize client in case of failed
negotiation"). But that still means that ever since we added
TLS support to qemu-nbd, we have been vulnerable to an ill-timed
port-scan being able to cause a denial of service by taking down
qemu-nbd before a real client has a chance to connect.
Since negotiation is now handled asynchronously via coroutines,
we no longer have a synchronous point of return by re-adding a
return value to nbd_client_new(). So this patch instead wires
things up to pass the negotiation status through the close_fn
callback function.
Simple test across two terminals:
$ qemu-nbd -f raw -p 30001 file
$ nmap 127.0.0.1 -p 30001 && \
qemu-io -c 'r 0 512' -f raw nbd://localhost:30001
Note that this patch does not change what constitutes successful
negotiation (thus, a client must enter transmission phase before
that client can be considered as a reason to terminate the server
when the connection ends). Perhaps we may want to tweak things
in a later patch to also treat a client that uses NBD_OPT_ABORT
as being a 'successful' negotiation (the client correctly talked
the NBD protocol, and informed us it was not going to use our
export after all), but that's a discussion for another day.
Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1451614
Signed-off-by: Eric Blake <eblake@redhat.com>
Message-Id: <20170608222617.20376-1-eblake@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
2017-06-08 22:26:17 +00:00
|
|
|
client->close_fn = close_fn;
|
2012-09-18 11:26:25 +00:00
|
|
|
|
2016-01-14 08:41:03 +00:00
|
|
|
data->client = client;
|
coroutine: move entry argument to qemu_coroutine_create
In practice the entry argument is always known at creation time, and
it is confusing that sometimes qemu_coroutine_enter is used with a
non-NULL argument to re-enter a coroutine (this happens in
block/sheepdog.c and tests/test-coroutine.c). So pass the opaque value
at creation time, for consistency with e.g. aio_bh_new.
Mostly done with the following semantic patch:
@ entry1 @
expression entry, arg, co;
@@
- co = qemu_coroutine_create(entry);
+ co = qemu_coroutine_create(entry, arg);
...
- qemu_coroutine_enter(co, arg);
+ qemu_coroutine_enter(co);
@ entry2 @
expression entry, arg;
identifier co;
@@
- Coroutine *co = qemu_coroutine_create(entry);
+ Coroutine *co = qemu_coroutine_create(entry, arg);
...
- qemu_coroutine_enter(co, arg);
+ qemu_coroutine_enter(co);
@ entry3 @
expression entry, arg;
@@
- qemu_coroutine_enter(qemu_coroutine_create(entry), arg);
+ qemu_coroutine_enter(qemu_coroutine_create(entry, arg));
@ reentry @
expression co;
@@
- qemu_coroutine_enter(co, NULL);
+ qemu_coroutine_enter(co);
except for the aforementioned few places where the semantic patch
stumbled (as expected) and for test_co_queue, which would otherwise
produce an uninitialized variable warning.
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Reviewed-by: Fam Zheng <famz@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
2016-07-04 17:10:01 +00:00
|
|
|
data->co = qemu_coroutine_create(nbd_co_client_start, data);
|
|
|
|
qemu_coroutine_enter(data->co);
|
2011-09-19 12:03:37 +00:00
|
|
|
}
|