WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
136087
[CSS Shapes] Negative raster shape height leads to crash
https://bugs.webkit.org/show_bug.cgi?id=136087
Summary
[CSS Shapes] Negative raster shape height leads to crash
Zoltan Horvath
Reported
2014-08-19 17:50:07 PDT
The original test was provided by the u-szeged fuzzer. I reduced the test case and handled the negative margin-IntRect case in the shapes code.
Attachments
backtrace
(19.10 KB, text/plain)
2014-08-19 17:53 PDT
,
Zoltan Horvath
no flags
Details
Patch
(6.51 KB, patch)
2014-08-19 18:11 PDT
,
Zoltan Horvath
no flags
Details
Formatted Diff
Diff
Cleaned up test case
(498 bytes, text/html)
2015-01-07 12:07 PST
,
Bem Jones-Bey
no flags
Details
Patch
(5.43 KB, patch)
2015-01-07 12:51 PST
,
Bem Jones-Bey
no flags
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Zoltan Horvath
Comment 1
2014-08-19 17:53:08 PDT
Created
attachment 236839
[details]
backtrace
Zoltan Horvath
Comment 2
2014-08-19 18:11:58 PDT
Created
attachment 236842
[details]
Patch
Bem Jones-Bey
Comment 3
2014-08-20 08:08:39 PDT
Comment on
attachment 236842
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=236842&action=review
Given the complexity of the test case, I'm not convinced this is the proper fix.
> Source/WebCore/ChangeLog:9 > + The original test was provided by the u-szeged fuzzer. I reduced the test > + case and handled the negative margin-IntRect case in the shapes code.
Can you give a better description of what this is fixing?
> LayoutTests/fast/shapes/shape-outside-floats/shape-outside-raster-shape-negative-height.html:3 > +* {
This test seems way too complex, especially since you're matching everything here. Are you sure you couldn't get something simpler? If you can't reduce it to simply a single float with shape-outside and a negative height margin box, I wonder if there is something more complex here than just having the height be negative. Is there a condition before it even gets to the shapes code that is failing?
> LayoutTests/fast/shapes/shape-outside-floats/shape-outside-raster-shape-negative-height.html:17 > +<button>a</button>
Does this have to be a button for this to fail, or would a <span> work? (Or do you even need it at all?)
> LayoutTests/fast/shapes/shape-outside-floats/shape-outside-raster-shape-negative-height.html:20 > +<p style="position: absolute; top: 100px;">
I wouldn't muck around with absolute positioning. Just have this paragraph in here, and dumpAsText() the entire test. You're not gaining anything from it being a ref test except complexity.
Bem Jones-Bey
Comment 4
2015-01-07 12:07:55 PST
Created
attachment 244174
[details]
Cleaned up test case
Bem Jones-Bey
Comment 5
2015-01-07 12:51:30 PST
Created
attachment 244179
[details]
Patch
Zoltan Horvath
Comment 6
2015-01-07 12:55:49 PST
Comment on
attachment 244179
[details]
Patch Thanks! Looks good!
WebKit Commit Bot
Comment 7
2015-01-07 14:30:33 PST
Comment on
attachment 244179
[details]
Patch Clearing flags on attachment: 244179 Committed
r178054
: <
http://trac.webkit.org/changeset/178054
>
WebKit Commit Bot
Comment 8
2015-01-07 14:30:37 PST
All reviewed patches have been landed. Closing bug.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug