Fix build breaks with clang 21 (#120644)

Contributes to #119706

Author: jkotas

src/coreclr/debug/di/rspriv.h

@@ -6397,8 +6397,8 @@ class CordbThread : public CordbBase, public ICorDebugThread,
     // Lazily initialized.
     EXCEPTION_RECORD *  m_pExceptionRecord;
 
-    static const CorDebugUserState kInvalidUserState = CorDebugUserState(-1);
-    CorDebugUserState     m_userState;  // This is the current state of the
+    static const int kInvalidUserState = -1;
+    int                   m_userState;  // This is the current state of the
                                         // thread, at the time that the
                                         // left side synchronized
 

src/coreclr/debug/di/rsthread.cpp

@@ -782,7 +782,7 @@ CorDebugUserState CordbThread::GetUserState()
         m_userState = pDAC->GetUserState(m_vmThreadToken);
     }
 
-    return m_userState;
+    return (CorDebugUserState)m_userState;
 }
 
 
@@ -886,7 +886,7 @@ HRESULT CordbThread::CreateStepper(ICorDebugStepper ** ppStepper)
 //Returns true if current user state of a thread is USER_WAIT_SLEEP_JOIN
 bool CordbThread::IsThreadWaitingOrSleeping()
 {
-    CorDebugUserState userState = m_userState;
+    int userState = m_userState;
     if (userState == kInvalidUserState)
     {
         //If m_userState is not ready, we'll read from DAC only part of it which

src/native/libs/System.Native/pal_networking.c

@@ -407,8 +407,10 @@ int32_t SystemNative_GetHostEntryForName(const uint8_t* address, int32_t address
     char name[_POSIX_HOST_NAME_MAX];
     result = gethostname((char*)name, _POSIX_HOST_NAME_MAX);
 
-    bool includeIPv4Loopback = true;
-    bool includeIPv6Loopback = true;
+    bool includeIPv4Loopback;
+    bool includeIPv6Loopback;
+    includeIPv4Loopback = true;
+    includeIPv6Loopback = true;
 
     if (result == 0 && strcasecmp((const char*)address, name) == 0)
     {
@@ -1526,7 +1528,7 @@ int32_t SystemNative_ReceiveSocketError(intptr_t socket, MessageHeader* messageH
 #if HAVE_LINUX_ERRQUEUE_H
     char buffer[sizeof(struct sock_extended_err) + sizeof(struct sockaddr_storage)];
     messageHeader->ControlBufferLen = sizeof(buffer);
-    messageHeader->ControlBuffer = (void*)buffer;
+    messageHeader->ControlBuffer = (uint8_t*)buffer;
 
     struct msghdr header;
     struct icmphdr icmph;

src/native/libs/System.Native/pal_process.c

@@ -254,7 +254,7 @@ int32_t SystemNative_ForkAndExecProcess(const char* filename,
 
     if (setCredentials && groupsLength > 0)
     {
-        getGroupsBuffer = malloc(sizeof(uint32_t) * Int32ToSizeT(groupsLength));
+        getGroupsBuffer = (uint32_t*)(malloc(sizeof(uint32_t) * Int32ToSizeT(groupsLength)));
         if (getGroupsBuffer == NULL)
         {
             success = false;

src/native/libs/System.Native/pal_signal.c

@@ -130,7 +130,7 @@ static bool TryConvertSignalCodeToPosixSignal(int signalCode, PosixSignal* posix
             return true;
 
         default:
-            *posixSignal = signalCode;
+            *posixSignal = (PosixSignal)signalCode;
             return false;
     }
 }

src/native/libs/System.Native/pal_threading.c

@@ -81,9 +81,9 @@ LowLevelMonitor* SystemNative_LowLevelMonitor_Create(void)
 
     error = pthread_cond_init(&monitor->Condition, &conditionAttributes);
 
-    int condAttrDestroyError = pthread_condattr_destroy(&conditionAttributes);
+    int condAttrDestroyError;
+    condAttrDestroyError = pthread_condattr_destroy(&conditionAttributes);
     assert(condAttrDestroyError == 0);
-    (void)condAttrDestroyError; // unused in release build
 #else
     error = pthread_cond_init(&monitor->Condition, NULL);
 #endif
@@ -117,18 +117,17 @@ void SystemNative_LowLevelMonitor_Destroy(LowLevelMonitor* monitor)
     error = pthread_mutex_destroy(&monitor->Mutex);
     assert(error == 0);
 
-    (void)error; // unused in release build
-
     free(monitor);
 }
 
 void SystemNative_LowLevelMonitor_Acquire(LowLevelMonitor* monitor)
 {
     assert(monitor != NULL);
 
-    int error = pthread_mutex_lock(&monitor->Mutex);
+    int error;
+
+    error = pthread_mutex_lock(&monitor->Mutex);
     assert(error == 0);
-    (void)error; // unused in release build
 
     SetIsLocked(monitor, true);
 }
@@ -139,9 +138,10 @@ void SystemNative_LowLevelMonitor_Release(LowLevelMonitor* monitor)
 
     SetIsLocked(monitor, false);
 
-    int error = pthread_mutex_unlock(&monitor->Mutex);
+    int error;
+
+    error = pthread_mutex_unlock(&monitor->Mutex);
     assert(error == 0);
-    (void)error; // unused in release build
 }
 
 void SystemNative_LowLevelMonitor_Wait(LowLevelMonitor* monitor)
@@ -150,9 +150,10 @@ void SystemNative_LowLevelMonitor_Wait(LowLevelMonitor* monitor)
 
     SetIsLocked(monitor, false);
 
-    int error = pthread_cond_wait(&monitor->Condition, &monitor->Mutex);
+    int error;
+
+    error = pthread_cond_wait(&monitor->Condition, &monitor->Mutex);
     assert(error == 0);
-    (void)error; // unused in release build
 
     SetIsLocked(monitor, true);
 }
@@ -212,8 +213,6 @@ void SystemNative_LowLevelMonitor_Signal_Release(LowLevelMonitor* monitor)
 
     error = pthread_mutex_unlock(&monitor->Mutex);
     assert(error == 0);
-
-    (void)error; // unused in release build
 }
 
 int32_t SystemNative_CreateThread(uintptr_t stackSize, void *(*startAddress)(void*), void *parameter)

src/native/libs/System.Net.Security.Native/pal_gssapi.c

@@ -244,7 +244,7 @@ uint32_t NetSecurityNative_ImportPrincipalName(uint32_t* minorStatus,
     // Principal name will usually be in the form SERVICE/HOST. But SPNEGO protocol prefers
     // GSS_C_NT_HOSTBASED_SERVICE format. That format uses '@' separator instead of '/' between
     // service name and host name. So convert input string into that format.
-    char* ptrSlash = memchr(inputName, '/', inputNameLen);
+    char* ptrSlash = (char*)memchr(inputName, '/', inputNameLen);
     char* inputNameCopy = NULL;
     if (ptrSlash != NULL)
     {

src/native/libs/System.Security.Cryptography.Native/memory_debug.c

@@ -164,7 +164,7 @@ static void do_track_entry(header_t* entry, int32_t add)
 
 static void* mallocFunction(size_t size, const char *file, int line)
 {
-    header_t* entry = malloc(size + sizeof(header_t));
+    header_t* entry = (header_t*)malloc(size + sizeof(header_t));
     if (entry != NULL)
     {
         init_memory_entry(entry, size, file, line);
@@ -187,7 +187,7 @@ static void* reallocFunction (void *ptr, size_t size, const char *file, int line
     }
 
     void* toReturn = NULL;
-    header_t* newEntry = (header_t*) realloc((void*)entry, size + sizeof(header_t));
+    header_t* newEntry = (header_t*)realloc((void*)entry, size + sizeof(header_t));
     if (newEntry != NULL)
     {
         entry = newEntry;
@@ -279,7 +279,7 @@ void CryptoNative_ForEachTrackedAllocation(void (*callback)(void* ptr, uint64_t
 
 static void init_tracking_lists(void)
 {
-    g_trackedMemory = malloc(kPartitionCount * sizeof(list_t));
+    g_trackedMemory = (list_t*)malloc(kPartitionCount * sizeof(list_t));
     for (uint32_t i = 0; i < kPartitionCount; i++)
     {
         list_init(&g_trackedMemory[i]);

src/native/libs/System.Security.Cryptography.Native/openssl.c

@@ -1371,6 +1371,7 @@ static int32_t EnsureOpenSsl10Initialized(void)
     int numLocks = 0;
     int locksInitialized = 0;
     int randPollResult = 0;
+    size_t allocationSize = 0;
 
     pthread_mutex_lock(&g_initLock);
 
@@ -1390,7 +1391,6 @@ static int32_t EnsureOpenSsl10Initialized(void)
     }
 
     // Create the locks array
-    size_t allocationSize = 0;
     if (!multiply_s(sizeof(pthread_mutex_t), (size_t)numLocks, &allocationSize))
     {
         ret = 2;

src/native/libs/System.Security.Cryptography.Native/pal_ecc_import_export.c

@@ -790,6 +790,8 @@ int32_t CryptoNative_EvpPKeyGetEcCurveParameters(
     EC_GROUP* group = NULL;
     size_t generatorBufferSize = 0;
     unsigned char* generatorBuffer = NULL;
+    int curveTypeNID;
+    int fieldTypeNID;
 
     // Exit if CryptoNative_EvpPKeyGetEcKeyParameters failed
     if (rc != 1)
@@ -798,7 +800,6 @@ int32_t CryptoNative_EvpPKeyGetEcCurveParameters(
     if (!xBn || !yBn)
         goto error;
 
-    int curveTypeNID;
     if (!CryptoNative_EvpPKeyGetEcGroupNid(pkey, &curveTypeNID) || !curveTypeNID)
         goto error;
 
@@ -819,7 +820,7 @@ int32_t CryptoNative_EvpPKeyGetEcCurveParameters(
     // and some providers seem to be ignoring OSSL_PKEY_PARAM_EC_FIELD_TYPE.
     // This is specifically true for tpm2 provider.
     // We can reliably get the field type from the EC_GROUP.
-    int fieldTypeNID = EC_GROUP_get_field_type(group);
+    fieldTypeNID = EC_GROUP_get_field_type(group);
 
     *curveType = NIDToCurveType(fieldTypeNID);
     if (*curveType == Unspecified)