From aa30ae4e84a2d44dea72e2f7b6fe8ccdbd4f70b1 Mon Sep 17 00:00:00 2001 From: Justin Pettit Date: Thu, 17 Jan 2013 00:31:20 -0800 Subject: [PATCH] ofproto: Add "actions=" to drop action in "ovs-appctl bridge/dump-flows". The function flow_stats_ds() did its own formatting when there were no actions, but this didn't include "action=". There's actually no reason to special-case this, since ofpact_format() properly handles actions with a zero length and prints it properly. Signed-off-by: Justin Pettit --- ofproto/ofproto.c | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/ofproto/ofproto.c b/ofproto/ofproto.c index ddf51f8b3..918b9b466 100644 --- a/ofproto/ofproto.c +++ b/ofproto/ofproto.c @@ -2809,11 +2809,7 @@ flow_stats_ds(struct rule *rule, struct ds *results) ds_put_format(results, "n_bytes=%"PRIu64", ", byte_count); cls_rule_format(&rule->cr, results); ds_put_char(results, ','); - if (rule->ofpacts_len > 0) { - ofpacts_format(rule->ofpacts, rule->ofpacts_len, results); - } else { - ds_put_cstr(results, "drop"); - } + ofpacts_format(rule->ofpacts, rule->ofpacts_len, results); ds_put_cstr(results, "\n"); } -- 2.43.0