diff options
author | Mithra Pattison <mithra.pattison@nokia.com> | 2012-01-06 12:54:26 +1000 |
---|---|---|
committer | Qt by Nokia <qt-info@nokia.com> | 2012-01-06 08:42:54 +0100 |
commit | 13d75ce063eb6438215c1bbb1cdda1bd2aab10cd (patch) | |
tree | 5dcffb14fca65dfceda6722bcd51df28d7559d13 | |
parent | 6b67a11032779ff3a7a78a57e8d2bb2cd8b7e525 (diff) |
Fix for QSoundEffect crash bug
When multiple QSoundEffect instances play the same wav source file
simultaneously, the system would crash due to some instances not
waiting for the underlying pulse audio stream to complete its
setup logic. QSoundEffect now waits for the stream to attain the
correct state before playing the sound.
Change-Id: Ib5a1e6bc3f1cc314054f9cdc89c10100ad546721
Reviewed-by: Ling Hu <ling.hu@nokia.com>
Reviewed-by: Michael Goddard <michael.goddard@nokia.com>
-rw-r--r-- | src/multimedia/audio/qsoundeffect_pulse_p.cpp | 3 | ||||
-rw-r--r-- | tests/auto/integration/qsound/test2.wav | bin | 38316 -> 0 bytes | |||
-rw-r--r-- | tests/auto/integration/qsound/tst_qsound.cpp | 2 |
3 files changed, 3 insertions, 2 deletions
diff --git a/src/multimedia/audio/qsoundeffect_pulse_p.cpp b/src/multimedia/audio/qsoundeffect_pulse_p.cpp index 373586ef5..58143fcc2 100644 --- a/src/multimedia/audio/qsoundeffect_pulse_p.cpp +++ b/src/multimedia/audio/qsoundeffect_pulse_p.cpp @@ -567,7 +567,7 @@ void QSoundEffectPrivate::play() return; PulseDaemonLocker locker; - if (!m_sampleReady || !m_pulseStream || m_stopping || m_emptying) { + if (!m_pulseStream || m_status != QSoundEffect::Ready || m_stopping || m_emptying) { #ifdef QT_PA_DEBUG qDebug() << this << "play deferred"; #endif @@ -794,6 +794,7 @@ void QSoundEffectPrivate::playSample() qDebug() << this << "playSample"; #endif Q_ASSERT(m_pulseStream); + Q_ASSERT(pa_stream_get_state(m_pulseStream) == PA_STREAM_READY); pa_operation_unref(pa_stream_cork(m_pulseStream, 0, 0, 0)); } diff --git a/tests/auto/integration/qsound/test2.wav b/tests/auto/integration/qsound/test2.wav Binary files differdeleted file mode 100644 index e4088a973..000000000 --- a/tests/auto/integration/qsound/test2.wav +++ /dev/null diff --git a/tests/auto/integration/qsound/tst_qsound.cpp b/tests/auto/integration/qsound/tst_qsound.cpp index 6f98c45bb..3cb060c19 100644 --- a/tests/auto/integration/qsound/tst_qsound.cpp +++ b/tests/auto/integration/qsound/tst_qsound.cpp @@ -123,7 +123,7 @@ void tst_QSound::testStop() void tst_QSound::testStaticPlay() { // Check that you hear sound with static play also. - const QString testFileName = QStringLiteral("test2.wav"); + const QString testFileName = QStringLiteral("test.wav"); const QString fullPath = QFINDTESTDATA(testFileName); QVERIFY2(!fullPath.isEmpty(), qPrintable(QStringLiteral("Unable to locate ") + testFileName)); |