From 3a9a1744e007dd2e3bd08982201e8f0cf63de128 Mon Sep 17 00:00:00 2001 From: Hassan Sahibzada Date: Fri, 6 Oct 2023 16:50:19 -0400 Subject: [PATCH] do not build aws sdk or related tests in ci, fix free order to free lock first and then cvar for jssWaitLock --- .github/build_windows_openssl.bat | 2 +- src/source/Signaling/Signaling.c | 8 ++++---- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/.github/build_windows_openssl.bat b/.github/build_windows_openssl.bat index c168aa96b4..3fb65e786d 100644 --- a/.github/build_windows_openssl.bat +++ b/.github/build_windows_openssl.bat @@ -2,5 +2,5 @@ call "C:\Program Files\Microsoft Visual Studio\2022\Enterprise\VC\Auxiliary\Buil mkdir build cd build cmd.exe /c cmake -G "NMake Makefiles" .. -cmake -G "NMake Makefiles" -DBUILD_TEST=TRUE -DEXT_PTHREAD_INCLUDE_DIR="C:/tools/pthreads-w32-2-9-1-release/Pre-built.2/include/" -DEXT_PTHREAD_LIBRARIES="C:/tools/pthreads-w32-2-9-1-release/Pre-built.2/lib/x64/libpthreadGC2.a" .. +cmake -G "NMake Makefiles" -DBUILD_TEST=TRUE -DENABLE_AWS_SDK_IN_TESTS=OFF -DEXT_PTHREAD_INCLUDE_DIR="C:/tools/pthreads-w32-2-9-1-release/Pre-built.2/include/" -DEXT_PTHREAD_LIBRARIES="C:/tools/pthreads-w32-2-9-1-release/Pre-built.2/lib/x64/libpthreadGC2.a" .. nmake \ No newline at end of file diff --git a/src/source/Signaling/Signaling.c b/src/source/Signaling/Signaling.c index 0b5ac78cdb..254b4f6a71 100644 --- a/src/source/Signaling/Signaling.c +++ b/src/source/Signaling/Signaling.c @@ -262,14 +262,14 @@ STATUS freeSignaling(PSignalingClient* ppSignalingClient) CVAR_FREE(pSignalingClient->receiveCvar); } - if (IS_VALID_CVAR_VALUE(pSignalingClient->jssWaitCvar)) { - CVAR_FREE(pSignalingClient->jssWaitCvar); - } - if (IS_VALID_MUTEX_VALUE(pSignalingClient->jssWaitLock)) { MUTEX_FREE(pSignalingClient->jssWaitLock); } + if (IS_VALID_CVAR_VALUE(pSignalingClient->jssWaitCvar)) { + CVAR_FREE(pSignalingClient->jssWaitCvar); + } + if (IS_VALID_MUTEX_VALUE(pSignalingClient->stateLock)) { MUTEX_FREE(pSignalingClient->stateLock); }