[Mesa-stable] [PATCH 1/4] i965/sync: Fix uninitalized usage and leak of mutex

Emil Velikov emil.l.velikov at gmail.com
Thu Oct 6 15:28:04 UTC 2016


Hi Chad,

On 4 October 2016 at 23:37, Chad Versace <chadversary at chromium.org> wrote:
> We locked an unitialized mutex in the callstack
>     glClientWaitSync
>     intel_gl_client_wait_sync
>     brw_fence_client_wait_sync
> because we forgot to initialize it in intel_gl_fence_sync.
> (The EGLSync codepath didn't have this bug. It initialized the mutex in
> intel_dri_create_sync).
>
> We also forgot to tear down (mtx_destroy) the mutex when destroying
> the sync object.
>
> Cc: mesa-stable at lists.freedesktop.org
Do you have a few minutes to roll an similar fix for i915 ? It should
be identical (barring naming fixes) to this.

Thanks
Emil


More information about the mesa-stable mailing list