From 8abe315896fe8e07f57b412f8f8c442a725f1296 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Higor=20Eur=C3=ADpedes?= Date: Tue, 19 May 2015 14:40:25 -0300 Subject: [PATCH 1/5] (runloop.c) Set g_extern to NULL on free --- runloop.c | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/runloop.c b/runloop.c index 8a06ddc37c..1b7bb8a588 100644 --- a/runloop.c +++ b/runloop.c @@ -949,19 +949,18 @@ void rarch_main_state_free(void) void rarch_main_global_free(void) { - global_t *global = NULL; - event_command(EVENT_CMD_TEMPORARY_CONTENT_DEINIT); event_command(EVENT_CMD_SUBSYSTEM_FULLPATHS_DEINIT); event_command(EVENT_CMD_RECORD_DEINIT); event_command(EVENT_CMD_LOG_FILE_DEINIT); - global = global_get_ptr(); + g_extern = global_get_ptr(); - if (!global) + if (!g_extern) return; - free(global); + free(g_extern); + g_extern = NULL; } bool rarch_main_verbosity(void) From 027b0d4ddd7bd456baeda6bf379892f6610f1db4 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Higor=20Eur=C3=ADpedes?= Date: Tue, 19 May 2015 14:55:21 -0300 Subject: [PATCH 2/5] (retroarch.c) Fix uninitialized variable usage error --- retroarch.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/retroarch.c b/retroarch.c index 9839e9572c..f1264cf391 100644 --- a/retroarch.c +++ b/retroarch.c @@ -1117,7 +1117,7 @@ int rarch_main_init(int argc, char *argv[]) if (global->verbosity) { - char str[PATH_MAX_LENGTH]; + char str[PATH_MAX_LENGTH] = {0}; RARCH_LOG_OUTPUT("=== Build ======================================="); rarch_info_get_capabilities(RARCH_CAPABILITIES_CPU, str, sizeof(str)); From 59e0f7221d5e4322a4be37c058c4a706258ab58b Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Higor=20Eur=C3=ADpedes?= Date: Tue, 19 May 2015 15:11:05 -0300 Subject: [PATCH 3/5] (gl_raster_font) Fix typo --- gfx/drivers_font/gl_raster_font.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/gfx/drivers_font/gl_raster_font.c b/gfx/drivers_font/gl_raster_font.c index 8693212f19..574f435cda 100644 --- a/gfx/drivers_font/gl_raster_font.c +++ b/gfx/drivers_font/gl_raster_font.c @@ -29,8 +29,8 @@ font_color[ 4 * (6 * i + c) + 1] = color[1]; \ font_color[ 4 * (6 * i + c) + 2] = color[2]; \ font_color[ 4 * (6 * i + c) + 3] = color[3]; \ - font_lut_tex_coord[ 4 * (6 * i + c) + 0] = gl->coords.lut_tex_coord[0]; \ - font_lut_tex_coord[ 4 * (6 * i + c) + 1] = gl->coords.lut_tex_coord[1]; \ + font_lut_tex_coord[ 2 * (6 * i + c) + 0] = gl->coords.lut_tex_coord[0]; \ + font_lut_tex_coord[ 2 * (6 * i + c) + 1] = gl->coords.lut_tex_coord[1]; \ } while(0) #define MAX_MSG_LEN_CHUNK 64 From 73ae2f0ad1b0d64d3bac25673fcb96a71a5ef717 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Higor=20Eur=C3=ADpedes?= Date: Tue, 19 May 2015 15:11:34 -0300 Subject: [PATCH 4/5] (Makefile) Add SANITIZER variable --- Makefile | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/Makefile b/Makefile index 0f83528380..da84e7db85 100644 --- a/Makefile +++ b/Makefile @@ -78,6 +78,12 @@ endif RARCH_OBJ := $(addprefix $(OBJDIR)/,$(OBJ)) RARCH_JOYCONFIG_OBJ := $(addprefix $(OBJDIR)/,$(JOYCONFIG_OBJ)) +ifneq ($(SANITIZER),) + CFLAGS := -fsanitize=$(SANITIZER) $(CFLAGS) + CXXFLAGS := -fsanitize=$(SANITIZER) $(CXXFLAGS) + LDFLAGS := -fsanitize=$(SANITIZER) $(LDLAGS) +endif + all: $(TARGET) $(JTARGET) config.mk -include $(RARCH_OBJ:.o=.d) $(RARCH_JOYCONFIG_OBJ:.o=.d) From f093c00e9cac1759f10f7a1b49c21debf187c1b6 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Higor=20Eur=C3=ADpedes?= Date: Tue, 19 May 2015 15:13:05 -0300 Subject: [PATCH 5/5] (runloop.c) Remove uneeded statement --- runloop.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/runloop.c b/runloop.c index 1b7bb8a588..0859a008b3 100644 --- a/runloop.c +++ b/runloop.c @@ -954,8 +954,6 @@ void rarch_main_global_free(void) event_command(EVENT_CMD_RECORD_DEINIT); event_command(EVENT_CMD_LOG_FILE_DEINIT); - g_extern = global_get_ptr(); - if (!g_extern) return;