--- sourceforge.net/trunk/rdesktop/rdp.c	2003/05/30 21:59:56	381
+++ sourceforge.net/trunk/rdesktop/rdp.c	2003/07/09 09:18:20	435
@@ -2,17 +2,17 @@
    rdesktop: A Remote Desktop Protocol client.
    Protocol services - RDP layer
    Copyright (C) Matthew Chapman 1999-2002
-   
+
    This program is free software; you can redistribute it and/or modify
    it under the terms of the GNU General Public License as published by
    the Free Software Foundation; either version 2 of the License, or
    (at your option) any later version.
-   
+
    This program is distributed in the hope that it will be useful,
    but WITHOUT ANY WARRANTY; without even the implied warranty of
    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
    GNU General Public License for more details.
-   
+
    You should have received a copy of the GNU General Public License
    along with this program; if not, write to the Free Software
    Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
@@ -28,6 +28,7 @@
 extern BOOL desktop_save;
 extern BOOL use_rdp5;
 extern uint16 server_rdp_version;
+extern int server_bpp;
 
 uint8 *next_packet;
 uint32 rdp_shareid;
@@ -170,37 +171,33 @@
 	{
 		flags |= RDP_LOGON_BLOB;
 		DEBUG_RDP5(("Sending RDP5-style Logon packet\n"));
-		packetlen = 4 + // Unknown uint32
-			4 + // flags
-			2 + // len_domain
-			2 + // len_user
-			(flags & RDP_LOGON_AUTO ? 2 : 0) + // len_password
-			(flags & RDP_LOGON_BLOB ? 2 : 0) + // Length of BLOB
-			2 + // len_program
-			2 + // len_directory
-			(0 < len_domain ? len_domain : 2) + // domain 
-			len_user +
-			(flags & RDP_LOGON_AUTO ? len_password : 0) +
-			0 + // We have no 512 byte BLOB. Perhaps we must?
-			(flags & RDP_LOGON_BLOB && !(flags & RDP_LOGON_AUTO) ? 2 : 0) + // After the BLOB is a unknown int16. If there is a BLOB, that is.
-			(0 < len_program ? len_program : 2) +
-			(0 < len_directory ? len_directory : 2) +
-			2 + // Unknown (2)
-			2 + // Client ip length
-			len_ip + // Client ip
-			2 +  // DLL string length
-			len_dll + // DLL string
-			2 +  // Unknown
-			2 +  // Unknown
-			64 +  // Time zone #0
-			2 +  // Unknown
-			64 +  // Time zone #1
-			32; // Unknown
-			
-		s = sec_init(sec_flags, packetlen);	
+		packetlen = 4 +	// Unknown uint32
+			4 +	// flags
+			2 +	// len_domain
+			2 +	// len_user
+			(flags & RDP_LOGON_AUTO ? 2 : 0) +	// len_password
+			(flags & RDP_LOGON_BLOB ? 2 : 0) +	// Length of BLOB
+			2 +	// len_program
+			2 +	// len_directory
+			(0 < len_domain ? len_domain : 2) +	// domain
+			len_user + (flags & RDP_LOGON_AUTO ? len_password : 0) + 0 +	// We have no 512 byte BLOB. Perhaps we must?
+			(flags & RDP_LOGON_BLOB && !(flags & RDP_LOGON_AUTO) ? 2 : 0) +	// After the BLOB is a unknown int16. If there is a BLOB, that is.
+			(0 < len_program ? len_program : 2) + (0 < len_directory ? len_directory : 2) + 2 +	// Unknown (2)
+			2 +	// Client ip length
+			len_ip +	// Client ip
+			2 +	// DLL string length
+			len_dll +	// DLL string
+			2 +	// Unknown
+			2 +	// Unknown
+			64 +	// Time zone #0
+			2 +	// Unknown
+			64 +	// Time zone #1
+			32;	// Unknown
+
+		s = sec_init(sec_flags, packetlen);
 		DEBUG_RDP5(("Called sec_init with packetlen %d\n", packetlen));
 
-		out_uint32(s, 0); // Unknown
+		out_uint32(s, 0);	// Unknown
 		out_uint32_le(s, flags);
 		out_uint16_le(s, len_domain);
 		out_uint16_le(s, len_user);
@@ -209,55 +206,61 @@
 			out_uint16_le(s, len_password);
 
 		}
