rtnetlink: Restore RTM_NEW/DELLINK notification behavior
authorMartin Willi <martin@strongswan.org>
Tue, 11 Apr 2023 07:43:19 +0000 (09:43 +0200)
committerJakub Kicinski <kuba@kernel.org>
Thu, 13 Apr 2023 03:47:58 +0000 (20:47 -0700)
commit59d3efd27c11c59b32291e5ebc307bed2edb65ee
treece16413675c1a6a99ecad9f1ddcdb61aed9e9371
parent136f36c74b0345d5d0087d4094894a006470bbd5
rtnetlink: Restore RTM_NEW/DELLINK notification behavior

The commits referenced below allows userspace to use the NLM_F_ECHO flag
for RTM_NEW/DELLINK operations to receive unicast notifications for the
affected link. Prior to these changes, applications may have relied on
multicast notifications to learn the same information without specifying
the NLM_F_ECHO flag.

For such applications, the mentioned commits changed the behavior for
requests not using NLM_F_ECHO. Multicast notifications are still received,
but now use the portid of the requester and the sequence number of the
request instead of zero values used previously. For the application, this
message may be unexpected and likely handled as a response to the
NLM_F_ACKed request, especially if it uses the same socket to handle
requests and notifications.

To fix existing applications relying on the old notification behavior,
set the portid and sequence number in the notification only if the
request included the NLM_F_ECHO flag. This restores the old behavior
for applications not using it, but allows unicasted notifications for
others.

Fixes: f3a63cce1b4f ("rtnetlink: Honour NLM_F_ECHO flag in rtnl_delete_link")
Fixes: d88e136cab37 ("rtnetlink: Honour NLM_F_ECHO flag in rtnl_newlink_create")
Signed-off-by: Martin Willi <martin@strongswan.org>
Acked-by: Guillaume Nault <gnault@redhat.com>
Acked-by: Hangbin Liu <liuhangbin@gmail.com>
Link: https://lore.kernel.org/r/20230411074319.24133-1-martin@strongswan.org
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
include/linux/rtnetlink.h
net/core/dev.c
net/core/rtnetlink.c