]> git.ipfire.org Git - thirdparty/libvirt.git/commitdiff
Fix building with -Og
authorMartin Kletzander <mkletzan@redhat.com>
Fri, 3 Jun 2016 11:26:30 +0000 (13:26 +0200)
committerMartin Kletzander <mkletzan@redhat.com>
Mon, 3 Oct 2016 15:40:03 +0000 (17:40 +0200)
When building using -Og, gcc sees that some variables can be used
uninitialized  It can be debatable whether it is possible with our
codeflow, but functions should be self-contained and initializations are
always good.  The return instead of goto is due to actualType being used
in the cleanup.

Signed-off-by: Martin Kletzander <mkletzan@redhat.com>
(cherry picked from commit 3470cd860d517760b13e26d97b6a842ff72687a1)

src/lxc/lxc_driver.c
src/nwfilter/nwfilter_ebiptables_driver.c
src/util/virbitmap.c

index 1a9550e438ae0dd9a5fb55bc48ca79cae5dba404..5618c0993932dee3541abaa9370523f4c2a91557 100644 (file)
@@ -4183,7 +4183,7 @@ lxcDomainAttachDeviceNetLive(virConnectPtr conn,
     if (!priv->initpid) {
         virReportError(VIR_ERR_OPERATION_INVALID, "%s",
                        _("Cannot attach disk until init PID is known"));
-        goto cleanup;
+        return -1;
     }
 
     /* preallocate new slot for device */
index 423d069e1b268eb60fc61b44e2b3244bacbeb596..b7be2917e29e309c6deefadccb5738b4d31bf98d 100644 (file)
@@ -1570,7 +1570,7 @@ iptablesCreateRuleInstanceStateCtrl(virFirewallPtr fw,
                                     const char *ifname,
                                     virNWFilterVarCombIterPtr vars)
 {
-    int rc;
+    int rc = 0;
     bool directionIn = false;
     char chainPrefix[2];
     bool maySkipICMP, inout = false;
index 57135a09f71eb088173e1c6f94a1cd0df3a9ef3f..c94764792fd185f19519c30cd1aa2c9c1cb6a487 100644 (file)
@@ -724,7 +724,7 @@ virBitmapLastSetBit(virBitmapPtr bitmap)
     ssize_t i;
     int unusedBits;
     ssize_t sz;
-    unsigned long bits;
+    unsigned long bits = 0;
 
     unusedBits = bitmap->map_len * VIR_BITMAP_BITS_PER_UNIT - bitmap->max_bit;