do not build aws sdk or related tests in ci, fix free order to free l…

…ock first and then cvar for jssWaitLock
awslabs · Oct 6, 2023 · 3a9a174 · 3a9a174
1 parent db08822
commit 3a9a174
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git 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
diff --git 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);
     }