Re: [libvirt] [PATCH 0/9] More Coverity fixes

2014-09-12 Thread John Ferlan
On 09/11/2014 08:05 PM, John Ferlan wrote: > There are two repeats from the last series (1 & 2). > > For patch 1, I went with my suggestion - I'm open to others > For patch 2, Coverity was complaining more about the way nparams > would be overwritten - fix that by adding a new variable > >

Re: [libvirt] [PATCH 0/9] More Coverity fixes

2014-09-11 Thread Eric Blake
On 09/11/2014 06:05 PM, John Ferlan wrote: > There are two repeats from the last series (1 & 2). > > For patch 1, I went with my suggestion - I'm open to others > For patch 2, Coverity was complaining more about the way nparams > would be overwritten - fix that by adding a new variable > > Ne

[libvirt] [PATCH 0/9] More Coverity fixes

2014-09-11 Thread John Ferlan
There are two repeats from the last series (1 & 2). For patch 1, I went with my suggestion - I'm open to others For patch 2, Coverity was complaining more about the way nparams would be overwritten - fix that by adding a new variable New patches 3 & 4 -> eblake helped out with these - especia