Re: [libvirt PATCH 06/11] qemu_monitor_json: explicitly ignore return values

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

 



On Mon, Nov 16, 2020 at 17:09:17 +0100, Pavel Hrdina wrote:
> On Mon, Nov 16, 2020 at 04:58:56PM +0100, Peter Krempa wrote:
> > On Mon, Nov 16, 2020 at 16:38:53 +0100, Pavel Hrdina wrote:
> > > Signed-off-by: Pavel Hrdina <phrdina@xxxxxxxxxx>
> > > ---
> > >  src/qemu/qemu_monitor_json.c | 4 ++--
> > >  1 file changed, 2 insertions(+), 2 deletions(-)
> > 
> > I presume this silences a coverity moan about "most of the callers check
> > the value". NACK in that case.
> 
> Yes it does. In that case we should remove ignore_value() from the
> remaining calls to that function that are not checked.

Either that, or for a full fix add G_GNUC_UNUSED to the function header.
Without that nothing would be really fixed and noting will be prevented
in the future.




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux