diff options
author | Denys Vlasenko <vda.linux@googlemail.com> | 2014-06-30 10:14:34 +0200 |
---|---|---|
committer | Denys Vlasenko <vda.linux@googlemail.com> | 2014-06-30 10:14:34 +0200 |
commit | a9dc7c2f59dc5e92870d2d46316ea5c1f14740e3 (patch) | |
tree | 0c3b2ff86a761578247db504d8e742d55071b4a8 | |
parent | 1b487ea8a69ac90b530e9ccd161a5b1b21e604c7 (diff) | |
download | busybox-w32-a9dc7c2f59dc5e92870d2d46316ea5c1f14740e3.tar.gz busybox-w32-a9dc7c2f59dc5e92870d2d46316ea5c1f14740e3.tar.bz2 busybox-w32-a9dc7c2f59dc5e92870d2d46316ea5c1f14740e3.zip |
lzop: add overflow check
See CVE-2014-4607
http://www.openwall.com/lists/oss-security/2014/06/26/20
function old new delta
lzo1x_decompress_safe 1010 1031 +21
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
-rw-r--r-- | archival/libarchive/liblzo.h | 2 | ||||
-rw-r--r-- | archival/libarchive/lzo1x_d.c | 3 |
2 files changed, 5 insertions, 0 deletions
diff --git a/archival/libarchive/liblzo.h b/archival/libarchive/liblzo.h index 843997cb9..4596620fe 100644 --- a/archival/libarchive/liblzo.h +++ b/archival/libarchive/liblzo.h | |||
@@ -76,11 +76,13 @@ | |||
76 | # define TEST_IP (ip < ip_end) | 76 | # define TEST_IP (ip < ip_end) |
77 | # define NEED_IP(x) \ | 77 | # define NEED_IP(x) \ |
78 | if ((unsigned)(ip_end - ip) < (unsigned)(x)) goto input_overrun | 78 | if ((unsigned)(ip_end - ip) < (unsigned)(x)) goto input_overrun |
79 | # define TEST_IV(x) if ((x) > (unsigned)0 - (511)) goto input_overrun | ||
79 | 80 | ||
80 | # undef TEST_OP /* don't need both of the tests here */ | 81 | # undef TEST_OP /* don't need both of the tests here */ |
81 | # define TEST_OP 1 | 82 | # define TEST_OP 1 |
82 | # define NEED_OP(x) \ | 83 | # define NEED_OP(x) \ |
83 | if ((unsigned)(op_end - op) < (unsigned)(x)) goto output_overrun | 84 | if ((unsigned)(op_end - op) < (unsigned)(x)) goto output_overrun |
85 | # define TEST_OV(x) if ((x) > (unsigned)0 - (511)) goto output_overrun | ||
84 | 86 | ||
85 | #define HAVE_ANY_OP 1 | 87 | #define HAVE_ANY_OP 1 |
86 | 88 | ||
diff --git a/archival/libarchive/lzo1x_d.c b/archival/libarchive/lzo1x_d.c index 9bc1270da..40b167e68 100644 --- a/archival/libarchive/lzo1x_d.c +++ b/archival/libarchive/lzo1x_d.c | |||
@@ -92,6 +92,7 @@ int lzo1x_decompress_safe(const uint8_t* in, unsigned in_len, | |||
92 | ip++; | 92 | ip++; |
93 | NEED_IP(1); | 93 | NEED_IP(1); |
94 | } | 94 | } |
95 | TEST_IV(t); | ||
95 | t += 15 + *ip++; | 96 | t += 15 + *ip++; |
96 | } | 97 | } |
97 | /* copy literals */ | 98 | /* copy literals */ |
@@ -224,6 +225,7 @@ int lzo1x_decompress_safe(const uint8_t* in, unsigned in_len, | |||
224 | ip++; | 225 | ip++; |
225 | NEED_IP(1); | 226 | NEED_IP(1); |
226 | } | 227 | } |
228 | TEST_IV(t); | ||
227 | t += 31 + *ip++; | 229 | t += 31 + *ip++; |
228 | } | 230 | } |
229 | #if defined(COPY_DICT) | 231 | #if defined(COPY_DICT) |
@@ -265,6 +267,7 @@ int lzo1x_decompress_safe(const uint8_t* in, unsigned in_len, | |||
265 | ip++; | 267 | ip++; |
266 | NEED_IP(1); | 268 | NEED_IP(1); |
267 | } | 269 | } |
270 | TEST_IV(t); | ||
268 | t += 7 + *ip++; | 271 | t += 7 + *ip++; |
269 | } | 272 | } |
270 | #if defined(COPY_DICT) | 273 | #if defined(COPY_DICT) |