Bug 14021 - REGRESSION: WebKit race condition vulnerability
Summary: REGRESSION: WebKit race condition vulnerability
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore JavaScript (show other bugs)
Version: 523.x (Safari 3)
Hardware: Mac OS X 10.4
: P1 Major
Assignee: Nobody
URL: http://geekable.com/vulnerable.png
Keywords: HasReduction, InRadar, Regression
Depends on:
Blocks:
 
Reported: 2007-06-06 19:39 PDT by Jeffrey Czerniak
Modified: 2007-07-14 00:43 PDT (History)
5 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jeffrey Czerniak 2007-06-06 19:39:24 PDT
I'll quote Michal Zalewski, who discovered the vulnerability (http://seclists.org/fulldisclosure/2007/Jun/0026.html):

When Javascript code instructs [the browser] to navigate away from a page 
   that meets same-domain origin policy (and hence can be scriptually 
   accessed and modified by the attacker) to an unrelated third-party 
   site, there is a window of opportunity for concurrently executed 
   Javascript to perform actions with the permissions for the old page, 
   but actual content for the newly loaded page, for example: 

     - Read or set victim.document.cookie, 

     - Arbitrarily alter document DOM, including changing form submission 
       URLs, injecting code, 

     - Read or write DOM structures that were not fully initialized, 
       prompting memory corruption and browser crash. 

Proof of concept located at http://lcamtuf.coredump.cx/ierace/

Confirmed vulnerable on fully-patched Tiger installation with nightly WebKit build r22026.
Comment 1 Mark Rowe (bdash) 2007-06-06 20:09:29 PDT
<rdar://problem/5255829>
Comment 2 Andrew Wellington 2007-06-06 20:41:34 PDT
I have been able to reproduce this on WebKit ToT, but not in shipping WebKit.
Comment 3 David Kilzer (:ddkilzer) 2007-06-06 22:23:52 PDT
(In reply to comment #2)
> I have been able to reproduce this on WebKit ToT, but not in shipping WebKit.

Therefore this is a regression.

Comment 4 Oliver Hunt 2007-07-14 00:43:29 PDT
Fix landed r23599