alsasink: fix occasional crash intersecting invalid values
authorVincent Penquerc'h <vincent.penquerch@collabora.co.uk>
Tue, 3 Jun 2014 14:10:33 +0000 (15:10 +0100)
committerVincent Penquerc'h <vincent.penquerch@collabora.co.uk>
Tue, 3 Jun 2014 14:17:20 +0000 (15:17 +0100)
commit3b2d5833732a29672689badf3edf69867679e052
tree162fe2716f119c97bb7f16f2e1357cf14a400d5c
parentd2ea3260e934a192e1d6e421fd2f90fc39c02c85
alsasink: fix occasional crash intersecting invalid values

When a pipeline using alsasink and push mode upstream fails
to preroll, the following state will be the case:
- A loop upstream will be PAUSED, pushing a first buffer
- alsasink will be READY, pending PAUSED, because async

On error, the pipeline will switch to NULL. alsasink is in
READY, so goes to NULL immediately. It zeroes its cached
caps. Meanwhile, the upstream loop can cause a caps query,
conccurent with the state change. This will use those cached
caps. If the zeroing happens between the NULL test and the
dereferencing, GStreamer will critical down in the GstValue
code.

Since it appears that such a gap between states (PAUSED
and pushing upstream, and NULL downstream) is expected, we
need to protect the read/write access to the cached caps.

This fixes the critical.

See https://bugzilla.gnome.org/show_bug.cgi?id=731121
ext/alsa/gstalsasink.c