Leahnaya/SDL
1
0
Fork 0
mirror of https://github.com/yawut/SDL.git synced 2026-10-02 05:06:46 -05:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fixed bug 1082 - Crash on startup when using empty command-line argument.

Browse Source 
yrizoud@gmail.com 2010-12-01 07:34:48 PST
Run a SDL program with "" as one of the command-line arguments: crash on
startup.

Program received signal SIGSEGV, Segmentation fault.
0x0047b2ea in ParseCommandLine (cmdline=<value optimized out>, argv=0x0)
    at ./src/main/win32/SDL_win32_main.c:100
100 while ( *bufp && ( *bufp != '"' || *lastp == '\\' ) ) {
(gdb) bt
#0  0x0047b2ea in ParseCommandLine (cmdline=<value optimized out>, argv=0x0)
    at ./src/main/win32/SDL_win32_main.c:100
#1  0x0047b5bb in WinMain@16 (hInst=0x400000, hPrev=0x0,
    szCmdLine=0x81c530e0 "a \"\" b", sw=10)
    at ./src/main/win32/SDL_win32_main.c:374
#2  0x0047af28 in main ()

---

The problem is that on Windows when you make a shortcut, and want it to accept
drag-and-dropped files, the good way to make it work work with files that have
spaces in their names or paths is make this argument "%1" (with the surrounding
quotes). But then when you run it without dropping a file into it, it's
resolved as "", and triggers this bug.

--HG--
branch : SDL-1.2
This commit is contained in:
Sam Lantinga
2011-12-30 06:54:58 -05:00
parent 3408e81a80
commit f27bf8110d
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/main/win32/SDL_win32_main.c
View File

@@ -98,7 +98,7 @@ static int ParseCommandLine(char *cmdline, char **argv)
++argc;
}
/* Skip over word */
while ( *bufp && ( *bufp != '"' || *lastp == '\\' ) ) {
while ( *bufp && ( *bufp != '"' || (lastp && *lastp == '\\') ) ) {
lastp = bufp;
++bufp;
}