Closed Bug 822854 Opened 7 years ago Closed 7 years ago

Crash [@ nsRuleNode::ComputeColumnData] with -moz-column-rule-color: inherit

Categories

(Core :: CSS Parsing and Computation, defect, critical)

x86_64
macOS
defect
Not set
critical

Tracking

()

RESOLVED FIXED

People

(Reporter: jruderman, Assigned: bugs)

References

(Blocks 1 open bug)

Details

(Keywords: crash, testcase)

Crash Data

Attachments

(3 files, 3 obsolete files)

No description provided.
Attached file stack
Assignee: nobody → bugs
Attached patch Fix (obsolete) — Splinter Review
Add a missing null-check
Attachment #693692 - Flags: review?(cam)
Attached file Bug fix. (obsolete) —
Adds a missing null-check. (+ Better patch file with 8 lines of context.)
Attachment #693692 - Attachment is obsolete: true
Attachment #693692 - Flags: review?(cam)
Attachment #693701 - Flags: review?(cam)
Attached patch Bug fix. (obsolete) — Splinter Review
Clicking the []patch checkbox this time.
Attachment #693701 - Attachment is obsolete: true
Attachment #693701 - Flags: review?(cam)
Attachment #693702 - Flags: review?(cam)
You can avoid uploading a new patch and update the "patch-ness" of the existing attachment in the attachment details page if you click the "(edit details)" link.
Comment on attachment 693702 [details] [diff] [review]
Bug fix.

In COMPUTE_START_RESET, if we find that presContext is null we construct a new (default) style struct for the parent's style to use those values.  I think if we take the else branch here if parent is null then we will compute the value to whatever column-rule-color is when mColumnRuleColorIsForeground is true.  I am not sure that mColumnRuleColor is valid in that case.  In nsStyleColumn's construct it's set to NS_RGB(0, 0, 0), but I don't know if it remains that colour.

In any case, this will be different from nsStyleColor's default mColor value, which is set to aPresContext->DefaultColor() in nsStyleColor's constructor.  I feel like we ought to be using that.  You can just create a local nsStyleColor to get this mColor out of.
Attachment #693702 - Flags: review?(cam)
Attachment #693702 - Attachment is obsolete: true
Attachment #698091 - Flags: review?(cam)
Comment on attachment 698091 [details] [diff] [review]
Updated patch per review comments.

Looks good.
Attachment #698091 - Flags: review?(cam) → review+
https://hg.mozilla.org/mozilla-central/rev/3077141dd57f
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.