diff options
| author | Daniel Stenberg <daniel@haxx.se> | 2019-05-31 23:00:06 +0200 |
|---|---|---|
| committer | Daniel Stenberg <daniel@haxx.se> | 2019-06-01 10:41:15 +0200 |
| commit | c36f42839d88b0e4c938e80ebd5da4c765b0264e (patch) | |
| tree | 6927df23be01456171daaeffc5eba25695a495b2 | |
| parent | d3d04a872c0d907547e96f2b4c89367d9f43e021 (diff) | |
| download | curl-c36f42839d88b0e4c938e80ebd5da4c765b0264e.tar.gz | |
singlesocket: use separate variable for inner loop
An inner loop within the singlesocket() function wrongly re-used the
variable for the outer loop which then could cause an infinite
loop. Change to using a separate variable!
Reported-by: Eric Wu
Fixes #3970
Closes #3973
| -rw-r--r-- | lib/multi.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/lib/multi.c b/lib/multi.c index ec0318735..02f846033 100644 --- a/lib/multi.c +++ b/lib/multi.c @@ -2252,14 +2252,14 @@ static CURLMcode singlesocket(struct Curl_multi *multi, actions[i] = action; if(entry) { /* check if new for this transfer */ - for(i = 0; i< data->numsocks; i++) { - if(s == data->sockets[i]) { - prevaction = data->actions[i]; + int j; + for(j = 0; j< data->numsocks; j++) { + if(s == data->sockets[j]) { + prevaction = data->actions[j]; sincebefore = TRUE; break; } } - } else { /* this is a socket we didn't have before, add it to the hash! */ |