]> git.puffer.fish Git - mirror/frr.git/commit
isisd: fix crash when entering "no ip[v6] router isis" twice
authorRenato Westphal <renato@opensourcerouting.org>
Wed, 20 Feb 2019 18:36:50 +0000 (15:36 -0300)
committerRenato Westphal <renato@opensourcerouting.org>
Thu, 21 Feb 2019 02:04:12 +0000 (23:04 -0300)
commit472c3dae39776873a018efa63cdfae634c072193
tree90b5c9fb6dc85be6697058211b4816dd666aa928
parent220c5830e7093f8c903a689fc61ef64b372778fe
isisd: fix crash when entering "no ip[v6] router isis" twice

isisd CLI has some housekeeping code that removes the
"frr-isisd:isis" container from the interface configuration when
IS-IS is disabled for both IPv4 and IPv6 in the corresponding
interface.

The problem is that the code was checking the values of the
"ipv4-routing" and "ipv6-routing" leafs without checking if the
parent "frr-isisd:isis" container was present. So, entering "no
ip[v6] router isis" twice would cause isisd to crash since the
"frr-isisd:isis" container wouldn't be present the second time the
command is processed. Fix this.

isisd aborted: vtysh -c "configure terminal" -c "interface eth99" -c "no ip router isis WORD"
isisd aborted: vtysh -c "configure terminal" -c "interface eth99" -c "no ipv6 router isis"

Signed-off-by: Renato Westphal <renato@opensourcerouting.org>
isisd/isis_cli.c