diff options
| author |   Benoît Ganne <bganne@cisco.com> | 2019-07-18 17:34:28 +0200 |
|---|---|---|
| committer |   Andrew Yourtchenko <ayourtch@gmail.com> | 2019-09-30 15:34:52 +0000 |
| commit | 59ffe5ea1cf585fa9ace7e527083aee4abe7aab7 (patch) | |
| tree | fb8c2601d24729c3428a273561c09ab50aa1d576 /src/vnet/adj/adj_mcast.c | |
| parent | 804a95cf8113ee2b226363eab71e98dbee0b6820 (diff) | |
fib: do not dump no-longer valid adjacencies
In some cases, we can refer to no-longer adjacencies (eg. in traces). Do
not dump them in this case as they are probably incorrect (memory can be
reused).
Type: fix
Change-Id: Ib653ba066bb6595ec6ec37d313a3124bce0eeed3
Signed-off-by: Benoît Ganne <bganne@cisco.com>
(cherry picked from commit 138c37af5291e2045075052a4db036be03be4a85)
Diffstat (limited to 'src/vnet/adj/adj_mcast.c')
| -rw-r--r-- | src/vnet/adj/adj_mcast.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/src/vnet/adj/adj_mcast.c b/src/vnet/adj/adj_mcast.c index 4454afec630..c94d28e8713 100644 --- a/src/vnet/adj/adj_mcast.c +++ b/src/vnet/adj/adj_mcast.c @@ -329,7 +329,12 @@ format_adj_mcast (u8* s, va_list *ap) { index_t index = va_arg(*ap, index_t); CLIB_UNUSED(u32 indent) = va_arg(*ap, u32); - ip_adjacency_t * adj = adj_get(index); + ip_adjacency_t * adj; + + if (!adj_is_valid(index)) + return format(s, "<invalid adjacency>"); + + adj = adj_get(index); s = format(s, "%U-mcast: ", format_fib_protocol, adj->ia_nh_proto); |