lp:~charlesk/indicator-application/hashtable-mem-bugs

Branch merges

Propose for merging

No branches dependent on this one.

Merged into lp:indicator-application/0.5

Lars Karlitski (community): Approve on 2012-03-20

Diff: 68 lines (+23/-21)

1 file modified

src/indicator-application.c (+23/-21)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Bug #957992: memory leak + dangling pointer use in indicator_application_dispose()

Medium

Fix Released

Link a bug report

Related blueprints

228. By Charles Kerr on 2012-03-17

Fix two memory bugs with clearing out out directories from our icon search path.

1. indicator_application_dispose() calls g_hash_table_get_keys() N times and leaks them all.

2. indicator_application_dispose() passes each of the hashtable's keys (owned by the hash) to function theme_dir_unref(key), which removes from the hashtable but then continues to use the now-dangling key.

227. By Charles Kerr on 2012-03-13

0.4.93

226. By Charles Kerr on 2012-03-13

in generate_id(), we have a handful of local guint8's that get left-shifted to the point of potential overflow -- so change them to guint32 instead. Bug reported by coverity in lp ticket #944234

225. By Charles Kerr on 2012-03-13

merge lp:~htorque/indicator-application/fix-small-memleak to fix a small memory leak described in lp bug #953585

224. By Ted Gould on 2012-03-09

0.4.92

223. By Charles Kerr on 2012-03-05

merge lp:~ted/indicator-application/lp944236 to fix the warning that Coverity reported in bug #944236

222. By Ted Gould on 2012-03-02

Add full type to GVariantBuilder

221. By Ted Gould on 2012-02-15

0.4.91

220. By Ted Gould on 2012-02-12

Unreffing the result of the approval message

219. By Ted Gould on 2012-02-08

0.4.90