package/tmux: add upstream security fix for CVE-2020-27347
authorPeter Korsgaard <peter@korsgaard.com>
Thu, 12 Nov 2020 09:21:45 +0000 (10:21 +0100)
committerPeter Korsgaard <peter@korsgaard.com>
Thu, 12 Nov 2020 20:49:21 +0000 (21:49 +0100)
Fixes CVE-2020-27347: The function input_csi_dispatch_sgr_colon() in file
input.c contained a stack-based buffer-overflow that can be exploited by
terminal output.

For details, see:
https://www.openwall.com/lists/oss-security/2020/11/05/3

Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
package/tmux/0001-Do-not-write-after-the-end-of-the-array-and-overwrit.patch [new file with mode: 0644]
package/tmux/tmux.mk

diff --git a/package/tmux/0001-Do-not-write-after-the-end-of-the-array-and-overwrit.patch b/package/tmux/0001-Do-not-write-after-the-end-of-the-array-and-overwrit.patch
new file mode 100644 (file)
index 0000000..d169322
--- /dev/null
@@ -0,0 +1,35 @@
+From a868bacb46e3c900530bed47a1c6f85b0fbe701c Mon Sep 17 00:00:00 2001
+From: nicm <nicm>
+Date: Thu, 29 Oct 2020 16:33:01 +0000
+Subject: [PATCH] Do not write after the end of the array and overwrite the
+ stack when colon-separated SGR sequences contain empty arguments. Reported by
+ Sergey Nizovtsev.
+
+[Peter: Fixes CVE-2020-27347]
+Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
+---
+ input.c | 7 ++++++-
+ 1 file changed, 6 insertions(+), 1 deletion(-)
+
+diff --git a/input.c b/input.c
+index 42a60c92..c280c0d9 100644
+--- a/input.c
++++ b/input.c
+@@ -1976,8 +1976,13 @@ input_csi_dispatch_sgr_colon(struct input_ctx *ictx, u_int i)
+                               free(copy);
+                               return;
+                       }
+-              } else
++              } else {
+                       n++;
++                      if (n == nitems(p)) {
++                              free(copy);
++                              return;
++                      }
++              }
+               log_debug("%s: %u = %d", __func__, n - 1, p[n - 1]);
+       }
+       free(copy);
+-- 
+2.20.1
+
index 169c9bb83fdd8f60b5349434ef2f71b27df982be..00e77ad7627c00dcb8bb1133d73ffb4affc618d0 100644 (file)
@@ -10,6 +10,9 @@ TMUX_LICENSE = ISC
 TMUX_LICENSE_FILES = COPYING
 TMUX_DEPENDENCIES = libevent ncurses host-pkgconf
 
+# 0001-Do-not-write-after-the-end-of-the-array-and-overwrit.patch
+TMUX_IGNORE_CVES += CVE-2020-27347
+
 # Add /usr/bin/tmux to /etc/shells otherwise some login tools like dropbear
 # can reject the user connection. See man shells.
 define TMUX_ADD_TMUX_TO_SHELLS