Remove PAL_Random and move palrt APIs into the minipal

src/coreclr/CMakeLists.txt

@@ -250,10 +250,6 @@ if(NOT CLR_CMAKE_HOST_MACCATALYST AND NOT CLR_CMAKE_HOST_IOS AND NOT CLR_CMAKE_H
   add_subdirectory(utilcode)
   add_subdirectory(inc)
 
-  if(CLR_CMAKE_HOST_UNIX)
-      add_subdirectory(palrt)
-  endif(CLR_CMAKE_HOST_UNIX)
-
   add_subdirectory(ilasm)
   add_subdirectory(ildasm)
   add_subdirectory(gcinfo)

src/coreclr/debug/createdump/crashinfo.cpp

@@ -1026,7 +1026,7 @@ ConvertString(const WCHAR* str)
     if (str == nullptr)
         return { };
 
-    size_t cch = u16_strlen(str) + 1;
+    size_t cch = minipal_u16_strlen((CHAR16_T*)str) + 1;
     int len = minipal_get_length_utf16_to_utf8((CHAR16_T*)str, cch, 0);
     if (len == 0)
         return { };

src/coreclr/debug/createdump/createdump.h

@@ -51,6 +51,7 @@ typedef int T_CONTEXT;
 #include <arrayholder.h>
 #include <releaseholder.h>
 #ifdef HOST_UNIX
+#include <minipal/strings.h>
 #include <minipal/utf8.h>
 #include <dn-u16.h>
 #include <dumpcommon.h>

src/coreclr/debug/createdump/createdumppal.cpp

@@ -217,14 +217,6 @@ RaiseException(
     throw;
 }
 
-size_t u16_strlen(const WCHAR* str)
-{
-    size_t nChar = 0;
-    while (*str++)
-        nChar++;
-    return nChar;
-}
-
 //
 // Used by _ASSERTE
 //

src/coreclr/debug/createdump/datatarget.cpp

@@ -102,7 +102,7 @@ DumpDataTarget::GetImageBase(
     *baseAddress = 0;
 
     char tempModuleName[MAX_PATH];
-    size_t cch = u16_strlen(moduleName) + 1;
+    size_t cch = minipal_u16_strlen((CHAR16_T*)moduleName) + 1;
     int length = minipal_convert_utf16_to_utf8((CHAR16_T*)moduleName, cch, tempModuleName, sizeof(tempModuleName), 0);
     if (length > 0)
     {

src/coreclr/debug/daccess/daccess.cpp

@@ -264,7 +264,7 @@ SplitFullName(_In_z_ PCWSTR fullName,
     else
     {
         *params = NULL;
-        memberEnd = fullName + (u16_strlen(fullName) - 1);
+        memberEnd = fullName + (minipal_u16_strlen((const CHAR16_T*)fullName) - 1);
     }
 
     if (syntax != SPLIT_TYPE)
@@ -5680,7 +5680,7 @@ static int FormatCLRStubName(
     // Compute the address as a string safely.
     WCHAR addrString[Max64BitHexString + 1];
     FormatInteger(addrString, ARRAY_SIZE(addrString), "%p", stubAddr);
-    size_t addStringLen = u16_strlen(addrString);
+    size_t addStringLen = minipal_u16_strlen((const CHAR16_T*)addrString);
 
     // Compute maximum length, include the null terminator.
     size_t formatName_MaxLen = ARRAY_SIZE(formatName_Prefix) // Include trailing null
@@ -5691,7 +5691,7 @@ static int FormatCLRStubName(
     size_t stubManagedNameLen = 0;
     if (stubNameMaybe != NULL)
     {
-        stubManagedNameLen = u16_strlen(stubNameMaybe);
+        stubManagedNameLen = minipal_u16_strlen((const CHAR16_T*)stubNameMaybe);
         formatName_MaxLen += ARRAY_SIZE(formatName_OpenBracket) - 1;
         formatName_MaxLen += ARRAY_SIZE(formatName_CloseBracket) - 1;
     }
@@ -6458,7 +6458,7 @@ ClrDataAccess::GetMetaDataFileInfoFromPEFile(PEAssembly *pPEAssembly,
     // It is possible that the module is in-memory. That is the wszFilePath here is empty.
     // We will try to use the module name instead in this case for hosting debugger
     // to find match.
-    if (u16_strlen(wszFilePath) == 0)
+    if (minipal_u16_strlen((const CHAR16_T*)wszFilePath) == 0)
     {
         mdImage->GetModuleFileNameHintForDAC().DacGetUnicode(cchFilePath, wszFilePath, &uniPathChars);
         if (uniPathChars > cchFilePath)
@@ -7259,9 +7259,9 @@ STDAPI OutOfProcessExceptionEventCallback(_In_ PDWORD pContext,
         return hr;
     }
 
-    if ((pwszEventName == NULL) || (*pchSize <= u16_strlen(gmb.wzEventTypeName)))
+    if ((pwszEventName == NULL) || (*pchSize <= minipal_u16_strlen((const CHAR16_T*)gmb.wzEventTypeName)))
     {
-        *pchSize = static_cast<DWORD>(u16_strlen(gmb.wzEventTypeName)) + 1;
+        *pchSize = static_cast<DWORD>(minipal_u16_strlen((const CHAR16_T*)gmb.wzEventTypeName)) + 1;
         return HRESULT_FROM_WIN32(ERROR_INSUFFICIENT_BUFFER);
     }
 
@@ -7383,9 +7383,9 @@ STDAPI OutOfProcessExceptionEventSignatureCallback(_In_ PDWORD pContext,
     // Return pwszName as an emptry string to let WER use localized version of "Parameter n"
     *pwszName = W('\0');
 
-    if ((pwszValue == NULL) || (*pchValue <= u16_strlen(pwszBucketValues[dwIndex])))
+    if ((pwszValue == NULL) || (*pchValue <= minipal_u16_strlen((const CHAR16_T*)pwszBucketValues[dwIndex])))
     {
-        *pchValue = static_cast<DWORD>(u16_strlen(pwszBucketValues[dwIndex]))+ 1;
+        *pchValue = static_cast<DWORD>(minipal_u16_strlen((const CHAR16_T*)pwszBucketValues[dwIndex]))+ 1;
         return HRESULT_FROM_WIN32(ERROR_INSUFFICIENT_BUFFER);
     }
 

src/coreclr/debug/daccess/enummem.cpp

@@ -233,14 +233,14 @@ HRESULT ClrDataAccess::EnumMemCLRStatic(IN CLRDataEnumMemoryFlags flags)
             WCHAR* path = entryAssemblyPath;
             if (path != NULL)
             {
-                size_t pathLen = u16_strlen(path) + 1;
+                size_t pathLen = minipal_u16_strlen((const CHAR16_T*)path) + 1;
 
                 // Get the file name based on the last directory separator
                 const WCHAR* name = u16_strrchr(path, DIRECTORY_SEPARATOR_CHAR_W);
                 if (name != NULL)
                 {
                     name += 1;
-                    size_t len = u16_strlen(name) + 1;
+                    size_t len = minipal_u16_strlen((const CHAR16_T*)name) + 1;
                     wcscpy_s(path, len, name);
 
                     // Null out the rest of the buffer

src/coreclr/debug/daccess/inspect.cpp

@@ -1045,7 +1045,7 @@ ClrDataValue::GetString(
 
             if (strLen)
             {
-                *strLen = static_cast<ULONG32>(u16_strlen(msgStr) + 1);
+                *strLen = static_cast<ULONG32>(minipal_u16_strlen((const CHAR16_T*)msgStr) + 1);
             }
             status = StringCchCopy(str, bufLen, msgStr) == S_OK ?
                 S_OK : S_FALSE;

src/coreclr/debug/daccess/request.cpp

@@ -700,7 +700,7 @@ ClrDataAccess::GetRegisterName(int regNum, unsigned int count, _Inout_updates_z_
     const WCHAR callerPrefix[] = W("caller.");
     // Include null terminator in prefixLen/regLen because wcscpy_s will fail otherwise
     unsigned int prefixLen = (unsigned int)ARRAY_SIZE(callerPrefix);
-    unsigned int regLen = (unsigned int)u16_strlen(regs[regNum]) + 1;
+    unsigned int regLen = (unsigned int)minipal_u16_strlen((const CHAR16_T*)regs[regNum]) + 1;
     unsigned int needed = (callerFrame ? prefixLen - 1 : 0) + regLen;
     if (pNeeded)
         *pNeeded = needed;
@@ -2140,7 +2140,7 @@ ClrDataAccess::GetFrameName(CLRDATA_ADDRESS vtable, unsigned int count, _Inout_u
     else
     {
         // Turn from bytes to wide characters
-        unsigned int len = (unsigned int)u16_strlen(pszName);
+        unsigned int len = (unsigned int)minipal_u16_strlen((const CHAR16_T*)pszName);
 
         if (frameName)
         {
@@ -2614,7 +2614,7 @@ ClrDataAccess::GetAppDomainName(CLRDATA_ADDRESS addr, unsigned int count, _Inout
         if (pAppDomain->m_friendlyName.IsValid())
         {
             LPCWSTR friendlyName = (LPCWSTR)pAppDomain->m_friendlyName;
-            size_t friendlyNameLen = u16_strlen(friendlyName);
+            size_t friendlyNameLen = minipal_u16_strlen((const CHAR16_T*)friendlyName);
 
             if (pNeeded)
             {

src/coreclr/debug/daccess/task.cpp

@@ -720,7 +720,7 @@ ClrDataAppDomain::GetName(
                 S_OK : S_FALSE;
             if (nameLen)
             {
-                size_t cchName = u16_strlen(rawName) + 1;
+                size_t cchName = minipal_u16_strlen((const CHAR16_T*)rawName) + 1;
                 if (FitsIn<ULONG32>(cchName))
                 {
                     *nameLen = (ULONG32) cchName;
@@ -4749,7 +4749,7 @@ ClrDataExceptionState::GetString(
             status = StringCchCopy(str, bufLen, msgStr) == S_OK ? S_OK : S_FALSE;
             if (strLen != NULL)
             {
-                size_t cchName = u16_strlen(msgStr) + 1;
+                size_t cchName = minipal_u16_strlen((const CHAR16_T*)msgStr) + 1;
                 if (FitsIn<ULONG32>(cchName))
                 {
                     *strLen = (ULONG32) cchName;

src/coreclr/debug/dbgutil/dbgutil.cpp

@@ -16,7 +16,7 @@
 #include <assert.h>
 #include <stdio.h>
 #include <memory>
-#include <dn-u16.h>
+#include <minipal/strings.h>
 #include "corhlpr.h"
 
 #ifdef HOST_WINDOWS
@@ -329,7 +329,7 @@ HRESULT GetNextLevelResourceEntryRVAByName(ICorDebugDataTarget* pDataTarget,
     DWORD* pNextLevelRva)
 {
     HRESULT hr = S_OK;
-    DWORD nameLength = (DWORD)u16_strlen(pwzName);
+    DWORD nameLength = (DWORD)minipal_u16_strlen((const CHAR16_T*)pwzName);
     WCHAR entryName[50];
     assert(nameLength < 50);     // this implementation won't support matching a name longer
     // than 50 characters. We only look up the hard coded name

src/coreclr/debug/di/module.cpp

@@ -1101,7 +1101,7 @@ void CordbModule::CopyRemoteMetaData(
 
     // Allocate space for the local copy of the metadata
     // No need to zero out the memory since we'll fill it all here.
-    LPVOID pRawBuffer = CoTaskMemAlloc(buffer.cbSize);
+    LPVOID pRawBuffer = minipal_co_task_mem_alloc(buffer.cbSize);
     if (pRawBuffer == NULL)
     {
         ThrowOutOfMemory();

src/coreclr/debug/di/process.cpp

@@ -9619,7 +9619,7 @@ void Ls_Rs_StringBuffer::CopyLSDataToRS(ICorDebugDataTarget * pTarget)
 
     // Now we know it's safe to call u16_strlen. The buffer is local, so we know the pages are there.
     // And we know there's a null capping the max length of the string.
-    SIZE_T dwActualLenWithNull = u16_strlen(pString) + 1;
+    SIZE_T dwActualLenWithNull = minipal_u16_strlen((const CHAR16_T*)pString) + 1;
     if (dwActualLenWithNull != dwExpectedLenWithNull)
     {
         ThrowHR(CORDBG_E_TARGET_INCONSISTENT);

src/coreclr/debug/di/publish.cpp

@@ -418,7 +418,7 @@ CorpubProcess::CorpubProcess(DWORD dwProcessId,
             if (ret > 0)
             {
                 // Recompute string length because we don't know if 'ret' is in bytes or char.
-                SIZE_T len = u16_strlen(szName) + 1;
+                SIZE_T len = minipal_u16_strlen((const CHAR16_T*)szName) + 1;
                 m_szProcessName = new (nothrow) WCHAR[len];
                 if (m_szProcessName != NULL)
                 {
@@ -602,7 +602,7 @@ HRESULT AllocateAndReadRemoteString(
     if (SUCCEEDED(hr))
     {
         // Ensure that the string we just read is actually null terminated.
-        // We can't call u16_strlen() on it yet, since that may AV on a non-null terminated string.
+        // We can't call minipal_u16_strlen((const CHAR16_T*)) on it yet, since that may AV on a non-null terminated string.
         WCHAR * pString = *ppNewLocalBuffer;
 
         if (pString[ceSize - 1] == W('\0'))
@@ -612,7 +612,7 @@ HRESULT AllocateAndReadRemoteString(
         }
         pString[ceSize - 1] = W('\0');
 
-        SIZE_T ceTestLen = u16_strlen(pString);
+        SIZE_T ceTestLen = minipal_u16_strlen((const CHAR16_T*)pString);
         if (ceTestLen == ceSize - 1)
         {
             // String was not previously null-terminated.

src/coreclr/debug/di/rsmda.cpp

@@ -105,7 +105,7 @@ HRESULT CordbMDA::QueryInterface(REFIID riid, void **ppInterface)
 HRESULT CopyOutString(LPCWSTR pInputString, ULONG32 cchName, ULONG32 * pcchName, _Out_writes_to_opt_(cchName, *pcchName) WCHAR szName[])
 {
     _ASSERTE(pInputString != NULL);
-    ULONG32 len = (ULONG32) u16_strlen(pInputString) + 1;
+    ULONG32 len = (ULONG32) minipal_u16_strlen((const CHAR16_T*)pInputString) + 1;
 
     if (cchName == 0)
     {

src/coreclr/debug/di/rsthread.cpp

@@ -10205,7 +10205,7 @@ HRESULT CordbEval::NewString(LPCWSTR string)
 {
     PUBLIC_API_ENTRY(this);
     FAIL_IF_NEUTERED(this);
-    return NewStringWithLength(string, (UINT)u16_strlen(string));
+    return NewStringWithLength(string, (UINT)minipal_u16_strlen((const CHAR16_T*)string));
 }
 
 //---------------------------------------------------------------------------------------

src/coreclr/debug/di/symbolinfo.cpp

@@ -258,7 +258,7 @@ STDMETHODIMP SymbolInfo::GetTypeDefProps (             // S_OK or error.
         *pdwTypeDefFlags=classInfo->flags;
 
 
-    SIZE_T cch=u16_strlen(classInfo->wszName)+1;
+    SIZE_T cch=minipal_u16_strlen((const CHAR16_T*)classInfo->wszName.GetUnicode())+1;
     if (cch > UINT32_MAX)
         return E_UNEXPECTED;
     *pchTypeDef=(ULONG)cch;
@@ -309,7 +309,7 @@ STDMETHODIMP SymbolInfo::GetMethodProps (
 
 
     *pClass=m_LastMethod.cls;
-    SIZE_T cch=u16_strlen(m_LastMethod.wszName)+1;
+    SIZE_T cch=minipal_u16_strlen((const CHAR16_T*)m_LastMethod.wszName.GetUnicode())+1;
     if(cch > UINT32_MAX)
         return E_UNEXPECTED;
     *pchMethod=(ULONG)cch;

src/coreclr/debug/inc/dbgappdomain.h

@@ -43,7 +43,7 @@ struct AppDomainInfo
         else
             m_szAppDomainName = W("<NoName>");
 
-        m_iNameLengthInBytes = (int) (u16_strlen(m_szAppDomainName) + 1) * sizeof(WCHAR);
+        m_iNameLengthInBytes = (int) (minipal_u16_strlen((const CHAR16_T*)m_szAppDomainName) + 1) * sizeof(WCHAR);
     }
 #endif
 };

src/coreclr/debug/inc/dbgipcevents.h

@@ -872,7 +872,7 @@ template <int nMaxLengthIncludingNull>
 class MSLAYOUT EmbeddedIPCString
 {
 public:
-    // Set, caller responsibility that u16_strlen(pData) < nMaxLengthIncludingNull
+    // Set, caller responsibility that minipal_u16_strlen((const CHAR16_T*)pData) < nMaxLengthIncludingNull
     void SetString(const WCHAR * pData)
     {
         // If the string doesn't fit into the buffer, that's an issue (and so this is a real

src/coreclr/debug/inc/dbgtransportsession.h

@@ -11,6 +11,8 @@
 
 #endif // !RIGHT_SIDE_COMPILE
 
+#include <minipal/guid.h>
+
 #if defined(FEATURE_DBGIPC_TRANSPORT_VM) || defined(FEATURE_DBGIPC_TRANSPORT_DI)
 
 #include <twowaypipe.h>
@@ -519,7 +521,7 @@ class DbgTransportSession
     // Struct defining the format of the data block sent with a SessionRequest.
     struct SessionRequestData
     {
-        GUID            m_sSessionID;   // Unique session ID. Treated as byte blob so no endian-ness
+        minipal_guid_t  m_sSessionID;   // Unique session ID. Treated as byte blob so no endian-ness
     };
 
     // Struct used to track a message that is being (or will soon be) sent but has not yet been acknowledged.
@@ -674,7 +676,7 @@ class DbgTransportSession
     // Session ID randomly allocated by the right side and sent over in the SessionRequest message. This
     // serves to disambiguate a re-send of the SessionRequest due to a network error versus a SessionRequest
     // from a different debugger.
-    GUID            m_sSessionID;
+    minipal_guid_t  m_sSessionID;
 
     // Lock used to synchronize sending messages and updating the session state. This ensures message bytes
     // don't become interleaved on the transport connection, the send queue is updated consistently across

src/coreclr/debug/inc/ddmarshalutil.h

@@ -90,7 +90,7 @@ class WriteBuffer : public BaseBuffer
         if (!fIsNull)
         {
             _ASSERTE(pString != NULL);
-            DWORD len = (DWORD) u16_strlen(pString);
+            DWORD len = (DWORD) minipal_u16_strlen((const CHAR16_T*)pString);
             DWORD cbCopy = (len + 1) * sizeof(WCHAR);
 
             EnsureSize(cbCopy);
@@ -179,7 +179,7 @@ class ReadBuffer : public BaseBuffer
         else
         {
             const WCHAR * pString = (WCHAR*) &m_pBuffer[m_idx];
-            DWORD len = (DWORD) u16_strlen(pString);
+            DWORD len = (DWORD) minipal_u16_strlen((const CHAR16_T*)pString);
             m_idx += (len + 1) * sizeof(WCHAR); // skip past null
             _ASSERTE(m_idx <= m_size);
             return pString;

src/coreclr/debug/shared/dbgtransportsession.cpp

@@ -89,9 +89,8 @@ HRESULT DbgTransportSession::Init(DebuggerIPCControlBlock *pDCB, AppDomainEnumer
     // The RS randomly allocates a session ID which is sent to the LS in the SessionRequest message. In the
     // case of network errors during session formation this allows the LS to tell SessionRequest re-sends from
     // a new request from a different RS.
-    HRESULT hr = CoCreateGuid(&m_sSessionID);
-    if (FAILED(hr))
-        return hr;
+    if (!minipal_guid_v4_create(&m_sSessionID))
+        return E_FAIL;
 #endif // RIGHT_SIDE_COMPILE
 
 

src/coreclr/debug/shared/stringcopyholder.cpp

@@ -68,7 +68,7 @@ HRESULT StringCopyHolder::AssignCopy(const WCHAR * pStringSrc)
     }
     else
     {
-        SIZE_T cchLen = u16_strlen(pStringSrc) + 1;
+        SIZE_T cchLen = minipal_u16_strlen((const CHAR16_T*)pStringSrc) + 1;
         m_szData = new (nothrow) WCHAR[cchLen];
         if (m_szData == NULL)
         {

src/coreclr/dlls/mscordac/CMakeLists.txt

@@ -175,7 +175,6 @@ else(CLR_CMAKE_HOST_WIN32)
         mscorrc
         ${START_WHOLE_ARCHIVE} # force all PAL objects to be included so all exports are available
         coreclrpal
-        palrt
         coreclrminipal
         ${END_WHOLE_ARCHIVE}
     )

src/coreclr/dlls/mscordac/mscordac_unixexports.src

@@ -40,7 +40,6 @@ nativeStringResourceTable_mscorrc
 #PAL_CloseProcessMemory
 #PAL_ReadProcessMemory
 #PAL_ProbeMemory
-#PAL_Random
 #PAL__wcstoui64
 #PAL_wcstoul
 #PAL_wcstod

src/coreclr/dlls/mscordbi/CMakeLists.txt

@@ -96,7 +96,6 @@ if(CLR_CMAKE_HOST_WIN32)
 elseif(CLR_CMAKE_HOST_UNIX)
 
     list(APPEND COREDBI_LIBRARIES
-        palrt
         # share the PAL in the dac module
         mscordaccore
     )

src/coreclr/dlls/mscoree/coreclr/CMakeLists.txt

@@ -140,7 +140,6 @@ else()
         coreclrpal
         ${END_WHOLE_ARCHIVE}
         mscorrc
-        palrt
     )
 endif(CLR_CMAKE_TARGET_WIN32)