From 237919f1c337b2ac0b5b30d65964dd4bbb126457 Mon Sep 17 00:00:00 2001 From: Aidan MacDonald Date: Mon, 19 Apr 2021 21:20:41 +0100 Subject: Fix invalid use of bool in doom Settings code tries to cast the bool pointer to an int pointer and then write through it, which makes AddressSanitizer whine. Change-Id: I1d6162fc98144cd6b40444016b3b48ef02967324 --- apps/plugins/doom/doomdef.c | 2 +- apps/plugins/doom/doomdef.h | 2 +- apps/plugins/doom/m_misc.c | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/apps/plugins/doom/doomdef.c b/apps/plugins/doom/doomdef.c index 3facfbb299..6ae4bb6894 100644 --- a/apps/plugins/doom/doomdef.c +++ b/apps/plugins/doom/doomdef.c @@ -34,7 +34,7 @@ // Location for any defines turned variables. #if(LCD_HEIGHT>LCD_WIDTH) -bool rotate_screen=0; +int rotate_screen=0; int SCREENWIDTH; int SCREENHEIGHT; #endif diff --git a/apps/plugins/doom/doomdef.h b/apps/plugins/doom/doomdef.h index 05346cd443..43114f75ef 100644 --- a/apps/plugins/doom/doomdef.h +++ b/apps/plugins/doom/doomdef.h @@ -94,7 +94,7 @@ typedef unsigned Language_t; // when multiple screen sizes are supported #if(LCD_HEIGHT>LCD_WIDTH) -extern bool rotate_screen; +extern int rotate_screen; // proff 08/17/98: Changed for high-res #define MAX_SCREENWIDTH LCD_HEIGHT #define MAX_SCREENHEIGHT LCD_HEIGHT diff --git a/apps/plugins/doom/m_misc.c b/apps/plugins/doom/m_misc.c index 586a4a1c2f..0318c9d741 100644 --- a/apps/plugins/doom/m_misc.c +++ b/apps/plugins/doom/m_misc.c @@ -308,7 +308,7 @@ default_t defaults[] = {"screen_height",{&desired_screenheight, NULL},{200, NULL},200,1200, def_int,ss_none, 0, 0},*/ #if(LCD_HEIGHT>LCD_WIDTH) - {"rotate_screen",{(void *)&rotate_screen, NULL},{0, NULL},0,1, + {"rotate_screen",{&rotate_screen, NULL},{0, NULL},0,1, def_bool,ss_none, 0, 0}, /* kwk - rotate the screen 90 degrees */ #endif {"fake_contrast",{&fake_contrast, NULL},{1, NULL},0,1, -- cgit v1.2.3