diff options
| author | Jakub Kicinski <[email protected]> | 2025-04-15 15:15:52 +0000 |
|---|---|---|
| committer | Jakub Kicinski <[email protected]> | 2025-04-17 01:28:11 +0000 |
| commit | 4798cfa2097f0833d54d8f5ce20ef14631917839 (patch) | |
| tree | c731ce67262b4d0d26934feac33f9d7ef583f58a /net/core/dev.c | |
| parent | ptp: ocp: fix start time alignment in ptp_ocp_signal_set (diff) | |
| download | kernel-4798cfa2097f0833d54d8f5ce20ef14631917839.tar.gz kernel-4798cfa2097f0833d54d8f5ce20ef14631917839.zip | |
net: don't try to ops lock uninitialized devs
We need to be careful when operating on dev while in rtnl_create_link().
Some devices (vxlan) initialize netdev_ops in ->newlink, so later on.
Avoid using netdev_lock_ops(), the device isn't registered so we
cannot legally call its ops or generate any notifications for it.
netdev_ops_assert_locked_or_invisible() is safe to use, it checks
registration status first.
Reported-by: [email protected]
Fixes: 04efcee6ef8d ("net: hold instance lock during NETDEV_CHANGE")
Acked-by: Stanislav Fomichev <[email protected]>
Reviewed-by: Kuniyuki Iwashima <[email protected]>
Link: https://patch.msgid.link/[email protected]
Signed-off-by: Jakub Kicinski <[email protected]>
Diffstat (limited to 'net/core/dev.c')
| -rw-r--r-- | net/core/dev.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/net/core/dev.c b/net/core/dev.c index 5fcbc66d865e..1be7cb73a602 100644 --- a/net/core/dev.c +++ b/net/core/dev.c @@ -1520,6 +1520,8 @@ EXPORT_SYMBOL(netdev_features_change); void netif_state_change(struct net_device *dev) { + netdev_ops_assert_locked_or_invisible(dev); + if (dev->flags & IFF_UP) { struct netdev_notifier_change_info change_info = { .info.dev = dev, |