[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH v2] qemu: Resolve Coverity DEADCODE.




On 11/12/2014 10:53 AM, Matthias Gatto wrote:
> reported here: http://www.redhat.com/archives/libvir-list/2014-November/msg00327.html
> 
> I could have just remove bool supportMaxOptions variable, but
> if I had do this, we could not check anymore if the nparams variable is
> superior to QEMU_NB_BLOCK_IO_TUNE_PARAM_MAX.
> 
> v2: change following this proposal:
> http://www.redhat.com/archives/libvir-list/2014-November/msg00379.html
> 
> ---
>  src/qemu/qemu_driver.c | 17 +++++++++--------
>  1 file changed, 9 insertions(+), 8 deletions(-)
> 

ACK

I'll push the two Coverity changes shortly, plus I'll post the test Jan
asked for...

John


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]