]> git.ipfire.org Git - thirdparty/libvirt.git/commit
qemu: Check return status for virDomainGraphicsListenSetAddress
authorJohn Ferlan <jferlan@redhat.com>
Tue, 9 Feb 2016 11:24:33 +0000 (06:24 -0500)
committerJohn Ferlan <jferlan@redhat.com>
Wed, 10 Feb 2016 20:18:54 +0000 (15:18 -0500)
commit336d4dace49939ef20698eca7379cdaf148d82f1
tree0c06a11583e14fd3e86bfa3e280175dbb5d662df
parent7b3adac4cfb00d95f691aa845a42e4d1e21530b8
qemu: Check return status for virDomainGraphicsListenSetAddress

Recent refactors in the vbox code to check the return status for the
function tipped Coverity's scales of justice for any functions that
do not check status - such as this one.

While I'm at it, since the call is essentially the same other than
whether starting from val or val+1 when val[0] = '[', just adjust
the val pointer by one and have one call instead of two.

Additionally, the call to virDomainGraphicsListenGetAddress is redundant
since it checking that the address field got filled.  It's a leftover
from the strndup -> ListenSetAddress conversion (commit id 'ef79fb5b5')

Signed-off-by: John Ferlan <jferlan@redhat.com>
src/qemu/qemu_command.c