From: Dave Barach <dave@barachs.net>
Date: Thu, 3 Jan 2019 14:30:00 +0000 (-0500)
Subject: Fix used-before-set in vl_api_bridge_flags_t_handler
X-Git-Tag: v19.04-rc0~55
X-Git-Url: https://gerrit.fd.io/r/gitweb?p=vpp.git;a=commitdiff_plain;h=98d6b61183ecb15932fbdae96a60e0d0fcf97466

Fix used-before-set in vl_api_bridge_flags_t_handler

The aarch64 cross-compiler complains about this when building an
optimized image. So we fix it...

Change-Id: Ib2b2089c0534753d7d470311012fa0073ab27d1d
Signed-off-by: Dave Barach <dave@barachs.net>
---

diff --git a/src/vnet/l2/l2_api.c b/src/vnet/l2/l2_api.c
index 9e3a47f772b..25f38a6d5b0 100644
--- a/src/vnet/l2/l2_api.c
+++ b/src/vnet/l2/l2_api.c
@@ -558,6 +558,7 @@ vl_api_bridge_flags_t_handler (vl_api_bridge_flags_t * mp)
   vlib_main_t *vm = vlib_get_main ();
   bd_main_t *bdm = &bd_main;
   vl_api_bridge_flags_reply_t *rmp;
+  u32 bitmap = 0;
   int rv = 0;
 
   bd_flags_t flags = bd_flags_decode (mp->flags);
@@ -575,7 +576,7 @@ vl_api_bridge_flags_t_handler (vl_api_bridge_flags_t * mp)
       goto out;
     }
 
-  u32 bitmap = bd_set_flags (vm, bd_index, flags, mp->is_set);
+  bitmap = bd_set_flags (vm, bd_index, flags, mp->is_set);
 
 out:
   /* *INDENT-OFF* */