conf: fix NULL check in virNetDevBandwidthParse

Found by coverity:
Error: REVERSE_INULL (CWE-476):
    libvirt-0.10.2/src/conf/netdev_bandwidth_conf.c:99: deref_ptr:
    Directly dereferencing pointer "node".
    libvirt-0.10.2/src/conf/netdev_bandwidth_conf.c:107:
    check_after_deref: Null-checking "node" suggests that it may be
    null, but it has already been dereferenced on all paths leading to
    the check.
This commit is contained in:
Ján Tomko 2012-11-29 10:37:53 +01:00 committed by Eric Blake
parent f4ea67f5b3
commit d5e8842538

View File

@ -96,7 +96,7 @@ virNetDevBandwidthPtr
virNetDevBandwidthParse(xmlNodePtr node)
{
virNetDevBandwidthPtr def = NULL;
xmlNodePtr cur = node->children;
xmlNodePtr cur;
xmlNodePtr in = NULL, out = NULL;
if (VIR_ALLOC(def) < 0) {
@ -110,6 +110,8 @@ virNetDevBandwidthParse(xmlNodePtr node)
goto error;
}
cur = node->children;
while (cur) {
if (cur->type == XML_ELEMENT_NODE) {
if (xmlStrEqual(cur->name, BAD_CAST "inbound")) {