On 08/22/2014 11:28 AM, John Ferlan wrote:
Resolve some issues seen by my daily Coverity run.
Interesting that they didn't show up on the internal Jenkins Coverity
scanner, but did show up on my daily run. Perhaps because I have 7.0.3
and the Jenkins has 7.0.0... I also see 7.5.0 is available - I'll give
that a go too...
John Ferlan (3):
commandtest: Resolve Coverity RESOURCE_LEAK
virnetsocket: Resolve Coverity RESOURCE_LEAK
xenconfig: Resolve Coverity RESOURCE_LEAK
src/rpc/virnetsocket.c | 8 ++++++--
src/xenconfig/xen_common.c | 2 ++
tests/commandtest.c | 1 +
3 files changed, 9 insertions(+), 2 deletions(-)
I've resolved the points from review and pushed.
Thanks for the quick reviews!
John
FWIW:
I should never have upgraded my coverity to 7.5.0 - it's *way too*
chatty! <sigh> Although it seems to have found some more "interesting"
latent things, but also it seems a new set of false positives. <double
sigh>... Only 164 things to look at!