Closed Bug 712226 Opened 8 years ago Closed 8 years ago

Crash when fclose(NULL)'ing a battery-status FILE*

Categories

(Core :: Hardware Abstraction Layer (HAL), defect)

ARM
Gonk (Firefox OS)
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla12

People

(Reporter: cjones, Assigned: cjones)

Details

Attachments

(1 file)

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.
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)
Attachment #583089 - Flags: review?(mwu) → review+
https://hg.mozilla.org/mozilla-central/rev/fa06927b6435
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.