Changeset 103444 in vbox for trunk/src/VBox/Runtime/common/string/RTStrCopy.cpp

Timestamp:

Feb 19, 2024 1:58:17 PM (10 months ago)

Author:

vboxsync

Message:

IPRT: Reverted r161541, r161577 & r161578 because the RTStrCopy2 and RTStrCat2 functions encourages writing unsafe code, given that RTStrCat[2] doesn't check for NULL destination buffers and will crash if the two are chained.

File:

• trunk/src/VBox/Runtime/common/string/RTStrCopy.cpp (modified) (2 diffs)

trunk/src/VBox/Runtime/common/string/RTStrCopy.cpp

@@ -45 +45 @@
 
 
-DECLINLINE(int) rtStrCopy(char *pszDst, size_t cbDst, const char *pszSrc)
+RTDECL(int) RTStrCopy(char *pszDst, size_t cbDst, const char *pszSrc)
 {
     size_t cchSrc = strlen(pszSrc);
@@ -61 +61 @@
     return VERR_BUFFER_OVERFLOW;
 }
-
-RTDECL(int) RTStrCopy(char *pszDst, size_t cbDst, const char *pszSrc)
-{
-    return rtStrCopy(pszDst, cbDst, pszSrc);
-}
 RT_EXPORT_SYMBOL(RTStrCopy);
 
-RTDECL(char *) RTStrCopy2(char *pszDst, size_t cbDst, const char *pszSrc)
-{
-    return RT_SUCCESS(rtStrCopy(pszDst, cbDst, pszSrc)) ? pszDst : NULL;
-}
-RT_EXPORT_SYMBOL(RTStrCopy2);
-