commit 687707d0b8659765c27a18a749299d8ad31be234
author Guillaume Solignac <gsoligna@cisco.com> Thu Jan 18 12:01:57 2024 +0100
committer Damjan Marion <dmarion@0xa5.net> Mon May 06 11:53:32 2024 +0000
tree f640a0a6ac6ba8b144b098e40bb8097cb5492fa6
parent 4aecd4869c47c3561fb4acde4726a0a832798cff

vlib: fix missing integer init

Some compilers were not clever enough to figure out these inits
weren't needed. Hence adding them.

Type: fix
Signed-off-by: Guillaume Solignac <gsoligna@cisco.com>
Change-Id: I50c4e978bfd2ef15ece7a29e517bb8ddba12f960
Signed-off-by: Pierre Pfister <ppfister@cisco.com>
Signed-off-by: Dave Wallace <dwallacelf@gmail.com>

src/vlib/pci/pci.c

diff --git a/src/vlib/pci/pci.c b/src/vlib/pci/pci.c
index 0bc90c5..7284e66 100644
--- a/src/vlib/pci/pci.c
+++ b/src/vlib/pci/pci.c
@@ -119,7 +119,7 @@
 {
   const vlib_pci_config_reg_command_t cmd = { .intx_disable = 1 };
   clib_error_t *err;
-  int already_set;
+  int already_set = 0;
 
   err = _vlib_pci_config_set_control_bit (vm, h, cmd.as_u16, 0, &already_set);
   log_debug (h, "interrupt%senabled", already_set ? " " : " already ");
@@ -131,7 +131,7 @@
 {
   const vlib_pci_config_reg_command_t cmd = { .intx_disable = 1 };
   clib_error_t *err;
-  int already_set;
+  int already_set = 0;
 
   err = _vlib_pci_config_set_control_bit (vm, h, cmd.as_u16, 1, &already_set);
   log_debug (h, "interrupt%sdisabled", already_set ? " " : " already ");
@@ -143,7 +143,7 @@
 {
   const vlib_pci_config_reg_command_t cmd = { .bus_master = 1 };
   clib_error_t *err;
-  int already_set;
+  int already_set = 0;
 
   err = _vlib_pci_config_set_control_bit (vm, h, cmd.as_u16, 1, &already_set);
   log_debug (h, "bus-master%senabled", already_set ? " " : " already ");
@@ -155,7 +155,7 @@
 {
   const vlib_pci_config_reg_command_t cmd = { .bus_master = 1 };
   clib_error_t *err;
-  int already_set;
+  int already_set = 0;
 
   err = _vlib_pci_config_set_control_bit (vm, h, cmd.as_u16, 0, &already_set);
   log_debug (h, "bus-master%sdisabled", already_set ? " " : " already ");