Changeset 40091 in vbox for trunk/src/VBox/Runtime/common

Timestamp:

Feb 13, 2012 10:14:00 AM (13 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

76231

Message:

Runtime/strings: add Utf-8 and Utf-16 sanitising to a white list of characters. Do not validate the string encoding in advance.

Location:

trunk/src/VBox/Runtime/common/string

Files:

• utf-16.cpp (modified) (2 diffs)
• utf-8.cpp (modified) (2 diffs)

trunk/src/VBox/Runtime/common/string/utf-16.cpp

@@ -255 +255 @@
     AssertReturn(chReplacement && (unsigned)chReplacement < 128, -1);
     /* Validate the encoding. */
-    if (RT_FAILURE(RTUtf16CalcUtf8LenEx(pwsz, RTSTR_MAX, NULL)))
-        return -1;
     for (;;)
     {
@@ -262 +260 @@
         PCRTUNICP pCp;
         PRTUTF16 pwszOld = pwsz;
-        RTUtf16GetCpEx((PCRTUTF16 *)&pwsz, &Cp);
+        if (RT_FAILURE(RTUtf16GetCpEx((PCRTUTF16 *)&pwsz, &Cp)))
+            return -1;
         if (!Cp)
             break;

trunk/src/VBox/Runtime/common/string/utf-8.cpp

@@ -356 +356 @@
     size_t cReplacements = 0;
     AssertReturn(chReplacement && (unsigned)chReplacement < 128, -1);
-    if (RT_FAILURE(RTStrValidateEncoding(psz)))
-        return -1;
     for (;;)
     {
@@ -363 +361 @@
         PCRTUNICP pCp;
         char *pszOld = psz;
-        RTStrGetCpEx((const char **)&psz, &Cp);
+        if (RT_FAILURE(RTStrGetCpEx((const char **)&psz, &Cp)))
+            return -1;
         if (!Cp)
             break;