Title: [149064] trunk/Source/WebKit2
Revision
149064
Author
[email protected]
Date
2013-04-24 14:33:43 -0700 (Wed, 24 Apr 2013)

Log Message

Crash when trying to determine the length of an invalid message
https://bugs.webkit.org/show_bug.cgi?id=115127
<rdar://problem/13700736>

Reviewed by Beth Dakin.

Use message instead of incomingMessage since incomingMessage will always be null.

* Platform/CoreIPC/Connection.cpp:
(CoreIPC::Connection::processIncomingMessage):

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (149063 => 149064)


--- trunk/Source/WebKit2/ChangeLog	2013-04-24 20:38:38 UTC (rev 149063)
+++ trunk/Source/WebKit2/ChangeLog	2013-04-24 21:33:43 UTC (rev 149064)
@@ -1,3 +1,16 @@
+2013-04-24  Anders Carlsson  <[email protected]>
+
+        Crash when trying to determine the length of an invalid message
+        https://bugs.webkit.org/show_bug.cgi?id=115127
+        <rdar://problem/13700736>
+
+        Reviewed by Beth Dakin.
+
+        Use message instead of incomingMessage since incomingMessage will always be null.
+
+        * Platform/CoreIPC/Connection.cpp:
+        (CoreIPC::Connection::processIncomingMessage):
+
 2013-04-24  Carlos Garcia Campos  <[email protected]>
 
         [GTK] Use stamp files for generate-forwarding-headers makefile rules

Modified: trunk/Source/WebKit2/Platform/CoreIPC/Connection.cpp (149063 => 149064)


--- trunk/Source/WebKit2/Platform/CoreIPC/Connection.cpp	2013-04-24 20:38:38 UTC (rev 149063)
+++ trunk/Source/WebKit2/Platform/CoreIPC/Connection.cpp	2013-04-24 21:33:43 UTC (rev 149064)
@@ -624,7 +624,7 @@
             // Something went wrong when decoding the message. Encode the message length so we can figure out if this
             // happens for certain message lengths.
             CString messageReceiverName = "<unknown message>";
-            CString messageName = String::format("<message length: %zu bytes>", incomingMessage->length()).utf8();
+            CString messageName = String::format("<message length: %zu bytes>", message->length()).utf8();
 
             m_clientRunLoop->dispatch(bind(&Connection::dispatchDidReceiveInvalidMessage, this, messageReceiverName, messageName));
             return;
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to