mirror of
https://github.com/void-linux/void-packages.git
synced 2025-07-26 23:42:57 +02:00
unzip: fix CVE-2018-18384.
This commit is contained in:
parent
125f385aa0
commit
fd9adfd49b
3 changed files with 40 additions and 30 deletions
|
@ -1,28 +0,0 @@
|
||||||
From: "Steven M. Schweda" <sms@antinode.info>
|
|
||||||
Subject: Fix CVE-2014-9913, buffer overflow in unzip
|
|
||||||
Bug: https://sourceforge.net/p/infozip/bugs/27/
|
|
||||||
Bug-Debian: https://bugs.debian.org/847485
|
|
||||||
Bug-Ubuntu: https://launchpad.net/bugs/387350
|
|
||||||
|
|
||||||
--- list.c
|
|
||||||
+++ list.c
|
|
||||||
@@ -339,7 +339,18 @@
|
|
||||||
G.crec.compression_method == ENHDEFLATED) {
|
|
||||||
methbuf[5] = dtype[(G.crec.general_purpose_bit_flag>>1) & 3];
|
|
||||||
} else if (methnum >= NUM_METHODS) {
|
|
||||||
- sprintf(&methbuf[4], "%03u", G.crec.compression_method);
|
|
||||||
+ /* 2013-02-26 SMS.
|
|
||||||
+ * http://sourceforge.net/p/infozip/bugs/27/ CVE-2014-9913.
|
|
||||||
+ * Unexpectedly large compression methods overflow
|
|
||||||
+ * &methbuf[]. Use the old, three-digit decimal format
|
|
||||||
+ * for values which fit. Otherwise, sacrifice the
|
|
||||||
+ * colon, and use four-digit hexadecimal.
|
|
||||||
+ */
|
|
||||||
+ if (G.crec.compression_method <= 999) {
|
|
||||||
+ sprintf( &methbuf[ 4], "%03u", G.crec.compression_method);
|
|
||||||
+ } else {
|
|
||||||
+ sprintf( &methbuf[ 3], "%04X", G.crec.compression_method);
|
|
||||||
+ }
|
|
||||||
}
|
|
||||||
|
|
||||||
#if 0 /* GRR/Euro: add this? */
|
|
|
@ -1,10 +1,32 @@
|
||||||
$NetBSD: patch-list.c,v 1.1 2015/01/06 14:12:45 wiz Exp $
|
$NetBSD: patch-list.c,v 1.3 2019/07/15 14:08:03 nia Exp $
|
||||||
|
|
||||||
|
chunk 1:
|
||||||
|
CVE-2018-18384 fix from
|
||||||
|
https://sourceforge.net/p/infozip/bugs/53/
|
||||||
|
and
|
||||||
|
https://sources.debian.org/patches/unzip/6.0-24/07-increase-size-of-cfactorstr.patch/
|
||||||
|
|
||||||
|
chunk 2:
|
||||||
Big-hammer fix for
|
Big-hammer fix for
|
||||||
http://seclists.org/oss-sec/2014/q4/497
|
http://seclists.org/oss-sec/2014/q4/497
|
||||||
|
|
||||||
|
chunk 3:
|
||||||
|
CVE-2014-9913 fix from
|
||||||
|
https://people.debian.org/~sanvila/unzip/cve-2014-9913/cve-2014-9913-unzip-buffer-overflow.txt
|
||||||
|
via
|
||||||
|
http://www.info-zip.org/phpBB3/viewtopic.php?f=7&t=529
|
||||||
|
|
||||||
--- list.c.orig 2009-02-08 17:11:34.000000000 +0000
|
--- list.c.orig 2009-02-08 17:11:34.000000000 +0000
|
||||||
+++ list.c
|
+++ list.c
|
||||||
|
@@ -97,7 +97,7 @@ int list_files(__G) /* return PK-type
|
||||||
|
{
|
||||||
|
int do_this_file=FALSE, cfactor, error, error_in_archive=PK_COOL;
|
||||||
|
#ifndef WINDLL
|
||||||
|
- char sgn, cfactorstr[10];
|
||||||
|
+ char sgn, cfactorstr[12];
|
||||||
|
int longhdr=(uO.vflag>1);
|
||||||
|
#endif
|
||||||
|
int date_format;
|
||||||
@@ -116,7 +116,7 @@ int list_files(__G) /* return PK-type
|
@@ -116,7 +116,7 @@ int list_files(__G) /* return PK-type
|
||||||
ulg acl_size, tot_aclsize=0L, tot_aclfiles=0L;
|
ulg acl_size, tot_aclsize=0L, tot_aclfiles=0L;
|
||||||
#endif
|
#endif
|
||||||
|
@ -14,3 +36,19 @@ http://seclists.org/oss-sec/2014/q4/497
|
||||||
static ZCONST char dtype[]="NXFS"; /* see zi_short() */
|
static ZCONST char dtype[]="NXFS"; /* see zi_short() */
|
||||||
static ZCONST char Far method[NUM_METHODS+1][8] =
|
static ZCONST char Far method[NUM_METHODS+1][8] =
|
||||||
{"Stored", "Shrunk", "Reduce1", "Reduce2", "Reduce3", "Reduce4",
|
{"Stored", "Shrunk", "Reduce1", "Reduce2", "Reduce3", "Reduce4",
|
||||||
|
@@ -339,7 +339,14 @@ int list_files(__G) /* return PK-type
|
||||||
|
G.crec.compression_method == ENHDEFLATED) {
|
||||||
|
methbuf[5] = dtype[(G.crec.general_purpose_bit_flag>>1) & 3];
|
||||||
|
} else if (methnum >= NUM_METHODS) {
|
||||||
|
- sprintf(&methbuf[4], "%03u", G.crec.compression_method);
|
||||||
|
+ /* Fix for CVE-2014-9913, similar to CVE-2016-9844.
|
||||||
|
+ * Use the old decimal format only for values which fit.
|
||||||
|
+ */
|
||||||
|
+ if (G.crec.compression_method <= 999) {
|
||||||
|
+ sprintf( &methbuf[ 4], "%03u", G.crec.compression_method);
|
||||||
|
+ } else {
|
||||||
|
+ sprintf( &methbuf[ 3], "%04X", G.crec.compression_method);
|
||||||
|
+ }
|
||||||
|
}
|
||||||
|
|
||||||
|
#if 0 /* GRR/Euro: add this? */
|
||||||
|
|
|
@ -1,7 +1,7 @@
|
||||||
# Template file for 'unzip'
|
# Template file for 'unzip'
|
||||||
pkgname=unzip
|
pkgname=unzip
|
||||||
version=6.0
|
version=6.0
|
||||||
revision=12
|
revision=13
|
||||||
wrksrc=unzip60
|
wrksrc=unzip60
|
||||||
makedepends="bzip2-devel"
|
makedepends="bzip2-devel"
|
||||||
short_desc="List, test and extract compressed files in a ZIP archive"
|
short_desc="List, test and extract compressed files in a ZIP archive"
|
||||||
|
|
Loading…
Add table
Reference in a new issue