Re: [PATCH] Cygwin: Make native clipboard layout same for 32- and 64-bit

[Corinna Vinschen <corinna-cygwin@cygwin.com>]

On Oct 24 08:58, Takashi Yano wrote:
> Hi Corinna,
> 
> On Fri, 22 Oct 2021 17:11:13 +0200
> Corinna Vinschen wrote:
> > Just to close this up prior to the 3.3.0 release...
> > 
> > Given we never actually strived for 32<->64 bit interoperability, it's
> > hard to argue why this should be different for the clipboard stuff.
> > 
> > Running 32 and 64 bit Cygwin versions in parallel doesn't actually make
> > much sense for most people anyway, unless they explicitely develop for
> > 32 and 64 bit systems under Cygwin.  From a productivity point of view
> > there's no good reason to run more than one arch.
> > 
> > So I agree with Ken here.  It's probably not worth the trouble.
> 
> Current code below in fhandler_clipboard.cc causes access violation
> if clipboard is accessed between 32 and 64 bit cygwin.
> 
>       cygcb_t *clipbuf = (cygcb_t *) cb_data;
> 
>       if (pos < (off_t) clipbuf->len)
>         {
>           ret = ((len > (clipbuf->len - pos)) ? (clipbuf->len - pos) : len);
>           memcpy (ptr, clipbuf->data + pos , ret);
>           pos += ret;
>         }
> 
> Don't you think this should be fixed?

Well, if you put it this way...

I didn't get that it's crashing, so yeah, in that case a fix would
be nice, of course.


Corinna