Skip to content

Commit

Permalink
l2tp: cleanup comparisons to NULL
Browse files Browse the repository at this point in the history
checkpatch warns about comparisons to NULL, e.g.

        CHECK: Comparison to NULL could be written "!rt"
        torvalds#474: FILE: net/l2tp/l2tp_ip.c:474:
        +       if (rt == NULL) {

These sort of comparisons are generally clearer and more readable
the way checkpatch suggests, so update l2tp accordingly.

Signed-off-by: Tom Parkin <[email protected]>
Signed-off-by: David S. Miller <[email protected]>
  • Loading branch information
tomparkin authored and davem330 committed Jul 23, 2020
1 parent 49b0aa1 commit 0febc7b
Show file tree
Hide file tree
Showing 6 changed files with 47 additions and 48 deletions.
20 changes: 10 additions & 10 deletions net/l2tp/l2tp_core.c
Original file line number Diff line number Diff line change
Expand Up @@ -412,7 +412,7 @@ static void l2tp_recv_dequeue_skb(struct l2tp_session *session, struct sk_buff *
}

/* call private receive handler */
if (session->recv_skb != NULL)
if (session->recv_skb)
(*session->recv_skb)(session, skb, L2TP_SKB_CB(skb)->length);
else
kfree_skb(skb);
Expand Down Expand Up @@ -911,7 +911,7 @@ int l2tp_udp_encap_recv(struct sock *sk, struct sk_buff *skb)
struct l2tp_tunnel *tunnel;

tunnel = rcu_dereference_sk_user_data(sk);
if (tunnel == NULL)
if (!tunnel)
goto pass_up;

l2tp_dbg(tunnel, L2TP_MSG_DATA, "%s: received %d bytes\n",
Expand Down Expand Up @@ -1150,7 +1150,7 @@ static void l2tp_tunnel_destruct(struct sock *sk)
{
struct l2tp_tunnel *tunnel = l2tp_tunnel(sk);

if (tunnel == NULL)
if (!tunnel)
goto end;

l2tp_info(tunnel, L2TP_MSG_CONTROL, "%s: closing...\n", tunnel->name);
Expand Down Expand Up @@ -1189,7 +1189,7 @@ static void l2tp_tunnel_closeall(struct l2tp_tunnel *tunnel)
struct hlist_node *tmp;
struct l2tp_session *session;

BUG_ON(tunnel == NULL);
BUG_ON(!tunnel);

l2tp_info(tunnel, L2TP_MSG_CONTROL, "%s: closing all sessions...\n",
tunnel->name);
Expand All @@ -1214,7 +1214,7 @@ static void l2tp_tunnel_closeall(struct l2tp_tunnel *tunnel)
__l2tp_session_unhash(session);
l2tp_session_queue_purge(session);

if (session->session_close != NULL)
if (session->session_close)
(*session->session_close)(session);

l2tp_session_dec_refcount(session);
Expand Down Expand Up @@ -1407,11 +1407,11 @@ int l2tp_tunnel_create(struct net *net, int fd, int version, u32 tunnel_id, u32
int err;
enum l2tp_encap_type encap = L2TP_ENCAPTYPE_UDP;

if (cfg != NULL)
if (cfg)
encap = cfg->encap;

tunnel = kzalloc(sizeof(struct l2tp_tunnel), GFP_KERNEL);
if (tunnel == NULL) {
if (!tunnel) {
err = -ENOMEM;
goto err;
}
Expand All @@ -1426,7 +1426,7 @@ int l2tp_tunnel_create(struct net *net, int fd, int version, u32 tunnel_id, u32
rwlock_init(&tunnel->hlist_lock);
tunnel->acpt_newsess = true;

if (cfg != NULL)
if (cfg)
tunnel->debug = cfg->debug;

tunnel->encap = encap;
Expand Down Expand Up @@ -1615,7 +1615,7 @@ int l2tp_session_delete(struct l2tp_session *session)

__l2tp_session_unhash(session);
l2tp_session_queue_purge(session);
if (session->session_close != NULL)
if (session->session_close)
(*session->session_close)(session);

l2tp_session_dec_refcount(session);
Expand Down Expand Up @@ -1648,7 +1648,7 @@ struct l2tp_session *l2tp_session_create(int priv_size, struct l2tp_tunnel *tunn
struct l2tp_session *session;

session = kzalloc(sizeof(struct l2tp_session) + priv_size, GFP_KERNEL);
if (session != NULL) {
if (session) {
session->magic = L2TP_SESSION_MAGIC;
session->tunnel = tunnel;

Expand Down
12 changes: 6 additions & 6 deletions net/l2tp/l2tp_debugfs.c
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ static void l2tp_dfs_next_session(struct l2tp_dfs_seq_data *pd)
pd->session = l2tp_session_get_nth(pd->tunnel, pd->session_idx);
pd->session_idx++;

if (pd->session == NULL) {
if (!pd->session) {
pd->session_idx = 0;
l2tp_dfs_next_tunnel(pd);
}
Expand All @@ -72,16 +72,16 @@ static void *l2tp_dfs_seq_start(struct seq_file *m, loff_t *offs)
if (!pos)
goto out;

BUG_ON(m->private == NULL);
BUG_ON(!m->private);
pd = m->private;

if (pd->tunnel == NULL)
if (!pd->tunnel)
l2tp_dfs_next_tunnel(pd);
else
l2tp_dfs_next_session(pd);

/* NULL tunnel and session indicates end of list */
if ((pd->tunnel == NULL) && (pd->session == NULL))
if (!pd->tunnel && !pd->session)
pd = NULL;

out:
Expand Down Expand Up @@ -221,7 +221,7 @@ static void l2tp_dfs_seq_session_show(struct seq_file *m, void *v)
atomic_long_read(&session->stats.rx_bytes),
atomic_long_read(&session->stats.rx_errors));

if (session->show != NULL)
if (session->show)
session->show(m, session);
}

Expand Down Expand Up @@ -268,7 +268,7 @@ static int l2tp_dfs_seq_open(struct inode *inode, struct file *file)
int rc = -ENOMEM;

pd = kzalloc(sizeof(*pd), GFP_KERNEL);
if (pd == NULL)
if (!pd)
goto out;

/* Derive the network namespace from the pid opening the
Expand Down
2 changes: 1 addition & 1 deletion net/l2tp/l2tp_ip.c
Original file line number Diff line number Diff line change
Expand Up @@ -471,7 +471,7 @@ static int l2tp_ip_sendmsg(struct sock *sk, struct msghdr *msg, size_t len)
rt = (struct rtable *)__sk_dst_check(sk, 0);

rcu_read_lock();
if (rt == NULL) {
if (!rt) {
const struct ip_options_rcu *inet_opt;

inet_opt = rcu_dereference(inet->inet_opt);
Expand Down
2 changes: 1 addition & 1 deletion net/l2tp/l2tp_ip6.c
Original file line number Diff line number Diff line change
Expand Up @@ -486,7 +486,7 @@ static int l2tp_ip6_push_pending_frames(struct sock *sk)
int err = 0;

skb = skb_peek(&sk->sk_write_queue);
if (skb == NULL)
if (!skb)
goto out;

transhdr = (__be32 *)skb_transport_header(skb);
Expand Down
23 changes: 11 additions & 12 deletions net/l2tp/l2tp_netlink.c
Original file line number Diff line number Diff line change
Expand Up @@ -333,7 +333,7 @@ static int l2tp_nl_tunnel_send(struct sk_buff *skb, u32 portid, u32 seq, int fla
goto nla_put_failure;

nest = nla_nest_start_noflag(skb, L2TP_ATTR_STATS);
if (nest == NULL)
if (!nest)
goto nla_put_failure;

if (nla_put_u64_64bit(skb, L2TP_ATTR_TX_PACKETS,
Expand Down Expand Up @@ -475,7 +475,7 @@ static int l2tp_nl_cmd_tunnel_dump(struct sk_buff *skb, struct netlink_callback

for (;;) {
tunnel = l2tp_tunnel_get_nth(net, ti);
if (tunnel == NULL)
if (!tunnel)
goto out;

if (l2tp_nl_tunnel_send(skb, NETLINK_CB(cb->skb).portid,
Expand Down Expand Up @@ -598,14 +598,13 @@ static int l2tp_nl_cmd_session_create(struct sk_buff *skb, struct genl_info *inf
cfg.reorder_timeout = nla_get_msecs(info->attrs[L2TP_ATTR_RECV_TIMEOUT]);

#ifdef CONFIG_MODULES
if (l2tp_nl_cmd_ops[cfg.pw_type] == NULL) {
if (!l2tp_nl_cmd_ops[cfg.pw_type]) {
genl_unlock();
request_module("net-l2tp-type-%u", cfg.pw_type);
genl_lock();
}
#endif
if ((l2tp_nl_cmd_ops[cfg.pw_type] == NULL) ||
(l2tp_nl_cmd_ops[cfg.pw_type]->session_create == NULL)) {
if (!l2tp_nl_cmd_ops[cfg.pw_type] || !l2tp_nl_cmd_ops[cfg.pw_type]->session_create) {
ret = -EPROTONOSUPPORT;
goto out_tunnel;
}
Expand Down Expand Up @@ -637,7 +636,7 @@ static int l2tp_nl_cmd_session_delete(struct sk_buff *skb, struct genl_info *inf
u16 pw_type;

session = l2tp_nl_session_get(info);
if (session == NULL) {
if (!session) {
ret = -ENODEV;
goto out;
}
Expand All @@ -662,7 +661,7 @@ static int l2tp_nl_cmd_session_modify(struct sk_buff *skb, struct genl_info *inf
struct l2tp_session *session;

session = l2tp_nl_session_get(info);
if (session == NULL) {
if (!session) {
ret = -ENODEV;
goto out;
}
Expand Down Expand Up @@ -729,7 +728,7 @@ static int l2tp_nl_session_send(struct sk_buff *skb, u32 portid, u32 seq, int fl
goto nla_put_failure;

nest = nla_nest_start_noflag(skb, L2TP_ATTR_STATS);
if (nest == NULL)
if (!nest)
goto nla_put_failure;

if (nla_put_u64_64bit(skb, L2TP_ATTR_TX_PACKETS,
Expand Down Expand Up @@ -774,7 +773,7 @@ static int l2tp_nl_cmd_session_get(struct sk_buff *skb, struct genl_info *info)
int ret;

session = l2tp_nl_session_get(info);
if (session == NULL) {
if (!session) {
ret = -ENODEV;
goto err;
}
Expand Down Expand Up @@ -813,14 +812,14 @@ static int l2tp_nl_cmd_session_dump(struct sk_buff *skb, struct netlink_callback
int si = cb->args[1];

for (;;) {
if (tunnel == NULL) {
if (!tunnel) {
tunnel = l2tp_tunnel_get_nth(net, ti);
if (tunnel == NULL)
if (!tunnel)
goto out;
}

session = l2tp_session_get_nth(tunnel, si);
if (session == NULL) {
if (!session) {
ti++;
l2tp_tunnel_dec_refcount(tunnel);
tunnel = NULL;
Expand Down
36 changes: 18 additions & 18 deletions net/l2tp/l2tp_ppp.c
Original file line number Diff line number Diff line change
Expand Up @@ -154,12 +154,12 @@ static inline struct l2tp_session *pppol2tp_sock_to_session(struct sock *sk)
{
struct l2tp_session *session;

if (sk == NULL)
if (!sk)
return NULL;

sock_hold(sk);
session = (struct l2tp_session *)(sk->sk_user_data);
if (session == NULL) {
if (!session) {
sock_put(sk);
goto out;
}
Expand Down Expand Up @@ -217,7 +217,7 @@ static void pppol2tp_recv(struct l2tp_session *session, struct sk_buff *skb, int
*/
rcu_read_lock();
sk = rcu_dereference(ps->sk);
if (sk == NULL)
if (!sk)
goto no_sock;

/* If the first two bytes are 0xFF03, consider that it is the PPP's
Expand Down Expand Up @@ -285,7 +285,7 @@ static int pppol2tp_sendmsg(struct socket *sock, struct msghdr *m,
/* Get session and tunnel contexts */
error = -EBADF;
session = pppol2tp_sock_to_session(sk);
if (session == NULL)
if (!session)
goto error;

tunnel = session->tunnel;
Expand Down Expand Up @@ -360,7 +360,7 @@ static int pppol2tp_xmit(struct ppp_channel *chan, struct sk_buff *skb)

/* Get session and tunnel contexts from the socket */
session = pppol2tp_sock_to_session(sk);
if (session == NULL)
if (!session)
goto abort;

tunnel = session->tunnel;
Expand Down Expand Up @@ -703,7 +703,7 @@ static int pppol2tp_connect(struct socket *sock, struct sockaddr *uservaddr,
* tunnel id.
*/
if (!info.session_id && !info.peer_session_id) {
if (tunnel == NULL) {
if (!tunnel) {
struct l2tp_tunnel_cfg tcfg = {
.encap = L2TP_ENCAPTYPE_UDP,
.debug = 0,
Expand Down Expand Up @@ -738,11 +738,11 @@ static int pppol2tp_connect(struct socket *sock, struct sockaddr *uservaddr,
} else {
/* Error if we can't find the tunnel */
error = -ENOENT;
if (tunnel == NULL)
if (!tunnel)
goto end;

/* Error if socket is not prepped */
if (tunnel->sock == NULL)
if (!tunnel->sock)
goto end;
}

Expand Down Expand Up @@ -911,14 +911,14 @@ static int pppol2tp_getname(struct socket *sock, struct sockaddr *uaddr,
struct pppol2tp_session *pls;

error = -ENOTCONN;
if (sk == NULL)
if (!sk)
goto end;
if (!(sk->sk_state & PPPOX_CONNECTED))
goto end;

error = -EBADF;
session = pppol2tp_sock_to_session(sk);
if (session == NULL)
if (!session)
goto end;

pls = l2tp_session_priv(session);
Expand Down Expand Up @@ -1263,13 +1263,13 @@ static int pppol2tp_setsockopt(struct socket *sock, int level, int optname,
return -EFAULT;

err = -ENOTCONN;
if (sk->sk_user_data == NULL)
if (!sk->sk_user_data)
goto end;

/* Get session context from the socket */
err = -EBADF;
session = pppol2tp_sock_to_session(sk);
if (session == NULL)
if (!session)
goto end;

/* Special case: if session_id == 0x0000, treat as operation on tunnel
Expand Down Expand Up @@ -1382,13 +1382,13 @@ static int pppol2tp_getsockopt(struct socket *sock, int level, int optname,
return -EINVAL;

err = -ENOTCONN;
if (sk->sk_user_data == NULL)
if (!sk->sk_user_data)
goto end;

/* Get the session context */
err = -EBADF;
session = pppol2tp_sock_to_session(sk);
if (session == NULL)
if (!session)
goto end;

/* Special case: if session_id == 0x0000, treat as operation on tunnel */
Expand Down Expand Up @@ -1464,7 +1464,7 @@ static void pppol2tp_next_session(struct net *net, struct pppol2tp_seq_data *pd)
pd->session = l2tp_session_get_nth(pd->tunnel, pd->session_idx);
pd->session_idx++;

if (pd->session == NULL) {
if (!pd->session) {
pd->session_idx = 0;
pppol2tp_next_tunnel(net, pd);
}
Expand All @@ -1479,17 +1479,17 @@ static void *pppol2tp_seq_start(struct seq_file *m, loff_t *offs)
if (!pos)
goto out;

BUG_ON(m->private == NULL);
BUG_ON(!m->private);
pd = m->private;
net = seq_file_net(m);

if (pd->tunnel == NULL)
if (!pd->tunnel)
pppol2tp_next_tunnel(net, pd);
else
pppol2tp_next_session(net, pd);

/* NULL tunnel and session indicates end of list */
if ((pd->tunnel == NULL) && (pd->session == NULL))
if (!pd->tunnel && !pd->session)
pd = NULL;

out:
Expand Down

0 comments on commit 0febc7b

Please sign in to comment.