[rdesktop-devel] [ rdesktop-Bugs-1102161 ] !!! broken clipboard support when using koi8-r

[SourceForge.net <no...@so...>]

Bugs item #1102161, was opened at 2005-01-14 10:21
Message generated for change (Comment added) made by ikonst
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=381347&aid=1102161&group_id=24366

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
>Status: Pending
Resolution: None
Priority: 5
Submitted By: Nobody/Anonymous (nobody)
Assigned to: Nobody/Anonymous (nobody)
Summary: !!! broken clipboard support when using koi8-r

Initial Comment:
At my environment XGetWindowProperty (xclip.c, 
xclip_handle_SelectionNotify()) return extra information:

data ::= 0x1B 0x25 0x2F 0x31 0x80 0x8B local_charset 0x02 
clipboard_data_in_local_charset

And rdesktop sends this data paket to the RDP server.

I my case local_charset is filled with string "koi8-r"


Possible solution:
        if (event->target != rdesktop_clipboard_formats_atom)
        {
                uint8 *translated_data;
                uint8 *skiped_data = data;
                uint32 length = nitems;
                /* 0x1B 0x25 0x2F 0x31 0x80 0x8B charset 0x02 */
                if ((length > 6 + 1 + 1)
                && data[0] == 0x1B
                && data[1] == 0x25
                && data[2] == 0x2F
                && data[3] == 0x31
                && data[4] == 0x80
                && data[5] == 0x8B) {
                        skiped_data += 6;
                        length -= 6;
                        while (length > 0) {
                                if (*skiped_data == 0x02) {
                                        length --;
                                        skiped_data ++;
                                        break;
                                }
                                length --;
                                skiped_data ++;
                        }
                        DUMP("skipped_data", skiped_data, length);
                }

DUMP macro is defined as:
#define DUMP(label, ptr, count)         {               int i;                                          printf("DUMPING (%s, %d bytes):\n",                     label, count);                          for (i=0; i<count; i++) {                               printf ("%02X ", ptr[i]);                       if ((i && (i % 16) == 0)                        || (i == count-1)) {                                    printf("\n");                           }                                       }                                       }


----------------------------------------------------------------------

>Comment By: Ilya Konstantinov (ikonst)
Date: 2005-10-28 12:30

Message:
Logged In: YES 
user_id=335423

There is a patch pending (for CVS version of rdesktop) that
switches the clipboard to Unicode support so all languages,
not only Russian, should be supported with it. I'd
appreciate if you could test it and tell me if it works for you:
Patch 1339819
http://sourceforge.net/tracker/index.php?func=detail&aid=1339819&group_id=24366&atid=381349

----------------------------------------------------------------------

Comment By: Nobody/Anonymous (nobody)
Date: 2005-06-03 09:02

Message:
Logged In: NO 

Any solutions about this issue ?

----------------------------------------------------------------------

Comment By: Nobody/Anonymous (nobody)
Date: 2005-01-14 10:21

Message:
Logged In: NO 

By the way, version I use is 1.3.1.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=381347&aid=1102161&group_id=24366