[Desktop-packages] [Bug 1750741] Re: Memory leak in g_dbus_proxy_new_for_bus_sync()

2018-02-27 Thread Dan Streetman
> so lets focus on pacemaker instead of glib2.0 sounds good - i will mark this as 'wont fix', but we can come back to it if needed. Also note, the existing bug 1545308 appears to be the same as this. ** Changed in: glib2.0 (Ubuntu Trusty) Status: Confirmed => Won't Fix -- You received

[Desktop-packages] [Bug 1750741] Re: Memory leak in g_dbus_proxy_new_for_bus_sync()

2018-02-26 Thread Seyeong Kim
@ddstreet After analyzing code. I don't think this fix is possible without any big changes for trusty. and glib2.0 affects many pkgs it seems. it is quite dangerous to fix this with large commits. so lets focus on pacemaker instead of glib2.0 how about this ddstreet? Thanks -- You

[Desktop-packages] [Bug 1750741] Re: Memory leak in g_dbus_proxy_new_for_bus_sync()

2018-02-22 Thread Launchpad Bug Tracker
Status changed to 'Confirmed' because the bug affects multiple users. ** Changed in: glib2.0 (Ubuntu Trusty) Status: New => Confirmed -- You received this bug notification because you are a member of Desktop Packages, which is subscribed to glib2.0 in Ubuntu.

[Desktop-packages] [Bug 1750741] Re: Memory leak in g_dbus_proxy_new_for_bus_sync()

2018-02-22 Thread Dan Streetman
Seyeong, + - d/p/0001-GDBusProxy-Fix-a-memory-leak-during- initialization.patch this fixes a memory leak, but in later code than the Trusty glib2.0 code + - d/p/0001-tests-gdatetime-Use-a-real-rather-than-invented- time.patch this fixes an autopkgtest regression, that's unrelated to

[Desktop-packages] [Bug 1750741] Re: Memory leak in g_dbus_proxy_new_for_bus_sync()

2018-02-22 Thread Seyeong Kim
Hello ddstreet yep, it is tricky.. In the beginning, I added only one line but build failed. 3 commits for building, only one line is related to memroy leak directly. I retried it and attaching build link on lp Thanks

[Desktop-packages] [Bug 1750741] Re: Memory leak in g_dbus_proxy_new_for_bus_sync()

2018-02-21 Thread Dan Streetman
Seyeong, the debdiff patch is very large, and contains more than just the 1-line memleak fix from the description...can you remove the extra changes from the debdiff so it only contains the memleak patch? Or explain why the other changes are needed? ** Tags added: sts-sponsor-ddstreet -- You

[Desktop-packages] [Bug 1750741] Re: Memory leak in g_dbus_proxy_new_for_bus_sync()

2018-02-21 Thread Dan Streetman
** Also affects: glib2.0 (Ubuntu Trusty) Importance: Undecided Status: New -- You received this bug notification because you are a member of Desktop Packages, which is subscribed to glib2.0 in Ubuntu. https://bugs.launchpad.net/bugs/1750741 Title: Memory leak in

[Desktop-packages] [Bug 1750741] Re: Memory leak in g_dbus_proxy_new_for_bus_sync()

2018-02-20 Thread Seyeong Kim
** Patch added: "lp1750741_trusty_glib2.0.debdiff" https://bugs.launchpad.net/ubuntu/+source/glib2.0/+bug/1750741/+attachment/5059468/+files/lp1750741_trusty_glib2.0.debdiff -- You received this bug notification because you are a member of Desktop Packages, which is subscribed to glib2.0 in