[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 6/6] io: Reply to ping frames
From: |
Brandon Carpenter |
Subject: |
Re: [Qemu-devel] [PATCH v2 6/6] io: Reply to ping frames |
Date: |
Mon, 11 Sep 2017 12:04:15 -0700 |
On Mon, Sep 11, 2017 at 10:10 AM, Daniel P. Berrange
<address@hidden> wrote:
It feels like this is still dangerous - the client simply has to
interleave each "ping" with a 1 byte binary frame to get around this
limit. We need to make sure we have an absolute cap on the output
buffer size.
Okay. I see that now that I look at it more closely. This breed of
asynchronous I/O is tricky because the conditions for reading/writing
are all over the place. There's a lot of context to keep in your head.
I have a fix. And I realized that I was missing a patch in the series
for RFC-compliant closing of websocket connections, which I must have
lost during a rebase. Should I submit v3 of the patch series or just
add those patches to this thread?
Thank you,
--
Brandon Carpenter | Software Engineer
Cypherpath, Inc.
400 Columbia Point Drive Ste 101 | Richland, Washington USA
Office: (650) 713-3060
--
CONFIDENTIALITY NOTICE: This e-mail message, including any attachments, is
for the sole use of the intended recipient(s) and may contain proprietary,
confidential or privileged information or otherwise be protected by law.
Any unauthorized review, use, disclosure or distribution is prohibited. If
you are not the intended recipient, please notify the sender and destroy
all copies and the original message.
- [Qemu-devel] [PATCH v2 1/6] io: Always remove an old channel watch before adding a new one, (continued)
- Message not available
- Message not available
- Message not available
- Message not available
- Message not available
- [Qemu-devel] [PATCH v2 6/6] io: Reply to ping frames, Brandon Carpenter, 2017/09/08
- Re: [Qemu-devel] [PATCH v2 6/6] io: Reply to ping frames, Daniel P. Berrange, 2017/09/11
- Re: [Qemu-devel] [PATCH v2 6/6] io: Reply to ping frames, Brandon Carpenter, 2017/09/11
- Re: [Qemu-devel] [PATCH v2 6/6] io: Reply to ping frames, Daniel P. Berrange, 2017/09/11
- Re: [Qemu-devel] [PATCH v2 6/6] io: Reply to ping frames,
Brandon Carpenter <=
- Re: [Qemu-devel] [PATCH v2 6/6] io: Reply to ping frames, Daniel P. Berrange, 2017/09/12
- Re: [Qemu-devel] [PATCH v2 6/6] io: Reply to ping frames, Daniel P. Berrange, 2017/09/11
- Re: [Qemu-devel] [PATCH v2 6/6] io: Reply to ping frames, Brandon Carpenter, 2017/09/11
- Re: [Qemu-devel] [PATCH v2 6/6] io: Reply to ping frames, Daniel P. Berrange, 2017/09/12
- Re: [Qemu-devel] [PATCH v2 6/6] io: Reply to ping frames, Brandon Carpenter, 2017/09/12
- Re: [Qemu-devel] [PATCH v2 0/6] Update websocket code to more fully support the RFC, Eric Blake, 2017/09/08