Re: [PATCH wayland 2/4] wl_array: Set data to null after free

2016-09-20 Thread Pekka Paalanen
On Mon, 19 Sep 2016 12:08:21 +0100 Eric Engestrom wrote: > On Mon, Sep 19, 2016 at 11:59:03AM +0100, Eric Engestrom wrote: > > On Fri, Sep 16, 2016 at 03:37:37PM -0700, Yong Bakos wrote: > > > From: Yong Bakos > > > > > > Explicitly set the data member to NULL during wl_array_release, > > >

Re: [PATCH wayland 2/4] wl_array: Set data to null after free

2016-09-19 Thread Yong Bakos
On Sep 19, 2016, at 4:08 AM, Eric Engestrom wrote: > > On Mon, Sep 19, 2016 at 11:59:03AM +0100, Eric Engestrom wrote: >> On Fri, Sep 16, 2016 at 03:37:37PM -0700, Yong Bakos wrote: >>> From: Yong Bakos >>> >>> Explicitly set the data member to NULL during wl_array_release, preventing >>> the

Re: [PATCH wayland 2/4] wl_array: Set data to null after free

2016-09-19 Thread Eric Engestrom
On Mon, Sep 19, 2016 at 11:59:03AM +0100, Eric Engestrom wrote: > On Fri, Sep 16, 2016 at 03:37:37PM -0700, Yong Bakos wrote: > > From: Yong Bakos > > > > Explicitly set the data member to NULL during wl_array_release, preventing > > the > > dangling pointer but, more importantly, making it test

Re: [PATCH wayland 2/4] wl_array: Set data to null after free

2016-09-19 Thread Eric Engestrom
On Fri, Sep 16, 2016 at 03:37:37PM -0700, Yong Bakos wrote: > From: Yong Bakos > > Explicitly set the data member to NULL during wl_array_release, preventing the > dangling pointer but, more importantly, making it testable. > > Signed-off-by: Yong Bakos > --- > src/wayland-util.c | 1 + > 1 fi

[PATCH wayland 2/4] wl_array: Set data to null after free

2016-09-16 Thread Yong Bakos
From: Yong Bakos Explicitly set the data member to NULL during wl_array_release, preventing the dangling pointer but, more importantly, making it testable. Signed-off-by: Yong Bakos --- src/wayland-util.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/wayland-util.c b/src/wayland-util