diff options
| author | Eric Dumazet <edumazet@google.com> | 2023-03-17 15:55:31 +0000 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2023-03-18 12:23:33 +0000 |
| commit | 68ac9a8b6e65c7cbbe96541353dab1b3f8de2043 (patch) | |
| tree | 3c16170c8c27636a63b6f338c4e6be616d65eda7 /net/packet | |
| parent | 94c540fbfc80ef95ec398fc77da1920de2946edb (diff) | |
af_packet: preserve const qualifier in pkt_sk()
We can change pkt_sk() to propagate const qualifier of its argument,
thanks to container_of_const()
This should avoid some potential errors caused by accidental
(const -> not_const) promotion.
Signed-off-by: Eric Dumazet <edumazet@google.com>
Cc: Willem de Bruijn <willemb@google.com>
Reviewed-by: Simon Horman <simon.horman@corigine.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/packet')
| -rw-r--r-- | net/packet/internal.h | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/net/packet/internal.h b/net/packet/internal.h index 680703dbce5e..e793e99646f1 100644 --- a/net/packet/internal.h +++ b/net/packet/internal.h @@ -133,10 +133,7 @@ struct packet_sock { atomic_t tp_drops ____cacheline_aligned_in_smp; }; -static inline struct packet_sock *pkt_sk(struct sock *sk) -{ - return (struct packet_sock *)sk; -} +#define pkt_sk(ptr) container_of_const(ptr, struct packet_sock, sk) enum packet_sock_flags { PACKET_SOCK_ORIGDEV, |