add stupid extra checks to see why this code overflows on pentoo...

This commit is contained in:
iceman1001 2023-07-27 16:26:36 +02:00
commit 7440e617e0

View file

@ -175,7 +175,7 @@ static char *filenamemcopy(const char *preferredName, const char *suffix) {
} }
static size_t path_size(savePaths_t a) { static size_t path_size(savePaths_t a) {
if (a == spItemCount) { if (a >= spItemCount) {
return 0; return 0;
} }
return strlen(g_session.defaultPaths[a]); return strlen(g_session.defaultPaths[a]);
@ -205,9 +205,10 @@ char *newfilenamemcopyEx(const char *preferredName, const char *suffix, savePath
// user preference save paths // user preference save paths
size_t save_path_len = path_size(e_save_path); size_t save_path_len = path_size(e_save_path);
if (save_path_len < FILE_PATH_SIZE ) { if (save_path_len && save_path_len < (FILE_PATH_SIZE - strlen(PATHSEP))) {
snprintf(pfn, len, "%.*s%s", (int)save_path_len, g_session.defaultPaths[e_save_path], PATHSEP); snprintf(pfn, len, "%s%s", g_session.defaultPaths[e_save_path], PATHSEP);
pfn += save_path_len + strlen(PATHSEP); pfn += save_path_len;
pfn += strlen(PATHSEP);
} }
// remove file extension if exist in name // remove file extension if exist in name