HUD: Small memory leak when switching windows
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Application Menu Indicator |
Fix Released
|
Low
|
Allison Karlitskaya | ||
indicator-appmenu (Ubuntu) |
Fix Released
|
Low
|
Allison Karlitskaya |
Bug Description
Valgrind reports this leak every time I switch between windows:
==1830== 12,808 (+6,592) bytes in 688 (+348) blocks are definitely lost in loss record 3,245 of 3,324
==1830== at 0x4C2B6CD: malloc (in /usr/lib/
==1830== by 0x5847A38: g_malloc (gmem.c:159)
==1830== by 0x585C2AD: g_strdup (gstrfuncs.c:356)
==1830== by 0x55E31EF: g_value_set_string (gvaluetypes.
==1830== by 0x63812B9: demarshal_basic (dbus-gvalue.c:619)
==1830== by 0x6382AD4: _dbus_gvalue_
==1830== by 0x637C37C: dbus_g_
==1830== by 0x637F69A: dbus_g_proxy_call (dbus-gproxy.
==1830== by 0x50521A5: bamf_view_get_icon (in /usr/lib/
==1830== by 0x406BB0: hud_window_
==1830== by 0x55BAC81: g_closure_invoke (gclosure.c:777)
==1830== by 0x55CBD50: signal_
==1830== by 0x55D4078: g_signal_
==1830== by 0x55D4221: g_signal_emit (gsignal.c:3352)
==1830== by 0x637D0A6: marshal_
==1830== by 0x55BAC81: g_closure_invoke (gclosure.c:777)
==1830== by 0x55CBD50: signal_
==1830== by 0x55D4078: g_signal_
==1830== by 0x55D4221: g_signal_emit (gsignal.c:3352)
==1830== by 0x637D6E5: dbus_g_
==1830== by 0x76489A5: dbus_connection
==1830== by 0x6376424: message_
==1830== by 0x5841C99: g_main_
==1830== by 0x584205F: g_main_
==1830== by 0x5842459: g_main_loop_run (gmain.c:3317)
==1830== by 0x404EBB: main (hud-service.c:320)
hudwindowsource
source->active_icon = bamf_view_get_icon (BAMF_VIEW (application));
Unfortunately I'm not able to provide a fix.
ProblemType: Bug
DistroRelease: Ubuntu 12.04
Package: indicator-appmenu 0.3.96-0ubuntu2 [modified: usr/lib/
ProcVersionSign
Uname: Linux 3.2.0-21-generic x86_64
ApportVersion: 2.0-0ubuntu3
Architecture: amd64
Date: Tue Apr 3 13:51:59 2012
InstallationMedia: Ubuntu 11.04 "Natty Narwhal" - Alpha amd64 (20110112)
ProcEnviron:
TERM=xterm
LANG=en_US.UTF-8
SHELL=/bin/bash
SourcePackage: indicator-appmenu
UpgradeStatus: No upgrade log present (probably fresh install)
Related branches
- Ted Gould (community): Approve
-
Diff: 2576 lines (+1367/-705)34 files modifiedconfigure.ac (+1/-1)
data/Makefile.am (+5/-13)
data/com.canonical.indicator.appmenu.gschema.xml (+18/-0)
data/com.canonical.indicator.appmenu.gschema.xml.in (+0/-15)
data/com.canonical.indicator.appmenu.hud.gschema.xml (+16/-0)
data/com.canonical.indicator.appmenu.hud.gschema.xml.in (+0/-14)
data/com.canonical.indicator.appmenu.hud.search.gschema.xml (+72/-0)
data/com.canonical.indicator.appmenu.hud.search.gschema.xml.in (+0/-90)
po/POTFILES.in (+4/-4)
src/Makefile.am (+2/-2)
src/distance.c (+0/-415)
src/distance.h (+0/-33)
src/hudappindicatorsource.c (+4/-4)
src/huddbusmenucollector.c (+3/-3)
src/huddebugsource.c (+3/-3)
src/hudindicatorsource.c (+3/-3)
src/huditem.c (+11/-0)
src/huditem.h (+2/-1)
src/hudmenumodelcollector.c (+3/-3)
src/hudquery.c (+9/-1)
src/hudresult.c (+13/-40)
src/hudresult.h (+12/-12)
src/hudsettings.c (+4/-15)
src/hudsettings.h (+1/-4)
src/hudsource.c (+3/-3)
src/hudsource.h (+13/-13)
src/hudsourcelist.c (+3/-3)
src/hudtoken.c (+394/-0)
src/hudtoken.h (+40/-0)
src/hudwindowsource.c (+9/-4)
tests/Makefile.am (+18/-0)
tests/hud-performance.c (+279/-0)
tests/test-distance.c (+58/-6)
tests/word-list.h (+364/-0)
- Ken VanDine: Pending requested
-
Diff: 1560 lines (+613/-245)32 files modifiedChangeLog (+255/-0)
Makefile.am (+1/-35)
Makefile.am.coverage (+48/-0)
Makefile.in (+45/-39)
configure (+106/-68)
configure.ac (+4/-1)
data/Makefile.in (+5/-11)
debian/changelog (+19/-1)
debian/control (+4/-0)
docs/devel/html/HudAppMenuRegistrar.html (+1/-1)
docs/devel/html/HudDbusmenuCollector.html (+3/-3)
docs/devel/html/HudResult.html (+4/-4)
docs/devel/html/HudSource.html (+7/-7)
docs/devel/html/ch01.html (+1/-1)
docs/devel/html/ch02.html (+1/-1)
docs/devel/html/ch03.html (+1/-1)
docs/devel/html/ch04.html (+1/-1)
docs/devel/html/ch05.html (+1/-1)
docs/devel/html/hud-HudSettings.html (+2/-20)
docs/man/hud-cli.1 (+2/-2)
docs/man/hud-dump-application.1 (+2/-2)
docs/man/hud-list-applications.1 (+2/-2)
docs/man/hud-verify-app-info.1 (+2/-2)
m4/gcov.m4 (+13/-10)
src/Makefile.in (+18/-18)
src/hudquery.c (+1/-1)
src/hudtoken.c (+6/-0)
src/hudtoken.h (+1/-0)
src/indicator-appmenu.c (+2/-0)
tests/Makefile.in (+53/-11)
tests/test-dbus-message-count.in (+1/-1)
tools-vala/hud-gtk.c (+1/-1)
Changed in indicator-appmenu (Ubuntu): | |
assignee: | nobody → Ryan Lortie (desrt) |
importance: | Undecided → Low |
Changed in indicator-appmenu (Ubuntu): | |
status: | New → Fix Committed |
Changed in indicator-appmenu: | |
status: | In Progress → Fix Released |
Changed in indicator-appmenu: | |
milestone: | none → 0.3.97 |
This is fixed in Ryan's HUD branch <https:/ /code.launchpad .net/~desrt/ indicator- appmenu/ hud-performance> which will be in this week's release.