CVS: rdesktop rdp5.c,1.17,1.18

Update of /cvsroot/rdesktop/rdesktop
In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv3157

Modified Files:
	rdp5.c 
Log Message:
update comments. Thanks to Scott Marovich (marovich at hpl.hp.com) for pointing out that 3 is probably a synchronise update PDU

Index: rdp5.c
===================================================================
RCS file: /cvsroot/rdesktop/rdesktop/rdp5.c,v
retrieving revision 1.17
retrieving revision 1.18
diff -C2 -d -r1.17 -r1.18
*** rdp5.c	17 Apr 2005 23:14:20 -0000	1.17
--- rdp5.c	26 Apr 2005 22:48:20 -0000	1.18
***************
*** 1,5 ****
  /* -*- c-basic-offset: 8 -*-
     rdesktop: A Remote Desktop Protocol client.
!    Protocol services - Multipoint Communications Service
     Copyright (C) Matthew Chapman 1999-2005
     Copyright (C) Erik Forsberg 2003
--- 1,5 ----
  /* -*- c-basic-offset: 8 -*-
     rdesktop: A Remote Desktop Protocol client.
!    Protocol services - RDP5 short form PDU processing
     Copyright (C) Matthew Chapman 1999-2005
     Copyright (C) Erik Forsberg 2003
***************
*** 81,108 ****
  		switch (type)
  		{
! 				/* Thanks to Jeroen Meijer <jdmeijer at yahoo
! 				   dot com> for finding out the meaning of
! 				   most of the opcodes here. Especially opcode
! 				   8! :) */
! 			case 0:	/* orders */
  				in_uint16_le(ts, count);
  				process_orders(ts, count);
  				break;
! 			case 1:	/* bitmap update (???) */
  				in_uint8s(ts, 2);	/* part length */
  				process_bitmap_updates(ts);
  				break;
! 			case 2:	/* palette */
  				in_uint8s(ts, 2);	/* uint16 = 2 */
  				process_palette(ts);
  				break;
! 			case 3:	/* probably an palette with offset 3. Weird */
  				break;
! 			case 5:
  				ui_set_null_cursor();
  				break;
! 			case 6:	/* TODO: W2K3 SP1 sends this on connect */
  				break;
! 			case 8:
  				in_uint16_le(ts, x);
  				in_uint16_le(ts, y);
--- 81,104 ----
  		switch (type)
  		{
! 			case 0:	/* update orders */
  				in_uint16_le(ts, count);
  				process_orders(ts, count);
  				break;
! 			case 1:	/* update bitmap */
  				in_uint8s(ts, 2);	/* part length */
  				process_bitmap_updates(ts);
  				break;
! 			case 2:	/* update palette */
  				in_uint8s(ts, 2);	/* uint16 = 2 */
  				process_palette(ts);
  				break;
! 			case 3:	/* update synchronize */
  				break;
! 			case 5: /* null pointer */
  				ui_set_null_cursor();
  				break;
! 			case 6:	/* default pointer */
  				break;
! 			case 8: /* pointer position */
  				in_uint16_le(ts, x);
  				in_uint16_le(ts, y);
***************
*** 110,117 ****
  					ui_move_pointer(x, y);
  				break;
! 			case 9:
  				process_colour_pointer_pdu(ts);
  				break;
! 			case 10:
  				process_cached_pointer_pdu(ts);
  				break;
--- 106,113 ----
  					ui_move_pointer(x, y);
  				break;
! 			case 9: /* color pointer */
  				process_colour_pointer_pdu(ts);
  				break;
! 			case 10: /* cached pointer */
  				process_cached_pointer_pdu(ts);
  				break;



-------------------------------------------------------
SF.Net email is sponsored by: Tell us your software development plans!
Take this survey and enter to win a one-year sub to SourceForge.net
Plus IDC's 2005 look-ahead and a copy of this survey
Click here to start!  http://www.idcswdc.com/cgi-bin/survey?id=105hix