Fix miniupnpc CVE-2017-8798
This commit is contained in:
parent
d8f402ad8f
commit
c65062ad5e
1 changed files with 7 additions and 5 deletions
12
external/miniupnpc/miniwget.c
vendored
12
external/miniupnpc/miniwget.c
vendored
|
@ -280,11 +280,12 @@ getHTTPResponse(int s, int * size, int * status_code)
|
||||||
goto end_of_stream;
|
goto end_of_stream;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
bytestocopy = ((int)chunksize < (n - i))?chunksize:(unsigned int)(n - i);
|
/* it is guaranteed that (n >= i) */
|
||||||
|
bytestocopy = (chunksize < (unsigned int)(n - i))?chunksize:(unsigned int)(n - i);
|
||||||
if((content_buf_used + bytestocopy) > content_buf_len)
|
if((content_buf_used + bytestocopy) > content_buf_len)
|
||||||
{
|
{
|
||||||
char * tmp;
|
char * tmp;
|
||||||
if(content_length >= (int)(content_buf_used + bytestocopy)) {
|
if((content_length >= 0) && ((unsigned int)content_length >= (content_buf_used + bytestocopy))) {
|
||||||
content_buf_len = content_length;
|
content_buf_len = content_length;
|
||||||
} else {
|
} else {
|
||||||
content_buf_len = content_buf_used + bytestocopy;
|
content_buf_len = content_buf_used + bytestocopy;
|
||||||
|
@ -309,14 +310,15 @@ getHTTPResponse(int s, int * size, int * status_code)
|
||||||
{
|
{
|
||||||
/* not chunked */
|
/* not chunked */
|
||||||
if(content_length > 0
|
if(content_length > 0
|
||||||
&& (int)(content_buf_used + n) > content_length) {
|
&& (content_buf_used + n) > (unsigned int)content_length) {
|
||||||
/* skipping additional bytes */
|
/* skipping additional bytes */
|
||||||
n = content_length - content_buf_used;
|
n = content_length - content_buf_used;
|
||||||
}
|
}
|
||||||
if(content_buf_used + n > content_buf_len)
|
if(content_buf_used + n > content_buf_len)
|
||||||
{
|
{
|
||||||
char * tmp;
|
char * tmp;
|
||||||
if(content_length >= (int)(content_buf_used + n)) {
|
if(content_length >= 0
|
||||||
|
&& (unsigned int)content_length >= (content_buf_used + n)) {
|
||||||
content_buf_len = content_length;
|
content_buf_len = content_length;
|
||||||
} else {
|
} else {
|
||||||
content_buf_len = content_buf_used + n;
|
content_buf_len = content_buf_used + n;
|
||||||
|
@ -336,7 +338,7 @@ getHTTPResponse(int s, int * size, int * status_code)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
/* use the Content-Length header value if available */
|
/* use the Content-Length header value if available */
|
||||||
if(content_length > 0 && (int)content_buf_used >= content_length)
|
if(content_length > 0 && content_buf_used >= (unsigned int)content_length)
|
||||||
{
|
{
|
||||||
#ifdef DEBUG
|
#ifdef DEBUG
|
||||||
printf("End of HTTP content\n");
|
printf("End of HTTP content\n");
|
||||||
|
|
Loading…
Reference in a new issue