diff options
| author |   Jakub Kicinski <kuba@kernel.org> | 2021-11-19 06:21:53 -0800 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2021-11-20 12:25:57 +0000 |
| commit | d07b26f5bbea9ade34dfd6abea7b3ca056c03cd1 (patch) | |
| tree | 4a907941337e5445e0834277647ba4c04269523e /include/linux/netdevice.h | |
| parent | net: unexport dev_addr_init() & dev_addr_flush() (diff) | |
| download | linux-dev-d07b26f5bbea9ade34dfd6abea7b3ca056c03cd1.tar.xz linux-dev-d07b26f5bbea9ade34dfd6abea7b3ca056c03cd1.zip | |
dev_addr: add a modification check
netdev->dev_addr should only be modified via helpers,
but someone may be casting off the const. Add a runtime
check to catch abuses.
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'include/linux/netdevice.h')
| -rw-r--r-- | include/linux/netdevice.h | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/include/linux/netdevice.h b/include/linux/netdevice.h index 2462195784a9..cb7f2661d187 100644 --- a/include/linux/netdevice.h +++ b/include/linux/netdevice.h @@ -1942,6 +1942,8 @@ enum netdev_ml_priv_type { * @unlink_list: As netif_addr_lock() can be called recursively, * keep a list of interfaces to be deleted. * + * @dev_addr_shadow: Copy of @dev_addr to catch direct writes. + * * FIXME: cleanup struct net_device such that network protocol info * moves out. */ @@ -2268,6 +2270,8 @@ struct net_device { /* protected by rtnl_lock */ struct bpf_xdp_entity xdp_state[__MAX_XDP_MODE]; + + u8 dev_addr_shadow[MAX_ADDR_LEN]; }; #define to_net_dev(d) container_of(d, struct net_device, dev) @@ -4288,6 +4292,7 @@ int dev_addr_del(struct net_device *dev, const unsigned char *addr, unsigned char addr_type); void dev_addr_flush(struct net_device *dev); int dev_addr_init(struct net_device *dev); +void dev_addr_check(struct net_device *dev); /* Functions used for unicast addresses handling */ int dev_uc_add(struct net_device *dev, const unsigned char *addr); |