Browse Source

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.
Philipp Wiesemann 10 years ago
parent
commit
8f4c2a8f9c
1 changed files with 3 additions and 2 deletions
  1. 3 2
      src/video/emscripten/SDL_emscriptenevents.c

+ 3 - 2
src/video/emscripten/SDL_emscriptenevents.c

@@ -447,8 +447,9 @@ EM_BOOL
 Emscripten_HandleKeyPress(int eventType, const EmscriptenKeyboardEvent *keyEvent, void *userData)
 Emscripten_HandleKeyPress(int eventType, const EmscriptenKeyboardEvent *keyEvent, void *userData)
 {
 {
     char text[5];
     char text[5];
-    Emscripten_ConvertUTF32toUTF8(keyEvent->charCode, text);
-    SDL_SendKeyboardText(text);
+    if (Emscripten_ConvertUTF32toUTF8(keyEvent->charCode, text)) {
+        SDL_SendKeyboardText(text);
+    }
     return 1;
     return 1;
 }
 }