diff options
author | Andy Grover <andy.grover@oracle.com> | 2010-01-12 14:56:44 -0500 |
---|---|---|
committer | Andy Grover <andy.grover@oracle.com> | 2010-09-08 21:11:32 -0400 |
commit | 8690bfa17aea4c42da1bcf90a7af93d161eca624 (patch) | |
tree | 88c4c5fa63aab0c18cf13228c4b3a6f980aa74be /net/rds/recv.c | |
parent | 2dc393573430f853e56e25bf4b41c34ba2aa8fd6 (diff) |
RDS: cleanup: remove "== NULL"s and "!= NULL"s in ptr comparisons
Favor "if (foo)" style over "if (foo != NULL)".
Signed-off-by: Andy Grover <andy.grover@oracle.com>
Diffstat (limited to 'net/rds/recv.c')
-rw-r--r-- | net/rds/recv.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/net/rds/recv.c b/net/rds/recv.c index c93588c2d553..88f1f5aecfa6 100644 --- a/net/rds/recv.c +++ b/net/rds/recv.c | |||
@@ -210,7 +210,7 @@ void rds_recv_incoming(struct rds_connection *conn, __be32 saddr, __be32 daddr, | |||
210 | } | 210 | } |
211 | 211 | ||
212 | rs = rds_find_bound(daddr, inc->i_hdr.h_dport); | 212 | rs = rds_find_bound(daddr, inc->i_hdr.h_dport); |
213 | if (rs == NULL) { | 213 | if (!rs) { |
214 | rds_stats_inc(s_recv_drop_no_sock); | 214 | rds_stats_inc(s_recv_drop_no_sock); |
215 | goto out; | 215 | goto out; |
216 | } | 216 | } |
@@ -251,7 +251,7 @@ static int rds_next_incoming(struct rds_sock *rs, struct rds_incoming **inc) | |||
251 | { | 251 | { |
252 | unsigned long flags; | 252 | unsigned long flags; |
253 | 253 | ||
254 | if (*inc == NULL) { | 254 | if (!*inc) { |
255 | read_lock_irqsave(&rs->rs_recv_lock, flags); | 255 | read_lock_irqsave(&rs->rs_recv_lock, flags); |
256 | if (!list_empty(&rs->rs_recv_queue)) { | 256 | if (!list_empty(&rs->rs_recv_queue)) { |
257 | *inc = list_entry(rs->rs_recv_queue.next, | 257 | *inc = list_entry(rs->rs_recv_queue.next, |