[PATCH 2/4] Coverity: dix/window.c: double_free: Double free of pointer "srcbits" and "mskbits" in call to "Xfree"

Oliver McFadden oliver.mcfadden at nokia.com
Mon Dec 28 02:02:56 PST 2009


freed_arg: Pointer "srcbits" and "mskbits" freed by function "AllocARGBCursor"

Signed-off-by: Oliver McFadden <oliver.mcfadden at nokia.com>
---
 dix/window.c |    5 -----
 1 files changed, 0 insertions(+), 5 deletions(-)

diff --git a/dix/window.c b/dix/window.c
index caff1cb..be90981 100644
--- a/dix/window.c
+++ b/dix/window.c
@@ -3316,11 +3316,6 @@ TileScreenSaver(int i, int kind)
 	    else
 		cursor = 0;
 	}
-	else
-	{
-	    xfree (srcbits);
-	    xfree (mskbits);
-	}
     }
 
     pWin = savedScreenInfo[i].pWindow =
-- 
1.6.1



More information about the xorg-devel mailing list