rpms/compiz-fusion/devel compiz-fusion.spec, 1.17, 1.18 workarounds-fullscreenfix.patch, 1.2, 1.3

[Adel Gadllah (drago01)]

Author: drago01

Update of /cvs/pkgs/rpms/compiz-fusion/devel
In directory cvs-int.fedora.redhat.com:/tmp/cvs-serv3998

Modified Files:
	compiz-fusion.spec workarounds-fullscreenfix.patch 
Log Message:
plug small memory leak


Index: compiz-fusion.spec
===================================================================
RCS file: /cvs/pkgs/rpms/compiz-fusion/devel/compiz-fusion.spec,v
retrieving revision 1.17
retrieving revision 1.18
diff -u -r1.17 -r1.18
--- compiz-fusion.spec	27 Dec 2007 14:45:33 -0000	1.17
+++ compiz-fusion.spec	27 Dec 2007 17:38:31 -0000	1.18
@@ -2,7 +2,7 @@
 
 Name: compiz-fusion
 Version: 0.6.0       
-Release: 9%{?dist}
+Release: 10%{?dist}
 Summary: Collection of Compiz Fusion plugins for Compiz      
 
 Group: User Interface/Desktops        
@@ -131,6 +131,9 @@
 
 
 %changelog
+* Thu Dec 27 2007 Adel Gadllah <adel.gadllah at gmail.com> 0.6.0-10
+- Plug small memory leak
+
 * Thu Dec 27 2007 Adel Gadllah <adel.gadllah at gmail.com> 0.6.0-9
 - Don't break legacy apps that want to unfullscreen themselves
 

workarounds-fullscreenfix.patch:

Index: workarounds-fullscreenfix.patch
===================================================================
RCS file: /cvs/pkgs/rpms/compiz-fusion/devel/workarounds-fullscreenfix.patch,v
retrieving revision 1.2
retrieving revision 1.3
diff -u -r1.2 -r1.3
--- workarounds-fullscreenfix.patch	27 Dec 2007 14:45:33 -0000	1.2
+++ workarounds-fullscreenfix.patch	27 Dec 2007 17:38:31 -0000	1.3
@@ -1,6 +1,6 @@
 diff -upNr compiz-fusion-plugins-main-0.6.0.orign/src/workarounds/workarounds.c compiz-fusion-plugins-main-0.6.0/src/workarounds/workarounds.c
 --- compiz-fusion-plugins-main-0.6.0.orign/src/workarounds/workarounds.c	2007-10-15 19:43:43.000000000 +0200
-+++ compiz-fusion-plugins-main-0.6.0/src/workarounds/workarounds.c	2007-12-27 15:32:04.000000000 +0100
++++ compiz-fusion-plugins-main-0.6.0/src/workarounds/workarounds.c	2007-12-27 18:32:55.000000000 +0100
 @@ -51,6 +51,16 @@ typedef struct _WorkaroundsWindow {
      Bool isFullscreen;
  } WorkaroundsWindow;
@@ -28,12 +28,12 @@
      {
  	unsigned int state = w->state & ~CompWindowStateFullscreenMask;
  
-@@ -211,8 +220,59 @@ workaroundsFixupFullscreen (CompWindow *
+@@ -211,8 +220,67 @@ workaroundsFixupFullscreen (CompWindow *
  	    recalcWindowType (w);
  	    recalcWindowActions (w);
  	    updateWindowAttributes (w, CompStackingUpdateModeNormal);
 +
-+	    /*keep track of windows that we interact with*/
++	    /* keep track of windows that we interact with */
 +
 +	    if(mfwList == NULL) 
 +	    {
@@ -57,9 +57,11 @@
  	}
      }
 +    else if(!isFullSize && (w->state & CompWindowStateFullscreenMask) && mfwList) {
-+	/*did we set the flag?*/
-+	WorkaroundsManagedFsWindow *mfw;
 +
++	/* did we set the flag? */
++
++	WorkaroundsManagedFsWindow *mfw,*mfwPrev;
++	mfwPrev = mfwList;
 +	for(mfw = mfwList; mfw->next != NULL; mfw = mfw->next) 
 +	{
 +		if(mfw->id == w->id) 
@@ -78,8 +80,14 @@
 +	 		   recalcWindowActions (w);
 +	 		   updateWindowAttributes (w, CompStackingUpdateModeNormal);
 +			}
++			
++			mfwPrev->next=mfw->next;
++	        	free(mfw);
++			
 +			break;
 +		}
++		
++		mfwPrev = mfw;
 +	
 +    	}
 +   }
@@ -88,7 +96,7 @@
  }
  
  static void
-@@ -356,6 +416,27 @@ workaroundsHandleEvent (CompDisplay *d,
+@@ -356,6 +424,27 @@ workaroundsHandleEvent (CompDisplay *d,
  	if (w && w->attrib.override_redirect)
  	    workaroundsDoFixes (w);
  	break;