[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 5/7] ui/vnc: clipboard support
From: |
Gerd Hoffmann |
Subject: |
Re: [PATCH 5/7] ui/vnc: clipboard support |
Date: |
Thu, 4 Mar 2021 09:58:11 +0100 |
On Wed, Mar 03, 2021 at 06:27:27PM +0400, Marc-André Lureau wrote:
> Hi
>
> On Wed, Mar 3, 2021 at 4:13 PM Gerd Hoffmann <kraxel@redhat.com> wrote:
>
> > Hi,
> >
> > > > + case Z_BUF_ERROR:
> > > > + out_len <<= 1;
> > > > + if (out_len > (1 << 20)) {
> > > >
> > >
> > > 1Mb isn't that much, is it? Well, since it handles only text for now it's
> > > probably enough. Would it make sense to make this a #define for clarity ?
> >
> > Yep. While talking about sizes: How does vdagent handles large
> > clipboard chunks? There is ...
> >
> > #define VD_AGENT_MAX_DATA_SIZE 2048
> >
> > ... but I suspect clipboard content isn't limited to that ...
> >
> >
> The client splits large data to send in many messages.
Ok. Is it documented anywhere how that split happens? I suspect it'll
involve VDIChunkHeader somehow?
> I don't see the
> agent doing the same (I might be missing something).
Hmm, ok. Guess I should better be prepared to receive messages larger
than VD_AGENT_MAX_DATA_SIZE ...
take care,
Gerd