|
 |
12208d |
From 8f70b086e6553b4d41aaff2c5fb4266859436626 Mon Sep 17 00:00:00 2001
|
|
|
837307 |
From: Thomas Bernard <miniupnp@free.fr>
|
|
|
837307 |
Date: Sun, 15 Nov 2020 17:02:51 +0100
|
|
|
12208d |
Subject: [PATCH] (CVE-2020-35521 CVE-2020-35522) enforce (configurable) memory
|
|
|
12208d |
limit in tiff2rgba
|
|
|
837307 |
|
|
|
837307 |
fixes #207
|
|
|
837307 |
fixes #209
|
|
|
12208d |
|
|
|
12208d |
(cherry picked from commit 98a254f5b92cea22f5436555ff7fceb12afee84d)
|
|
|
837307 |
---
|
|
|
837307 |
tools/tiff2rgba.c | 25 +++++++++++++++++++++++--
|
|
|
837307 |
1 file changed, 23 insertions(+), 2 deletions(-)
|
|
|
837307 |
|
|
|
837307 |
diff --git a/tools/tiff2rgba.c b/tools/tiff2rgba.c
|
|
|
12208d |
index 4de96aec..e6de2209 100644
|
|
|
837307 |
--- a/tools/tiff2rgba.c
|
|
|
837307 |
+++ b/tools/tiff2rgba.c
|
|
|
837307 |
@@ -55,6 +55,10 @@ uint32 rowsperstrip = (uint32) -1;
|
|
|
837307 |
int process_by_block = 0; /* default is whole image at once */
|
|
|
837307 |
int no_alpha = 0;
|
|
|
837307 |
int bigtiff_output = 0;
|
|
|
837307 |
+#define DEFAULT_MAX_MALLOC (256 * 1024 * 1024)
|
|
|
837307 |
+/* malloc size limit (in bytes)
|
|
|
837307 |
+ * disabled when set to 0 */
|
|
|
837307 |
+static tmsize_t maxMalloc = DEFAULT_MAX_MALLOC;
|
|
|
837307 |
|
|
|
837307 |
|
|
|
837307 |
static int tiffcvt(TIFF* in, TIFF* out);
|
|
|
837307 |
@@ -70,8 +74,11 @@ main(int argc, char* argv[])
|
|
|
837307 |
extern char *optarg;
|
|
|
837307 |
#endif
|
|
|
837307 |
|
|
|
837307 |
- while ((c = getopt(argc, argv, "c:r:t:bn8")) != -1)
|
|
|
837307 |
+ while ((c = getopt(argc, argv, "c:r:t:bn8M:")) != -1)
|
|
|
837307 |
switch (c) {
|
|
|
837307 |
+ case 'M':
|
|
|
837307 |
+ maxMalloc = (tmsize_t)strtoul(optarg, NULL, 0) << 20;
|
|
|
837307 |
+ break;
|
|
|
837307 |
case 'b':
|
|
|
837307 |
process_by_block = 1;
|
|
|
837307 |
break;
|
|
|
837307 |
@@ -397,6 +404,12 @@ cvt_whole_image( TIFF *in, TIFF *out )
|
|
|
837307 |
(unsigned long)width, (unsigned long)height);
|
|
|
837307 |
return 0;
|
|
|
837307 |
}
|
|
|
837307 |
+ if (maxMalloc != 0 && (tmsize_t)pixel_count * (tmsize_t)sizeof(uint32) > maxMalloc) {
|
|
|
837307 |
+ TIFFError(TIFFFileName(in),
|
|
|
837307 |
+ "Raster size " TIFF_UINT64_FORMAT " over memory limit (" TIFF_UINT64_FORMAT "), try -b option.",
|
|
|
837307 |
+ (uint64)pixel_count * sizeof(uint32), (uint64)maxMalloc);
|
|
|
837307 |
+ return 0;
|
|
|
837307 |
+ }
|
|
|
837307 |
|
|
|
837307 |
rowsperstrip = TIFFDefaultStripSize(out, rowsperstrip);
|
|
|
837307 |
TIFFSetField(out, TIFFTAG_ROWSPERSTRIP, rowsperstrip);
|
|
|
837307 |
@@ -522,6 +535,13 @@ tiffcvt(TIFF* in, TIFF* out)
|
|
|
837307 |
TIFFSetField(out, TIFFTAG_SOFTWARE, TIFFGetVersion());
|
|
|
837307 |
CopyField(TIFFTAG_DOCUMENTNAME, stringv);
|
|
|
837307 |
|
|
|
837307 |
+ if (maxMalloc != 0 && TIFFStripSize(in) > maxMalloc)
|
|
|
837307 |
+ {
|
|
|
837307 |
+ TIFFError(TIFFFileName(in),
|
|
|
837307 |
+ "Strip Size " TIFF_UINT64_FORMAT " over memory limit (" TIFF_UINT64_FORMAT ")",
|
|
|
837307 |
+ (uint64)TIFFStripSize(in), (uint64)maxMalloc);
|
|
|
837307 |
+ return 0;
|
|
|
837307 |
+ }
|
|
|
837307 |
if( process_by_block && TIFFIsTiled( in ) )
|
|
|
837307 |
return( cvt_by_tile( in, out ) );
|
|
|
837307 |
else if( process_by_block )
|
|
|
837307 |
@@ -531,7 +551,7 @@ tiffcvt(TIFF* in, TIFF* out)
|
|
|
837307 |
}
|
|
|
837307 |
|
|
|
837307 |
static char* stuff[] = {
|
|
|
837307 |
- "usage: tiff2rgba [-c comp] [-r rows] [-b] [-n] [-8] input... output",
|
|
|
837307 |
+ "usage: tiff2rgba [-c comp] [-r rows] [-b] [-n] [-8] [-M size] input... output",
|
|
|
837307 |
"where comp is one of the following compression algorithms:",
|
|
|
837307 |
" jpeg\t\tJPEG encoding",
|
|
|
837307 |
" zip\t\tZip/Deflate encoding",
|
|
|
837307 |
@@ -543,6 +563,7 @@ static char* stuff[] = {
|
|
|
837307 |
" -b (progress by block rather than as a whole image)",
|
|
|
837307 |
" -n don't emit alpha component.",
|
|
|
837307 |
" -8 write BigTIFF file instead of ClassicTIFF",
|
|
|
837307 |
+ " -M set the memory allocation limit in MiB. 0 to disable limit",
|
|
|
837307 |
NULL
|
|
|
837307 |
};
|
|
|
837307 |
|