ipcpd: Fix bad memory handling in LFA policy

The Loop-Free Alternates policy had bad memory management in two
places. In the calculation of the LFAs a table was freed in the first
iteration of a loop, whereas it was still needed in the other
iterations. It is now freed outside of the loop. In the alternate PFF
the address structs were not freed upon shutdown, this has been added
as well. It also fixes some bad initialization in the LFA calculation
function.

Signed-off-by: Sander Vrijders <sander.vrijders@ugent.be>
Signed-off-by: Dimitri Staessens <dimitri.staessens@ugent.be>

1 files changed, 1 insertions, 0 deletions

diff --git a/src/ipcpd/normal/pol/alternate_pff.c b/src/ipcpd/normal/pol/alternate_pff.c
index cb438c5f..3cb99d9c 100644
--- a/src/ipcpd/normal/pol/alternate_pff.c
+++ b/src/ipcpd/normal/pol/alternate_pff.c
@@ -234,6 +234,7 @@ void alternate_pff_destroy(struct pff_i * pff_i)
 
         htable_destroy(pff_i->table);
         del_nhops_down(pff_i);
+        del_addrs(pff_i);
         pthread_rwlock_destroy(&pff_i->lock);
         free(pff_i);
 }