]> de.git.xonotic.org Git - xonotic/darkplaces.git/blobdiff - keys.c
Fix crash in inputline.
[xonotic/darkplaces.git] / keys.c
diff --git a/keys.c b/keys.c
index 28f74a42b0dfcbb343ba094af647fdf1c2d89761..62db167516af5338caa42a311852266af01c0a71 100644 (file)
--- a/keys.c
+++ b/keys.c
@@ -23,6 +23,7 @@
 #include "quakedef.h"
 #include "cl_video.h"
 #include "utf8lib.h"
+#include "csprogs.h"
 
 cvar_t con_closeontoggleconsole = {CVAR_SAVE, "con_closeontoggleconsole","1", "allows toggleconsole binds to close the console as well; when set to 2, this even works when not at the start of the line in console input; when set to 3, this works even if the toggleconsole key is the color tag"};
 
@@ -51,6 +52,8 @@ static void Key_History_Init(void)
        qfile_t *historyfile;
        ConBuffer_Init(&history, HIST_TEXTSIZE, HIST_MAXLINES, zonemempool);
 
+// not necessary for mobile
+#ifndef DP_MOBILETOUCH
        historyfile = FS_OpenRealFile("darkplaces_history.txt", "rb", false); // rb to handle unix line endings on windows too
        if(historyfile)
        {
@@ -82,6 +85,7 @@ static void Key_History_Init(void)
 
                FS_Close(historyfile);
        }
+#endif
 
        history_line = -1;
 }
@@ -90,6 +94,8 @@ static void Key_History_Shutdown(void)
 {
        // TODO write history to a file
 
+// not necessary for mobile
+#ifndef DP_MOBILETOUCH
        qfile_t *historyfile = FS_OpenRealFile("darkplaces_history.txt", "w", false);
        if(historyfile)
        {
@@ -98,6 +104,7 @@ static void Key_History_Shutdown(void)
                        FS_Printf(historyfile, "%s\n", ConBuffer_GetLine(&history, i));
                FS_Close(historyfile);
        }
+#endif
 
        ConBuffer_Shutdown(&history);
 }
@@ -107,19 +114,21 @@ static void Key_History_Push(void)
        if(key_line[1]) // empty?
        if(strcmp(key_line, "]quit")) // putting these into the history just sucks
        if(strncmp(key_line, "]quit ", 6)) // putting these into the history just sucks
-               ConBuffer_AddLine(&history, key_line + 1, strlen(key_line) - 1, 0);
+       if(strcmp(key_line, "]rcon_password")) // putting these into the history just sucks
+       if(strncmp(key_line, "]rcon_password ", 15)) // putting these into the history just sucks
+               ConBuffer_AddLine(&history, key_line + 1, (int)strlen(key_line) - 1, 0);
        Con_Printf("%s\n", key_line); // don't mark empty lines as history
        history_line = -1;
        if (history_matchfound)
                history_matchfound = false;
 }
 
-qboolean Key_History_Get_foundCommand(void)
+static qboolean Key_History_Get_foundCommand(void)
 {
        if (!history_matchfound)
                return false;
        strlcpy(key_line + 1, ConBuffer_GetLine(&history, history_line), sizeof(key_line) - 1);
-       key_linepos = strlen(key_line);
+       key_linepos = (int)strlen(key_line);
        history_matchfound = false;
        return true;
 }
@@ -138,14 +147,14 @@ static void Key_History_Up(void)
                if(history_line != -1)
                {
                        strlcpy(key_line + 1, ConBuffer_GetLine(&history, history_line), sizeof(key_line) - 1);
-                       key_linepos = strlen(key_line);
+                       key_linepos = (int)strlen(key_line);
                }
        }
        else if(history_line > 0)
        {
                --history_line; // this also does -1 -> 0, so it is good
                strlcpy(key_line + 1, ConBuffer_GetLine(&history, history_line), sizeof(key_line) - 1);
-               key_linepos = strlen(key_line);
+               key_linepos = (int)strlen(key_line);
        }
 }
 
@@ -168,7 +177,7 @@ static void Key_History_Down(void)
                strlcpy(key_line + 1, history_savedline, sizeof(key_line) - 1);
        }
 
-       key_linepos = strlen(key_line);
+       key_linepos = (int)strlen(key_line);
 }
 
 static void Key_History_First(void)
