[PATCH 0/5] GDBus and memleak fixes

Michal Privoznik posted 5 patches 3 years, 5 months ago
Test syntax-check failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1601630385.git.mprivozn@redhat.com
NEWS.rst                |  6 ++++++
src/rpc/virnetdaemon.c  |  4 ++--
src/util/virfirewalld.c | 16 ++++++++--------
src/util/virgdbus.c     | 29 ++++++++++++++++++++---------
src/util/virgdbus.h     |  4 ++--
src/util/virpolkit.c    |  4 ++--
src/util/virsystemd.c   | 23 ++++++++---------------
src/vmx/vmx.c           |  1 +
tests/meson.build       |  1 +
tests/virfirewalltest.c |  2 +-
10 files changed, 51 insertions(+), 39 deletions(-)
[PATCH 0/5] GDBus and memleak fixes
Posted by Michal Privoznik 3 years, 5 months ago
The first patch fixes issue raised by Cole here:

https://www.redhat.com/archives/libvir-list/2020-October/msg00037.html

The second one sets the environment variable which will cause our tests
fail should we run into similar problem. To see it in action just switch
order with patch 1/5 and observe test suite failing.

The last two patches fix memleaks I've found during debugging the
original problem.

https://gitlab.com/MichalPrivoznik/libvirt/-/pipelines/197284472

Michal Prívozník (5):
  lib: Don't unref message passed to virGDBusCallMethod{WithFD}()
  tests: Set G_DEBUG environment variable
  virfirewalltest: Don't duplicate string when adding it onto stringlist
  vmx; Free @checkMACAddress in virVMXParseEthernet()
  NEWS: Document recent GLib DBus fix

 NEWS.rst                |  6 ++++++
 src/rpc/virnetdaemon.c  |  4 ++--
 src/util/virfirewalld.c | 16 ++++++++--------
 src/util/virgdbus.c     | 29 ++++++++++++++++++++---------
 src/util/virgdbus.h     |  4 ++--
 src/util/virpolkit.c    |  4 ++--
 src/util/virsystemd.c   | 23 ++++++++---------------
 src/vmx/vmx.c           |  1 +
 tests/meson.build       |  1 +
 tests/virfirewalltest.c |  2 +-
 10 files changed, 51 insertions(+), 39 deletions(-)

-- 
2.26.2