[PATCH] Fix a memory leak in qxl_screen_init

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



From: Longguang Yue <bigclouds@xxxxxxx>

qxl->uxa was allocated twice, once in qxl_screen_init() and once in
qxl_uxa_init(). This commit removes the redundant allocation in
qxl_screen_init().
---

Here is a this same patch in git-format-patch format and with a whitespace
change removed. ACK from me for the patch. Longguang, let me know if your name/email
and the commit log are fine with you before this gets pushed.

Christophe

 src/qxl_driver.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/src/qxl_driver.c b/src/qxl_driver.c
index 29b1551..aa969e8 100644
--- a/src/qxl_driver.c
+++ b/src/qxl_driver.c
@@ -749,8 +749,6 @@ qxl_screen_init (SCREEN_INIT_ARGS_DECL)
 	}
     }
     
-    qxl->uxa = uxa_driver_alloc ();
-    
 #ifndef XSPICE
     qxl->io_pages = (void *)((unsigned long)qxl->ram);
     qxl->io_pages_physical = (void *)((unsigned long)qxl->ram_physical);
-- 
1.9.0

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]