From 754137e70cf58a64ad524b704a86b651ba0cde07 Mon Sep 17 00:00:00 2001
From: Petr Stodulka <pstodulk@redhat.com>
Date: Wed, 14 Dec 2016 16:30:36 +0100
Subject: [PATCH] Fix CVE-2016-9844 (rhbz#1404283)

Fixes buffer overflow in zipinfo in similar way like fix for
CVE-2014-9913 provided by upstream.
---
 zipinfo.c | 14 +++++++++++++-
 1 file changed, 13 insertions(+), 1 deletion(-)

diff --git a/zipinfo.c b/zipinfo.c
index c03620e..accca2a 100644
--- a/zipinfo.c
+++ b/zipinfo.c
@@ -1984,7 +1984,19 @@ static int zi_short(__G)   /* return PK-type error code */
         ush  dnum=(ush)((G.crec.general_purpose_bit_flag>>1) & 3);
         methbuf[3] = dtype[dnum];
     } else if (methnum >= NUM_METHODS) {   /* unknown */
-        sprintf(&methbuf[1], "%03u", G.crec.compression_method);
+        /* 2016-12-05 SMS.
+         * https://launchpad.net/bugs/1643750
+         * Unexpectedly large compression methods overflow
+         * &methbuf[].  Use the old, three-digit decimal format
+         * for values which fit.  Otherwise, sacrifice the "u",
+         * and use four-digit hexadecimal.
+         */
+        if (G.crec.compression_method <= 999) {
+              sprintf( &methbuf[ 1], "%03u", G.crec.compression_method);
+        } else {
+              sprintf( &methbuf[ 0], "%04X", G.crec.compression_method);
+        }
+
     }
 
     for (k = 0;  k < 15;  ++k)
-- 
2.5.5

063297e6de84a9a2477150a6ce9416d8'>commit</a><a href='/guix/diff/gnu/packages/android.scm?id=4ace6455063297e6de84a9a2477150a6ce9416d8'>diff</a></td><td class='form'><form class='right' method='get' action='/guix/log/gnu/packages/android.scm'>
<input type='hidden' name='id' value='4ace6455063297e6de84a9a2477150a6ce9416d8'/><select name='qt'>
<option value='grep'>log msg</option>
<option value='author'>author</option>
<option value='committer'>committer</option>
<option value='range'>range</option>
</select>
<input class='txt' type='search' size='10' name='q' value=''/>
<input type='submit' value='search'/>
</form>
</td></tr></table>
<div class='path'>path: <a href='/guix/log/?id=4ace6455063297e6de84a9a2477150a6ce9416d8'>root</a>/<a href='/guix/log/gnu?id=4ace6455063297e6de84a9a2477150a6ce9416d8'>gnu</a>/<a href='/guix/log/gnu/packages?id=4ace6455063297e6de84a9a2477150a6ce9416d8'>packages</a>/<a href='/guix/log/gnu/packages/android.scm?id=4ace6455063297e6de84a9a2477150a6ce9416d8'>android.scm</a></div><div class='content'><table class='list nowrap'><tr class='nohover'><th class='left'>Age</th><th class='left'>Commit message (<a href='/guix/log/gnu/packages/android.scm?id=4ace6455063297e6de84a9a2477150a6ce9416d8&amp;showmsg=1'>Expand</a>)</th><th class='left'>Author</th></tr>
<tr><td><span title='2021-10-04 13:31:33 +0200'>2021-10-04</span></td><td><a href='/guix/commit/gnu/packages/android.scm?id=d5750d565f69ac78588f9ddb8709642e5f7de273'>gnu: android-udev-rules: Update to 20210501.</a><span class='msg-avail'>...</span></td><td>Tobias Geerinckx-Rice</td></tr>
<tr><td><span title='2021-09-25 09:34:29 +0200'>2021-09-25</span></td><td><a href='/guix/commit/gnu/packages/android.scm?id=a5b2474037d749f42a816c60f0393941ba97444e'>gnu: Consolidate duplicate copyright names.</a><span class='msg-avail'>...</span></td><td>Greg Hogan</td></tr>
<tr><td><span title='2021-07-27 12:34:57 -0400'>2021-07-27</span></td><td><a href='/guix/commit/gnu/packages/android.scm?id=05effbbfc2fc6223aafacf8a3cb2b2d970b6bb66'>gnu: adb: Make compatible with OpenSSL 1.1.</a><span class='msg-avail'>...</span></td><td>Giovanni Biscuolo</td></tr>