diff options
author | G. Paul Ziemba <paulz@labn.net> | 2023-08-18 20:11:17 +0200 |
---|---|---|
committer | G. Paul Ziemba <paulz@labn.net> | 2023-08-18 20:11:17 +0200 |
commit | 6182675e7ebcfd9d809e5825207bc9d1e83ff4f8 (patch) | |
tree | f9d23e222fa3bee90dd04db221f7571b3735a17a /pbrd | |
parent | Merge pull request #14181 from opensourcerouting/fix/bgpd_labeled_unicast_set... (diff) | |
download | frr-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.c | 9 |
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 |