[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH v2 6/7] nbd-client: Stricter enforcing of struct
From: |
Eric Blake |
Subject: |
Re: [Qemu-block] [PATCH v2 6/7] nbd-client: Stricter enforcing of structured reply spec |
Date: |
Thu, 9 Nov 2017 08:45:49 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0 |
On 11/09/2017 03:37 AM, Vladimir Sementsov-Ogievskiy wrote:
> 09.11.2017 00:57, Eric Blake wrote:
>> Ensure that the server is not sending unexpected chunk lengths
>> for either the NONE or the OFFSET_DATA chunk, nor unexpected
>> hole length for OFFSET_HOLE. This will flag any server that
>> responds to a zero-length read with an OFFSET_DATA as broken,
>
> or OFFSET_HOLE
True, even though our implementation was not doing that. I can tweak
the commit message.
>
>> even though we previously fixed our client to never be able to
>> send such a request over the wire.
>>
>> Signed-off-by: Eric Blake <address@hidden>
>> ---
>> block/nbd-client.c | 12 +++++++++---
>> 1 file changed, 9 insertions(+), 3 deletions(-)
>>
>> @@ -281,7 +281,8 @@ static int
>> nbd_co_receive_offset_data_payload(NBDClientSession *s,
>>
>> assert(nbd_reply_is_structured(&s->reply));
>>
>> - if (chunk->length < sizeof(offset)) {
>> + /* The NBD spec requires at least one byte of payload */
>> + if (chunk->length <= sizeof(offset)) {
>> error_setg(errp, "Protocol error: invalid payload for "
>> "NBD_REPLY_TYPE_OFFSET_DATA");
>> return -EINVAL;
>> @@ -293,7 +294,7 @@ static int
>> nbd_co_receive_offset_data_payload(NBDClientSession *s,
>> be64_to_cpus(&offset);
>>
>> data_size = chunk->length - sizeof(offset);
>> - if (offset < orig_offset || data_size > qiov->size ||
>> + if (!data_size || offset < orig_offset || data_size > qiov->size ||
>
> !data_size - always false here (because of previous check), and even if
> it could be zero it
> isn't correspond to error message.
>
> without this, or with an assert instead:
I like the assert instead.
>
> Reviewed-by: Vladimir Sementsov-Ogievskiy <address@hidden>
>
>
Thanks for the reviews; I'll queue this series onto my NBD tree with
your suggestions incorporated, and send a pull request for 2.11 shortly.
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.org | libvirt.org
signature.asc
Description: OpenPGP digital signature
- Re: [Qemu-block] [PATCH v2 1/7] nbd-client: Fix error message typos, (continued)
- [Qemu-block] [PATCH v2 3/7] nbd/client: Nicer trace of structured reply, Eric Blake, 2017/11/08
- [Qemu-block] [PATCH v2 4/7] nbd: Fix struct name for structured reads, Eric Blake, 2017/11/08
- [Qemu-block] [PATCH v2 5/7] nbd-client: Short-circuit 0-length operations, Eric Blake, 2017/11/08
- [Qemu-block] [PATCH v2 6/7] nbd-client: Stricter enforcing of structured reply spec, Eric Blake, 2017/11/08
- [Qemu-block] [PATCH v2 7/7] nbd/server: Fix structured read of length 0, Eric Blake, 2017/11/08
- Re: [Qemu-block] [Qemu-devel] [PATCH v2 0/7] various NBD fixes for 2.11, no-reply, 2017/11/08