--- sourceforge.net/trunk/rdesktop/channels.c	2003/08/01 17:01:58	438
+++ sourceforge.net/trunk/rdesktop/channels.c	2004/06/17 09:42:58	717
@@ -81,13 +81,19 @@
 {
 	uint32 length, flags;
 	uint32 thislength, remaining;
-	char *data;
+	uint8 *data;
 
 	/* first fragment sent in-place */
 	s_pop_layer(s, channel_hdr);
 	length = s->end - s->p - 8;
 
+	DEBUG_CLIPBOARD(("channel_send, length = %d\n", length));
+
 	thislength = MIN(length, CHANNEL_CHUNK_LENGTH);
+/* Note: In the original clipboard implementation, this number was
+   1592, not 1600. However, I don't remember the reason and 1600 seems
+   to work so.. This applies only to *this* length, not the length of
+   continuation or ending packets. */
 	remaining = length - thislength;
 	flags = (remaining == 0) ? CHANNEL_FLAG_FIRST | CHANNEL_FLAG_LAST : CHANNEL_FLAG_FIRST;
 	if (channel->flags & CHANNEL_OPTION_SHOW_PROTOCOL)
@@ -96,6 +102,7 @@
 	out_uint32_le(s, length);
 	out_uint32_le(s, flags);
 	data = s->end = s->p + thislength;
+	DEBUG_CLIPBOARD(("Sending %d bytes with FLAG_FIRST\n", thislength));
 	sec_send_to_channel(s, g_encryption ? SEC_ENCRYPT : 0, channel->mcs_id);
 
 	/* subsequent segments copied (otherwise would have to generate headers backwards) */
@@ -104,6 +111,10 @@
 		thislength = MIN(remaining, CHANNEL_CHUNK_LENGTH);
 		remaining -= thislength;
 		flags = (remaining == 0) ? CHANNEL_FLAG_LAST : 0;
+		if (channel->flags & CHANNEL_OPTION_SHOW_PROTOCOL)
+			flags |= CHANNEL_FLAG_SHOW_PROTOCOL;
+
+		DEBUG_CLIPBOARD(("Sending %d bytes with flags %d\n", thislength, flags));
 
 		s = sec_init(g_encryption ? SEC_ENCRYPT : 0, thislength + 8);
 		out_uint32_le(s, length);
@@ -121,7 +132,7 @@
 {
 	uint32 length, flags;
 	uint32 thislength;
-	VCHANNEL *channel;
+	VCHANNEL *channel = NULL;
 	unsigned int i;
 	STREAM in;
 
@@ -150,19 +161,19 @@
 		{
 			if (length > in->size)
 			{
-				in->data = xrealloc(in->data, length);
+				in->data = (uint8 *) xrealloc(in->data, length);
 				in->size = length;
 			}
 			in->p = in->data;
 		}
 
-		thislength = s->end - s->p;
+		thislength = MIN(s->end - s->p, in->data + in->size - in->p);
 		memcpy(in->p, s->p, thislength);
-		s->p += thislength;
-		s->end += thislength;
+		in->p += thislength;
 
 		if (flags & CHANNEL_FLAG_LAST)
 		{
+			in->end = in->p;
 			in->p = in->data;
 			channel->process(in);
 		}