1143540 - crash in `anonymous namespace''::wasapi_stream_start(cubeb_stream*)

Status: RESOLVED DUPLICATE of bug 1136107

(Core :: Audio/Video, defect)

Description

[(Away)]

This bug was filed from the Socorro interface and is 
report bp-2c46b73e-2e78-41db-96ed-345212150308.
=============================================================

Hard to say which of these is failing, but it's not |stm| since the code had just locked |stm->stream_reset_lock| on the line before:
  XASSERT(stm && !stm->thread && !stm->shutdown_event);

http://hg.mozilla.org/releases/mozilla-aurora/file/104dbec6acc2/media/libcubeb/src/cubeb_wasapi.cpp#l1192

0 	xul.dll 	`anonymous namespace'::wasapi_stream_start(cubeb_stream*) 	media/libcubeb/src/cubeb_wasapi.cpp
1 	xul.dll 	cubeb_stream_start 	media/libcubeb/src/cubeb.c
2 	xul.dll 	mozilla::AudioStream::Resume() 	dom/media/AudioStream.cpp
3 	xul.dll 	mozilla::AudioSink::IsPlaybackContinuing() 	dom/media/AudioSink.cpp
4 	xul.dll 	mozilla::AudioSink::AudioLoop() 	dom/media/AudioSink.cpp
5 	nss3.dll 	PR_EnterMonitor 	nsprpub/pr/src/threads/prmon.c
6 	xul.dll 	nsRunnableMethodImpl<nsresult ( mozilla::net::CacheFileIOManager::*)(void), void, 1>::Run() 	xpcom/glue/nsThreadUtils.h
7 	ntdll.dll 	RtlpLowFragHeapAllocFromContext 	
8 	xul.dll 	NS_ProcessNextEvent(nsIThread*, bool) 	xpcom/glue/nsThreadUtils.cpp
9 	xul.dll 	mozilla::ipc::MessagePumpForNonMainThreads::Run(base::MessagePump::Delegate*) 	ipc/glue/MessagePump.cpp

I assume this signature is also related: bp-90d71680-261c-4a26-80aa-e69382150308 -- in that one we seem to have a null |stm->client| in wasapi_stream_stop.

http://hg.mozilla.org/releases/mozilla-aurora/file/bcfac4ff0a06/media/libcubeb/src/cubeb_wasapi.cpp#l1223

Comment 1

[(Away)]

[Tracking Requested - why for this release]: Top crash in aurora 38

Comment 2

[Sylvestre Ledru [:Sylvestre]]

Top crash, tracking.

Comment 3

[Matthew Gregan [:kinetik]]

Fixed in bug 1136107, but it was uplifted to aurora on the 12th (i.e. after the builds these crashes are reported in).

Comment 4

[(Away)]

Ah okay. This assumption was wrong:

> I assume this signature is also related: bp-90d71680-261c-4a26-80aa-e69382150308
> null |stm->client| in wasapi_stream_stop.

The wasapi_stream_start issue has been fixed but wasapi_stream_stop is still crashing. I've separated the latter into bug 1143968.

Comment 5

[Liz Henry (:lizzard) (relman/hg->git project)]

OK, untracking this for 38 and 39 and tracking bug 1143968 instead.