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!