Bug 179919

Summary:

ContentExtensionsBackend::actionsForResourceLoad() doesn't handle unicode urls

Product:

WebKit

Reporter:

Ms2ger (he/him; ⌚ UTC+1/+2) <Ms2ger>

Component:

Page Loading

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED DUPLICATE

Severity:

Normal

CC:

achristensen, beidson

Priority:

Version:

WebKit Nightly Build

Hardware:

Unspecified

OS:

Unspecified

Bug Depends on:

Bug Blocks:

172337

Attachments:

Description	Flags
Test	none

Ms2ger (he/him; ⌚ UTC+1/+2)

Reported 2017-11-21 06:11:07 PST

Created attachment 327405 [details] Test The attached test case triggers an assertion if ENABLE(CONTENT_EXTENSIONS) is true. ASSERTION FAILED: A decoded URL should only contain ASCII characters. The matching algorithm assumes the input is ASCII. urlString.containsOnlyASCII() /Volumes/Data/EWS/WebKit/Source/WebCore/contentextensions/ContentExtensionsBackend.cpp(83) : std::pair<Vector<Action>, Vector<String> > WebCore::ContentExtensions::ContentExtensionsBackend::actionsForResourceLoad(const WebCore::ResourceLoadInfo &) const 1 0x111626990 WTFCrash 2 0x11b2ff492 WebCore::ContentExtensions::ContentExtensionsBackend::actionsForResourceLoad(WebCore::ResourceLoadInfo const&) const 3 0x11bfa3bff WebCore::UserContentProvider::actionsForResourceLoad(WebCore::ResourceLoadInfo const&, WebCore::DocumentLoader&) 4 0x11be9453a WebCore::DOMWindow::open(WebCore::DOMWindow&, WebCore::DOMWindow&, WTF::String const&, WTF::AtomicString const&, WTF::String const&) 5 0x11a0dcddd WebCore::jsDOMWindowInstanceFunctionOpenBody(JSC::ExecState*, WebCore::JSDOMWindow*, JSC::ThrowScope&) 6 0x11a09cd07 long long WebCore::IDLOperation<WebCore::JSDOMWindow>::call<&(WebCore::jsDOMWindowInstanceFunctionOpenBody(JSC::ExecState*, WebCore::JSDOMWindow*, JSC::ThrowScope&)), (WebCore::CastedThisErrorBehavior)0>(JSC::ExecState&, char const*) 7 0x11a02520c WebCore::jsDOMWindowInstanceFunctionOpen(JSC::ExecState*)

Attachments
Test (58 bytes, text/html) 2017-11-21 06:11 PST, Ms2ger (he/him; ⌚ UTC+1/+2)	no flags	Details
View All Add attachment proposed patch, testcase, etc.

Alex Christensen

Comment 1 2017-11-27 12:23:47 PST

*** This bug has been marked as a duplicate of bug 172337 ***

Note You need to log in before you can comment on or make changes to this bug.