diff options
| author | Mat Martineau <mathew.j.martineau@linux.intel.com> | 2020-01-09 16:59:14 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2020-01-10 03:41:40 +0100 |
| commit | e9cdced78dc20c1592c1fb98ed064943007a46c5 (patch) | |
| tree | 733872a3d2f4cc0aac217f383b1d69e91879ba61 /net/ax25 | |
| parent | flow_dissector: fix document for skb_flow_get_icmp_tci (diff) | |
| download | linux-e9cdced78dc20c1592c1fb98ed064943007a46c5.tar.xz linux-e9cdced78dc20c1592c1fb98ed064943007a46c5.zip | |
net: Make sock protocol value checks more specific
SK_PROTOCOL_MAX is only used in two places, for DECNet and AX.25. The
limits have more to do with the those protocol definitions than they do
with the data type of sk_protocol, so remove SK_PROTOCOL_MAX and use
U8_MAX directly.
Reviewed-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: Mat Martineau <mathew.j.martineau@linux.intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ax25')
| -rw-r--r-- | net/ax25/af_ax25.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/ax25/af_ax25.c b/net/ax25/af_ax25.c index 324306d6fde0..ff57ea89c27e 100644 --- a/net/ax25/af_ax25.c +++ b/net/ax25/af_ax25.c @@ -808,7 +808,7 @@ static int ax25_create(struct net *net, struct socket *sock, int protocol, struct sock *sk; ax25_cb *ax25; - if (protocol < 0 || protocol > SK_PROTOCOL_MAX) + if (protocol < 0 || protocol > U8_MAX) return -EINVAL; if (!net_eq(net, &init_net)) |