Hi,

Signed-off-by: Daniel Walton <[email protected]>
Reviewed-by:   Donald Sharp <[email protected]>

There was a crash from not NULLing out peer->hostname but I cleaned
up a bunch of other suspect ones as well.

Indicates it'd be a good idea to add a helper to do the freeing, and have it take a double-pointer, so it can at least null out the caller's reference.

On Wed, 21 Oct 2015, Daniel Walton wrote:


            if (filter->dlist[i].name)
-             free (filter->dlist[i].name);

Boo, not going through our alloc layer.

regards,
--
Paul Jakma      [email protected]  @pjakma Key ID: 64A2FF6A
Fortune:
If you are honest because honesty is the best policy, your honesty is corrupt.

_______________________________________________
Quagga-dev mailing list
[email protected]
https://lists.quagga.net/mailman/listinfo/quagga-dev

Reply via email to