Changeset 16369 in vbox for trunk/src/VBox/Main/glue

Timestamp:

Jan 29, 2009 2:26:18 PM (16 years ago)

Author:

vboxsync

Message:

Utf8Str::substr error handling

File:

• trunk/src/VBox/Main/glue/string.cpp (modified) (1 diff)

trunk/src/VBox/Main/glue/string.cpp

@@ -44 +44 @@
     if (n)
     {
-        const char *psz = c_str();
-        RTUNICP cp;
+        const char *psz;
 
-        // walk the UTF-8 characters until where the caller wants to start
-        size_t i = pos;
-        while (i--)
-            RTStrGetCpEx(&psz, &cp);
+        if ((psz = c_str()))
+        {
+            RTUNICP cp;
 
-        const char *pFirst = psz;
+            // walk the UTF-8 characters until where the caller wants to start
+            size_t i = pos;
+            while (*psz && i--)
+                if (!(RT_SUCCESS(RTStrGetCpEx(&psz, &cp))))
+                    return ret;     // return empty string on bad encoding
 
-        if (n == npos)
-            // all the rest:
-            ret = pFirst;
-        else
-        {
-            i = n;
-            while (i--)
-                RTStrGetCpEx(&psz, &cp);
+            const char *pFirst = psz;
 
-            size_t len = psz - pFirst;
-            char *psz = (char*)RTMemAlloc(len + 1);
-            memcpy(psz, pFirst, len);
-            psz[len] = '\0';
-            ret = psz;
-            RTMemFree(psz);
+            if (n == npos)
+                // all the rest:
+                ret = pFirst;
+            else
+            {
+                i = n;
+                while (*psz && i--)
+                    if (!(RT_SUCCESS(RTStrGetCpEx(&psz, &cp))))
+                        return ret;     // return empty string on bad encoding
+
+                size_t len = psz - pFirst;
+                char *psz = (char*)RTMemAlloc(len + 1);
+                memcpy(psz, pFirst, len);
+                psz[len] = '\0';
+                ret = psz;
+                RTMemFree(psz);
+            }
         }
     }