@@ -180,7 +189,7 @@ static void Key_History_First(void)
        {
                history_line = 0;
                strlcpy(key_line + 1, ConBuffer_GetLine(&history, history_line), sizeof(key_line) - 1);
-               key_linepos = strlen(key_line);
+               key_linepos = (int)strlen(key_line);
        }
 }
 
@@ -193,7 +202,7 @@ static void Key_History_Last(void)
        {
                history_line = CONBUFFER_LINES_COUNT(&history) - 1;
                strlcpy(key_line + 1, ConBuffer_GetLine(&history, history_line), sizeof(key_line) - 1);
-               key_linepos = strlen(key_line);
+               key_linepos = (int)strlen(key_line);
        }
 }
 
@@ -201,7 +210,8 @@ static void Key_History_Find_Backwards(void)
 {
        int i;
        const char *partial = key_line + 1;
-       size_t digits = strlen(va("%i", HIST_MAXLINES));
+       char vabuf[1024];
+       size_t digits = strlen(va(vabuf, sizeof(vabuf), "%i", HIST_MAXLINES));
 
        if (history_line == -1) // editing the "new" line
                strlcpy(history_savedline, key_line + 1, sizeof(history_savedline));
@@ -219,7 +229,7 @@ static void Key_History_Find_Backwards(void)
        if (!*partial)
                partial = "*";
        else if (!( strchr(partial, '*') || strchr(partial, '?') )) // no pattern?
-               partial = va("*%s*", partial);
+               partial = va(vabuf, sizeof(vabuf), "*%s*", partial);
 
        for ( ; i >= 0; i--)
                if (matchpattern_with_separator(ConBuffer_GetLine(&history, i), partial, true, "", false))
@@ -235,7 +245,8 @@ static void Key_History_Find_Forwards(void)
 {
        int i;
        const char *partial = key_line + 1;
-       size_t digits = strlen(va("%i", HIST_MAXLINES));
+       char vabuf[1024];
+       size_t digits = strlen(va(vabuf, sizeof(vabuf), "%i", HIST_MAXLINES));
 
        if (history_line == -1) // editing the "new" line
                return;
@@ -250,7 +261,7 @@ static void Key_History_Find_Forwards(void)
        if (!*partial)
                partial = "*";
        else if (!( strchr(partial, '*') || strchr(partial, '?') )) // no pattern?
-               partial = va("*%s*", partial);
+               partial = va(vabuf, sizeof(vabuf), "*%s*", partial);
 
        for ( ; i < CONBUFFER_LINES_COUNT(&history); i++)
                if (matchpattern_with_separator(ConBuffer_GetLine(&history, i), partial, true, "", false))
@@ -266,13 +277,14 @@ static void Key_History_Find_All(void)
 {
        const char *partial = key_line + 1;
        int i, count = 0;
-       size_t digits = strlen(va("%i", HIST_MAXLINES));
+       char vabuf[1024];
+       size_t digits = strlen(va(vabuf, sizeof(vabuf), "%i", HIST_MAXLINES));
        Con_Printf("History commands containing \"%s\":\n", key_line + 1);
 
        if (!*partial)
                partial = "*";
        else if (!( strchr(partial, '*') || strchr(partial, '?') )) // no pattern?
-               partial = va("*%s*", partial);
+               partial = va(vabuf, sizeof(vabuf), "*%s*", partial);
 
        for (i=0; i<CONBUFFER_LINES_COUNT(&history); i++)
                if (matchpattern_with_separator(ConBuffer_GetLine(&history, i), partial, true, "", false))
@@ -287,7 +299,8 @@ static void Key_History_f(void)
 {
        char *errchar = NULL;
        int i = 0;
-       size_t digits = strlen(va("%i", HIST_MAXLINES));
+       char vabuf[1024];
+       size_t digits = strlen(va(vabuf, sizeof(vabuf), "%i", HIST_MAXLINES));
 
        if (Cmd_Argc () > 1)
        {
@@ -838,12 +851,12 @@ Key_Console (int key, int unicode)
                        
                        // save the content of the variable in cvar_str
                        cvar_str = Cvar_VariableString(cvar);
-                       cvar_str_len = strlen(cvar_str);
+                       cvar_str_len = (int)strlen(cvar_str);
                        if (cvar_str_len==0)
                                return;
                        
                        // insert space and cvar_str in key_line
-                       chars_to_move = strlen(&key_line[key_linepos]);
+                       chars_to_move = (int)strlen(&key_line[key_linepos]);
                        if (key_linepos + 1 + cvar_str_len + chars_to_move < MAX_INPUTLINE)
                        {
                                if (chars_to_move)
@@ -901,7 +914,7 @@ Key_Console (int key, int unicode)
                {
                        int             pos;
                        size_t          inchar = 0;
-                       pos = u8_prevbyte(key_line+1, key_linepos-1) + 1; // do NOT give the ']' to u8_prevbyte
+                       pos = (int)u8_prevbyte(key_line+1, key_linepos-1) + 1; // do NOT give the ']' to u8_prevbyte
                        while (pos)
                                if(pos-1 > 0 && key_line[pos-1] == STRING_COLOR_TAG && isdigit(key_line[pos]))
                                        pos-=2;
@@ -917,11 +930,11 @@ Key_Console (int key, int unicode)
                                }
                        // we need to move to the beginning of the character when in a wide character:
                        u8_charidx(key_line, pos + 1, &inchar);
-                       key_linepos = pos + 1 - inchar;
+                       key_linepos = (int)(pos + 1 - inchar);
                }
                else
                {
-                       key_linepos = u8_prevbyte(key_line+1, key_linepos-1) + 1; // do NOT give the ']' to u8_prevbyte
+                       key_linepos = (int)u8_prevbyte(key_line+1, key_linepos-1) + 1; // do NOT give the ']' to u8_prevbyte
                }
                return;
        }
@@ -931,7 +944,7 @@ Key_Console (int key, int unicode)
        {
                if (key_linepos > 1)
                {
-                       int newpos = u8_prevbyte(key_line+1, key_linepos-1) + 1; // do NOT give the ']' to u8_prevbyte
+                       int newpos = (int)u8_prevbyte(key_line+1, key_linepos-1) + 1; // do NOT give the ']' to u8_prevbyte
                        strlcpy(key_line + newpos, key_line + key_linepos, sizeof(key_line) + 1 - key_linepos);
                        key_linepos = newpos;
                }
@@ -998,7 +1011,7 @@ Key_Console (int key, int unicode)
                        // skip the char
                        if (key_line[pos] == STRING_COLOR_TAG && key_line[pos+1] == STRING_COLOR_TAG) // consider ^^ as a character
                                pos++;
-                       pos += u8_bytelen(key_line + pos, 1);
+                       pos += (int)u8_bytelen(key_line + pos, 1);
                        
                        // now go beyond all next consecutive color tags, if any
                        if(pos < len)
@@ -1014,7 +1027,7 @@ Key_Console (int key, int unicode)
                        key_linepos = pos;
                }
                else
-                       key_linepos += u8_bytelen(key_line + key_linepos, 1);
+                       key_linepos += (int)u8_bytelen(key_line + key_linepos, 1);
                return;
        }
 
@@ -1176,9 +1189,17 @@ Key_Console (int key, int unicode)
                        // can't use strcpy to move string to right
                        len++;
                        //memmove(&key_line[key_linepos + u8_bytelen(key_line + key_linepos, 1)], &key_line[key_linepos], len);
+                       if (key_linepos + blen + len >= MAX_INPUTLINE)
+                               return;
                        memmove(&key_line[key_linepos + blen], &key_line[key_linepos], len);
                }
+               // FIXME: This is not proper overwriting with utf8.
+               if (key_linepos + blen >= MAX_INPUTLINE)
+                       return;
                memcpy(key_line + key_linepos, buf, blen);
+               if (blen > len)
+                       key_line[key_linepos + blen] = 0;
+               // END OF FIXME
                key_linepos += blen;
                //key_linepos += u8_fromchar(unicode, key_line + key_linepos, sizeof(key_line) - key_linepos - 1);
                //key_line[key_linepos] = ascii;
@@ -1192,17 +1213,16 @@ int chat_mode;
 char           chat_buffer[MAX_INPUTLINE];
 unsigned int   chat_bufferlen = 0;
 
-extern int Nicks_CompleteChatLine(char *buffer, size_t size, unsigned int pos);
-
 static void
 Key_Message (int key, int ascii)
 {
+       char vabuf[1024];
        if (key == K_ENTER || ascii == 10 || ascii == 13)
        {
                if(chat_mode < 0)
                        Cmd_ExecuteString(chat_buffer, src_command, true); // not Cbuf_AddText to allow semiclons in args; however, this allows no variables then. Use aliases!
                else
-                       Cmd_ForwardStringToServer(va("%s %s", chat_mode ? "say_team" : "say ", chat_buffer));
+                       Cmd_ForwardStringToServer(va(vabuf, sizeof(vabuf), "%s %s", chat_mode ? "say_team" : "say ", chat_buffer));
 
                key_dest = key_game;
                chat_bufferlen = 0;
@@ -1221,7 +1241,7 @@ Key_Message (int key, int ascii)
 
        if (key == K_BACKSPACE) {
                if (chat_bufferlen) {
-                       chat_bufferlen = u8_prevbyte(chat_buffer, chat_bufferlen);
+                       chat_bufferlen = (unsigned int)u8_prevbyte(chat_buffer, chat_bufferlen);
                        chat_buffer[chat_bufferlen] = 0;
                }
                return;
@@ -1283,10 +1303,9 @@ FIXME: handle quote special (general escape sequence?)
 ===================
 */
 const char *
-Key_KeynumToString (int keynum)
+Key_KeynumToString (int keynum, char *tinystr, size_t tinystrlength)
 {
        const keyname_t  *kn;
-       static char tinystr[2];
 
        // -1 is an invalid code
        if (keynum < 0)
@@ -1300,8 +1319,11 @@ Key_KeynumToString (int keynum)
        // if it is printable, output it as a single character
        if (keynum > 32 && keynum < 256)
        {
-               tinystr[0] = keynum;
-               tinystr[1] = 0;
+               if (tinystrlength >= 2)
+               {
+                       tinystr[0] = keynum;
+                       tinystr[1] = 0;
+               }
                return tinystr;
        }
 
@@ -1494,6 +1516,7 @@ static void
 Key_PrintBindList(int j)
 {
        char bindbuf[MAX_INPUTLINE];
+       char tinystr[2];
        const char *p;
        int i;
 
@@ -1504,9 +1527,9 @@ Key_PrintBindList(int j)
                {
                        Cmd_QuoteString(bindbuf, sizeof(bindbuf), p, "\"\\", false);
                        if (j == 0)
-                               Con_Printf("^2%s ^7= \"%s\"\n", Key_KeynumToString (i), bindbuf);
+                               Con_Printf("^2%s ^7= \"%s\"\n", Key_KeynumToString (i, tinystr, sizeof(tinystr)), bindbuf);
                        else
-                               Con_Printf("^3bindmap %d: ^2%s ^7= \"%s\"\n", j, Key_KeynumToString (i), bindbuf);
+                               Con_Printf("^3bindmap %d: ^2%s ^7= \"%s\"\n", j, Key_KeynumToString (i, tinystr, sizeof(tinystr)), bindbuf);
                }
        }
 }
@@ -1586,6 +1609,7 @@ Key_WriteBindings (qfile_t *f)
 {
        int         i, j;
        char bindbuf[MAX_INPUTLINE];
+       char tinystr[2];
        const char *p;
 
        for (j = 0; j < MAX_BINDMAPS; j++)
@@ -1597,9 +1621,9 @@ Key_WriteBindings (qfile_t *f)
                        {
                                Cmd_QuoteString(bindbuf, sizeof(bindbuf), p, "\"\\", false); // don't need to escape $ because cvars are not expanded inside bind
                                if (j == 0)
-                                       FS_Printf(f, "bind %s \"%s\"\n", Key_KeynumToString (i), bindbuf);
+                                       FS_Printf(f, "bind %s \"%s\"\n", Key_KeynumToString (i, tinystr, sizeof(tinystr)), bindbuf);
                                else
-                                       FS_Printf(f, "in_bind %d %s \"%s\"\n", j, Key_KeynumToString (i), bindbuf);
+                                       FS_Printf(f, "in_bind %d %s \"%s\"\n", j, Key_KeynumToString (i, tinystr, sizeof(tinystr)), bindbuf);
                        }
                }
        }
@@ -1686,8 +1710,6 @@ void Key_FindKeysForCommand (const char *command, int *keys, int numkeys, int bi
        }
 }
 
-extern qboolean CL_VM_InputEvent (int eventtype, int x, int y);
-
 /*
 ===================
 Called by the system between frames for both key up and key down events
@@ -1741,6 +1763,7 @@ Key_Event (int key, int ascii, qboolean down)
        const char *bind;
        qboolean q;
        keydest_t keydest = key_dest;
+       char vabuf[1024];
 
        if (key < 0 || key >= MAX_KEYS)
                return;
@@ -1751,11 +1774,6 @@ Key_Event (int key, int ascii, qboolean down)
                return;
        }
 
-       if (ascii == 0x80 && utf8_enable.integer) // pressing AltGr-5 (or AltGr-e) and for some reason we get windows-1252 encoding?
-               ascii = 0x20AC; // we want the Euro currency sign
-               // TODO find out which vid_ drivers do it and fix it there
-               // but catching U+0080 here is no loss as that char is not useful anyway
-
        // get key binding
        bind = keybindings[key_bmap][key];
        if (!bind)
@@ -1766,7 +1784,7 @@ Key_Event (int key, int ascii, qboolean down)
 
        if(key_consoleactive)
                keydest = key_console;
-       
+
        if (down)
        {
                // increment key repeat count each time a down is received so that things
@@ -1790,7 +1808,7 @@ Key_Event (int key, int ascii, qboolean down)
 
        if(keydest == key_void)
                return;
-       
+
        // key_consoleactive is a flag not a key_dest because the console is a
        // high priority overlay ontop of the normal screen (designed as a safety
        // feature so that developers and users can rescue themselves from a bad
@@ -1833,7 +1851,9 @@ Key_Event (int key, int ascii, qboolean down)
                                        if(key_consoleactive & KEY_CONSOLEACTIVE_FORCED)
                                        {
                                                key_consoleactive &= ~KEY_CONSOLEACTIVE_USER;
+#ifdef CONFIG_MENU
                                                MR_ToggleMenu(1);
+#endif
                                        }
                                        else
                                                Con_ToggleConsole_f();
@@ -1847,14 +1867,18 @@ Key_Event (int key, int ascii, qboolean down)
 
                        case key_menu:
                        case key_menu_grabbed:
+#ifdef CONFIG_MENU
                                MR_KeyEvent (key, ascii, down);
+#endif
                                break;
 
                        case key_game:
                                // csqc has priority over toggle menu if it wants to (e.g. handling escape for UI stuff in-game.. :sick:)
                                q = CL_VM_InputEvent(down ? 0 : 1, key, ascii);
+#ifdef CONFIG_MENU
                                if (!q && down)
                                        MR_ToggleMenu(1);
+#endif
                                break;
 
                        default:
@@ -1874,14 +1898,14 @@ Key_Event (int key, int ascii, qboolean down)
                        {
                                // button commands add keynum as a parm
                                if (bind[0] == '+')
-                                       Cbuf_AddText (va("%s %i\n", bind, key));
+                                       Cbuf_AddText (va(vabuf, sizeof(vabuf), "%s %i\n", bind, key));
                                else
                                {
                                        Cbuf_AddText (bind);
                                        Cbuf_AddText ("\n");
                                }
                        } else if(bind[0] == '+' && !down && keydown[key] == 0)
-                               Cbuf_AddText(va("-%s %i\n", bind + 1, key));
+                               Cbuf_AddText(va(vabuf, sizeof(vabuf), "-%s %i\n", bind + 1, key));
                }
                return;
        }
@@ -1922,7 +1946,12 @@ Key_Event (int key, int ascii, qboolean down)
        if (cl_videoplaying)
        {
                if (gamemode == GAME_BLOODOMNICIDE) // menu controls key events
+#ifdef CONFIG_MENU
                        MR_KeyEvent(key, ascii, down);
+#else
+                       {
+                       }
+#endif
                else
                        CL_Video_KeyEvent (key, ascii, keydown[key] != 0);
                return;
@@ -1937,7 +1966,9 @@ Key_Event (int key, int ascii, qboolean down)
                        break;
                case key_menu:
                case key_menu_grabbed:
+#ifdef CONFIG_MENU
                        MR_KeyEvent (key, ascii, down);
+#endif
                        break;
                case key_game:
                        q = CL_VM_InputEvent(down ? 0 : 1, key, ascii);
@@ -1948,14 +1979,14 @@ Key_Event (int key, int ascii, qboolean down)
                                {
                                        // button commands add keynum as a parm
                                        if (bind[0] == '+')
-                                               Cbuf_AddText (va("%s %i\n", bind, key));
+                                               Cbuf_AddText (va(vabuf, sizeof(vabuf), "%s %i\n", bind, key));
                                        else
                                        {
                                                Cbuf_AddText (bind);
                                                Cbuf_AddText ("\n");
                                        }
                                } else if(bind[0] == '+' && !down && keydown[key] == 0)
-                                       Cbuf_AddText(va("-%s %i\n", bind + 1, key));
+                                       Cbuf_AddText(va(vabuf, sizeof(vabuf), "-%s %i\n", bind + 1, key));
                        }
                        break;
                default: