Closed Bug 769294 Opened 12 years ago Closed 12 years ago

Crash when dumping layers in the console

Categories

(Core :: Graphics: Layers, defect)

defect
Not set
trivial

Tracking

()

RESOLVED FIXED
mozilla16

People

(Reporter: nical, Assigned: nical)

Details

Attachments

(1 file)

Attached patch Trivial fixSplinter Review
The Dump method in LayerManager has a small bug: if the argument aFile is nil it is supposed to use terminal output by default. It is handled by the call to FILEOrDefault at the beginning of the method. However one of the fprintf calls that follow use the 'aFile' variable instead of 'file', resulting in a segfault when aFile is null, rather than writing in stdout.
Attachment #637523 - Flags: review?(joe)
Attachment #637523 - Flags: review?(joe) → review+
Keywords: checkin-needed
Assignee: nobody → nsilva
https://hg.mozilla.org/integration/mozilla-inbound/rev/a52d79ad75da
Flags: in-testsuite-
Keywords: checkin-needed
Target Milestone: --- → mozilla16
https://hg.mozilla.org/mozilla-central/rev/a52d79ad75da
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: