Re: [libvirt] [PATCH 7/8] qemu: Resolve Coverity BAD_SIZEOF

2014-09-15 Thread John Ferlan
On 09/15/2014 04:09 AM, Peter Krempa wrote: > On 09/13/14 15:27, John Ferlan wrote: >> Coverity complains about the calculation of the buf & len within >> the PROBE macro. So to quiet things down, do the calculation prior >> to usage in either write() or qemuMonitorIOWriteWithFD() calls and >> t

Re: [libvirt] [PATCH 7/8] qemu: Resolve Coverity BAD_SIZEOF

2014-09-15 Thread Peter Krempa
On 09/15/14 16:57, John Ferlan wrote: > > > On 09/15/2014 04:09 AM, Peter Krempa wrote: >> On 09/13/14 15:27, John Ferlan wrote: >>> Coverity complains about the calculation of the buf & len within >>> the PROBE macro. So to quiet things down, do the calculation prior >>> to usage in either writ

Re: [libvirt] [PATCH 7/8] qemu: Resolve Coverity BAD_SIZEOF

2014-09-15 Thread John Ferlan
On 09/15/2014 04:09 AM, Peter Krempa wrote: > On 09/13/14 15:27, John Ferlan wrote: >> Coverity complains about the calculation of the buf & len within >> the PROBE macro. So to quiet things down, do the calculation prior >> to usage in either write() or qemuMonitorIOWriteWithFD() calls and >> t

Re: [libvirt] [PATCH 7/8] qemu: Resolve Coverity BAD_SIZEOF

2014-09-15 Thread Peter Krempa
On 09/13/14 15:27, John Ferlan wrote: > Coverity complains about the calculation of the buf & len within > the PROBE macro. So to quiet things down, do the calculation prior > to usage in either write() or qemuMonitorIOWriteWithFD() calls and > then have the PROBE use the calculated values - which

[libvirt] [PATCH 7/8] qemu: Resolve Coverity BAD_SIZEOF

2014-09-13 Thread John Ferlan
Coverity complains about the calculation of the buf & len within the PROBE macro. So to quiet things down, do the calculation prior to usage in either write() or qemuMonitorIOWriteWithFD() calls and then have the PROBE use the calculated values - which works. Signed-off-by: John Ferlan --- src/