Bug 70012

Summary:

webkit aborts on assert in XSSAuditor.cpp

Product:

WebKit

Reporter:

Peter Hatina <phatina>

Component:

WebCore Misc.

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED WORKSFORME

Severity:

Normal

CC:

abarth, dbates, tsepez

Priority:

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Attachments:

Description	Flags
Patch	abarth: review-
test case	none

Peter Hatina

Reported 2011-10-13 03:37:15 PDT

Webkit quits on assertation condition in XSSAuditor.cpp, when calling static String fullyDecodeString(const String& string, const TextResourceDecoder* decoder) with empty string passed. Steps to reproduce (gtk+): 1. use WebKitWebView 2. change user-stylesheet-uri property in WebKitWebSettings, I passed this setting ("* { font: 7pt curier };") 3. load uri http://www.abclinuxu.cz Expected behavior: properly load specified uri and use specified css supplied by user Current behavior: program using webkit aborts on ASSERT(!workingString.isEmpty());

Attachments
Patch (1.62 KB, patch) 2011-10-13 03:48 PDT, Peter Hatina	abarth: review-	Details Formatted Diff Diff
test case (1.41 KB, application/octet-stream) 2011-10-18 03:47 PDT, Peter Hatina	no flags	Details
View All Add attachment proposed patch, testcase, etc.

Peter Hatina

Comment 1 2011-10-13 03:48:36 PDT

Created attachment 110820 [details] Patch

Alexey Proskuryakov

Comment 2 2011-10-13 11:12:29 PDT

Is this testable?

Adam Barth

Comment 3 2011-10-13 11:28:50 PDT

Comment on attachment 110820 [details] Patch Please include a test.

Peter Hatina

Comment 4 2011-10-18 03:47:12 PDT

Created attachment 111417 [details] test case I'm sorry, now I can not reproduce the issue - last week, the site had different content, which could be the reason, why I got assertation failure.

Thomas Sepez

Comment 5 2012-02-23 16:34:47 PST

This assert no longer exists in XSSAuditor.cpp.

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