Re: [libvirt] [PATCH 3/3] qemumonitortestutils: Don't skip va_end() on error path

2013-08-02 Thread Peter Krempa
On 08/01/13 19:57, John Ferlan wrote: On 08/01/2013 10:39 AM, Peter Krempa wrote: When allocation of memory failed the error path didn't call va_end() causing a coverity failure. Reported by John Ferlan. --- tests/qemumonitortestutils.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

Re: [libvirt] [PATCH 3/3] qemumonitortestutils: Don't skip va_end() on error path

2013-08-01 Thread John Ferlan
On 08/01/2013 10:39 AM, Peter Krempa wrote: > When allocation of memory failed the error path didn't call va_end() > causing a coverity failure. > > Reported by John Ferlan. > --- > tests/qemumonitortestutils.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > ACK John -- libvir-list

Re: [libvirt] [PATCH 3/3] qemumonitortestutils: Don't skip va_end() on error path

2013-08-01 Thread Eric Blake
On 08/01/2013 08:39 AM, Peter Krempa wrote: > When allocation of memory failed the error path didn't call va_end() > causing a coverity failure. > > Reported by John Ferlan. > --- > tests/qemumonitortestutils.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) ACK. > > diff --git a/tests

[libvirt] [PATCH 3/3] qemumonitortestutils: Don't skip va_end() on error path

2013-08-01 Thread Peter Krempa
When allocation of memory failed the error path didn't call va_end() causing a coverity failure. Reported by John Ferlan. --- tests/qemumonitortestutils.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/qemumonitortestutils.c b/tests/qemumonitortestutils.c index fb76156.