diff options
author | paul <paul> | 2003-05-14 18:29:13 +0000 |
---|---|---|
committer | paul <paul> | 2003-05-14 18:29:13 +0000 |
commit | 143a953120b07e7f4347a21089020b15ac831ebe (patch) | |
tree | f08e1df6f285281322dc24f1cf510226b681fd21 /ospf6d/ospf6d.c | |
parent | 786ad3990f79ba1c80c4760208ccb4495eafcbb3 (diff) | |
download | quagga-143a953120b07e7f4347a21089020b15ac831ebe.tar.bz2 quagga-143a953120b07e7f4347a21089020b15ac831ebe.tar.xz |
From: Gilad Arnold <gilad arnold at terayon com>
Subject: [zebra 19080] suspected memory leakage upon static route deletion
upon deletion of a static route entry, at the end of
zebra/zebra_rib.c/static_delete_ipv4(), there's a call to XFREE to
deallocate the 'struct static_ipv4'. However, in the case of a static with
ifname as nexthop (type==STATIC_IPV4_IFNAME), this struct holds a pointer to
a dynamically allocated ifname string buffer (si->gate.ifname, see its
allocation within static_add_ipv4() using XSTRDUP). IMO, the attached patch
is required.
Notes: the same applies to IPv6 statics as well (fix included in patch);
and, I admit I haven't yet tested that, but it seems simple enough to be
working... ;->
Diffstat (limited to 'ospf6d/ospf6d.c')
0 files changed, 0 insertions, 0 deletions