From 410d70d965d68ead2daa83ee7f0b694cebd0dad2 Mon Sep 17 00:00:00 2001 From: henrika Date: Mon, 3 May 2021 09:16:44 +0200 Subject: [PATCH] Fixes minor issue in CreateDeviceInternal (ADM2 on Windows) Resolves an old (unresolved) review comment in https://webrtc-review.googlesource.com/c/src/+/160050/6/modules/audio_device/win/core_audio_utility_win.cc#325 The mistake was pointed out by dkirovbroadsoft@gmail.com. Thanks! Tbr: thaloun@chromium.org Bug: webrtc:11107 Change-Id: Ib732eaea8b07c6d6fb0b8963b00c3b009ccb8fee Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/217120 Reviewed-by: Henrik Andreassson Commit-Queue: Henrik Andreassson Cr-Commit-Position: refs/heads/master@{#33893} --- modules/audio_device/win/core_audio_utility_win.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/modules/audio_device/win/core_audio_utility_win.cc b/modules/audio_device/win/core_audio_utility_win.cc index f17ee99143..289abe9d26 100644 --- a/modules/audio_device/win/core_audio_utility_win.cc +++ b/modules/audio_device/win/core_audio_utility_win.cc @@ -323,7 +323,7 @@ ComPtr CreateDeviceInternal(const std::string& device_id, // Verify that the audio endpoint device is active, i.e., that the audio // adapter that connects to the endpoint device is present and enabled. - if (SUCCEEDED(error.Error()) && !audio_endpoint_device.Get() && + if (SUCCEEDED(error.Error()) && audio_endpoint_device.Get() && !IsDeviceActive(audio_endpoint_device.Get())) { RTC_LOG(LS_WARNING) << "Selected endpoint device is not active"; audio_endpoint_device.Reset();