[Webkit-unassigned] [Bug 47498] Crash while processing ill-formed SVG with cycles.
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Thu Oct 14 23:29:12 PDT 2010
https://bugs.webkit.org/show_bug.cgi?id=47498
Csaba Osztrogonac <ossy at webkit.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #70819|review? |review-
Flag| |
--- Comment #7 from Csaba Osztrogonac <ossy at webkit.org> 2010-10-14 23:29:12 PST ---
(From update of attachment 70819)
View in context: https://bugs.webkit.org/attachment.cgi?id=70819&action=review
I'm not familiar with SVG, so I can't review the logic of the patch.
r- due to build error
> WebCore/rendering/SVGResources.cpp:453
> + RenderSVGResourceType resourceType = clipper->resourceType();
> + ASSERT(resourceType == ClipperResourceType);
> +
To avoid "unused variable" warning(error), you should
use ASSERT_UNUSED, or don't use a new local variable.
But I propose avoiding unnecessary local variable,
which is dead code in relase mode:
ASSERT(clipper->resourceType() == ClipperResourceType);
> WebCore/rendering/SVGResources.cpp:476
> + RenderSVGResourceType resourceType = filter->resourceType();
> + ASSERT(resourceType == FilterResourceType);
> +
ditto
> WebCore/rendering/SVGResources.cpp:498
> + RenderSVGResourceType resourceType = markerStart->resourceType();
> + ASSERT(resourceType == MarkerResourceType);
ditto
> WebCore/rendering/SVGResources.cpp:520
> + RenderSVGResourceType resourceType = markerMid->resourceType();
> + ASSERT(resourceType == MarkerResourceType);
ditto
> WebCore/rendering/SVGResources.cpp:542
> + RenderSVGResourceType resourceType = markerEnd->resourceType();
> + ASSERT(resourceType == MarkerResourceType);
ditto
> WebCore/rendering/SVGResources.cpp:564
> + RenderSVGResourceType resourceType = masker->resourceType();
> + ASSERT(resourceType == MaskerResourceType);
ditto
> WebCore/rendering/SVGResources.cpp:586
> + RenderSVGResourceType resourceType = fill->resourceType();
> + ASSERT(resourceType == PatternResourceType || resourceType == LinearGradientResourceType || resourceType == RadialGradientResourceType);
ditto
> WebCore/rendering/SVGResources.cpp:608
> + RenderSVGResourceType resourceType = stroke->resourceType();
> + ASSERT(resourceType == PatternResourceType || resourceType == LinearGradientResourceType || resourceType == RadialGradientResourceType);
ditto
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list