]> git.ipfire.org Git - thirdparty/libvirt.git/commit
util: add missing error log messages when failing to get netlink VFINFO
authorLaine Stump <laine@laine.org>
Thu, 20 Dec 2012 19:52:41 +0000 (14:52 -0500)
committerLaine Stump <laine@laine.org>
Tue, 8 Jan 2013 17:22:16 +0000 (12:22 -0500)
commit6b789ea3e03db52ef6f5694dca34cff57ce8201d
tree35813c29e01d77ee858572b909bb49fe96cdd05f
parent52fca883d235a484708bc8108b54e61d0b14f3dc
util: add missing error log messages when failing to get netlink VFINFO

This patch fixes the lack of error messages when libvirt fails to find
VFINFO in a returned netlinke response message.

https://bugzilla.redhat.com/show_bug.cgi?id=827519#c10 is an example
of the error message that was previously logged when the
IFLA_VFINFO_LIST object was missing from the netlink response. The
reason for this failure is detailed in

   https://bugzilla.redhat.com/show_bug.cgi?id=889319

Even though that root problem has been fixed, the experience of
finding the root cause shows us how important it is to properly log an
error message in these cases. This patch *seems* to replace the entire
function, but really most of the changes are due to moving code that
was previously inside an if() statement out to the top level of the
function (the original if() was reversed and made to log an error and
return).
(cherry picked from commit 846770e5ff959f7819e2c32857598cb88e2e2f0e)
src/util/virnetdev.c