Merge pull request #242 from sureshsundriyal/coverity

[coverity] Avoid a buffer-overrun.
9 years ago · 14755f6f75
parent 87ec101f9c b4f2c1683a
commit 14755f6f75
1 changed files with 4 additions and 1 deletions
--- a/src/lnav.cc
+++ b/src/lnav.cc
@ -3863,7 +3863,10 @@ static void looper(void)
                    while ((ch = getch()) != ERR) {
                        alerter::singleton().new_input(ch);

-                        if (escape_index >= sizeof(escape_buffer) - 1) {
+                        /* Check to make sure there is enough space for a
+                         * character and a string terminator.
+                         */
+                        if (escape_index >= sizeof(escape_buffer) - 2) {
                            escape_index = 0;
                        }
                        else if (escape_index > 0) {