output/wasapi: release the COMWorker if OpenDevice() fails

Fixes assertion failure in the Thread destructor.

output/wasapi: release the COMWorker if OpenDevice() fails
8b41c4f3 · Max Kellermann · 17f7098e · 8b41c4f3
Commit 8b41c4f3 authored Mar 05, 2021 by Max Kellermann
Hide whitespace changes
Inline Side-by-side

Showing with 7 additions and 1 deletion

WasapiOutputPlugin.cxx src/output/plugins/wasapi/WasapiOutputPlugin.cxx +7 -1

No files found.
--- a/src/output/plugins/wasapi/WasapiOutputPlugin.cxx
+++ b/src/output/plugins/wasapi/WasapiOutputPlugin.cxx
@@ -205,7 +205,13 @@ public:
 	WasapiOutput(const ConfigBlock &block);
 	void Enable() override {
 		COMWorker::Aquire();
-		COMWorker::Async([&]() { OpenDevice(); }).get();
+
+		try {
+			COMWorker::Async([&]() { OpenDevice(); }).get();
+		} catch (...) {
+			COMWorker::Release();
+			throw;
+		}
 	}
 	void Disable() noexcept override {
 		COMWorker::Async([&]() { DoDisable(); }).get();