Fix nasa#1461, Move data lock to inside of if block

thnkslprpt · Nov 8, 2022 · e300c81 · e300c81
1 parent 7a220ae
commit e300c81
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/modules/sb/fsw/src/cfe_sb_api.c b/modules/sb/fsw/src/cfe_sb_api.c
@@ -2096,16 +2096,16 @@ CFE_Status_t CFE_SB_ReleaseMessageBuffer(CFE_SB_Buffer_t *BufPtr)
 
     Status = CFE_SB_ZeroCopyBufferValidate(BufPtr, &BufDscPtr);
 
-    CFE_SB_LockSharedData(__func__, __LINE__);
-
     if (Status == CFE_SUCCESS)
     {
+        CFE_SB_LockSharedData(__func__, __LINE__);
+
         /* Clear the ownership app ID and decrement use count (may also free) */
         BufDscPtr->AppId = CFE_ES_APPID_UNDEFINED;
         CFE_SB_DecrBufUseCnt(BufDscPtr);
-    }
 
-    CFE_SB_UnlockSharedData(__func__, __LINE__);
+        CFE_SB_UnlockSharedData(__func__, __LINE__);
+    }
 
     return Status;
 }