diff options
author | Octavian Purdila <opurdila@ixiacom.com> | 2008-07-03 06:31:21 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-07-03 06:31:21 -0400 |
commit | 374e7b59498ce0785b3727794b351221528a5159 (patch) | |
tree | dca488eae546ff8a113e614670e23460a23dc7ea /net/ipv4/tcp.c | |
parent | 81b23b4a7acd9b37a269c62d02479d4f645dd20a (diff) |
tcp: fix a size_t < 0 comparison in tcp_read_sock
<used> should be of type int (not size_t) since recv_actor can return
negative values and it is also used in a < 0 comparison.
Signed-off-by: Octavian Purdila <opurdila@ixiacom.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv4/tcp.c')
-rw-r--r-- | net/ipv4/tcp.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/net/ipv4/tcp.c b/net/ipv4/tcp.c index de53024664e4..1d723de18686 100644 --- a/net/ipv4/tcp.c +++ b/net/ipv4/tcp.c | |||
@@ -1209,7 +1209,8 @@ int tcp_read_sock(struct sock *sk, read_descriptor_t *desc, | |||
1209 | return -ENOTCONN; | 1209 | return -ENOTCONN; |
1210 | while ((skb = tcp_recv_skb(sk, seq, &offset)) != NULL) { | 1210 | while ((skb = tcp_recv_skb(sk, seq, &offset)) != NULL) { |
1211 | if (offset < skb->len) { | 1211 | if (offset < skb->len) { |
1212 | size_t used, len; | 1212 | int used; |
1213 | size_t len; | ||
1213 | 1214 | ||
1214 | len = skb->len - offset; | 1215 | len = skb->len - offset; |
1215 | /* Stop reading if we hit a patch of urgent data */ | 1216 | /* Stop reading if we hit a patch of urgent data */ |