8bit-unaware String usage in NetscapePluginModuleX11.cpp
authorkbalazs@webkit.org <kbalazs@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 17 Nov 2012 12:58:17 +0000 (12:58 +0000)
committerkbalazs@webkit.org <kbalazs@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 17 Nov 2012 12:58:17 +0000 (12:58 +0000)
https://bugs.webkit.org/show_bug.cgi?id=102482

Reviewed by Michael Saboff.

String::fromUTF8 is not guaranteed to give a 16 bit string.

* Shared/Plugins/Netscape/x11/NetscapePluginModuleX11.cpp:
(WebKit::truncateToSingleLine): Do an explicit 16 bit conversion.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@135043 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebKit2/ChangeLog
Source/WebKit2/Shared/Plugins/Netscape/x11/NetscapePluginModuleX11.cpp

index 321b158..171a74a 100644 (file)
@@ -1,3 +1,15 @@
+2012-11-17  Balazs Kelemen  <kbalazs@webkit.org>
+
+        8bit-unaware String usage in NetscapePluginModuleX11.cpp
+        https://bugs.webkit.org/show_bug.cgi?id=102482
+
+        Reviewed by Michael Saboff.
+
+        String::fromUTF8 is not guaranteed to give a 16 bit string.
+
+        * Shared/Plugins/Netscape/x11/NetscapePluginModuleX11.cpp:
+        (WebKit::truncateToSingleLine): Do an explicit 16 bit conversion.
+
 2012-11-16  Jaehun Lim  <ljaehun.lim@samsung.com>
 
         [EFL][WK2] Remove unnecessary #include statemtns
index f92fb0b..d6f4536 100644 (file)
@@ -167,7 +167,8 @@ static String truncateToSingleLine(const String& string)
     String stringBuffer(StringImpl::createUninitialized(oldLength + 1, buffer));
 
     unsigned newLength = 0;
-    for (const UChar* c = string.characters16(); c < string.characters16() + oldLength; ++c) {
+    const UChar* start = string.characters();
+    for (const UChar* c = start; c < start + oldLength; ++c) {
         if (*c != UChar('\n'))
             buffer[newLength++] = *c;
     }