Mirrors_Framework/SDL
1
0
Fork 0
mirror of https://github.com/libsdl-org/SDL.git synced 2026-05-30 21:37:43 +08:00
Code Issues Actions 5 Packages Projects Releases Wiki Activity

SDL_iconv_string: simplify recomputation of outbuf and outbytesleft

Browse Source 
Noticed this in SDL-1.2 where gcc-13 emits a -Wuse-after-free warning.
No such warning in SDL2 and SDL3, because unlike SDL1.2, SDL_realloc()
is not a macro expanding to libc realloc(). It warns, of course, if
SDL_realloc() is replaced with plain realloc():

src/stdlib/SDL_iconv.c: In function 'SDL_iconv_string_REAL':
src/stdlib/SDL_iconv.c:824:39: warning: pointer 'oldstring' may be used after 'realloc' [-Wuse-after-free]
  824 |             outbuf = string + (outbuf - oldstring);
      |                               ~~~~~~~~^~~~~~~~~~~~
src/stdlib/SDL_iconv.c:818:30: note: call to 'realloc' here
  818 |             string = (char *)realloc(string, stringsize + sizeof(Uint32));
      |                              ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
This commit is contained in:
Ozkan Sezer
2024-05-10 17:55:00 +03:00
committed by Ozkan Sezer
parent b2fca55e5c
commit 2205626816
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/stdlib/SDL_iconv.c
+3 -2
View File
@@ -813,6 +813,7 @@ char *SDL_iconv_string(const char *tocode, const char *fromcode, const char *inb
switch (retCode) { switch (retCode) {
case SDL_ICONV_E2BIG: case SDL_ICONV_E2BIG:
{ {
const ptrdiff_t diff = (ptrdiff_t) (outbuf - string);
char *oldstring = string; char *oldstring = string;
stringsize *= 2; stringsize *= 2;
string = (char *)SDL_realloc(string, stringsize + sizeof(Uint32)); string = (char *)SDL_realloc(string, stringsize + sizeof(Uint32));
@@ -821,8 +822,8 @@ char *SDL_iconv_string(const char *tocode, const char *fromcode, const char *inb
SDL_iconv_close(cd); SDL_iconv_close(cd);
return NULL; return NULL;
} }
outbuf = string + (outbuf - oldstring); outbuf = string + diff;
outbytesleft = stringsize - (outbuf - string); outbytesleft = stringsize - diff;
SDL_memset(outbuf, 0, sizeof(Uint32)); SDL_memset(outbuf, 0, sizeof(Uint32));
continue; continue;
} }