diff options
author | Cary Clark <cary@android.com> | 2010-08-31 09:20:16 -0400 |
---|---|---|
committer | Cary Clark <cary@android.com> | 2010-08-31 09:28:43 -0400 |
commit | ec079933211eb47479b2624cb6be72257d178c01 (patch) | |
tree | 81193c99368f40090836f6408456d64e027982a3 /WebCore/loader/ImageLoader.h | |
parent | 7fbc2518b3a834153730d32761429507c03e7a22 (diff) | |
download | external_webkit-ec079933211eb47479b2624cb6be72257d178c01.zip external_webkit-ec079933211eb47479b2624cb6be72257d178c01.tar.gz external_webkit-ec079933211eb47479b2624cb6be72257d178c01.tar.bz2 |
fix monkey crash in nav cache
WebView::motionUp() got the latest nav cache, extracted
some nodes from it with findAt(), then called setNavBounds()
which got a newer cache. The older cache node was sent
to CachedRoot::setCursor() which crashed trying to use
the state pointer.
The flaw was that, although motionUp requests the newest
cache up front, and then setNavBounds() requests the stale
cache, in rare circumstances it can receive a newer cache
as well.
The fix is to replace the setNavBounds() function with a
direct call on the valid cache.
Change-Id: If23ee9222f2b701d916911f4b667185f1c3d3d18
http://b/2316138
Diffstat (limited to 'WebCore/loader/ImageLoader.h')
0 files changed, 0 insertions, 0 deletions