Re: [PATCH wayland] server: fix potential memleak and NULL deref

2014-05-06 Thread Kristian Høgsberg
On Mon, May 05, 2014 at 04:28:26PM -0700, U. Artie Eoff wrote: > If for some reason that errno is neither value (ENOMEM or > EINVAL), then prior to this patch, there would be a NULL > deref in wl_closure_lookup(...) at the "else if" conditional > when closure == NULL. Also, closure might not be NUL

[PATCH wayland] server: fix potential memleak and NULL deref

2014-05-05 Thread U. Artie Eoff
If for some reason that errno is neither value (ENOMEM or EINVAL), then prior to this patch, there would be a NULL deref in wl_closure_lookup(...) at the "else if" conditional when closure == NULL. Also, closure might not be NULL but still fall into the block due to the wl_closure_lookup < 0 condit