From 760c645af081bceb4b17ce136d40141d0839f227 Mon Sep 17 00:00:00 2001 From: Victor Vazquez Date: Tue, 25 Aug 2020 22:52:02 +0000 Subject: [PATCH] remove double checking mutex (#528) --- sdk/core/azure-core/src/http/curl/curl.cpp | 3 --- 1 file changed, 3 deletions(-) diff --git a/sdk/core/azure-core/src/http/curl/curl.cpp b/sdk/core/azure-core/src/http/curl/curl.cpp index ffd9c9188..8aa9daa7e 100644 --- a/sdk/core/azure-core/src/http/curl/curl.cpp +++ b/sdk/core/azure-core/src/http/curl/curl.cpp @@ -786,9 +786,6 @@ std::unique_ptr CurlSession::GetCurlConnection(Requ { std::string const& host = request.GetHost(); - // Double-check locking. Check if there is any available connection before locking mutex - auto& hostPoolFirstCheck = s_connectionPoolIndex[host]; - if (hostPoolFirstCheck.size() > 0) { // Critical section. Needs to own s_connectionPoolMutex before executing // Lock mutex to access connection pool. mutex is unlock as soon as lock is out of scope