Bug 143103 - Crash when laying out (char)0
Summary: Crash when laying out (char)0
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Myles C. Maxfield
URL:
Keywords: InRadar, Regression
Depends on:
Blocks:
 
Reported: 2015-03-26 10:07 PDT by Myles C. Maxfield
Modified: 2015-03-26 10:33 PDT (History)
2 users (show)

See Also:


Attachments
Patch (1.84 KB, patch)
2015-03-26 10:12 PDT, Myles C. Maxfield
no flags Details | Formatted Diff | Diff
Patch (2.05 KB, patch)
2015-03-26 10:29 PDT, Myles C. Maxfield
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Myles C. Maxfield 2015-03-26 10:07:46 PDT
Crash when laying out (char)0
Comment 1 Myles C. Maxfield 2015-03-26 10:12:58 PDT
Created attachment 249494 [details]
Patch
Comment 2 Myles C. Maxfield 2015-03-26 10:16:48 PDT
<rdar://problem/20220552>
Comment 3 Simon Fraser (smfr) 2015-03-26 10:21:46 PDT
Comment on attachment 249494 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=249494&action=review

> Source/WebCore/ChangeLog:13
> +        No new tests, for now. I'm having trouble creating a test because
> +        the site that causes this bug generates their page using script,
> +        and the script is all minified, and difficult to understand. I
> +        will contact the owner of the site and ask for and unminified
> +        version of their sources. However, I don't want to that to block
> +        this tiny fix from going in.

This doesn't explain what the fix is.
Comment 4 Myles C. Maxfield 2015-03-26 10:29:08 PDT
Created attachment 249495 [details]
Patch
Comment 5 Myles C. Maxfield 2015-03-26 10:30:35 PDT
Committed r182015: <http://trac.webkit.org/changeset/182015>
Comment 6 mitz 2015-03-26 10:33:29 PDT
A better title for this bug would have been “REGRESSION (r179368): Null deref in WebCore::Font::systemFallbackFontForCharacter loading http://paste.ofcode.org