Re: [Spice-devel] patch[1/1] fix a memory leak in qxl_screen_init

2014-04-22 Thread Jeremy White
Subject: Re: [Spice-devel] patch[1/1] fix a memory leak in qxl_screen_init On 11/06/2013 08:48 AM, bigclouds wrote: it is needed to detect the return of qxl_uxa_init in qxl_screen_init . I don't think we have understood each other. There is a duplicate allocation; you are trying to fix that. As I

Re: [Spice-devel] patch[1/1] fix a memory leak in qxl_screen_init

2014-04-07 Thread Jeremy White
On 03/26/2014 11:30 AM, longguang.yue wrote: please forgive my ignore. jwhite you are right, i post a new patch as you suggest. Yes, this patch does look correct. I have been meaning to get time to put it through some further testing, and then push it. I hope to do that this week.

Re: [Spice-devel] patch[1/1] fix a memory leak in qxl_screen_init

2014-03-26 Thread longguang.yue
If you can finish that. - Original Message - From: Jeremy White jwh...@codeweavers.com To: bigclouds bigclo...@163.com Cc: spice-devel@lists.freedesktop.org Sent: Wednesday, November 6, 2013 4:28:45 PM Subject: Re: [Spice-devel] patch[1/1] fix a memory leak in qxl_screen_init On 11

Re: [Spice-devel] patch[1/1] fix a memory leak in qxl_screen_init

2013-11-19 Thread Marian Krcmarik
Sent: Wednesday, November 6, 2013 4:28:45 PM Subject: Re: [Spice-devel] patch[1/1] fix a memory leak in qxl_screen_init On 11/06/2013 08:48 AM, bigclouds wrote: it is needed to detect the return of qxl_uxa_init in qxl_screen_init . I don't think we have understood each other

Re: [Spice-devel] patch[1/1] fix a memory leak in qxl_screen_init

2013-11-06 Thread bigclouds
it is needed to detect the return of qxl_uxa_init in qxl_screen_init . At 2013-11-06 21:59:58,Jeremy White jwh...@codeweavers.com wrote: Nice catch! On 11/06/2013 03:37 AM, bigclouds wrote: hi, it allocate twice memory for qxl-uxa in function qxl_screen_init and qxl_uxa_init