Fix DCOP interface to window suspend/resume functions

This resolves Bug 1354

twin/KWinInterface.h

@@ -12,9 +12,9 @@ class KWinInterface : virtual public DCOPObject
     virtual ASYNC cascadeDesktop() = 0;
     virtual ASYNC unclutterDesktop() = 0;
     virtual ASYNC reconfigure() = 0;
-    virtual ASYNC killWindow() = 0;
-    virtual ASYNC suspendWindow() = 0;
-    virtual ASYNC resumeWindow() = 0;
+    virtual ASYNC killWindowId(unsigned long winId) = 0;
+    virtual ASYNC suspendWindowId(unsigned long winId) = 0;
+    virtual ASYNC resumeWindowId(unsigned long winId) = 0;
     virtual void refresh() = 0;
     virtual void doNotManage(TQString)= 0;
     virtual void showWindowMenuAt(unsigned long winId, int x, int y)= 0;

twin/workspace.cpp

@@ -1894,8 +1894,9 @@ void Workspace::killWindowId( Window window_to_kill )
         client = findClient( FrameIdMatchPredicate( window ));
         if( client != NULL ) // found the client
             break;
-        Window parent, root;
-        Window* children;
+        Window parent = NULL;
+        Window root = NULL;
+        Window* children = NULL;
         unsigned int children_count;
         XQueryTree( tqt_xdisplay(), window, &root, &parent, &children, &children_count );
         if( children != NULL )
@@ -1903,6 +1904,8 @@ void Workspace::killWindowId( Window window_to_kill )
         if( window == root ) // we didn't find the client, probably an override-redirect window
             break;
         window = parent; // go up
+        if( window == NULL )
+            break;
         }
     if( client != NULL )
         client->killWindow();
@@ -1921,8 +1924,9 @@ void Workspace::suspendWindowId( Window window_to_suspend )
         client = findClient( FrameIdMatchPredicate( window ));
         if( client != NULL ) // found the client
             break;
-        Window parent, root;
-        Window* children;
+        Window parent = NULL;
+        Window root = NULL;
+        Window* children = NULL;
         unsigned int children_count;
         XQueryTree( tqt_xdisplay(), window, &root, &parent, &children, &children_count );
         if( children != NULL )
@@ -1930,6 +1934,8 @@ void Workspace::suspendWindowId( Window window_to_suspend )
         if( window == root ) // we didn't find the client, probably an override-redirect window
             break;
         window = parent; // go up
+        if( window == NULL )
+            break;
         }
     if( client != NULL )
         client->suspendWindow();
@@ -1948,8 +1954,9 @@ void Workspace::resumeWindowId( Window window_to_resume )
         client = findClient( FrameIdMatchPredicate( window ));
         if( client != NULL ) // found the client
             break;
-        Window parent, root;
-        Window* children;
+        Window parent = NULL;
+        Window root = NULL;
+        Window* children = NULL;
         unsigned int children_count;
         XQueryTree( tqt_xdisplay(), window, &root, &parent, &children, &children_count );
         if( children != NULL )
@@ -1957,6 +1964,8 @@ void Workspace::resumeWindowId( Window window_to_resume )
         if( window == root ) // we didn't find the client, probably an override-redirect window
             break;
         window = parent; // go up
+        if( window == NULL )
+            break;
         }
     if( client != NULL )
         client->resumeWindow();