-		if (flags & RDP_LOGON_BLOB && !(flags & RDP_LOGON_AUTO)) {
+		if (flags & RDP_LOGON_BLOB && !(flags & RDP_LOGON_AUTO))
+		{
 			out_uint16_le(s, 0);
 		}
 		out_uint16_le(s, len_program);
 		out_uint16_le(s, len_directory);
 		if (0 < len_domain)
 			rdp_out_unistr(s, domain, len_domain);
-		else 
+		else
 			out_uint16_le(s, 0);
 		rdp_out_unistr(s, user, len_user);
 		if (flags & RDP_LOGON_AUTO)
 		{
 			rdp_out_unistr(s, password, len_password);
 		}
-		if (flags & RDP_LOGON_BLOB && !(flags & RDP_LOGON_AUTO))  {
+		if (flags & RDP_LOGON_BLOB && !(flags & RDP_LOGON_AUTO))
+		{
 			out_uint16_le(s, 0);
 		}
-		if (0 < len_program) {
+		if (0 < len_program)
+		{
 			rdp_out_unistr(s, program, len_program);
-			
-		} else {
+
+		}
+		else
+		{
 			out_uint16_le(s, 0);
 		}
-		if (0 < len_directory) {
+		if (0 < len_directory)
+		{
 			rdp_out_unistr(s, directory, len_directory);
-		} else {
+		}
+		else
+		{
 			out_uint16_le(s, 0);
 		}
 		out_uint16_le(s, 2);
-		out_uint16_le(s, len_ip+2); // Length of client ip
+		out_uint16_le(s, len_ip + 2);	// Length of client ip
 		rdp_out_unistr(s, "127.0.0.1", len_ip);
-		out_uint16_le(s, len_dll+2);
+		out_uint16_le(s, len_dll + 2);
 		rdp_out_unistr(s, "C:\\WINNT\\System32\\mstscax.dll", len_dll);
 		out_uint16_le(s, 0xffc4);
 		out_uint16_le(s, 0xffff);
-		rdp_out_unistr(s, "GTB, normaltid", 
-			       2*strlen("GTB, normaltid"));
-		out_uint8s(s, 62-2*strlen("GTB, normaltid"));
-			       
+		rdp_out_unistr(s, "GTB, normaltid", 2 * strlen("GTB, normaltid"));
+		out_uint8s(s, 62 - 2 * strlen("GTB, normaltid"));
+
 
 		out_uint32_le(s, 0x0a0000);
 		out_uint32_le(s, 0x050000);
 		out_uint32_le(s, 3);
 		out_uint32_le(s, 0);
 		out_uint32_le(s, 0);
-		
-		rdp_out_unistr(s, "GTB, sommartid", 
-			       2*strlen("GTB, sommartid"));
-		out_uint8s(s, 62-2*strlen("GTB, sommartid"));
+
+		rdp_out_unistr(s, "GTB, sommartid", 2 * strlen("GTB, sommartid"));
+		out_uint8s(s, 62 - 2 * strlen("GTB, sommartid"));
 
 		out_uint32_le(s, 0x30000);
 		out_uint32_le(s, 0x050000);
@@ -432,16 +435,18 @@
 static void
 rdp_out_bmpcache_caps(STREAM s)
 {
+	int Bpp;
 	out_uint16_le(s, RDP_CAPSET_BMPCACHE);
 	out_uint16_le(s, RDP_CAPLEN_BMPCACHE);
 
+	Bpp = (server_bpp + 7) / 8;
 	out_uint8s(s, 24);	/* unused */
 	out_uint16_le(s, 0x258);	/* entries */
-	out_uint16_le(s, 0x100);	/* max cell size */
+	out_uint16_le(s, 0x100 * Bpp);	/* max cell size */
 	out_uint16_le(s, 0x12c);	/* entries */
-	out_uint16_le(s, 0x400);	/* max cell size */
+	out_uint16_le(s, 0x400 * Bpp);	/* max cell size */
 	out_uint16_le(s, 0x106);	/* entries */
-	out_uint16_le(s, 0x1000);	/* max cell size */
+	out_uint16_le(s, 0x1000 * Bpp);	/* max cell size */
 }
 
 /* Output control capability set */
@@ -710,7 +715,7 @@
 		if (!compress)
 		{
 			int y;
-			bmpdata = (uint8*)xmalloc(width * height * Bpp);
+			bmpdata = (uint8 *) xmalloc(width * height * Bpp);
 			for (y = 0; y < height; y++)
 			{
 				in_uint8a(s, &bmpdata[(height - y - 1) * (width * Bpp)],
@@ -733,7 +738,7 @@
 			in_uint8s(s, 4);	/* line_size, final_size */
 		}
 		in_uint8p(s, data, size);
-		bmpdata = (uint8*)xmalloc(width * height * Bpp);
+		bmpdata = (uint8 *) xmalloc(width * height * Bpp);
 		if (bitmap_decompress(bmpdata, width, height, data, size, Bpp))
 		{
 			ui_paint_bitmap(left, top, cx, cy, width, height, bmpdata);
@@ -760,7 +765,7 @@
 	in_uint16_le(s, map.ncolours);
 	in_uint8s(s, 2);	/* pad */
 
-	map.colours = (COLOURENTRY*)xmalloc(3 * map.ncolours);
+	map.colours = (COLOURENTRY *) xmalloc(3 * map.ncolours);
 
 	DEBUG(("PALETTE(c=%d)\n", map.ncolours));
 
@@ -847,7 +852,7 @@
 }
 
 /* Process incoming packets */
-void
+BOOL
 rdp_main_loop(void)
 {
 	uint8 type;
@@ -862,6 +867,13 @@
 				break;
 
 			case RDP_PDU_DEACTIVATE:
+				DEBUG(("RDP_PDU_DEACTIVATE\n"));
+				/* We thought we could detect a clean
+				   shutdown of the session by this
+				   packet, but it seems Windows 2003
+				   is sending us one of these when we
+				   reconnect to a disconnected session
+				   return True; */
 				break;
 
 			case RDP_PDU_DATA:
@@ -875,6 +887,10 @@
 				unimpl("PDU %d\n", type);
 		}
 	}
+	return True;
+	/* We want to detect if we got a clean shutdown, but we
+	   can't. Se above.  
+	   return False;  */
 }
 
 /* Establish a connection up to the RDP layer */