Closed
Bug 712226
Opened 13 years ago
Closed 13 years ago
Crash when fclose(NULL)'ing a battery-status FILE*
Categories
(Core :: Hardware Abstraction Layer (HAL), defect)
Tracking
()
RESOLVED
FIXED
mozilla12
People
(Reporter: cjones, Assigned: cjones)
Details
Attachments
(1 file)
|
870 bytes,
patch
|
mwu
:
review+
|
Details | Diff | Splinter Review |
On the maguro device, we don't have the same /sys/class/power_supply/battery/charging_source file, so opening it fails. Then we crash trying to close a null FILE*. Simple fix, "real" fix to read the right file coming up later.
|
Assignee | |
Comment 1•13 years ago
|
||
I'm happy to land this after the uplift. Would prefer to not lose more blame history.
Assignee: nobody → jones.chris.g
Attachment #583089 -
Flags: review?(mwu)
|
||
Updated•13 years ago
|
Attachment #583089 -
Flags: review?(mwu) → review+
|
|
||
Comment 2•13 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/fa06927b6435
Target Milestone: --- → mozilla12
|
||
Comment 3•13 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/fa06927b6435
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•