From 8f4c2a8f9c98186a38be6e33c58af48cd4dc0849 Mon Sep 17 00:00:00 2001 From: Philipp Wiesemann Date: Fri, 6 Mar 2015 21:29:25 +0100 Subject: [PATCH] Emscripten: Fixed ignoring return value of internal function. If the function Emscripten_ConvertUTF32toUTF8() failed (should currently not be possible) a not terminated string would have been sent as text input event. --- src/video/emscripten/SDL_emscriptenevents.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/video/emscripten/SDL_emscriptenevents.c b/src/video/emscripten/SDL_emscriptenevents.c index d61ff4fe0bb86..6a04d4cc8ffc2 100644 --- a/src/video/emscripten/SDL_emscriptenevents.c +++ b/src/video/emscripten/SDL_emscriptenevents.c @@ -447,8 +447,9 @@ EM_BOOL Emscripten_HandleKeyPress(int eventType, const EmscriptenKeyboardEvent *keyEvent, void *userData) { char text[5]; - Emscripten_ConvertUTF32toUTF8(keyEvent->charCode, text); - SDL_SendKeyboardText(text); + if (Emscripten_ConvertUTF32toUTF8(keyEvent->charCode, text)) { + SDL_SendKeyboardText(text); + } return 1; }