diff options
| author | Dan Carpenter <error27@gmail.com> | 2010-05-22 12:43:42 +0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2010-05-31 09:24:16 +0200 |
| commit | f5d72af9f3dfea67f014f880d00665f96622f57f (patch) | |
| tree | 9285fc7ee17a3f26dcbcce37ed2d79282fd254cc /net/caif | |
| parent | drivers/isdn: Use memdup_user (diff) | |
| download | linux-f5d72af9f3dfea67f014f880d00665f96622f57f.tar.xz linux-f5d72af9f3dfea67f014f880d00665f96622f57f.zip | |
caif: remove unneeded null check in caif_connect()
We already dereferenced uaddr towards the start of the function when we
checked that "uaddr->sa_family != AF_CAIF". Both the check here and the
earlier check were added in bece7b2398d0: "caif: Rewritten socket
implementation". Before that patch, we assumed that we recieved a valid
pointer for uaddr, and based on that, I have removed this check.
Signed-off-by: Dan Carpenter <error27@gmail.com>
Acked-by: Sjur Braendeland <sjur.brandeland@stericsson.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/caif')
| -rw-r--r-- | net/caif/caif_socket.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/net/caif/caif_socket.c b/net/caif/caif_socket.c index 3d0e09584fae..791249316ef3 100644 --- a/net/caif/caif_socket.c +++ b/net/caif/caif_socket.c @@ -874,8 +874,7 @@ static int caif_connect(struct socket *sock, struct sockaddr *uaddr, sk_stream_kill_queues(&cf_sk->sk); err = -EINVAL; - if (addr_len != sizeof(struct sockaddr_caif) || - !uaddr) + if (addr_len != sizeof(struct sockaddr_caif)) goto out; memcpy(&cf_sk->conn_req.sockaddr, uaddr, |