[SCM] WebKit Debian packaging branch, debian/unstable, updated. debian/1.1.15-1-40151-g37bb677
darin
darin at 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Sat Sep 26 06:51:33 UTC 2009
The following commit has been merged in the debian/unstable branch:
commit 83e15c8f6fc9cfef2c2755e4417f6bc4105dac0e
Author: darin <darin at 268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date: Mon Oct 21 00:41:34 2002 +0000
- fixed 3007961 -- Crash in DOM::HTMLImageElementImpl::parseAttribute
* khtml/rendering/render_image.cpp: (RenderImage::updateFromElement):
Be sure to call ref on the new image before calling deref on the old
image. That's because deref on the old image causes a cache flush,
which might cause the image to go away in certain cases.
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@2386 268f45cc-cd09-0410-ab3c-d52691b4dbfc
diff --git a/WebCore/ChangeLog-2002-12-03 b/WebCore/ChangeLog-2002-12-03
index c86428e..96d08af 100644
--- a/WebCore/ChangeLog-2002-12-03
+++ b/WebCore/ChangeLog-2002-12-03
@@ -1,3 +1,12 @@
+2002-10-20 Darin Adler <darin at apple.com>
+
+ - fixed 3007961 -- Crash in DOM::HTMLImageElementImpl::parseAttribute
+
+ * khtml/rendering/render_image.cpp: (RenderImage::updateFromElement):
+ Be sure to call ref on the new image before calling deref on the old
+ image. That's because deref on the old image causes a cache flush,
+ which might cause the image to go away in certain cases.
+
2002-10-20 Chris Blumenberg <cblu at apple.com>
Fixed possible build failure.
diff --git a/WebCore/ChangeLog-2003-10-25 b/WebCore/ChangeLog-2003-10-25
index c86428e..96d08af 100644
--- a/WebCore/ChangeLog-2003-10-25
+++ b/WebCore/ChangeLog-2003-10-25
@@ -1,3 +1,12 @@
+2002-10-20 Darin Adler <darin at apple.com>
+
+ - fixed 3007961 -- Crash in DOM::HTMLImageElementImpl::parseAttribute
+
+ * khtml/rendering/render_image.cpp: (RenderImage::updateFromElement):
+ Be sure to call ref on the new image before calling deref on the old
+ image. That's because deref on the old image causes a cache flush,
+ which might cause the image to go away in certain cases.
+
2002-10-20 Chris Blumenberg <cblu at apple.com>
Fixed possible build failure.
diff --git a/WebCore/ChangeLog-2005-08-23 b/WebCore/ChangeLog-2005-08-23
index c86428e..96d08af 100644
--- a/WebCore/ChangeLog-2005-08-23
+++ b/WebCore/ChangeLog-2005-08-23
@@ -1,3 +1,12 @@
+2002-10-20 Darin Adler <darin at apple.com>
+
+ - fixed 3007961 -- Crash in DOM::HTMLImageElementImpl::parseAttribute
+
+ * khtml/rendering/render_image.cpp: (RenderImage::updateFromElement):
+ Be sure to call ref on the new image before calling deref on the old
+ image. That's because deref on the old image causes a cache flush,
+ which might cause the image to go away in certain cases.
+
2002-10-20 Chris Blumenberg <cblu at apple.com>
Fixed possible build failure.
diff --git a/WebCore/khtml/rendering/render_image.cpp b/WebCore/khtml/rendering/render_image.cpp
index d13ff20..07a4244 100644
--- a/WebCore/khtml/rendering/render_image.cpp
+++ b/WebCore/khtml/rendering/render_image.cpp
@@ -360,7 +360,6 @@ bool RenderImage::nodeAtPoint(NodeInfo& info, int _x, int _y, int _tx, int _ty)
void RenderImage::updateFromElement()
{
-#ifdef APPLE_CHANGES
// Treat a lack of src or empty string for src as no image at all, not the page itself
// loaded as an image.
DOMString attr = element()->getAttribute(ATTR_SRC);
@@ -369,16 +368,12 @@ void RenderImage::updateFromElement()
new_image = NULL;
else
new_image = element()->getDocument()->docLoader()->requestImage(khtml::parseURL(attr));
-#else
- CachedImage *new_image = element()->getDocument()->docLoader()->
- requestImage(khtml::parseURL(element()->getAttribute(ATTR_SRC)));
-#endif
if(new_image && new_image != image && (!style() || !style()->contentObject())) {
loadEventSent = false;
+ new_image->ref(this);
if(image) image->deref(this);
image = new_image;
- image->ref(this);
berrorPic = image->isErrorImage();
}
--
WebKit Debian packaging
More information about the Pkg-webkit-commits
mailing list