github/azure-sdk-for-cpp
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

remove double checking mutex (#528)

Browse Source
This commit is contained in:
Victor Vazquez 2020-08-25 22:52:02 +00:00 committed by GitHub
parent 80f2c2f407
commit 760c645af0
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 0 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
sdk/core/azure-core/src/http/curl/curl.cpp
View File

@ -786,9 +786,6 @@ std::unique_ptr<CurlSession::CurlConnection> 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