Re: [PATCH] server: Fix possible wl_display_add_socket_fd memleak

2016-02-01 Thread Bryce Harrington
On Mon, Feb 01, 2016 at 02:57:08PM -0600, Derek Foreman wrote: > On 01/02/16 12:35 PM, Sergi Granell wrote: > > If wl_event_loop_add_fd failed, the fail path didn't free the > > newly allocated struct wl_socket. > > Looks good to me: > Reviewed-by: Derek Foreman Pushed: To ssh://git.freedesktop.

Re: [PATCH] server: Fix possible wl_display_add_socket_fd memleak

2016-02-01 Thread Derek Foreman
On 01/02/16 12:35 PM, Sergi Granell wrote: > If wl_event_loop_add_fd failed, the fail path didn't free the > newly allocated struct wl_socket. Looks good to me: Reviewed-by: Derek Foreman > --- > src/wayland-server.c | 9 + > 1 file changed, 5 insertions(+), 4 deletions(-) > > diff --g

[PATCH] server: Fix possible wl_display_add_socket_fd memleak

2016-02-01 Thread Sergi Granell
If wl_event_loop_add_fd failed, the fail path didn't free the newly allocated struct wl_socket. --- src/wayland-server.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/src/wayland-server.c b/src/wayland-server.c index 6654cd7..3ba8a5d 100644 --- a/src/wayland-server.c