diff options
| author | anlan_cs <vic.lan@pica8.com> | 2023-06-07 17:36:40 +0800 |
|---|---|---|
| committer | anlan_cs <vic.lan@pica8.com> | 2023-06-07 22:15:29 +0800 |
| commit | c33473f8222f5e5c0816f4d10328ae5bd7184bb2 (patch) | |
| tree | ed8b1820c4d4e20f14f77b5894c04c2fc5e8bbb5 /zebra/interface.c | |
| parent | fecc953b3c0535fe85191ed12da3bd3ba0b22054 (diff) | |
pimd: Fix wrong protocol for SSM
Consider LHR in `SSM` case, and its `oif` goes from up to down.
As the `oif` changes down, the `channel_oil` is still wrongly reserved
with `PIM_OIF_FLAG_PROTO_STAR` protocol.
The mechanism of adding `STAR` protocol for <S,G> entry without `oif`
is introduced by commit `2164ed5`, which should be only used for ASM.
PR #13699 wants to clean this `STAR` protocol after the `oif` becomes up
again. That case is for SSM, as the `oif` changes down, the `channel_oil`
should be removed, so the fix of that PR is wrong.
In `pim_ifchannel_delete()` we should check the `ch->parent` before that
mechanism of adding `STAR` protocol. If `ch->parent` is NULL, we should
skip that mechanism with SSM.
Fixes #13696
Signed-off-by: anlan_cs <vic.lan@pica8.com>
Diffstat (limited to 'zebra/interface.c')
0 files changed, 0 insertions, 0 deletions
