Mercurial > sdl-ios-xcode

changeset 717:42ed44b2c8b6

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
Date: Sun, 14 Sep 2003 17:04:55 -0400 From: Philip D.S. Thoren <pthoren@cs.uml.edu> Reply-To: "sdl@libsdl.org" <sdl@libsdl.org> To: "sdl@libsdl.org" <sdl@libsdl.org> Subject: [SDL] BUG in src/video/windx5/SDL_dx5events.c I tracked down a bug that crashed my program on Windows *all* the time. FILE: src/video/windx5/SDL_dx5events.c FUNCTION: handle_mouse BUG: SDL_PublicSurface is being dereferenced when it is NULL. I added the following to the very begining of handle_mouse and it solved my crashing problems: if (SDL_PublicSurface == NULL) return; ~ Philip D.S. Thoren
author Ryan C. Gordon <icculus@icculus.org>
date Mon, 15 Sep 2003 13:51:25 +0000
parents f25e3334d583
children cbc0f7fabd1c
files src/video/windx5/SDL_dx5events.c
diffstat 1 files changed, 5 insertions(+), 0 deletions(-) [+]
line wrap: on
line diff
--- a/src/video/windx5/SDL_dx5events.c	Sat Sep 13 02:20:32 2003 +0000
+++ b/src/video/windx5/SDL_dx5events.c	Mon Sep 15 13:51:25 2003 +0000
@@ -277,6 +277,11 @@
 	Uint8 button;
 	DWORD timestamp = 0;
 
+	/* Sanity check. Mailing list reports this being NULL unexpectedly. */
+	if (SDL_PublicSurface == NULL) {
+		return;
+	}
+
 	/* If we are in windowed mode, Windows is taking care of the mouse */
 	if (  (SDL_PublicSurface->flags & SDL_OPENGL) ||
 	     !(SDL_PublicSurface->flags & SDL_FULLSCREEN) ) {