geemili
/
SDL
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

log: Don't stack-allocate the message buffer. 

It's 4 kilobytes, so I could see this failing if you have a thread with
a tiny stack that unexpectedly logs something.
main
Ryan C. Gordon 2022-04-27 08:06:28 -04:00
parent 12f15aaa74
commit 2a42952281
No known key found for this signature in database
GPG Key ID: FA148B892AB48044
1 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/SDL_log.c
View File

@ -282,8 +282,7 @@ SDL_LogMessageV(int category, SDL_LogPriority priority, const char *fmt, va_list
return;
}
/* !!! FIXME: why not just "char message[SDL_MAX_LOG_MESSAGE];" ? */
message = SDL_stack_alloc(char, SDL_MAX_LOG_MESSAGE);
message = (char *) SDL_malloc(SDL_MAX_LOG_MESSAGE);
if (!message) {
return;
}
@ -300,7 +299,7 @@ SDL_LogMessageV(int category, SDL_LogPriority priority, const char *fmt, va_list
}
SDL_log_function(SDL_log_userdata, category, priority, message);
SDL_stack_free(message);
SDL_free(message);
}
#if defined(__WIN32__) && !defined(HAVE_STDIO_H) && !defined(__WINRT__)