• Sam Lantinga's avatar
    Fixed bug #1056 (Frequent crashes in Touch events by simply touching the screen) · e64d0e8d
    Sam Lantinga authored
     Joseba García Echebarria      2010-12-15 01:55:22 PST
    
    I believe the crash is caused by a check not being performed on wether an
    SDL_Touch element is NULL before using it in the SDL_SendTouchMotion function
    in src/events/SDL_touch.c around line 400.
    Judging from the rest of the code, there's a missing
    
        if (!touch) {
            return 0;
        }
    
    before using "touch" as SDL_GetFinger(), SDL_GetFingerIndexId() use
    touch->num_fingers without checking.
    
    I can attach a patch if you like. It seems pretty straightforward, though.
    
    I have yet to discover why touch is being returned as NULL as this error is
    only triggered when an actual gesture has been performed, maybe something
    related to SDL_AddTouch()?
    e64d0e8d
Name
Last commit
Last update
..
SDL_clipboardevents.c Loading commit data...
SDL_clipboardevents_c.h Loading commit data...
SDL_events.c Loading commit data...
SDL_events_c.h Loading commit data...
SDL_gesture.c Loading commit data...
SDL_gesture_c.h Loading commit data...
SDL_keyboard.c Loading commit data...
SDL_keyboard_c.h Loading commit data...
SDL_mouse.c Loading commit data...
SDL_mouse_c.h Loading commit data...
SDL_quit.c Loading commit data...
SDL_sysevents.h Loading commit data...
SDL_touch.c Loading commit data...
SDL_touch_c.h Loading commit data...
SDL_windowevents.c Loading commit data...
SDL_windowevents_c.h Loading commit data...
blank_cursor.h Loading commit data...
default_cursor.h Loading commit data...
scancodes_darwin.h Loading commit data...
scancodes_linux.h Loading commit data...
scancodes_windows.h Loading commit data...
scancodes_xfree86.h Loading commit data...