Re: [PATCH wayland v2 2/4] wl_array: Set data to invalid address after free

2016-09-29 Thread Eric Engestrom
On Thu, Sep 29, 2016 at 07:17:37AM -0700, Yong Bakos wrote: > Thanks Eric. Would you mind cc'ing the wayland-devel list, so that Patchwork > catches your RB? Sorry, on MLs I usually just hit reply-all without looking at the list of recipients. So, for patchwork: The series is: Reviewed-by:

Re: [PATCH wayland v2 2/4] wl_array: Set data to invalid address after free

2016-09-28 Thread Eric Engestrom
On Tue, Sep 27, 2016 at 01:03:48PM -0500, Yong Bakos wrote: > From: Yong Bakos > > Explicitly set the data member to an invalid memory address during > wl_array_release, such that re-using a freed wl_array without re-initializing > causes a crash. In addition, this

[PATCH wayland v2 2/4] wl_array: Set data to invalid address after free

2016-09-27 Thread Yong Bakos
From: Yong Bakos Explicitly set the data member to an invalid memory address during wl_array_release, such that re-using a freed wl_array without re-initializing causes a crash. In addition, this pointer assignment makes wl_array_release testable. Define a constant for