After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 639167 - memory leak in DBus.bus.acquire_name
memory leak in DBus.bus.acquire_name
Status: RESOLVED FIXED
Product: gjs
Classification: Bindings
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: gjs-maint
gjs-maint
Depends on:
Blocks:
 
 
Reported: 2011-01-10 18:01 UTC by Tommi Komulainen
Modified: 2011-02-22 12:15 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
dbus: fix memory leak in DBus.bus.acquire_name (1.49 KB, patch)
2011-01-10 18:01 UTC, Tommi Komulainen 		committed Details | Review

Description Tommi Komulainen 2011-01-10 18:01:30 UTC 
After running make valgrind-check

14 bytes in 1 blocks are definitely lost in loss record 301 of 1,270
   at 0x4024F20: malloc (vg_replace_malloc.c:236)
   by 0x40FAD63: g_malloc (gmem.c:164)
   by 0x4113C18: g_strdup (gstrfuncs.c:102)
   by 0x4037F11: gjs_string_get_ascii (jsapi-util-string.c:272)
   by 0x5BE20D4: gjs_js_dbus_acquire_name (dbus.c:1178)
   by 0x420A016: js_Interpret (in /opt/big/lib/libmozjs.so)
   by 0x4213700: js_Execute (in /opt/big/lib/libmozjs.so)
   by 0x41B8894: JS_EvaluateUCScriptForPrincipals (in /opt/big/lib/libmozjs.so)
   by 0x41B897E: JS_EvaluateUCScript (in /opt/big/lib/libmozjs.so)
   by 0x41BCF2F: JS_EvaluateScript (in /opt/big/lib/libmozjs.so)
   by 0x4031BE3: gjs_context_eval (context.c:912)
   by 0x4031EA7: gjs_context_eval_file (context.c:993)
Comment 1 Tommi Komulainen 2011-01-10 18:01:33 UTC 
Created attachment 177954 [details] [review]
dbus: fix memory leak in DBus.bus.acquire_name

As reported by valgrind-check.

14 bytes in 1 blocks are definitely lost in loss record 301 of 1,270
   at 0x4024F20: malloc (vg_replace_malloc.c:236)
   by 0x40FAD63: g_malloc (gmem.c:164)
   by 0x4113C18: g_strdup (gstrfuncs.c:102)
   by 0x4037F11: gjs_string_get_ascii (jsapi-util-string.c:272)
   by 0x5BE20D4: gjs_js_dbus_acquire_name (dbus.c:1178)
   by 0x420A016: js_Interpret (in /opt/big/lib/libmozjs.so)
   by 0x4213700: js_Execute (in /opt/big/lib/libmozjs.so)
   by 0x41B8894: JS_EvaluateUCScriptForPrincipals (in /opt/big/lib/libmozjs.so)
   by 0x41B897E: JS_EvaluateUCScript (in /opt/big/lib/libmozjs.so)
   by 0x41BCF2F: JS_EvaluateScript (in /opt/big/lib/libmozjs.so)
   by 0x4031BE3: gjs_context_eval (context.c:912)
   by 0x4031EA7: gjs_context_eval_file (context.c:993)
Comment 2 Johan (not receiving bugmail) Dahlin 2011-02-17 17:19:03 UTC 
Review of attachment 177954 [details] [review]:

Sure. Thanks.
Comment 3 Tommi Komulainen 2011-02-22 12:15:08 UTC 
Attachment 177954 [details] pushed as 7859892 - dbus: fix memory leak in DBus.bus.acquire_name