diff options
5 files changed, 19 insertions, 236 deletions
diff --git a/meta/recipes-support/libexif/libexif/CVE-2016-6328.patch b/meta/recipes-support/libexif/libexif/CVE-2016-6328.patch deleted file mode 100644 index a6f307439b..0000000000 --- a/meta/recipes-support/libexif/libexif/CVE-2016-6328.patch +++ /dev/null | |||
| @@ -1,64 +0,0 @@ | |||
| 1 | CVE: CVE-2016-6328 | ||
| 2 | Upstream-Status: Backport | ||
| 3 | Signed-off-by: Ross Burton <ross.burton@intel.com> | ||
| 4 | |||
| 5 | From 41bd04234b104312f54d25822f68738ba8d7133d Mon Sep 17 00:00:00 2001 | ||
| 6 | From: Marcus Meissner <marcus@jet.franken.de> | ||
| 7 | Date: Tue, 25 Jul 2017 23:44:44 +0200 | ||
| 8 | Subject: [PATCH] fixes some (not all) buffer overreads during decoding pentax | ||
| 9 | makernote entries. | ||
| 10 | |||
| 11 | This should fix: | ||
| 12 | https://sourceforge.net/p/libexif/bugs/125/ CVE-2016-6328 | ||
| 13 | --- | ||
| 14 | libexif/pentax/mnote-pentax-entry.c | 16 +++++++++++++--- | ||
| 15 | 1 file changed, 13 insertions(+), 3 deletions(-) | ||
| 16 | |||
| 17 | diff --git a/libexif/pentax/mnote-pentax-entry.c b/libexif/pentax/mnote-pentax-entry.c | ||
| 18 | index d03d159..ea0429a 100644 | ||
| 19 | --- a/libexif/pentax/mnote-pentax-entry.c | ||
| 20 | +++ b/libexif/pentax/mnote-pentax-entry.c | ||
| 21 | @@ -425,24 +425,34 @@ mnote_pentax_entry_get_value (MnotePentaxEntry *entry, | ||
| 22 | case EXIF_FORMAT_SHORT: | ||
| 23 | { | ||
| 24 | const unsigned char *data = entry->data; | ||
| 25 | - size_t k, len = strlen(val); | ||
| 26 | + size_t k, len = strlen(val), sizeleft; | ||
| 27 | + | ||
| 28 | + sizeleft = entry->size; | ||
| 29 | for(k=0; k<entry->components; k++) { | ||
| 30 | + if (sizeleft < 2) | ||
| 31 | + break; | ||
| 32 | vs = exif_get_short (data, entry->order); | ||
| 33 | snprintf (val+len, maxlen-len, "%i ", vs); | ||
| 34 | len = strlen(val); | ||
| 35 | data += 2; | ||
| 36 | + sizeleft -= 2; | ||
| 37 | } | ||
| 38 | } | ||
| 39 | break; | ||
| 40 | case EXIF_FORMAT_LONG: | ||
| 41 | { | ||
| 42 | const unsigned char *data = entry->data; | ||
| 43 | - size_t k, len = strlen(val); | ||
| 44 | + size_t k, len = strlen(val), sizeleft; | ||
| 45 | + | ||
| 46 | + sizeleft = entry->size; | ||
| 47 | for(k=0; k<entry->components; k++) { | ||
| 48 | + if (sizeleft < 4) | ||
| 49 | + break; | ||
| 50 | vl = exif_get_long (data, entry->order); | ||
| 51 | snprintf (val+len, maxlen-len, "%li", (long int) vl); | ||
| 52 | len = strlen(val); | ||
| 53 | data += 4; | ||
| 54 | + sizeleft -= 4; | ||
| 55 | } | ||
| 56 | } | ||
| 57 | break; | ||
| 58 | @@ -455,5 +465,5 @@ mnote_pentax_entry_get_value (MnotePentaxEntry *entry, | ||
| 59 | break; | ||
| 60 | } | ||
| 61 | |||
| 62 | - return (val); | ||
| 63 | + return val; | ||
| 64 | } | ||
diff --git a/meta/recipes-support/libexif/libexif/CVE-2017-7544.patch b/meta/recipes-support/libexif/libexif/CVE-2017-7544.patch deleted file mode 100644 index e49481ff84..0000000000 --- a/meta/recipes-support/libexif/libexif/CVE-2017-7544.patch +++ /dev/null | |||
| @@ -1,40 +0,0 @@ | |||
| 1 | From 8a92f964a66d476ca8907234359e92a70fc1325b Mon Sep 17 00:00:00 2001 | ||
| 2 | From: Changqing Li <changqing.li@windriver.com> | ||
| 3 | Date: Tue, 28 Aug 2018 15:12:10 +0800 | ||
| 4 | Subject: [PATCH] On saving makernotes, make sure the makernote container tags | ||
| 5 | has a type with 1 byte components. | ||
| 6 | |||
| 7 | Fixes (at least): | ||
| 8 | https://sourceforge.net/p/libexif/bugs/130 | ||
| 9 | https://sourceforge.net/p/libexif/bugs/129 | ||
| 10 | |||
| 11 | Upstream-Status: Backport[https://github.com/libexif/libexif/commit/ | ||
| 12 | c39acd1692023b26290778a02a9232c873f9d71a#diff-830e348923810f00726700b083ec00cd] | ||
| 13 | |||
| 14 | CVE: CVE-2017-7544 | ||
| 15 | |||
| 16 | Signed-off-by: Changqing Li <changqing.li@windriver.com> | ||
| 17 | --- | ||
| 18 | libexif/exif-data.c | 6 ++++++ | ||
| 19 | 1 file changed, 6 insertions(+) | ||
| 20 | |||
| 21 | diff --git a/libexif/exif-data.c b/libexif/exif-data.c | ||
| 22 | index 67df4db..6bf89eb 100644 | ||
| 23 | --- a/libexif/exif-data.c | ||
| 24 | +++ b/libexif/exif-data.c | ||
| 25 | @@ -255,6 +255,12 @@ exif_data_save_data_entry (ExifData *data, ExifEntry *e, | ||
| 26 | exif_mnote_data_set_offset (data->priv->md, *ds - 6); | ||
| 27 | exif_mnote_data_save (data->priv->md, &e->data, &e->size); | ||
| 28 | e->components = e->size; | ||
| 29 | + if (exif_format_get_size (e->format) != 1) { | ||
| 30 | + /* e->format is taken from input code, | ||
| 31 | + * but we need to make sure it is a 1 byte | ||
| 32 | + * entity due to the multiplication below. */ | ||
| 33 | + e->format = EXIF_FORMAT_UNDEFINED; | ||
| 34 | + } | ||
| 35 | } | ||
| 36 | } | ||
| 37 | |||
| 38 | -- | ||
| 39 | 2.7.4 | ||
| 40 | |||
diff --git a/meta/recipes-support/libexif/libexif/CVE-2018-20030.patch b/meta/recipes-support/libexif/libexif/CVE-2018-20030.patch deleted file mode 100644 index 76233e6dc9..0000000000 --- a/meta/recipes-support/libexif/libexif/CVE-2018-20030.patch +++ /dev/null | |||
| @@ -1,115 +0,0 @@ | |||
| 1 | CVE: CVE-2018-20030 | ||
| 2 | Upstream-Status: Backport | ||
| 3 | Signed-off-by: Ross Burton <ross.burton@intel.com> | ||
| 4 | |||
| 5 | From 6aa11df549114ebda520dde4cdaea2f9357b2c89 Mon Sep 17 00:00:00 2001 | ||
| 6 | From: Dan Fandrich <dan@coneharvesters.com> | ||
| 7 | Date: Fri, 12 Oct 2018 16:01:45 +0200 | ||
| 8 | Subject: [PATCH] Improve deep recursion detection in | ||
| 9 | exif_data_load_data_content. | ||
| 10 | |||
| 11 | The existing detection was still vulnerable to pathological cases | ||
| 12 | causing DoS by wasting CPU. The new algorithm takes the number of tags | ||
| 13 | into account to make it harder to abuse by cases using shallow recursion | ||
| 14 | but with a very large number of tags. This improves on commit 5d28011c | ||
| 15 | which wasn't sufficient to counter this kind of case. | ||
| 16 | |||
| 17 | The limitation in the previous fix was discovered by Laurent Delosieres, | ||
| 18 | Secunia Research at Flexera (Secunia Advisory SA84652) and is assigned | ||
| 19 | the identifier CVE-2018-20030. | ||
| 20 | |||
| 21 | diff --git a/libexif/exif-data.c b/libexif/exif-data.c | ||
| 22 | index 67df4db..8d9897e 100644 | ||
| 23 | --- a/libexif/exif-data.c | ||
| 24 | +++ b/libexif/exif-data.c | ||
| 25 | @@ -35,6 +35,7 @@ | ||
| 26 | #include <libexif/olympus/exif-mnote-data-olympus.h> | ||
| 27 | #include <libexif/pentax/exif-mnote-data-pentax.h> | ||
| 28 | |||
| 29 | +#include <math.h> | ||
| 30 | #include <stdlib.h> | ||
| 31 | #include <stdio.h> | ||
| 32 | #include <string.h> | ||
| 33 | @@ -344,6 +345,20 @@ if (data->ifd[(i)]->count) { \ | ||
| 34 | break; \ | ||
| 35 | } | ||
| 36 | |||
| 37 | +/*! Calculate the recursion cost added by one level of IFD loading. | ||
| 38 | + * | ||
| 39 | + * The work performed is related to the cost in the exponential relation | ||
| 40 | + * work=1.1**cost | ||
| 41 | + */ | ||
| 42 | +static unsigned int | ||
| 43 | +level_cost(unsigned int n) | ||
| 44 | +{ | ||
| 45 | + static const double log_1_1 = 0.09531017980432493; | ||
| 46 | + | ||
| 47 | + /* Adding 0.1 protects against the case where n==1 */ | ||
| 48 | + return ceil(log(n + 0.1)/log_1_1); | ||
| 49 | +} | ||
| 50 | + | ||
| 51 | /*! Load data for an IFD. | ||
| 52 | * | ||
| 53 | * \param[in,out] data #ExifData | ||
| 54 | @@ -351,13 +366,13 @@ if (data->ifd[(i)]->count) { \ | ||
| 55 | * \param[in] d pointer to buffer containing raw IFD data | ||
| 56 | * \param[in] ds size of raw data in buffer at \c d | ||
| 57 | * \param[in] offset offset into buffer at \c d at which IFD starts | ||
| 58 | - * \param[in] recursion_depth number of times this function has been | ||
| 59 | - * recursively called without returning | ||
| 60 | + * \param[in] recursion_cost factor indicating how expensive this recursive | ||
| 61 | + * call could be | ||
| 62 | */ | ||
| 63 | static void | ||
| 64 | exif_data_load_data_content (ExifData *data, ExifIfd ifd, | ||
| 65 | const unsigned char *d, | ||
| 66 | - unsigned int ds, unsigned int offset, unsigned int recursion_depth) | ||
| 67 | + unsigned int ds, unsigned int offset, unsigned int recursion_cost) | ||
| 68 | { | ||
| 69 | ExifLong o, thumbnail_offset = 0, thumbnail_length = 0; | ||
| 70 | ExifShort n; | ||
| 71 | @@ -372,9 +387,20 @@ exif_data_load_data_content (ExifData *data, ExifIfd ifd, | ||
| 72 | if ((((int)ifd) < 0) || ( ((int)ifd) >= EXIF_IFD_COUNT)) | ||
| 73 | return; | ||
| 74 | |||
| 75 | - if (recursion_depth > 30) { | ||
| 76 | + if (recursion_cost > 170) { | ||
| 77 | + /* | ||
| 78 | + * recursion_cost is a logarithmic-scale indicator of how expensive this | ||
| 79 | + * recursive call might end up being. It is an indicator of the depth of | ||
| 80 | + * recursion as well as the potential for worst-case future recursive | ||
| 81 | + * calls. Since it's difficult to tell ahead of time how often recursion | ||
| 82 | + * will occur, this assumes the worst by assuming every tag could end up | ||
| 83 | + * causing recursion. | ||
| 84 | + * The value of 170 was chosen to limit typical EXIF structures to a | ||
| 85 | + * recursive depth of about 6, but pathological ones (those with very | ||
| 86 | + * many tags) to only 2. | ||
| 87 | + */ | ||
| 88 | exif_log (data->priv->log, EXIF_LOG_CODE_CORRUPT_DATA, "ExifData", | ||
| 89 | - "Deep recursion detected!"); | ||
| 90 | + "Deep/expensive recursion detected!"); | ||
| 91 | return; | ||
| 92 | } | ||
| 93 | |||
| 94 | @@ -416,15 +442,18 @@ exif_data_load_data_content (ExifData *data, ExifIfd ifd, | ||
| 95 | switch (tag) { | ||
| 96 | case EXIF_TAG_EXIF_IFD_POINTER: | ||
| 97 | CHECK_REC (EXIF_IFD_EXIF); | ||
| 98 | - exif_data_load_data_content (data, EXIF_IFD_EXIF, d, ds, o, recursion_depth + 1); | ||
| 99 | + exif_data_load_data_content (data, EXIF_IFD_EXIF, d, ds, o, | ||
| 100 | + recursion_cost + level_cost(n)); | ||
| 101 | break; | ||
| 102 | case EXIF_TAG_GPS_INFO_IFD_POINTER: | ||
| 103 | CHECK_REC (EXIF_IFD_GPS); | ||
| 104 | - exif_data_load_data_content (data, EXIF_IFD_GPS, d, ds, o, recursion_depth + 1); | ||
| 105 | + exif_data_load_data_content (data, EXIF_IFD_GPS, d, ds, o, | ||
| 106 | + recursion_cost + level_cost(n)); | ||
| 107 | break; | ||
| 108 | case EXIF_TAG_INTEROPERABILITY_IFD_POINTER: | ||
| 109 | CHECK_REC (EXIF_IFD_INTEROPERABILITY); | ||
| 110 | - exif_data_load_data_content (data, EXIF_IFD_INTEROPERABILITY, d, ds, o, recursion_depth + 1); | ||
| 111 | + exif_data_load_data_content (data, EXIF_IFD_INTEROPERABILITY, d, ds, o, | ||
| 112 | + recursion_cost + level_cost(n)); | ||
| 113 | break; | ||
| 114 | case EXIF_TAG_JPEG_INTERCHANGE_FORMAT: | ||
| 115 | thumbnail_offset = o; | ||
diff --git a/meta/recipes-support/libexif/libexif_0.6.21.bb b/meta/recipes-support/libexif/libexif_0.6.21.bb deleted file mode 100644 index d847beab18..0000000000 --- a/meta/recipes-support/libexif/libexif_0.6.21.bb +++ /dev/null | |||
| @@ -1,17 +0,0 @@ | |||
| 1 | SUMMARY = "Library for reading extended image information (EXIF) from JPEG files" | ||
| 2 | HOMEPAGE = "http://sourceforge.net/projects/libexif" | ||
| 3 | SECTION = "libs" | ||
| 4 | LICENSE = "LGPLv2.1" | ||
| 5 | LIC_FILES_CHKSUM = "file://COPYING;md5=243b725d71bb5df4a1e5920b344b86ad" | ||
| 6 | |||
| 7 | SRC_URI = "${SOURCEFORGE_MIRROR}/libexif/libexif-${PV}.tar.bz2 \ | ||
| 8 | file://CVE-2017-7544.patch \ | ||
| 9 | file://CVE-2016-6328.patch \ | ||
| 10 | file://CVE-2018-20030.patch" | ||
| 11 | |||
| 12 | SRC_URI[md5sum] = "27339b89850f28c8f1c237f233e05b27" | ||
| 13 | SRC_URI[sha256sum] = "16cdaeb62eb3e6dfab2435f7d7bccd2f37438d21c5218ec4e58efa9157d4d41a" | ||
| 14 | |||
| 15 | inherit autotools gettext | ||
| 16 | |||
| 17 | EXTRA_OECONF += "--disable-docs" | ||
diff --git a/meta/recipes-support/libexif/libexif_0.6.22.bb b/meta/recipes-support/libexif/libexif_0.6.22.bb new file mode 100644 index 0000000000..a520d5c9f9 --- /dev/null +++ b/meta/recipes-support/libexif/libexif_0.6.22.bb | |||
| @@ -0,0 +1,19 @@ | |||
| 1 | SUMMARY = "Library for reading extended image information (EXIF) from JPEG files" | ||
| 2 | HOMEPAGE = "https://libexif.github.io/" | ||
| 3 | SECTION = "libs" | ||
| 4 | LICENSE = "LGPLv2.1" | ||
| 5 | LIC_FILES_CHKSUM = "file://COPYING;md5=243b725d71bb5df4a1e5920b344b86ad" | ||
| 6 | |||
| 7 | def version_underscore(v): | ||
| 8 | return "_".join(v.split(".")) | ||
| 9 | |||
| 10 | SRC_URI = "https://github.com/libexif/libexif/releases/download/libexif-${@version_underscore("${PV}")}-release/libexif-${PV}.tar.xz \ | ||
| 11 | " | ||
| 12 | |||
| 13 | SRC_URI[sha256sum] = "5048f1c8fc509cc636c2f97f4b40c293338b6041a5652082d5ee2cf54b530c56" | ||
| 14 | |||
| 15 | UPSTREAM_CHECK_URI = "https://github.com/libexif/libexif/releases/" | ||
| 16 | |||
| 17 | inherit autotools gettext | ||
| 18 | |||
| 19 | EXTRA_OECONF += "--disable-docs" | ||
