gitea/SDL3_fork
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

events: Fix undefined behavior when disabling some event types

Browse Source 
Shifting a signed int left by 31 is UB.
This commit is contained in:
Cameron Gutman
2025-04-29 00:07:36 -05:00
parent 510c7edd9b
commit 39d3148185
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
src/events/SDL_events.c
View File

@@ -1844,7 +1844,7 @@ void SDL_SetEventEnabled(Uint32 type, bool enabled)
Uint8 lo = (type & 0xff); Uint8 lo = (type & 0xff);
if (SDL_disabled_events[hi] && if (SDL_disabled_events[hi] &&
(SDL_disabled_events[hi]->bits[lo / 32] & (1 << (lo & 31)))) { (SDL_disabled_events[hi]->bits[lo / 32] & (1U << (lo & 31)))) {
current_state = false; current_state = false;
} else { } else {
current_state = true; current_state = true;
@@ -1853,7 +1853,7 @@ void SDL_SetEventEnabled(Uint32 type, bool enabled)
if ((enabled != false) != current_state) { if ((enabled != false) != current_state) {
if (enabled) { if (enabled) {
SDL_assert(SDL_disabled_events[hi] != NULL); SDL_assert(SDL_disabled_events[hi] != NULL);
SDL_disabled_events[hi]->bits[lo / 32] &= ~(1 << (lo & 31)); SDL_disabled_events[hi]->bits[lo / 32] &= ~(1U << (lo & 31));
// Gamepad events depend on joystick events // Gamepad events depend on joystick events
switch (type) { switch (type) {
@@ -1884,7 +1884,7 @@ void SDL_SetEventEnabled(Uint32 type, bool enabled)
} }
// Out of memory, nothing we can do... // Out of memory, nothing we can do...
if (SDL_disabled_events[hi]) { if (SDL_disabled_events[hi]) {
SDL_disabled_events[hi]->bits[lo / 32] |= (1 << (lo & 31)); SDL_disabled_events[hi]->bits[lo / 32] |= (1U << (lo & 31));
SDL_FlushEvent(type); SDL_FlushEvent(type);
} }
} }
@@ -1903,7 +1903,7 @@ bool SDL_EventEnabled(Uint32 type)
Uint8 lo = (type & 0xff); Uint8 lo = (type & 0xff);
if (SDL_disabled_events[hi] && if (SDL_disabled_events[hi] &&
(SDL_disabled_events[hi]->bits[lo / 32] & (1 << (lo & 31)))) { (SDL_disabled_events[hi]->bits[lo / 32] & (1U << (lo & 31)))) {
return false; return false;
} else { } else {
return true; return true;