Message ID | 20191122215052.155595-1-zenczykowski@gmail.com |
---|---|
State | Accepted |
Delegated to: | David Miller |
Headers | show |
Series | net: inet_is_local_reserved_port() should return bool not int | expand |
On Fri, 22 Nov 2019 13:50:52 -0800, Maciej Żenczykowski wrote: > From: Maciej Żenczykowski <maze@google.com> > > Cc: Eric Dumazet <edumazet@google.com> > Signed-off-by: Maciej Żenczykowski <maze@google.com> Applied. It would had been nice to see net-next in the subject and a commit message, you know :/
> On Fri, 22 Nov 2019 13:50:52 -0800, Maciej Żenczykowski wrote: > > From: Maciej Żenczykowski <maze@google.com> > > > > Cc: Eric Dumazet <edumazet@google.com> > > Signed-off-by: Maciej Żenczykowski <maze@google.com> > > Applied. > > It would had been nice to see net-next in the subject and a commit > message, you know :/ Sorry, about that. I'm never entirely certain what's going to net-next, what isn't. Should I just default to net-next? As for the commit message, there simply didn't appear to be anything to write, the commit title was pretty self explanatory I thought. - Maciej
diff --git a/include/net/ip.h b/include/net/ip.h index a2c61c36dc4a..cebf3e10def1 100644 --- a/include/net/ip.h +++ b/include/net/ip.h @@ -339,10 +339,10 @@ static inline u64 snmp_fold_field64(void __percpu *mib, int offt, size_t syncp_o void inet_get_local_port_range(struct net *net, int *low, int *high); #ifdef CONFIG_SYSCTL -static inline int inet_is_local_reserved_port(struct net *net, int port) +static inline bool inet_is_local_reserved_port(struct net *net, int port) { if (!net->ipv4.sysctl_local_reserved_ports) - return 0; + return false; return test_bit(port, net->ipv4.sysctl_local_reserved_ports); } @@ -357,9 +357,9 @@ static inline int inet_prot_sock(struct net *net) } #else -static inline int inet_is_local_reserved_port(struct net *net, int port) +static inline bool inet_is_local_reserved_port(struct net *net, int port) { - return 0; + return false; } static inline int inet_prot_sock(struct net *net)