Commit 35f981d5 authored by antma's avatar antma

replace sprintf call with snprintf for downloaded part name

parent c1557971
...@@ -1822,11 +1822,16 @@ struct query_methods download_methods = { ...@@ -1822,11 +1822,16 @@ struct query_methods download_methods = {
void load_next_part (struct download *D) { void load_next_part (struct download *D) {
if (!D->offset) { if (!D->offset) {
static char buf[1000]; static char buf[PATH_MAX];
int l;
if (!D->id) { if (!D->id) {
sprintf (buf, "%s/download_%lld_%d", get_downloads_directory (), D->volume, D->local_id); l = snprintf (buf, sizeof (buf), "%s/download_%lld_%d", get_downloads_directory (), D->volume, D->local_id);
} else { } else {
sprintf (buf, "%s/download_%lld", get_downloads_directory (), D->id); l = snprintf (buf, sizeof (buf), "%s/download_%lld", get_downloads_directory (), D->id);
}
if (l >= (int) sizeof (buf)) {
logprintf ("Download filename is too long");
exit (1);
} }
D->name = tstrdup (buf); D->name = tstrdup (buf);
struct stat st; struct stat st;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment