Commit 13914dd3 authored by weinig@apple.com's avatar weinig@apple.com

2011-05-12 Sam Weinig <sam@webkit.org>

        Reviewed by Gavin Barraclough.

        WebProcess sandbox denies file:/// accesses after you navigate to a nonexistent file
        <rdar://problem/9427752>
        https://bugs.webkit.org/show_bug.cgi?id=60745

        * WebProcess/WebPage/WebPage.cpp:
        (WebKit::WebPage::SandboxExtensionTracker::didStartProvisionalLoad):
        Don't attempt to re-use a non-existent committed sandbox extension. This can
        happen when the previous load was file: url, but did not require an extension,
        such as an error page.


git-svn-id: http://svn.webkit.org/repository/webkit/trunk@86409 268f45cc-cd09-0410-ab3c-d52691b4dbfc
parent 560ea591
2011-05-12 Sam Weinig <sam@webkit.org>
Reviewed by Gavin Barraclough.
WebProcess sandbox denies file:/// accesses after you navigate to a nonexistent file
<rdar://problem/9427752>
https://bugs.webkit.org/show_bug.cgi?id=60745
* WebProcess/WebPage/WebPage.cpp:
(WebKit::WebPage::SandboxExtensionTracker::didStartProvisionalLoad):
Don't attempt to re-use a non-existent committed sandbox extension. This can
happen when the previous load was file: url, but did not require an extension,
such as an error page.
2011-05-12 Jia Pu <jpu@apple.com>
Reviewed by Darin Adler.
......
......@@ -2101,7 +2101,9 @@ void WebPage::SandboxExtensionTracker::didStartProvisionalLoad(WebFrame* frame)
if (!frame->isMainFrame())
return;
if (shouldReuseCommittedSandboxExtension(frame)) {
// We should only reuse the commited sandbox extension if it is not null. It can be
// null if the last load was for an error page.
if (m_committedSandboxExtension && shouldReuseCommittedSandboxExtension(frame)) {
m_pendingProvisionalSandboxExtension = m_committedSandboxExtension.release();
ASSERT(!m_committedSandboxExtension);
}
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment