[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 2/7] ui/vdagent: core infrastructure
From: |
Gerd Hoffmann |
Subject: |
Re: [PATCH v2 2/7] ui/vdagent: core infrastructure |
Date: |
Wed, 24 Mar 2021 10:46:58 +0100 |
On Mon, Mar 22, 2021 at 11:27:17AM +0100, Gerd Hoffmann wrote:
> Hi,
>
> > > + if (vd->msgsize != msg->size + sizeof(*msg)) {
> > > + /* FIXME: handle parse messages splitted into multiple chunks */
> > > + fprintf(stderr, "%s: size mismatch: chunk %d, msg %d (+%zd)\n",
> > > + __func__, vd->msgsize, msg->size, sizeof(*msg));
> > >
> >
> > Not fixing? You handle chunking on sending but not on receiving?
>
> linux vdagent doesn't do chunking on send, so no need (and also no
> testcase) so far.
>
> Didn't try windows guests (yet), but that is next on my clipboard
> todo list.
Hmm, windows guest agent doesn't has VD_AGENT_CAP_CLIPBOARD_SELECTION,
so I have to deal with that.
Windows guests do actually send large messages in chunks, so I have
something to test with, good.
What are VD_AGENT_CAP_GUEST_LINEEND_LF + VD_AGENT_CAP_GUEST_LINEEND_CRLF
are good for btw? Are linefeeds converted automatically between dos and
unix conventions? If so, who is supposed to handle that?
take care,
Gerd
[PATCH v2 4/7] ui/vdagent: add clipboard support, Gerd Hoffmann, 2021/03/18
[PATCH v2 3/7] ui/vdagent: add mouse support, Gerd Hoffmann, 2021/03/18
[PATCH v2 7/7] ui/gtk: add clipboard support, Gerd Hoffmann, 2021/03/18