Re: [Spice-devel] [PATCH 3/3] server/red_worker: fix possible leak of self_bitmap

2012-05-15 Thread Yonit Halperin
Hi, Instead of this patch series and the previous self_bitmap patch, I think we should do the following: Both GLZDrawable and Drawable share references to RedDrawable and self_bitmap, and self_bitmap life time is equal to RedDrawable's one. So we need to have a new type which warps RedDrawable

Re: [Spice-devel] [PATCH 3/3] server/red_worker: fix possible leak of self_bitmap

2012-05-15 Thread Alon Levy
On Tue, May 15, 2012 at 11:57:05AM +0300, Yonit Halperin wrote: Hi, Instead of this patch series and the previous self_bitmap patch, I think we should do the following: Both GLZDrawable and Drawable share references to RedDrawable and self_bitmap, and self_bitmap life time is equal to

Re: [Spice-devel] [PATCH 3/3] server/red_worker: fix possible leak of self_bitmap

2012-05-15 Thread Uri Lublin
On 05/15/2012 11:57 AM, Yonit Halperin wrote: Hi, Instead of this patch series and the previous self_bitmap patch, I think we should do the following: Both GLZDrawable and Drawable share references to RedDrawable and self_bitmap, and self_bitmap life time is equal to RedDrawable's one. So we

Re: [Spice-devel] [PATCH 3/3] server/red_worker: fix possible leak of self_bitmap

2012-05-15 Thread Yonit Halperin
On 05/15/2012 12:06 PM, Alon Levy wrote: On Tue, May 15, 2012 at 11:57:05AM +0300, Yonit Halperin wrote: Hi, Instead of this patch series and the previous self_bitmap patch, I think we should do the following: Both GLZDrawable and Drawable share references to RedDrawable and self_bitmap, and

Re: [Spice-devel] [PATCH 3/3] server/red_worker: fix possible leak of self_bitmap

2012-05-15 Thread Alon Levy
On Tue, May 15, 2012 at 12:27:24PM +0300, Yonit Halperin wrote: On 05/15/2012 12:06 PM, Alon Levy wrote: On Tue, May 15, 2012 at 11:57:05AM +0300, Yonit Halperin wrote: Hi, Instead of this patch series and the previous self_bitmap patch, I think we should do the following: Both GLZDrawable

Re: [Spice-devel] [PATCH 3/3] server/red_worker: fix possible leak of self_bitmap

2012-05-15 Thread Alon Levy
On Tue, May 15, 2012 at 12:18:44PM +0300, Uri Lublin wrote: On 05/15/2012 11:57 AM, Yonit Halperin wrote: Hi, Instead of this patch series and the previous self_bitmap patch, I think we should do the following: Both GLZDrawable and Drawable share references to RedDrawable and self_bitmap,