Fix "Cleanup some string operating functions" PR

src/coreclr/debug/daccess/inspect.cpp

@@ -1047,8 +1047,8 @@ ClrDataValue::GetString(
             {
                 *strLen = static_cast<ULONG32>(u16_strlen(msgStr) + 1);
             }
-            status = StringCchCopy(str, bufLen, msgStr) == S_OK ?
-                S_OK : S_FALSE;
+
+            status = u16_strcpy_s(str, bufLen, msgStr) != NULL ? S_OK : S_FALSE;
         }
         else
         {

src/coreclr/debug/daccess/stack.cpp

@@ -830,7 +830,7 @@ ClrDataFrame::GetArgumentByIndex(
                     *nameLen = 5;
                 }
 
-                StringCchCopy(name, bufLen, W("this"));
+                u16_strcpy_s(name, bufLen, W("this"));
             }
             else
             {

src/coreclr/debug/daccess/stdafx.h

@@ -50,11 +50,6 @@
 #include "dacimpl.h"
 
 
-#define STRSAFE_NO_DEPRECATE
-#include <strsafe.h>
-#undef _ftcscat
-#undef _ftcscpy
-
 // from ntstatus.h
 #define STATUS_STOWED_EXCEPTION          ((NTSTATUS)0xC000027BL)
 

src/coreclr/debug/daccess/task.cpp

@@ -732,8 +732,7 @@ ClrDataAppDomain::GetName(
             }
             else
             {
-                status = StringCchCopy(name, bufLen, (PCWSTR)rawName) == S_OK ?
-                    S_OK : S_FALSE;
+                status = u16_strcpy_s(name, bufLen, (PCWSTR)rawName) != NULL ? S_OK : S_FALSE;
                 if (nameLen)
                 {
                     size_t cchName = u16_strlen((PCWSTR)rawName) + 1;
@@ -4763,7 +4762,7 @@ ClrDataExceptionState::GetString(
                                                  message->GetStringLength(),
                                                  true);
 
-            status = StringCchCopy(str, bufLen, msgStr) == S_OK ? S_OK : S_FALSE;
+            status = u16_strcpy_s(str, bufLen, msgStr) != NULL ? S_OK : S_FALSE;
             if (strLen != NULL)
             {
                 size_t cchName = u16_strlen(msgStr) + 1;

src/coreclr/dlls/mscordac/mscordac_unixexports.src

@@ -63,6 +63,9 @@ nativeStringResourceTable_mscorrc
 #PAL_wcscat
 #PAL_wcsstr
 
+#minipal_get_length_utf16_to_utf8
+#minipal_convert_utf16_to_utf8
+
 #_wcsicmp
 #sprintf_s
 #vsprintf_s

src/coreclr/ilasm/main.cpp

@@ -11,7 +11,6 @@
 #include "clrversion.h"
 #include "shimload.h"
 
-#include "strsafe.h"
 #define ASSERTE_ALL_BUILDS(expr) _ASSERTE_ALL_BUILDS((expr))
 
 WCHAR* EqualOrColon(_In_ __nullterminated WCHAR* szArg)

src/coreclr/inc/clr/fs/path.h

@@ -10,8 +10,6 @@
 
 #include "clrtypes.h"
 
-#include "strsafe.h"
-
 #include "clr/str.h"
 
 namespace clr

src/coreclr/inc/corhlprpriv.h

@@ -11,7 +11,7 @@
 #define __CORHLPRPRIV_H__
 
 #include "corhlpr.h"
-#include "fstring.h"
+#include <minipal/utf8.h>
 
 #if defined(_MSC_VER) && defined(HOST_X86)
 #pragma optimize("y", on)		// If routines don't get inlined, don't pay the EBP frame penalty
@@ -225,71 +225,45 @@ class CQuickMemoryBase
         iSize = cbTotal;
     }
 
-
-    // Convert UTF8 string to UNICODE string, optimized for speed
-    HRESULT ConvertUtf8_UnicodeNoThrow(const char * utf8str)
-    {
-        bool allAscii;
-        DWORD length;
-
-        HRESULT hr = FString::Utf8_Unicode_Length(utf8str, & allAscii, & length);
-
-        if (SUCCEEDED(hr))
-        {
-            LPWSTR buffer = (LPWSTR) AllocNoThrow((length + 1) * sizeof(WCHAR));
-
-            if (buffer == NULL)
-            {
-                hr = E_OUTOFMEMORY;
-            }
-            else
-            {
-                hr = FString::Utf8_Unicode(utf8str, allAscii, buffer, length);
-            }
-        }
-
-        return hr;
-    }
-
     // Convert UTF8 string to UNICODE string, optimized for speed
     void ConvertUtf8_Unicode(const char * utf8str)
     {
-        bool allAscii;
-        DWORD length;
-
-        HRESULT hr = FString::Utf8_Unicode_Length(utf8str, & allAscii, & length);
-
-        if (SUCCEEDED(hr))
+        size_t sourceLen = 0;
+        size_t destLen = 0;
+        if (utf8str != NULL)
         {
-            LPWSTR buffer = (LPWSTR) AllocThrows((length + 1) * sizeof(WCHAR));
-
-            hr = FString::Utf8_Unicode(utf8str, allAscii, buffer, length);
+            sourceLen = strlen(utf8str) + 1;
+            destLen = minipal_get_length_utf8_to_utf16(utf8str, sourceLen, /* flags */ 0);
         }
 
-        if (FAILED(hr))
+        CHAR16_T* buffer = (CHAR16_T*) AllocThrows((destLen + 1) * sizeof(CHAR16_T));
+        buffer[destLen] = W('\0');
+
+        if (destLen > 0
+            && !minipal_convert_utf8_to_utf16(utf8str, sourceLen, buffer, destLen, /* flags */ 0))
         {
-            ThrowHR(hr);
+            ThrowHR(EMAKEHR(errno));
         }
     }
 
     // Convert UNICODE string to UTF8 string, optimized for speed
     void ConvertUnicode_Utf8(const WCHAR * pString)
     {
-        bool allAscii;
-        DWORD length;
-
-        HRESULT hr = FString::Unicode_Utf8_Length(pString, & allAscii, & length);
-
-        if (SUCCEEDED(hr))
+        size_t sourceLen = 0;
+        size_t destLen = 0;
+        if (pString != NULL)
         {
-            LPSTR buffer = (LPSTR) AllocThrows((length + 1) * sizeof(char));
-
-            hr = FString::Unicode_Utf8(pString, allAscii, buffer, length);
+            sourceLen = u16_strlen(pString) + 1;
+            destLen = minipal_get_length_utf16_to_utf8((const CHAR16_T*)pString, sourceLen, /* flags */ 0);
         }
 
-        if (FAILED(hr))
+        LPSTR buffer = (LPSTR) AllocThrows((destLen + 1) * sizeof(char));
+        buffer[destLen] = '\0';
+
+        if (destLen > 0
+            && !minipal_convert_utf16_to_utf8((const CHAR16_T*)pString, sourceLen, buffer, destLen, /* flags */ 0))
         {
-            ThrowHR(hr);
+            ThrowHR(EMAKEHR(errno));
         }
     }
 

src/coreclr/inc/sstring.h

@@ -477,7 +477,7 @@ class EMPTY_BASES_DECL SString : private SBuffer
 
     // Converts/copies into the given output string
     void ConvertToUnicode(SString &dest) const;
-    COUNT_T ConvertToUTF8(SString &dest) const;
+    void ConvertToUTF8(SString &dest) const;
 
     //-------------------------------------------------------------------
     // Accessing the string contents directly

src/coreclr/inc/utilcode.h

@@ -2834,27 +2834,6 @@ template <class T> class CChainedHash
 };
 
 
-//*****************************************************************************
-//
-//********** String helper functions.
-//
-//*****************************************************************************
-
-//*****************************************************************************
-// Checks if string length exceeds the specified limit
-//*****************************************************************************
-inline BOOL IsStrLongerThan(_In_ _In_z_ char* pstr, unsigned N)
-{
-    LIMITED_METHOD_CONTRACT;
-    unsigned i = 0;
-    if(pstr)
-    {
-        for(i=0; (i < N)&&(pstr[i]); i++);
-    }
-    return (i >= N);
-}
-
-
 //*****************************************************************************
 // Class to parse a list of simple assembly names and then find a match
 //*****************************************************************************

src/coreclr/minipal/Unix/CMakeLists.txt

@@ -7,6 +7,12 @@ if(NOT CLR_CROSS_COMPONENTS_BUILD)
     list(APPEND SOURCES
         ${CLR_SRC_NATIVE_DIR}/minipal/cpufeatures.c
     )
+
+    if(CLR_CMAKE_HOST_OSX)
+        list(APPEND SOURCES
+            ${CLR_SRC_NATIVE_DIR}/minipal/utf8.c
+        )
+    endif()
 endif()
 
 add_library(coreclrminipal