summaryrefslogtreecommitdiffstats
path: root/pbrd
diff options
context:
space:
mode:
authorG. Paul Ziemba <paulz@labn.net>2023-08-18 20:11:17 +0200
committerG. Paul Ziemba <paulz@labn.net>2023-08-18 20:11:17 +0200
commit6182675e7ebcfd9d809e5825207bc9d1e83ff4f8 (patch)
treef9d23e222fa3bee90dd04db221f7571b3735a17a /pbrd
parentMerge pull request #14181 from opensourcerouting/fix/bgpd_labeled_unicast_set... (diff)
downloadfrr-6182675e7ebcfd9d809e5825207bc9d1e83ff4f8.tar.xz
frr-6182675e7ebcfd9d809e5825207bc9d1e83ff4f8.zip
pbrd: address 230815 coverity: r.action.flags reordering
Signed-off-by: G. Paul Ziemba <paulz@labn.net>
Diffstat (limited to 'pbrd')
-rw-r--r--pbrd/pbr_zebra.c9
1 files changed, 2 insertions, 7 deletions
diff --git a/pbrd/pbr_zebra.c b/pbrd/pbr_zebra.c
index 35c771469..ee17a193f 100644
--- a/pbrd/pbr_zebra.c
+++ b/pbrd/pbr_zebra.c
@@ -568,15 +568,10 @@ static bool pbr_encode_pbr_map_sequence(struct stream *s,
/* actions */
- SET_FLAG(r.action.flags, PBR_ACTION_TABLE); /* always valid */
-
- /*
- * PBR should maintain its own set of action flags that we
- * can copy here instead of trying to infer from magic values.
- */
-
r.action.flags = pbrms->action_bm;
+ SET_FLAG(r.action.flags, PBR_ACTION_TABLE); /* always valid */
+
/*
* if the user does not use the command "set vrf name unchanged"
* then pbr_encode_pbr_map_sequence_vrf will not be called