vlib: fix missing integer init 66/40766/4
authorGuillaume Solignac <[email protected]>
Thu, 18 Jan 2024 11:01:57 +0000 (12:01 +0100)
committerDamjan Marion <[email protected]>
Mon, 6 May 2024 11:53:32 +0000 (11:53 +0000)
Some compilers were not clever enough to figure out these inits
weren't needed. Hence adding them.

Type: fix
Signed-off-by: Guillaume Solignac <[email protected]>
Change-Id: I50c4e978bfd2ef15ece7a29e517bb8ddba12f960
Signed-off-by: Pierre Pfister <[email protected]>
Signed-off-by: Dave Wallace <[email protected]>
src/vlib/pci/pci.c

index 0bc90c5..7284e66 100644 (file)
@@ -119,7 +119,7 @@ vlib_pci_intr_enable (vlib_main_t *vm, vlib_pci_dev_handle_t h)
 {
   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 @@ vlib_pci_intr_disable (vlib_main_t *vm, vlib_pci_dev_handle_t h)
 {
   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 @@ vlib_pci_bus_master_enable (vlib_main_t *vm, vlib_pci_dev_handle_t h)
 {
   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 @@ vlib_pci_bus_master_disable (vlib_main_t *vm, vlib_pci_dev_handle_t h)
 {
   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 ");