Fixed: <rdar://problem/4020110> Safari crashes in setAllData while taking a www...
authorcblu <cblu@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 25 Feb 2005 00:00:43 +0000 (00:00 +0000)
committercblu <cblu@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 25 Feb 2005 00:00:43 +0000 (00:00 +0000)
        Reviewed by rjw.

        * kwq/KWQResourceLoader.mm:
        (-[KWQResourceLoader finishJobAndHandle:]): clear the job after we've deleted to avoid reentrancy

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

WebCore/ChangeLog-2005-08-23
WebCore/kwq/KWQResourceLoader.mm

index 079cadab970f20b9e78cf771fe0daef63a9c58bd..8712029770932d6d4252b9a0d18511235c3e1f3f 100644 (file)
@@ -1,3 +1,12 @@
+2005-02-24  Chris Blumenberg  <cblu@apple.com>
+
+       Fixed: <rdar://problem/4020110> Safari crashes in setAllData while taking a www.zoomerang.com survey
+
+        Reviewed by rjw.
+
+        * kwq/KWQResourceLoader.mm:
+        (-[KWQResourceLoader finishJobAndHandle:]): clear the job after we've deleted to avoid reentrancy
+
 2005-02-24  Darin Adler  <darin@apple.com>
 
         Reviewed by Vicki.
index 2ff977fe9a92519de4c120b14b60235c08fc79fe..f4b2ce204bcce5fc88d80cbace2329a683855d25 100644 (file)
@@ -91,9 +91,10 @@ using KIO::TransferJob;
 
     if (job) {
         job->emitResult(data);
+        [handle release];
     }
     delete job;
-    [handle release];
+    job = 0;
 }
 
 - (void)cancel