diff options
| author |   Neale Ranns <neale@graphiant.com> | 2021-04-02 07:34:39 +0000 |
|---|---|---|
| committer |   Matthew Smith <mgsmith@netgate.com> | 2021-04-05 00:56:47 +0000 |
| commit | e2b6736e1f5c9a841e37ea1e0c3db4c1989a10ba (patch) | |
| tree | 26740cb0392570b67a02ff3e86810b60cceabe2a /src | |
| parent | 014dba38cb9579808a2134fd10a071e4f8c4e213 (diff) | |
ip6-nd: Solicitation reply only if target is our link-local
Type: fix
The fib source IP6_ND is used for all link-local entries, hence
solicitation responses were sent for a peer's address. Constrain the
source check to also in clude the LOCAL flag, which indicates that the
link-local address is ours.
Signed-off-by: Neale Ranns <neale@graphiant.com>
Change-Id: Iba7e66049e4d89ee3f36d77aeb09310b978d70de
Diffstat (limited to 'src')
| -rw-r--r-- | src/vnet/ip6-nd/ip6_nd.c | 13 |
1 files changed, 9 insertions, 4 deletions
diff --git a/src/vnet/ip6-nd/ip6_nd.c b/src/vnet/ip6-nd/ip6_nd.c index 917abddf7bb..311cbf743f7 100644 --- a/src/vnet/ip6-nd/ip6_nd.c +++ b/src/vnet/ip6-nd/ip6_nd.c @@ -215,10 +215,15 @@ icmp6_neighbor_solicitation_or_advertisement (vlib_main_t * vm, /* It's an address that belongs to one of our interfaces * that's good. */ } - else - if (fib_entry_is_sourced - (fei, FIB_SOURCE_IP6_ND_PROXY) || - fib_entry_is_sourced (fei, FIB_SOURCE_IP6_ND)) + else if (FIB_ENTRY_FLAG_LOCAL & + fib_entry_get_flags_for_source ( + fei, FIB_SOURCE_IP6_ND)) + { + /* It's one of our link local addresses + * that's good. */ + } + else if (fib_entry_is_sourced (fei, + FIB_SOURCE_IP6_ND_PROXY)) { /* The address was added by IPv6 Proxy ND config. * We should only respond to these if the NS arrived on |