Re: [PATCH] desktop-shell: fix bug referencing invalid surface list

2015-08-05 Thread Ryo Munakata
Hi Derek. On 08/05, Derek Foreman wrote: > On 29/07/15 01:12 PM, Ryo Munakata wrote: > > In shell_destroy_shell_surface, we remove the link of a shell_surface. > > There is a case which the link references shell_client::surface_list > > which is already free'd. > > Therefore we have to remove shel

Re: [PATCH] desktop-shell: fix bug referencing invalid surface list

2015-08-05 Thread Derek Foreman
On 29/07/15 01:12 PM, Ryo Munakata wrote: > In shell_destroy_shell_surface, we remove the link of a shell_surface. > There is a case which the link references shell_client::surface_list > which is already free'd. > Therefore we have to remove shell_client::surface_list when freeing > a shell_client

[PATCH] desktop-shell: fix bug referencing invalid surface list

2015-07-29 Thread Ryo Munakata
In shell_destroy_shell_surface, we remove the link of a shell_surface. There is a case which the link references shell_client::surface_list which is already free'd. Therefore we have to remove shell_client::surface_list when freeing a shell_client. Signed-off-by: Ryo Munakata --- desktop-shell/s