Closed Bug 611138 Opened 14 years ago Closed 14 years ago

"ABORT: Very, very bad - mData corrupt" with path seg manipulation

Categories

(Core :: SVG, defect)

x86
macOS
defect
Not set
critical

Tracking

()

RESOLVED FIXED
Tracking Status
blocking2.0 --- final+

People

(Reporter: jruderman, Assigned: jwatt)

References

Details

(Keywords: assertion, testcase)

Attachments

(3 files)

Attached image testcase 1
###!!! ABORT: Very, very bad - mData corrupt: 'i == mData.Length()', file content/svg/content/src/SVGPathData.cpp, line 146
Attached image testcase 2
###!!! ABORT: Seg type not recognized: 'IsValidType(type)', file content/svg/content/src/SVGPathSegUtils.h, line 118
These are new assertions from bug 522306.
Blocks: 522306
Assignee: nobody → jwatt
Status: NEW → ASSIGNED
blocking2.0: --- → final+
Both testcases here replace the first segment in a path with a new non-move command (making the path invalid, or at least not valid for painting).

Might be related to bug 610990 (which also involves creating paths that, at least temporarily, have a non-move command as the first segment).  That bug's testcases don't trigger any assertions/aborts, though.
Attached patch patchSplinter Review
Attachment #489698 - Flags: review?(dholbert)
Attachment #489698 - Flags: review?(dholbert) → review+
Pushed http://hg.mozilla.org/mozilla-central/rev/faf02793fe35

The test in there is a bit different, but I double checked that it also triggered this bug.
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Group: core-security
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: