On 6/16/20 8:25 AM, Peter Krempa wrote: > On Tue, Jun 16, 2020 at 08:07:01 -0400, John Ferlan wrote: >> Even though I changed roles, the cron job still runs. I recently >> updated to a newer coverity version and it found some new stuff and >> even more false positives. It's still very unhappy with the usage >> of GLIB_DEPRECATED_ENUMERATOR_IN_* macros for *glib/gspawn.h and >> *gobject/gparam.h, but I can work around that. I had to disable >> the use_after_free type checking because the virObject to GObject >> change just caused a large number of possible false positives for >> virObjectUnref usages (coverity cannot keep track of all the counts). >> >> Anyway, I had a few spare cycles, so I figured I'd pass along at >> least the memory leak and usage things that were found. I haven't >> posted in a while so hopefully I haven't missed some subtlety. >> >> FWIW: I no longer have commit access so I'm at the mercy of someone >> else pushing my patches now. >> >> >> John Ferlan (9): >> util: Fix memory leak in virAuthGetCredential >> util: Fix memory leak in virAuthConfigLookup >> lxc: Fix memory leak in virLXCControllerPopulateDevices >> util: Fix memory leak in virPCIProbeStubDriver >> test: Fix memory leak in testParseXMLDocFromFile >> conf: Fix memory leak in openvzWriteConfigParam >> conf: Fix memory leak in openvzReadFSConf >> conf: Fix memory leak in virCPUDefParseXML > > Patches 1-8: > > Reviewed-by: Peter Krempa <pkrempa@xxxxxxxxxx> > > And I'll push them shortly with the fix mentioned in one of them. > OK - thanks! John I see why I commented the way I did for @absFile <sigh>, but yeah it's a problem either way... Probably was more focused on the fact it was my original blunder... >> util: Avoid using wrong free function >> >> src/conf/cpu_conf.c | 2 +- >> src/lxc/lxc_controller.c | 3 ++- >> src/openvz/openvz_conf.c | 7 ++----- >> src/remote/remote_driver.c | 2 +- >> src/test/test_driver.c | 2 +- >> src/util/iohelper.c | 7 +++++++ >> src/util/virauth.c | 5 +---- >> src/util/virauthconfig.c | 4 ++-- >> src/util/virauthconfig.h | 2 +- >> src/util/virpci.c | 1 + >> tests/virauthconfigtest.c | 2 +- >> 11 files changed, 20 insertions(+), 17 deletions(-) >> >> -- >> 2.25.4 >>