A debug message was supposed to print the node name, old state, new
state, and reason for the change, in that order. However, the parameters
that were printed were out of order. This resulted in nonsense output.

Signed-off-by: Mark Michelson <mmich...@redhat.com>
---
 lib/inc-proc-eng.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/lib/inc-proc-eng.c b/lib/inc-proc-eng.c
index 3bcdf95ad..78187c8fd 100644
--- a/lib/inc-proc-eng.c
+++ b/lib/inc-proc-eng.c
@@ -325,9 +325,8 @@ engine_set_node_state(struct engine_node *node,
         va_start(args, reason_fmt);
         char *reason = xvasprintf(reason_fmt, args);
         VLOG_DBG("node: %s, old_state %s, new_state %s, reason: %s.",
-                 reason, node->name,
-                 engine_node_state_name[node->state],
-                 engine_node_state_name[state]);
+                 node->name, engine_node_state_name[node->state],
+                 engine_node_state_name[state], reason);
         va_end(args);
         free(reason);
     }
-- 
2.49.0

_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to