Closed
Bug 601131
Opened 14 years ago
Closed 11 years ago
Firefox hangs with feTurbulence which has huge numOctaves
Categories
(Core :: SVG, defect)
Core
SVG
Tracking
()
RESOLVED
FIXED
mozilla24
People
(Reporter: takenspc, Assigned: longsonr)
References
(Depends on 1 open bug)
Details
(Keywords: hang, perf)
Attachments
(2 files)
|
(deleted),
image/svg+xml
|
Details | |
|
(deleted),
patch
|
roc
:
review+
|
Details | Diff | Splinter Review |
Testcase (1 billion octaves)
SVG 1.1 allows content authors to specify loop count in |turblence| by numOctaves attribute.
http://www.w3.org/TR/SVG11/filters.html#feTurbulenceNumOctavesAttribute
And we use its value without any limitations.
http://mxr.mozilla.org/mozilla-central/source/content/svg/content/src/nsSVGFilters.cpp#3470
As a result, firefox hangs when loading an SVG file which contains feTurbulence which has huge numOctaves value.
We may be able to limit the loop count to some value.
|
Reporter | |
Comment 1•14 years ago
|
||
|
Assignee | |
Comment 2•11 years ago
|
||
octaves changes have a smaller and smaller effect as they grow larger. 10 was the largest value I found out in the wild.
Assignee: nobody → longsonr
Attachment #766406 -
Flags: review?(roc)
Attachment #766406 -
Flags: review?(roc) → review+
|
|
Assignee | |
Comment 3•11 years ago
|
||
|
|
Assignee | |
Comment 4•11 years ago
|
||
|
||
Comment 5•11 years ago
|
||
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla25
|
|
||
Comment 6•11 years ago
|
||
This landed on mozilla-central whilst it was still mozilla24, however mcMerge references the bugzilla target milestone field ordering, which was updated pre-emptively before the merge had occurred.
Target Milestone: mozilla25 → mozilla24
You need to log in
before you can comment on or make changes to this bug.
Description
•