git://git.webkit.org / WebKit-https.git / commitdiff
? search:
summary | shortlog | log | commit | commitdiff | tree
raw | patch | inline | side by side (parent: b9fd06f)
2007-07-06 Tony Chang <idealisms@gmail.com>
authorbdash <bdash@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 6 Jul 2007 10:03:38 +0000 (10:03 +0000)
committerbdash <bdash@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 6 Jul 2007 10:03:38 +0000 (10:03 +0000)
        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.

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

LayoutTests/ChangeLog patch | blob | history
LayoutTests/http/tests/multipart/resources/multipart-nodashes.php [new file with mode: 0644] patch | blob
LayoutTests/http/tests/multipart/win-boundary-crash-expected.txt [new file with mode: 0644] patch | blob
LayoutTests/http/tests/multipart/win-boundary-crash.html [new file with mode: 0644] patch | blob
WebCore/ChangeLog patch | blob | history
WebCore/loader/ResourceLoader.cpp patch | blob | history

diff --git a/LayoutTests/ChangeLog b/LayoutTests/ChangeLog
index 65f695b2dc56fb7640debc661a90627ca9e0cb39..07dd59bcfd24d6eba937b2c53368194ee637ec20 100644 (file)
--- a/LayoutTests/ChangeLog
+++ b/LayoutTests/ChangeLog
@@ -1,3 +1,14 @@
+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>
 
         Reviewed by Alexey.
diff --git a/LayoutTests/http/tests/multipart/resources/multipart-nodashes.php b/LayoutTests/http/tests/multipart/resources/multipart-nodashes.php
new file mode 100644 (file)
index 0000000..f0891f1
--- /dev/null
+++ b/LayoutTests/http/tests/multipart/resources/multipart-nodashes.php
@@ -0,0 +1,10 @@
+<?php
+    # Generates a multipart/x-mixed-replace response but doesn't
+    # include the -- before the boundary.
+
+    $boundary = "cutHere";
+    header("Content-Type: multipart/x-mixed-replace; boundary=$boundary");
+    echo("$boundary\r\n");
+    echo("Content-Type: image/png\r\n\r\n");
+    echo(file_get_contents("green-100x100.png"));
+?>
diff --git a/LayoutTests/http/tests/multipart/win-boundary-crash-expected.txt b/LayoutTests/http/tests/multipart/win-boundary-crash-expected.txt
new file mode 100644 (file)
index 0000000..32219ad
--- /dev/null
+++ b/LayoutTests/http/tests/multipart/win-boundary-crash-expected.txt
@@ -0,0 +1,5 @@
+Test for http://bugs.webkit.org/show_bug.cgi?id=14516 crash loading multipart/x-mixed-replace data on windows safari
+
+If WebKit does not crash when loading this img, then it passed.
+
+
diff --git a/LayoutTests/http/tests/multipart/win-boundary-crash.html b/LayoutTests/http/tests/multipart/win-boundary-crash.html
new file mode 100644 (file)
index 0000000..78f4254
--- /dev/null
+++ b/LayoutTests/http/tests/multipart/win-boundary-crash.html
@@ -0,0 +1,14 @@
+<html>
+<head>
+<script>
+    if (window.layoutTestController)
+        layoutTestController.dumpAsText();
+</script>
+</head>
+<body>
+<p>Test for <i><a href="http://bugs.webkit.org/show_bug.cgi?id=14516">http://bugs.webkit.org/show_bug.cgi?id=14516</a>
+crash loading multipart/x-mixed-replace data on windows safari</i></p>
+<p>If WebKit does not crash when loading this img, then it passed.</p>
+<img src="resources/multipart-nodashes.php" />
+</body>
+</html>
diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index 8daa8c83f68805731dd3b0bc61b62433a8b4a84a..08c7a7aa9f46d95edbd394ca30d15ee1475d5bab 100644 (file)
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -1,3 +1,15 @@
+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>
 
         Reviewed by Alexey.
diff --git a/WebCore/loader/ResourceLoader.cpp b/WebCore/loader/ResourceLoader.cpp
index 8ba007d2e3e7b03cca74c1eb76b126cc169a5150..ebbb88724e44c5370fb4a981c4a1277cfe28c754 100644 (file)
--- a/WebCore/loader/ResourceLoader.cpp
+++ b/WebCore/loader/ResourceLoader.cpp
@@ -161,7 +161,8 @@ void ResourceLoader::addData(const char* data, int length, bool allAtOnce)
 
 void ResourceLoader::clearResourceData()
 {
-    m_resourceData->clear();
+    if (m_resourceData)
+        m_resourceData->clear();
 }
 
 void ResourceLoader::willSendRequest(ResourceRequest& request, const ResourceResponse& redirectResponse)
Mirror of the WebKit open source project from svn.webkit.org.