Seems to stop it from crashing on non-theme QR scan. (#96)

This seems to stop Non-theme QR crash.
This commit is contained in:
Luís Marques
2017-10-16 14:46:53 +01:00
committed by Alex Taber
parent b5c78cf68b
commit 6c3fdbaf31

View File

@@ -221,6 +221,7 @@ Result http_get(char *url, char *path)
free(content_disposition); free(content_disposition);
free(new_url); free(new_url);
free(buf); free(buf);
return ret;
} }
char *filename; char *filename;
@@ -228,6 +229,13 @@ Result http_get(char *url, char *path)
filename = strtok(NULL, "\""); filename = strtok(NULL, "\"");
char *illegal_characters = "\"?;:/\\+"; char *illegal_characters = "\"?;:/\\+";
if(!filename)
{
free(content_disposition);
free(new_url);
free(buf);
return -1;
}
for (size_t i = 0; i < strlen(filename); i++) for (size_t i = 0; i < strlen(filename); i++)
{ {
for (size_t n = 0; n < strlen(illegal_characters); n++) for (size_t n = 0; n < strlen(illegal_characters); n++)
@@ -250,6 +258,8 @@ Result http_get(char *url, char *path)
if (buf == NULL) if (buf == NULL)
{ {
httpcCloseContext(&context); httpcCloseContext(&context);
free(content_disposition);
free(new_url);
free(last_buf); free(last_buf);
return ret; return ret;
} }
@@ -261,6 +271,8 @@ Result http_get(char *url, char *path)
if (buf == NULL) if (buf == NULL)
{ {
httpcCloseContext(&context); httpcCloseContext(&context);
free(content_disposition);
free(new_url);
free(last_buf); free(last_buf);
return -1; return -1;
} }
@@ -276,5 +288,9 @@ Result http_get(char *url, char *path)
exit_qr(); exit_qr();
free(content_disposition);
free(new_url);
free(buf);
return 0; return 0;
} }