From 26869b6df1d9b3492a19b4c5879c6e154633680b Mon Sep 17 00:00:00 2001 From: Daniel Baumann Date: Mon, 10 Sep 2018 10:55:40 +0200 Subject: Adding patch from upstream to fix a buffer overrun in zcat [CVE-2018-1000637] (Closes: #902936). Signed-off-by: Daniel Baumann --- debian/patches/series | 1 + debian/patches/upstream/0001-zcat-buffer-overrun.patch | 18 ++++++++++++++++++ 2 files changed, 19 insertions(+) create mode 100644 debian/patches/upstream/0001-zcat-buffer-overrun.patch diff --git a/debian/patches/series b/debian/patches/series index a353e88..fdb9b2d 100644 --- a/debian/patches/series +++ b/debian/patches/series @@ -1,2 +1,3 @@ debian/0001-build.patch debian/0002-zupdate.patch +upstream/0001-zcat-buffer-overrun.patch diff --git a/debian/patches/upstream/0001-zcat-buffer-overrun.patch b/debian/patches/upstream/0001-zcat-buffer-overrun.patch new file mode 100644 index 0000000..230987f --- /dev/null +++ b/debian/patches/upstream/0001-zcat-buffer-overrun.patch @@ -0,0 +1,18 @@ +Author: Antonio Diaz-Diaz +Description: zcat.cc: Fixed a buffer overrun on outbuf when '-v' is used [CVE-2018-1000637] (Closes: #902936). + +diff -Naurp zutils.orig/zcat.cc zutils/zcat.cc +--- zutils.orig/zcat.cc ++++ zutils/zcat.cc +@@ -229,8 +229,9 @@ int cat( int infd, const int format_inde + enum { buffer_size = 4096 }; + // buffer with space for sentinel newline at the end + uint8_t * const inbuf = new uint8_t[buffer_size+1]; +- // buffer with space for character quoting and 255-digit line number +- uint8_t * const outbuf = new uint8_t[(4*buffer_size)+256]; ++ // buffer with space for character quoting, 255-digit line number and ++ // worst case flushing respect to inbuf. ++ uint8_t * const outbuf = new uint8_t[(5*buffer_size)+256]; + int retval = 0; + Children children; + if( !set_data_feeder( &infd, children, format_index ) ) retval = 1; -- cgit v1.2.3