moparisthebest/curl
1
0
Fork 0
mirror of https://github.com/moparisthebest/curl synced 2024-12-24 17:18:48 -05:00
Code Issues Releases Wiki Activity

limit-rate: kick in even before "limit" data has been received

Browse Source 
... and make sure to avoid integer overflows with really large values.

Reported-by: 刘佩东
Fixes #2371
Closes #2373
This commit is contained in:
Daniel Stenberg 2018-03-10 14:07:38 +01:00
parent 029ae11034
commit 72a0f6251a
No known key found for this signature in database
GPG Key ID: 5CC908FDB71E12C2
2 changed files with 29 additions and 23 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

30
lib/progress.c
View File

@ -238,8 +238,8 @@ void Curl_pgrsStartNow(struct Curl_easy *data)
} }
/* /*
* This is used to handle speed limits, calculating how much milliseconds we * This is used to handle speed limits, calculating how many milliseconds to
* need to wait until we're back under the speed limit, if needed. * wait until we're back under the speed limit, if needed.
* *
* The way it works is by having a "starting point" (time & amount of data * The way it works is by having a "starting point" (time & amount of data
* transferred by then) used in the speed computation, to be used instead of * transferred by then) used in the speed computation, to be used instead of
@ -251,12 +251,11 @@ void Curl_pgrsStartNow(struct Curl_easy *data)
* the starting point, the limit (in bytes/s), the time of the starting point * the starting point, the limit (in bytes/s), the time of the starting point
* and the current time. * and the current time.
* *
* Returns -1 if no waiting is needed (not enough data transferred since * Returns 0 if no waiting is needed or when no waiting is needed but the
* starting point yet), 0 when no waiting is needed but the starting point * starting point should be reset (to current); or the number of milliseconds
* should be reset (to current), or the number of milliseconds to wait to get * to wait to get back under the speed limit.
* back under the speed limit.
*/ */
long Curl_pgrsLimitWaitTime(curl_off_t cursize, timediff_t Curl_pgrsLimitWaitTime(curl_off_t cursize,
curl_off_t startsize, curl_off_t startsize,
curl_off_t limit, curl_off_t limit,
struct curltime start, struct curltime start,
@ -270,16 +269,23 @@ long Curl_pgrsLimitWaitTime(curl_off_t cursize,
if(start.tv_sec == 0 && start.tv_usec == 0) if(start.tv_sec == 0 && start.tv_usec == 0)
return 0; return 0;
/* not enough data yet */ if(!limit)
if(size < limit) return 0;
return -1;
if(size < CURL_OFF_T_MAX/1000)
minimum = (time_t) (CURL_OFF_T_C(1000) * size / limit); minimum = (time_t) (CURL_OFF_T_C(1000) * size / limit);
else {
minimum = (time_t) (size / limit);
if(minimum < TIME_T_MAX/1000)
minimum *= CURL_OFF_T_C(1000);
else
minimum = TIME_T_MAX;
}
actual = Curl_timediff(now, start); actual = Curl_timediff(now, start);
if(actual < minimum) if(actual < minimum)
/* this is a conversion on some systems (64bit time_t => 32bit long) */ return (minimum - actual);
return (long)(minimum - actual);
return 0; return 0;
} }

4
lib/progress.h
View File

@ -7,7 +7,7 @@
* | (__| |_| | _ <| |___ * | (__| |_| | _ <| |___
* \___|\___/|_| \_\_____| * \___|\___/|_| \_\_____|
* *
* Copyright (C) 1998 - 2017, Daniel Stenberg, <daniel@haxx.se>, et al. * Copyright (C) 1998 - 2018, Daniel Stenberg, <daniel@haxx.se>, et al.
* *
* This software is licensed as described in the file COPYING, which * This software is licensed as described in the file COPYING, which
* you should have received as part of this distribution. The terms * you should have received as part of this distribution. The terms
@ -49,7 +49,7 @@ void Curl_pgrsSetUploadCounter(struct Curl_easy *data, curl_off_t size);
int Curl_pgrsUpdate(struct connectdata *); int Curl_pgrsUpdate(struct connectdata *);
void Curl_pgrsResetTransferSizes(struct Curl_easy *data); void Curl_pgrsResetTransferSizes(struct Curl_easy *data);
void Curl_pgrsTime(struct Curl_easy *data, timerid timer); void Curl_pgrsTime(struct Curl_easy *data, timerid timer);
long Curl_pgrsLimitWaitTime(curl_off_t cursize, timediff_t Curl_pgrsLimitWaitTime(curl_off_t cursize,
curl_off_t startsize, curl_off_t startsize,
curl_off_t limit, curl_off_t limit,
struct curltime start, struct curltime start,