Merge lp://staging/~seb128/indicator-power/dont-leak-content into lp://staging/indicator-power/14.04
Status: | Merged |
---|---|
Approved by: | Charles Kerr |
Approved revision: | 209 |
Merged at revision: | 209 |
Proposed branch: | lp://staging/~seb128/indicator-power/dont-leak-content |
Merge into: | lp://staging/indicator-power/14.04 |
Diff against target: |
11 lines (+1/-0) 1 file modified
src/ib-brightness-control.c (+1/-0) |
To merge this branch: | bzr merge lp://staging/~seb128/indicator-power/dont-leak-content |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Charles Kerr (community) | Approve | ||
PS Jenkins bot (community) | continuous-integration | Approve | |
Review via email: mp+192581@code.staging.launchpad.net |
Commit message
Don't leak the g_file_get_contents value, reported by valgrind
Description of the change
Don't leak the g_file_get_contents value, reported by valgrind
==17468== 4,097 bytes in 1 blocks are definitely lost in loss record 1,921 of 1,924
==17468== at 0x40299D8: malloc (in /usr/lib/
==17468== by 0x428562F: g_try_malloc (gmem.c:255)
==17468== by 0x426C7CE: g_file_get_contents (gfileutils.c:752)
==17468== by 0x8059DEE: ib_brightness_
==17468== by 0x8059E76: ib_brightness_
==17468== by 0x8057E53: action_
==17468== by 0x8057E7A: update_
==17468== by 0x80592FF: on_bus_acquired (service.c:526)
==17468== by 0x4143137: connection_get_cb (gdbusnameownin
==17468== by 0x40DF294: g_simple_
==17468== by 0x40DF2FA: complete_in_idle_cb (gsimpleasyncre
==17468== by 0x427C59F: g_idle_dispatch (gmain.c:5250)
==17468== by 0x427F83D: g_main_
==17468== by 0x427FBE7: g_main_
==17468== by 0x428004A: g_main_loop_run (gmain.c:3906)
==17468== by 0x4372904: (below main) (libc-start.c:260)
PASSED: Continuous integration, rev:209 jenkins. qa.ubuntu. com/job/ indicator- power-ci/ 64/ jenkins. qa.ubuntu. com/job/ indicator- power-trusty- amd64-ci/ 1 jenkins. qa.ubuntu. com/job/ indicator- power-trusty- armhf-ci/ 1 jenkins. qa.ubuntu. com/job/ indicator- power-trusty- armhf-ci/ 1/artifact/ work/output/ *zip*/output. zip
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
deb: http://
Click here to trigger a rebuild: 10.97.0. 26:8080/ job/indicator- power-ci/ 64/rebuild
http://