include/SDL_config_psp.h
author Sam Lantinga <slouken@libsdl.org>
Thu, 16 May 2013 00:43:22 -0700
changeset 7180 3733e68edbc3
parent 7116 d8f301fbc676
child 7191 75360622e65f
permissions -rw-r--r--
Fixed bug 1846 - _allmul implementation in SDL_stdlib.c doesn't clean up the stack

Colin Barrett

I see this manifest itself (VS2012 x86) as:

"Run-Time Check Failure #0 - The value of ESP was not properly saved across a function call. This is usually a result of calling a function declared with one calling convention with a function pointer declared with a different calling convention."

in the first call to SDL_GetTicks in my application. The disassembly at the problem line is:

hires_now.QuadPart *= 1000;
00AD0792 push 0
00AD0794 push 3E8h
00AD0799 mov eax,dword ptr [ebp-10h]
00AD079C push eax
00AD079D mov ecx,dword ptr [hires_now]
00AD07A0 push ecx
00AD07A1 call _allmul (0AE7D40h)
00AD07A6 mov dword ptr [hires_now],eax
00AD07A9 mov dword ptr [ebp-10h],edx

Apparently _allmul should be popping the stack but isn't (other similar functions in SDL_stdlib.c - _alldiv and whatnot - DO pop the stack).

A 'ret 10h' at the end of _allmul appears to do the trick
     1 /*
     2   Simple DirectMedia Layer
     3   Copyright (C) 1997-2013 Sam Lantinga <slouken@libsdl.org>
     4 
     5   This software is provided 'as-is', without any express or implied
     6   warranty.  In no event will the authors be held liable for any damages
     7   arising from the use of this software.
     8 
     9   Permission is granted to anyone to use this software for any purpose,
    10   including commercial applications, and to alter it and redistribute it
    11   freely, subject to the following restrictions:
    12 
    13   1. The origin of this software must not be misrepresented; you must not
    14      claim that you wrote the original software. If you use this software
    15      in a product, an acknowledgment in the product documentation would be
    16      appreciated but is not required.
    17   2. Altered source versions must be plainly marked as such, and must not be
    18      misrepresented as being the original software.
    19   3. This notice may not be removed or altered from any source distribution.
    20 */
    21 
    22 #ifndef _SDL_config_psp_h
    23 #define _SDL_config_psp_h
    24 
    25 #include "SDL_platform.h"
    26 
    27 
    28 
    29 #ifdef __GNUC__
    30 #define HAVE_GCC_SYNC_LOCK_TEST_AND_SET 1
    31 #endif
    32 
    33 #define HAVE_GCC_ATOMICS	1
    34 
    35 #define HAVE_ALLOCA_H		1
    36 #define HAVE_SYS_TYPES_H	1
    37 #define HAVE_STDIO_H	1
    38 #define STDC_HEADERS	1
    39 #define HAVE_STRING_H	1
    40 #define HAVE_INTTYPES_H	1
    41 #define HAVE_STDINT_H	1
    42 #define HAVE_CTYPE_H	1
    43 #define HAVE_MATH_H	1
    44 #define HAVE_SIGNAL_H	1
    45 
    46 /* C library functions */
    47 #define HAVE_MALLOC	1
    48 #define HAVE_CALLOC	1
    49 #define HAVE_REALLOC	1
    50 #define HAVE_FREE	1
    51 #define HAVE_ALLOCA	1
    52 #define HAVE_GETENV	1
    53 #define HAVE_SETENV	1
    54 #define HAVE_PUTENV	1
    55 #define HAVE_SETENV	1
    56 #define HAVE_UNSETENV	1
    57 #define HAVE_QSORT	1
    58 #define HAVE_ABS	1
    59 #define HAVE_BCOPY	1
    60 #define HAVE_MEMSET	1
    61 #define HAVE_MEMCPY	1
    62 #define HAVE_MEMMOVE	1
    63 #define HAVE_MEMCMP	1
    64 #define HAVE_STRLEN	1
    65 #define HAVE_STRLCPY	1
    66 #define HAVE_STRLCAT	1
    67 #define HAVE_STRDUP	1
    68 #define HAVE_STRCHR	1
    69 #define HAVE_STRRCHR	1
    70 #define HAVE_STRSTR	1
    71 #define HAVE_STRTOL	1
    72 #define HAVE_STRTOUL	1
    73 #define HAVE_STRTOLL	1
    74 #define HAVE_STRTOULL	1
    75 #define HAVE_STRTOD	1
    76 #define HAVE_ATOI	1
    77 #define HAVE_ATOF	1
    78 #define HAVE_STRCMP	1
    79 #define HAVE_STRNCMP	1
    80 #define HAVE_STRCASECMP	1
    81 #define HAVE_STRNCASECMP 1
    82 #define HAVE_SSCANF	1
    83 #define HAVE_SNPRINTF	1
    84 #define HAVE_VSNPRINTF	1
    85 #define HAVE_M_PI	1
    86 #define HAVE_ATAN	1
    87 #define HAVE_ATAN2	1
    88 #define HAVE_CEIL	1
    89 #define HAVE_COPYSIGN	1
    90 #define HAVE_COS	1
    91 #define HAVE_COSF	1
    92 #define HAVE_FABS	1
    93 #define HAVE_FLOOR	1
    94 #define HAVE_LOG	1
    95 #define HAVE_POW	1
    96 #define HAVE_SCALBN	1
    97 #define HAVE_SIN	1
    98 #define HAVE_SINF	1
    99 #define HAVE_SQRT	1
   100 #define HAVE_SETJMP	1
   101 #define HAVE_NANOSLEEP	1
   102 //#define HAVE_SYSCONF	1
   103 //#define HAVE_SIGACTION	1
   104 
   105 
   106 /* PSP isn't that sophisticated */
   107 #define LACKS_SYS_MMAN_H 1
   108 
   109 /* Enable the stub thread support (src/thread/psp/\*.c) */
   110 #define SDL_THREAD_PSP	1
   111 
   112 /* Enable the stub timer support (src/timer/psp/\*.c) */
   113 #define SDL_TIMERS_PSP	1
   114 
   115 /* Enable the stub joystick driver (src/joystick/psp/\*.c) */
   116 #define SDL_JOYSTICK_PSP		1
   117 
   118 /* Enable the stub audio driver (src/audio/psp/\*.c) */
   119 #define SDL_AUDIO_DRIVER_PSP	1
   120 
   121 /* PSP video dirver */
   122 #define SDL_VIDEO_DRIVER_PSP   1
   123 
   124 /* PSP render dirver */
   125 #define SDL_VIDEO_RENDER_PSP   1
   126 
   127 #define SDL_POWER_PSP          1
   128 
   129 /* PSP doesn't have haptic device (src/haptic/dummy/\*.c) */
   130 #define SDL_HAPTIC_DISABLED	   1
   131 
   132 /* PSP can't load shared object (src/loadso/dummy/\*.c) */
   133 #define SDL_LOADSO_DISABLED	   1
   134 
   135 
   136 #endif /* _SDL_config_psp_h */