History

When we don't make an audo injector (because we're at a limit), make sure the value returned to scripts is null.

Howard Stearns committed 10 years ago

Commit fcdc6dd1 parent 04543e71 file · +5 −2Browse files

Showing the changes to libraries/script-engine/src/AudioScriptingInterface.cpp (1 of 1 file). Show all

Unified · Split

libraries/script-engine/src/AudioScriptingInterface.cpp+5−2
@@ -45,8 +45,11 @@ ScriptAudioInjector* AudioScriptingInterface::playSound(Sound* sound, const Audi · 44 unmodified lines
4545 // stereo option isn't set from script, this comes from sound metadata or filename
4646 AudioInjectorOptions optionsCopy = injectorOptions;
4747 optionsCopy.stereo = sound->isStereo();
48−
49− return new ScriptAudioInjector(AudioInjector::playSound(sound->getByteArray(), optionsCopy, _localAudioInterface));
48+ auto injector = AudioInjector::playSound(sound->getByteArray(), optionsCopy, _localAudioInterface);
49+ if (!injector) {
50+ return NULL;
51+ }
52+ return new ScriptAudioInjector(injector);
5053
5154 } else {
5255 qCDebug(scriptengine) << "AudioScriptingInterface::playSound called with null Sound object.";