Changeset 24053 in webkit

Timestamp:

Jul 6, 2007 3:03:38 AM (17 years ago)

Author:

bdash

Message:

2007-07-06 Tony Chang <​idealisms@gmail.com>

Reviewed by Maciej.

​http://bugs.webkit.org/show_bug.cgi?id=14516
Bug 14516: crash loading multipart/x-mixed-replace data on windows safari

NULL check m_resourceData to prevent a crash in Safari on Windows.

• loader/ResourceLoader.cpp: (WebCore::ResourceLoader::clearResourceData):

2007-07-06 Tony Chang <​idealisms@gmail.com>

Reviewed by Maciej.

​http://bugs.webkit.org/show_bug.cgi?id=14516
Bug 14516: crash loading multipart/x-mixed-replace data on windows safari

• http/tests/multipart/resources/multipart-nodashes.php: Added.
• http/tests/multipart/win-boundary-crash-expected.txt: Added.
• http/tests/multipart/win-boundary-crash.html: Added.

Location:

trunk

Files:

• LayoutTests/ChangeLog (modified) (1 diff)
• LayoutTests/http/tests/multipart/resources/multipart-nodashes.php (added)
• LayoutTests/http/tests/multipart/win-boundary-crash-expected.txt (added)
• LayoutTests/http/tests/multipart/win-boundary-crash.html (added)
• WebCore/ChangeLog (modified) (1 diff)
• WebCore/loader/ResourceLoader.cpp (modified) (1 diff)

trunk/LayoutTests/ChangeLog

@@ +1 @@
+2007-07-06  Tony Chang  <idealisms@gmail.com>
+
+        Reviewed by Maciej.
+
+        http://bugs.webkit.org/show_bug.cgi?id=14516
+        Bug 14516: crash loading multipart/x-mixed-replace data on windows safari
+
+        * http/tests/multipart/resources/multipart-nodashes.php: Added.
+        * http/tests/multipart/win-boundary-crash-expected.txt: Added.
+        * http/tests/multipart/win-boundary-crash.html: Added.
+
 2007-07-06  Jungshik Shin <jungshik.shin@gmail.com>
 

trunk/WebCore/ChangeLog

@@ +1 @@
+2007-07-06  Tony Chang  <idealisms@gmail.com>
+
+        Reviewed by Maciej.
+
+        http://bugs.webkit.org/show_bug.cgi?id=14516
+        Bug 14516: crash loading multipart/x-mixed-replace data on windows safari
+
+        NULL check m_resourceData to prevent a crash in Safari on Windows.
+
+        * loader/ResourceLoader.cpp:
+        (WebCore::ResourceLoader::clearResourceData):
+
 2007-07-06  Jungshik Shin  <jungshik.shin@gmail.com>
 

trunk/WebCore/loader/ResourceLoader.cpp

@@ -162 +162 @@
 void ResourceLoader::clearResourceData()
 {
-    m_resourceData->clear();
+    if (m_resourceData)
+        m_resourceData->clear();
 }