Re: [PATCH wayland 2/2] cursor: fix some leaks

2012-07-10 Thread Pekka Paalanen
On Mon, 9 Jul 2012 18:02:51 -0400 Kristian Høgsberg wrote: > On Mon, Jul 09, 2012 at 01:34:35PM +0300, Pekka Paalanen wrote: > > - don't leak memory in shm_pool_create() on failure > > - don't leak fd in shm_pool_destroy() > > - return NULL from wl_cursor_theme_load() if pool fails > > > > Tha l

Re: [PATCH wayland 2/2] cursor: fix some leaks

2012-07-09 Thread Kristian Høgsberg
On Mon, Jul 09, 2012 at 01:34:35PM +0300, Pekka Paalanen wrote: > - don't leak memory in shm_pool_create() on failure > - don't leak fd in shm_pool_destroy() > - return NULL from wl_cursor_theme_load() if pool fails > > Tha last one fixes a segfault, when shm_pool_create() has failed. I think Mar

[PATCH wayland 2/2] cursor: fix some leaks

2012-07-09 Thread Pekka Paalanen
- don't leak memory in shm_pool_create() on failure - don't leak fd in shm_pool_destroy() - return NULL from wl_cursor_theme_load() if pool fails Tha last one fixes a segfault, when shm_pool_create() has failed. Signed-off-by: Pekka Paalanen --- cursor/wayland-cursor.c | 11 ++- 1 fil