CVS: rdesktop xwin.c,1.205.2.6,1.205.2.7

Peter Åstrand <[email protected]>
Newsgroups gmane.network.rdesktop.cvs
Message-ID <[email protected]>
Update of /cvsroot/rdesktop/rdesktop
In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv10109

Modified Files:
      Tag: seamlessrdp-branch
	xwin.c 
Log Message:
Checking return value from seamless_get_window_* everywhere

Index: xwin.c
===================================================================
RCS file: /cvsroot/rdesktop/rdesktop/xwin.c,v
retrieving revision 1.205.2.6
retrieving revision 1.205.2.7
diff -C2 -d -r1.205.2.6 -r1.205.2.7
*** xwin.c	10 Mar 2006 13:46:15 -0000	1.205.2.6
--- xwin.c	10 Mar 2006 14:09:16 -0000	1.205.2.7
***************
*** 1904,1907 ****
--- 1904,1912 ----
  							  xevent.xexpose.height, xevent.xexpose.x,
  							  xevent.xexpose.y);
+ 					else
+ 					{
+ 						error("Expose for unknown window 0x%lx\n",
+ 						      xevent.xexpose.window);
+ 					}
  				}
  
***************
*** 3059,3064 ****
  {
  	seamless_window *sw;
- 	sw = seamless_get_window_by_id(id);
  
  	if (!sw)
  	{
--- 3064,3069 ----
  {
  	seamless_window *sw;
  
+ 	sw = seamless_get_window_by_id(id);
  	if (!sw)
  	{
***************
*** 3078,3082 ****
  
  	sw = seamless_get_window_by_id(id);
- 
  	if (!sw)
  	{
--- 3083,3086 ----
***************
*** 3111,3114 ****
--- 3115,3123 ----
  
  	sw = seamless_get_window_by_id(id);
+ 	if (!sw)
+ 	{
+ 		warning("ui_seamless_settitle: No information for window 0x%lx\n", id);
+ 		return;
+ 	}
  
  	XStoreName(g_display, sw->wnd, title);
***************
*** 3122,3125 ****
--- 3131,3139 ----
  
  	sw = seamless_get_window_by_id(id);
+ 	if (!sw)
+ 	{
+ 		warning("ui_seamless_setstate: No information for window 0x%lx\n", id);
+ 		return;
+ 	}
  
  	switch (state)



-------------------------------------------------------
This SF.Net email is sponsored by xPML, a groundbreaking scripting language
that extends applications into web and mobile media. Attend the live webcast
and join the prime developer group breaking into this new coding territory!
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=110944&bid=241720&dat=121642
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.