diff options
| author | Andy Grover <andy.grover@oracle.com> | 2010-01-12 20:56:44 +0100 |
|---|---|---|
| committer | Andy Grover <andy.grover@oracle.com> | 2010-09-09 03:11:32 +0200 |
| commit | 8690bfa17aea4c42da1bcf90a7af93d161eca624 (patch) | |
| tree | 88c4c5fa63aab0c18cf13228c4b3a6f980aa74be /net/rds/bind.c | |
| parent | RDS: move rds_shutdown_worker impl. to rds_conn_shutdown (diff) | |
| download | linux-8690bfa17aea4c42da1bcf90a7af93d161eca624.tar.xz linux-8690bfa17aea4c42da1bcf90a7af93d161eca624.zip | |
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/bind.c')
| -rw-r--r-- | net/rds/bind.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/net/rds/bind.c b/net/rds/bind.c index 5d95fc007f1a..65de5cbdb576 100644 --- a/net/rds/bind.c +++ b/net/rds/bind.c @@ -121,7 +121,7 @@ static int rds_add_bound(struct rds_sock *rs, __be32 addr, __be16 *port) do { if (rover == 0) rover++; - if (rds_bind_tree_walk(addr, cpu_to_be16(rover), rs) == NULL) { + if (!rds_bind_tree_walk(addr, cpu_to_be16(rover), rs)) { *port = cpu_to_be16(rover); ret = 0; break; @@ -184,7 +184,7 @@ int rds_bind(struct socket *sock, struct sockaddr *uaddr, int addr_len) goto out; trans = rds_trans_get_preferred(sin->sin_addr.s_addr); - if (trans == NULL) { + if (!trans) { ret = -EADDRNOTAVAIL; rds_remove_bound(rs); if (printk_ratelimit()) |