diff options
| author |   Neale Ranns <neale.ranns@cisco.com> | 2017-01-05 01:01:47 -0800 |
|---|---|---|
| committer |   John Lo <loj@cisco.com> | 2017-01-05 15:10:05 +0000 |
| commit | c0790cfef0bd1c56f4c75dc4f959584148386258 (patch) | |
| tree | 08abc77cb599b15f1ea9d80bdf44bf14855a1360 /src/vnet/fib/fib_entry.c | |
| parent | ce1aae4e88d8ed2c73e8213115cf9f3a4e37370f (diff) | |
FIB memory leaks (VPP-578)
1) vec_free the fe_srcs of a fib_entry_t when the fib_entry_t is itself reed
2) in the load-balance fixup if a drop path is required add this to a new vector of next-hops 'fixed_nhs'. This vector is managed by the load-balance function. The caller continues to manage its own set. The function is now const implying that the caller is safe to assume the next-hops do not change.
Change-Id: I0f29203ee16b9a270f40edf237488fa99ba65320
Signed-off-by: Neale Ranns <nranns@cisco.com>
Signed-off-by: Neale Ranns <neale.ranns@cisco.com>
Diffstat (limited to 'src/vnet/fib/fib_entry.c')
| -rw-r--r-- | src/vnet/fib/fib_entry.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/vnet/fib/fib_entry.c b/src/vnet/fib/fib_entry.c index 24b506379ac..3aa3632c596 100644 --- a/src/vnet/fib/fib_entry.c +++ b/src/vnet/fib/fib_entry.c @@ -230,6 +230,7 @@ fib_entry_last_lock_gone (fib_node_t *node) ASSERT(0 == vec_len(fib_entry->fe_delegates)); vec_free(fib_entry->fe_delegates); + vec_free(fib_entry->fe_srcs); pool_put(fib_entry_pool, fib_entry); } |