Fix potential null dereference
This commit is contained in:
@@ -975,8 +975,8 @@ redirect: // goto here if we need to redirect
|
|||||||
if (*redirect_url == '/') // if relative URL
|
if (*redirect_url == '/') // if relative URL
|
||||||
{
|
{
|
||||||
strcpy(new_url, url);
|
strcpy(new_url, url);
|
||||||
// this is good code, i promise
|
char * last_slash = strchr(strchr(strchr(new_url, '/') + 1, '/') + 1, '/');
|
||||||
*(strchr(strchr(strchr(new_url, '/') + 1, '/') + 1, '/')) = '\0';
|
if (last_slash) *last_slash = '\0'; // prevents a NULL deref in case the original domain was not /-delimited
|
||||||
strncat(new_url, redirect_url, 0x824 - strlen(new_url));
|
strncat(new_url, redirect_url, 0x824 - strlen(new_url));
|
||||||
url = new_url;
|
url = new_url;
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user