]> git.baikalelectronics.ru Git - kernel.git/commit
ipv4: fib: Fix NULL pointer deref during fib_sync_down_dev()
authorIdo Schimmel <idosch@mellanox.com>
Fri, 28 Jul 2017 20:27:44 +0000 (23:27 +0300)
committerDavid S. Miller <davem@davemloft.net>
Tue, 1 Aug 2017 00:51:11 +0000 (17:51 -0700)
commit9c04b3a9378a3e68b1f3879412040edcfaac5ec8
tree2d4154df8757c876a60c261357cf8ba1a85a9aa6
parent36f5fceb13f226cbb16871a4eafcda0957e10390
ipv4: fib: Fix NULL pointer deref during fib_sync_down_dev()

Michał reported a NULL pointer deref during fib_sync_down_dev() when
unregistering a netdevice. The problem is that we don't check for
'in_dev' being NULL, which can happen in very specific cases.

Usually routes are flushed upon NETDEV_DOWN sent in either the netdev or
the inetaddr notification chains. However, if an interface isn't
configured with any IP address, then it's possible for host routes to be
flushed following NETDEV_UNREGISTER, after NULLing dev->ip_ptr in
inetdev_destroy().

To reproduce:
$ ip link add type dummy
$ ip route add local 1.1.1.0/24 dev dummy0
$ ip link del dev dummy0

Fix this by checking for the presence of 'in_dev' before referencing it.

Fixes: 98329e1fb2a6 ("ipv4: fib: Notify about nexthop status changes")
Signed-off-by: Ido Schimmel <idosch@mellanox.com>
Reported-by: Michał Mirosław <mirq-linux@rere.qmqm.pl>
Tested-by: Michał Mirosław <mirq-linux@rere.qmqm.pl>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/ipv4/fib_semantics.c