[waffle] [PATCH] core: Zero-initialize wcore_info struct.

Chad Versace chad.versace at linux.intel.com
Tue Aug 20 12:29:28 PDT 2013


On 08/20/2013 08:55 AM, Paul Berry wrote:
> Without this, it's possible that the garbage memory returned by malloc
> will have wcore_info::is_init == true, causing wcore_tinfo_init() to
> skip initialization.  That in turn causes a likely segfault the first
> time wcore_info::error is accessed.
>
> Fixes piglit crashes introduced by commit 6fcbe52 ("cmake: Refactor
> compiler flags into new module").
>

Thanks. Committed to master.

> Note: I'm not exactly sure why the piglit crashes only started
> occurring with 6fcbe52.  My best guess is that the non-TLS code path
> has always had this bug, and the cmake refactor caused me to switch
> from building with TLS to building without TLS.
>
> If that's the case then it may be worth fixing as well :)

You're right. My build stopped using TLS too. I'm looking into that now.





More information about the waffle mailing list