[Bug 1750741] Re: Memory leak in g_dbus_proxy_new_for_bus_sync()
> 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 this bug notification because you are a member of Ubuntu Desktop Bugs, which is subscribed to glib2.0 in Ubuntu. https://bugs.launchpad.net/bugs/1750741 Title: Memory leak in g_dbus_proxy_new_for_bus_sync() To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/glib2.0/+bug/1750741/+subscriptions -- desktop-bugs mailing list desktop-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/desktop-bugs
[Bug 1750741] Re: Memory leak in g_dbus_proxy_new_for_bus_sync()
@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 received this bug notification because you are a member of Ubuntu Desktop Bugs, which is subscribed to glib2.0 in Ubuntu. https://bugs.launchpad.net/bugs/1750741 Title: Memory leak in g_dbus_proxy_new_for_bus_sync() To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/glib2.0/+bug/1750741/+subscriptions -- desktop-bugs mailing list desktop-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/desktop-bugs
[Bug 1750741] Re: Memory leak in g_dbus_proxy_new_for_bus_sync()
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 Ubuntu Desktop Bugs, which is subscribed to glib2.0 in Ubuntu. https://bugs.launchpad.net/bugs/1750741 Title: Memory leak in g_dbus_proxy_new_for_bus_sync() To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/glib2.0/+bug/1750741/+subscriptions -- desktop-bugs mailing list desktop-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/desktop-bugs
[Bug 1750741] Re: Memory leak in g_dbus_proxy_new_for_bus_sync()
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 this memleak bug - it's ok to include in the same upload, but it needs its own lp bug. This is also needed in xenial glib2.0. + - d/p/0001-gio-Add-names-to-idles-and-timeouts.patch + - d/p/0002-gio-belatedly-port-gdbus-from-GSimpleAsyncResult-to-.patch these patches are huge backports, that update the glib2.0 code to use a different function that appears to *introduce* the memleak that your first patch "fixes". Are you sure there is actually a memleak in the Trusty glib2.0 code? If so, it does not seem to be the one that your first patch fixes. In any case, this huge backport is not acceptable as a SRU for only a memleak. -- You received this bug notification because you are a member of Ubuntu Desktop Bugs, which is subscribed to glib2.0 in Ubuntu. https://bugs.launchpad.net/bugs/1750741 Title: Memory leak in g_dbus_proxy_new_for_bus_sync() To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/glib2.0/+bug/1750741/+subscriptions -- desktop-bugs mailing list desktop-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/desktop-bugs
[Bug 1750741] Re: Memory leak in g_dbus_proxy_new_for_bus_sync()
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 https://launchpad.net/~xtrusia/+archive/ubuntu/sf160627-glib/+build/14383194/+files /buildlog_ubuntu-trusty- amd64.glib2.0_2.40.2-0ubuntu1+sf160627v20180222.0_BUILDING.txt.gz -- You received this bug notification because you are a member of Ubuntu Desktop Bugs, which is subscribed to glib2.0 in Ubuntu. https://bugs.launchpad.net/bugs/1750741 Title: Memory leak in g_dbus_proxy_new_for_bus_sync() To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/glib2.0/+bug/1750741/+subscriptions -- desktop-bugs mailing list desktop-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/desktop-bugs
[Bug 1750741] Re: Memory leak in g_dbus_proxy_new_for_bus_sync()
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 received this bug notification because you are a member of Ubuntu Desktop Bugs, which is subscribed to glib2.0 in Ubuntu. https://bugs.launchpad.net/bugs/1750741 Title: Memory leak in g_dbus_proxy_new_for_bus_sync() To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/glib2.0/+bug/1750741/+subscriptions -- desktop-bugs mailing list desktop-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/desktop-bugs
[Bug 1750741] Re: Memory leak in g_dbus_proxy_new_for_bus_sync()
** Also affects: glib2.0 (Ubuntu Trusty) Importance: Undecided Status: New -- You received this bug notification because you are a member of Ubuntu Desktop Bugs, which is subscribed to glib2.0 in Ubuntu. https://bugs.launchpad.net/bugs/1750741 Title: Memory leak in g_dbus_proxy_new_for_bus_sync() To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/glib2.0/+bug/1750741/+subscriptions -- desktop-bugs mailing list desktop-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/desktop-bugs
[Bug 1750741] Re: Memory leak in g_dbus_proxy_new_for_bus_sync()
** 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 Ubuntu Desktop Bugs, which is subscribed to glib2.0 in Ubuntu. https://bugs.launchpad.net/bugs/1750741 Title: Memory leak in g_dbus_proxy_new_for_bus_sync() To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/glib2.0/+bug/1750741/+subscriptions -- desktop-bugs mailing list desktop-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/desktop-bugs
