diff mbox series

[ovs-dev,V2,3/8] ovs-ofctl: Fix 'usable_protocols' may be used uninitialized

Message ID 20221116104727.915151-4-elibr@nvidia.com
State Deferred
Headers show
Series Fix warnings and enable Werror | expand

Checks

Context Check Description
ovsrobot/apply-robot success apply and check: success
ovsrobot/github-robot-_Build_and_Test success github build: passed
ovsrobot/intel-ovs-compilation success test: success

Commit Message

Eli Britstein Nov. 16, 2022, 10:47 a.m. UTC
With --enable-Werror and --with-dpdk=no:

../utilities/ovs-ofctl.c: In function 'ofctl_group_mod_file':
../utilities/ovs-ofctl.c:3107:16: error: 'usable_protocols' may be used uninitialized in this function [-Werror=maybe-uninitialized]
 3107 |     protocol = open_vconn_for_flow_mod(remote, &vconn, usable_protocols);
      |                ^
../utilities/ovs-ofctl.c:3124:27: note: 'usable_protocols' was declared here
 3124 |     enum ofputil_protocol usable_protocols;
      |                           ^

Fixes: 69185eb25acb ("ovs-ofctl: Only allow usable protocols for group commands")
Signed-off-by: Eli Britstein <elibr@nvidia.com>
---
 utilities/ovs-ofctl.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff mbox series

Patch

diff --git a/utilities/ovs-ofctl.c b/utilities/ovs-ofctl.c
index fe9114580..6805140d6 100644
--- a/utilities/ovs-ofctl.c
+++ b/utilities/ovs-ofctl.c
@@ -3121,7 +3121,7 @@  static void
 ofctl_group_mod_file(int argc OVS_UNUSED, char *argv[], int command)
 {
     struct ofputil_group_mod *gms = NULL;
-    enum ofputil_protocol usable_protocols;
+    enum ofputil_protocol usable_protocols = 0;
     size_t n_gms = 0;
     char *error;