diff options
| -rw-r--r-- | meta/recipes-multimedia/libtiff/files/0001-fix-the-FPE-in-tiffcrop-415-427-and-428.patch | 184 | ||||
| -rw-r--r-- | meta/recipes-multimedia/libtiff/tiff_4.4.0.bb | 3 |
2 files changed, 186 insertions, 1 deletions
diff --git a/meta/recipes-multimedia/libtiff/files/0001-fix-the-FPE-in-tiffcrop-415-427-and-428.patch b/meta/recipes-multimedia/libtiff/files/0001-fix-the-FPE-in-tiffcrop-415-427-and-428.patch new file mode 100644 index 0000000000..c7c5f616ed --- /dev/null +++ b/meta/recipes-multimedia/libtiff/files/0001-fix-the-FPE-in-tiffcrop-415-427-and-428.patch | |||
| @@ -0,0 +1,184 @@ | |||
| 1 | CVE: CVE-2022-2056 CVE-2022-2057 CVE-2022-2058 | ||
| 2 | Upstream-Status: Backport | ||
| 3 | Signed-off-by: Ross Burton <ross.burton@arm.com> | ||
| 4 | |||
| 5 | From 22a205da86ca2d038d0066e1d70752d117258fb4 Mon Sep 17 00:00:00 2001 | ||
| 6 | From: 4ugustus <wangdw.augustus@qq.com> | ||
| 7 | Date: Sat, 11 Jun 2022 09:31:43 +0000 | ||
| 8 | Subject: [PATCH] fix the FPE in tiffcrop (#415, #427, and #428) | ||
| 9 | |||
| 10 | --- | ||
| 11 | libtiff/tif_aux.c | 9 +++++++ | ||
| 12 | libtiff/tiffiop.h | 1 + | ||
| 13 | tools/tiffcrop.c | 62 ++++++++++++++++++++++++++--------------------- | ||
| 14 | 3 files changed, 44 insertions(+), 28 deletions(-) | ||
| 15 | |||
| 16 | diff --git a/libtiff/tif_aux.c b/libtiff/tif_aux.c | ||
| 17 | index 140f26c7..5b88c8d0 100644 | ||
| 18 | --- a/libtiff/tif_aux.c | ||
| 19 | +++ b/libtiff/tif_aux.c | ||
| 20 | @@ -402,6 +402,15 @@ float _TIFFClampDoubleToFloat( double val ) | ||
| 21 | return (float)val; | ||
| 22 | } | ||
| 23 | |||
| 24 | +uint32_t _TIFFClampDoubleToUInt32(double val) | ||
| 25 | +{ | ||
| 26 | + if( val < 0 ) | ||
| 27 | + return 0; | ||
| 28 | + if( val > 0xFFFFFFFFU || val != val ) | ||
| 29 | + return 0xFFFFFFFFU; | ||
| 30 | + return (uint32_t)val; | ||
| 31 | +} | ||
| 32 | + | ||
| 33 | int _TIFFSeekOK(TIFF* tif, toff_t off) | ||
| 34 | { | ||
| 35 | /* Huge offsets, especially -1 / UINT64_MAX, can cause issues */ | ||
| 36 | diff --git a/libtiff/tiffiop.h b/libtiff/tiffiop.h | ||
| 37 | index e3af461d..4e8bdac2 100644 | ||
| 38 | --- a/libtiff/tiffiop.h | ||
| 39 | +++ b/libtiff/tiffiop.h | ||
| 40 | @@ -365,6 +365,7 @@ extern double _TIFFUInt64ToDouble(uint64_t); | ||
| 41 | extern float _TIFFUInt64ToFloat(uint64_t); | ||
| 42 | |||
| 43 | extern float _TIFFClampDoubleToFloat(double); | ||
| 44 | +extern uint32_t _TIFFClampDoubleToUInt32(double); | ||
| 45 | |||
| 46 | extern tmsize_t | ||
| 47 | _TIFFReadEncodedStripAndAllocBuffer(TIFF* tif, uint32_t strip, | ||
| 48 | diff --git a/tools/tiffcrop.c b/tools/tiffcrop.c | ||
| 49 | index 1f827b2b..90286a5e 100644 | ||
| 50 | --- a/tools/tiffcrop.c | ||
| 51 | +++ b/tools/tiffcrop.c | ||
| 52 | @@ -5268,17 +5268,17 @@ computeInputPixelOffsets(struct crop_mask *crop, struct image_data *image, | ||
| 53 | { | ||
| 54 | if ((crop->res_unit == RESUNIT_INCH) || (crop->res_unit == RESUNIT_CENTIMETER)) | ||
| 55 | { | ||
| 56 | - x1 = (uint32_t) (crop->corners[i].X1 * scale * xres); | ||
| 57 | - x2 = (uint32_t) (crop->corners[i].X2 * scale * xres); | ||
| 58 | - y1 = (uint32_t) (crop->corners[i].Y1 * scale * yres); | ||
| 59 | - y2 = (uint32_t) (crop->corners[i].Y2 * scale * yres); | ||
| 60 | + x1 = _TIFFClampDoubleToUInt32(crop->corners[i].X1 * scale * xres); | ||
| 61 | + x2 = _TIFFClampDoubleToUInt32(crop->corners[i].X2 * scale * xres); | ||
| 62 | + y1 = _TIFFClampDoubleToUInt32(crop->corners[i].Y1 * scale * yres); | ||
| 63 | + y2 = _TIFFClampDoubleToUInt32(crop->corners[i].Y2 * scale * yres); | ||
| 64 | } | ||
| 65 | else | ||
| 66 | { | ||
| 67 | - x1 = (uint32_t) (crop->corners[i].X1); | ||
| 68 | - x2 = (uint32_t) (crop->corners[i].X2); | ||
| 69 | - y1 = (uint32_t) (crop->corners[i].Y1); | ||
| 70 | - y2 = (uint32_t) (crop->corners[i].Y2); | ||
| 71 | + x1 = _TIFFClampDoubleToUInt32(crop->corners[i].X1); | ||
| 72 | + x2 = _TIFFClampDoubleToUInt32(crop->corners[i].X2); | ||
| 73 | + y1 = _TIFFClampDoubleToUInt32(crop->corners[i].Y1); | ||
| 74 | + y2 = _TIFFClampDoubleToUInt32(crop->corners[i].Y2); | ||
| 75 | } | ||
| 76 | /* a) Region needs to be within image sizes 0.. width-1; 0..length-1 | ||
| 77 | * b) Corners are expected to be submitted as top-left to bottom-right. | ||
| 78 | @@ -5357,17 +5357,17 @@ computeInputPixelOffsets(struct crop_mask *crop, struct image_data *image, | ||
| 79 | { | ||
| 80 | if (crop->res_unit != RESUNIT_INCH && crop->res_unit != RESUNIT_CENTIMETER) | ||
| 81 | { /* User has specified pixels as reference unit */ | ||
| 82 | - tmargin = (uint32_t)(crop->margins[0]); | ||
| 83 | - lmargin = (uint32_t)(crop->margins[1]); | ||
| 84 | - bmargin = (uint32_t)(crop->margins[2]); | ||
| 85 | - rmargin = (uint32_t)(crop->margins[3]); | ||
| 86 | + tmargin = _TIFFClampDoubleToUInt32(crop->margins[0]); | ||
| 87 | + lmargin = _TIFFClampDoubleToUInt32(crop->margins[1]); | ||
| 88 | + bmargin = _TIFFClampDoubleToUInt32(crop->margins[2]); | ||
| 89 | + rmargin = _TIFFClampDoubleToUInt32(crop->margins[3]); | ||
| 90 | } | ||
| 91 | else | ||
| 92 | { /* inches or centimeters specified */ | ||
| 93 | - tmargin = (uint32_t)(crop->margins[0] * scale * yres); | ||
| 94 | - lmargin = (uint32_t)(crop->margins[1] * scale * xres); | ||
| 95 | - bmargin = (uint32_t)(crop->margins[2] * scale * yres); | ||
| 96 | - rmargin = (uint32_t)(crop->margins[3] * scale * xres); | ||
| 97 | + tmargin = _TIFFClampDoubleToUInt32(crop->margins[0] * scale * yres); | ||
| 98 | + lmargin = _TIFFClampDoubleToUInt32(crop->margins[1] * scale * xres); | ||
| 99 | + bmargin = _TIFFClampDoubleToUInt32(crop->margins[2] * scale * yres); | ||
| 100 | + rmargin = _TIFFClampDoubleToUInt32(crop->margins[3] * scale * xres); | ||
| 101 | } | ||
| 102 | |||
| 103 | if ((lmargin + rmargin) > image->width) | ||
| 104 | @@ -5397,24 +5397,24 @@ computeInputPixelOffsets(struct crop_mask *crop, struct image_data *image, | ||
| 105 | if (crop->res_unit != RESUNIT_INCH && crop->res_unit != RESUNIT_CENTIMETER) | ||
| 106 | { | ||
| 107 | if (crop->crop_mode & CROP_WIDTH) | ||
| 108 | - width = (uint32_t)crop->width; | ||
| 109 | + width = _TIFFClampDoubleToUInt32(crop->width); | ||
| 110 | else | ||
| 111 | width = image->width - lmargin - rmargin; | ||
| 112 | |||
| 113 | if (crop->crop_mode & CROP_LENGTH) | ||
| 114 | - length = (uint32_t)crop->length; | ||
| 115 | + length = _TIFFClampDoubleToUInt32(crop->length); | ||
| 116 | else | ||
| 117 | length = image->length - tmargin - bmargin; | ||
| 118 | } | ||
| 119 | else | ||
| 120 | { | ||
| 121 | if (crop->crop_mode & CROP_WIDTH) | ||
| 122 | - width = (uint32_t)(crop->width * scale * image->xres); | ||
| 123 | + width = _TIFFClampDoubleToUInt32(crop->width * scale * image->xres); | ||
| 124 | else | ||
| 125 | width = image->width - lmargin - rmargin; | ||
| 126 | |||
| 127 | if (crop->crop_mode & CROP_LENGTH) | ||
| 128 | - length = (uint32_t)(crop->length * scale * image->yres); | ||
| 129 | + length = _TIFFClampDoubleToUInt32(crop->length * scale * image->yres); | ||
| 130 | else | ||
| 131 | length = image->length - tmargin - bmargin; | ||
| 132 | } | ||
| 133 | @@ -5868,13 +5868,13 @@ computeOutputPixelOffsets (struct crop_mask *crop, struct image_data *image, | ||
| 134 | { | ||
| 135 | if (page->res_unit == RESUNIT_INCH || page->res_unit == RESUNIT_CENTIMETER) | ||
| 136 | { /* inches or centimeters specified */ | ||
| 137 | - hmargin = (uint32_t)(page->hmargin * scale * page->hres * ((image->bps + 7) / 8)); | ||
| 138 | - vmargin = (uint32_t)(page->vmargin * scale * page->vres * ((image->bps + 7) / 8)); | ||
| 139 | + hmargin = _TIFFClampDoubleToUInt32(page->hmargin * scale * page->hres * ((image->bps + 7) / 8)); | ||
| 140 | + vmargin = _TIFFClampDoubleToUInt32(page->vmargin * scale * page->vres * ((image->bps + 7) / 8)); | ||
| 141 | } | ||
| 142 | else | ||
| 143 | { /* Otherwise user has specified pixels as reference unit */ | ||
| 144 | - hmargin = (uint32_t)(page->hmargin * scale * ((image->bps + 7) / 8)); | ||
| 145 | - vmargin = (uint32_t)(page->vmargin * scale * ((image->bps + 7) / 8)); | ||
| 146 | + hmargin = _TIFFClampDoubleToUInt32(page->hmargin * scale * ((image->bps + 7) / 8)); | ||
| 147 | + vmargin = _TIFFClampDoubleToUInt32(page->vmargin * scale * ((image->bps + 7) / 8)); | ||
| 148 | } | ||
| 149 | |||
| 150 | if ((hmargin * 2.0) > (pwidth * page->hres)) | ||
| 151 | @@ -5912,13 +5912,13 @@ computeOutputPixelOffsets (struct crop_mask *crop, struct image_data *image, | ||
| 152 | { | ||
| 153 | if (page->mode & PAGE_MODE_PAPERSIZE ) | ||
| 154 | { | ||
| 155 | - owidth = (uint32_t)((pwidth * page->hres) - (hmargin * 2)); | ||
| 156 | - olength = (uint32_t)((plength * page->vres) - (vmargin * 2)); | ||
| 157 | + owidth = _TIFFClampDoubleToUInt32((pwidth * page->hres) - (hmargin * 2)); | ||
| 158 | + olength = _TIFFClampDoubleToUInt32((plength * page->vres) - (vmargin * 2)); | ||
| 159 | } | ||
| 160 | else | ||
| 161 | { | ||
| 162 | - owidth = (uint32_t)(iwidth - (hmargin * 2 * page->hres)); | ||
| 163 | - olength = (uint32_t)(ilength - (vmargin * 2 * page->vres)); | ||
| 164 | + owidth = _TIFFClampDoubleToUInt32(iwidth - (hmargin * 2 * page->hres)); | ||
| 165 | + olength = _TIFFClampDoubleToUInt32(ilength - (vmargin * 2 * page->vres)); | ||
| 166 | } | ||
| 167 | } | ||
| 168 | |||
| 169 | @@ -5927,6 +5927,12 @@ computeOutputPixelOffsets (struct crop_mask *crop, struct image_data *image, | ||
| 170 | if (olength > ilength) | ||
| 171 | olength = ilength; | ||
| 172 | |||
| 173 | + if (owidth == 0 || olength == 0) | ||
| 174 | + { | ||
| 175 | + TIFFError("computeOutputPixelOffsets", "Integer overflow when calculating the number of pages"); | ||
| 176 | + exit(EXIT_FAILURE); | ||
| 177 | + } | ||
| 178 | + | ||
| 179 | /* Compute the number of pages required for Portrait or Landscape */ | ||
| 180 | switch (page->orient) | ||
| 181 | { | ||
| 182 | -- | ||
| 183 | 2.34.1 | ||
| 184 | |||
diff --git a/meta/recipes-multimedia/libtiff/tiff_4.4.0.bb b/meta/recipes-multimedia/libtiff/tiff_4.4.0.bb index c82965ffa1..0af956a8f0 100644 --- a/meta/recipes-multimedia/libtiff/tiff_4.4.0.bb +++ b/meta/recipes-multimedia/libtiff/tiff_4.4.0.bb | |||
| @@ -8,7 +8,8 @@ LIC_FILES_CHKSUM = "file://COPYRIGHT;md5=34da3db46fab7501992f9615d7e158cf" | |||
| 8 | 8 | ||
| 9 | CVE_PRODUCT = "libtiff" | 9 | CVE_PRODUCT = "libtiff" |
| 10 | 10 | ||
| 11 | SRC_URI = "http://download.osgeo.org/libtiff/tiff-${PV}.tar.gz" | 11 | SRC_URI = "http://download.osgeo.org/libtiff/tiff-${PV}.tar.gz \ |
| 12 | file://0001-fix-the-FPE-in-tiffcrop-415-427-and-428.patch" | ||
| 12 | 13 | ||
| 13 | SRC_URI[sha256sum] = "917223b37538959aca3b790d2d73aa6e626b688e02dcda272aec24c2f498abed" | 14 | SRC_URI[sha256sum] = "917223b37538959aca3b790d2d73aa6e626b688e02dcda272aec24c2f498abed" |
| 14 | 15 | ||
