]> git.baikalelectronics.ru Git - kernel.git/commit
vxlan: fix unexpected failure of vxlan_changelink()
authorTaehee Yoo <ap420073@gmail.com>
Wed, 30 Oct 2019 08:15:12 +0000 (08:15 +0000)
committerDavid S. Miller <davem@davemloft.net>
Wed, 30 Oct 2019 18:52:47 +0000 (11:52 -0700)
commita3ecf1ad088d3d28050379d66fb2d33fa0aec8d9
tree2239aae2ad437c9a5160029dee85b3f6335e7fcf
parent3cb06665dc7a1a520b3d0504b4746f5a78f3cc7f
vxlan: fix unexpected failure of vxlan_changelink()

After commit e3f889100f51 ("vxlan: add adjacent link to limit depth
level"), vxlan_changelink() could fail because of
netdev_adjacent_change_prepare().
netdev_adjacent_change_prepare() returns -EEXIST when old lower device
and new lower device are same.
(old lower device is "dst->remote_dev" and new lower device is "lowerdev")
So, before calling it, lowerdev should be NULL if these devices are same.

Test command1:
    ip link add dummy0 type dummy
    ip link add vxlan0 type vxlan dev dummy0 dstport 4789 vni 1
    ip link set vxlan0 type vxlan ttl 5
    RTNETLINK answers: File exists

Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Fixes: e3f889100f51 ("vxlan: add adjacent link to limit depth level")
Signed-off-by: Taehee Yoo <ap420073@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/vxlan.c