]> git.proxmox.com Git - mirror_corosync.git/commitdiff
totemknet: Initialize return value in setup_nozzle
authorJan Friesse <jfriesse@redhat.com>
Tue, 11 Jun 2019 09:00:03 +0000 (11:00 +0200)
committerJan Friesse <jfriesse@redhat.com>
Wed, 12 Jun 2019 13:40:51 +0000 (15:40 +0200)
Also add comment why return value is currently not used.

Signed-off-by: Jan Friesse <jfriesse@redhat.com>
Reviewed-by: Christine Caulfield <ccaulfie@redhat.com>
exec/totemknet.c

index eb2ed2cd435c191f27ca6cb143c7223da924e279..e6e04d3a769898a0f4f4598b3571af943f1f256c 100644 (file)
@@ -1428,8 +1428,10 @@ int totemknet_reconfigure (
        }
 
 #ifdef HAVE_LIBNOZZLE
-       /* Set up nozzle device(s) */
-       setup_nozzle(instance);
+       /* Set up nozzle device(s). Return code is ignored, because unability
+        * configure nozzle is not fatal problem, errors are logged and
+        * there is not much else we can do */
+       (void)setup_nozzle(instance);
 #endif
        return (res);
 }
@@ -1793,7 +1795,7 @@ static int setup_nozzle(void *knet_context)
        char mac[32];
        int name_res;
        int macaddr_res;
-       int res;
+       int res = -1;
 
        /*
         * Return value ignored on purpose. icmap_get_string changes