* gnu/packages/patches/libexif-CVE-2016-6328.patch: New file. * gnu/local.mk (dist_patch_DATA): Add it. * gnu/packages/photo.scm (libexif)[source]: Use it.
		
			
				
	
	
		
			72 lines
		
	
	
	
		
			2 KiB
		
	
	
	
		
			Diff
		
	
	
	
	
	
			
		
		
	
	
			72 lines
		
	
	
	
		
			2 KiB
		
	
	
	
		
			Diff
		
	
	
	
	
	
| Fix CVE-2016-6328:
 | |
| 
 | |
| https://bugzilla.redhat.com/show_bug.cgi?id=1366239
 | |
| https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2016-6328
 | |
| 
 | |
| Patch copied from upstream source repository:
 | |
| 
 | |
| https://github.com/libexif/libexif/commit/41bd04234b104312f54d25822f68738ba8d7133d
 | |
| 
 | |
| From 41bd04234b104312f54d25822f68738ba8d7133d Mon Sep 17 00:00:00 2001
 | |
| From: Marcus Meissner <marcus@jet.franken.de>
 | |
| Date: Tue, 25 Jul 2017 23:44:44 +0200
 | |
| Subject: [PATCH] fixes some (not all) buffer overreads during decoding pentax
 | |
|  makernote entries.
 | |
| 
 | |
| This should fix:
 | |
| https://sourceforge.net/p/libexif/bugs/125/ CVE-2016-6328
 | |
| ---
 | |
|  libexif/pentax/mnote-pentax-entry.c | 16 +++++++++++++---
 | |
|  1 file changed, 13 insertions(+), 3 deletions(-)
 | |
| 
 | |
| diff --git a/libexif/pentax/mnote-pentax-entry.c b/libexif/pentax/mnote-pentax-entry.c
 | |
| index d03d159..ea0429a 100644
 | |
| --- a/libexif/pentax/mnote-pentax-entry.c
 | |
| +++ b/libexif/pentax/mnote-pentax-entry.c
 | |
| @@ -425,24 +425,34 @@ mnote_pentax_entry_get_value (MnotePentaxEntry *entry,
 | |
|  		case EXIF_FORMAT_SHORT:
 | |
|  		  {
 | |
|  			const unsigned char *data = entry->data;
 | |
| -		  	size_t k, len = strlen(val);
 | |
| +		  	size_t k, len = strlen(val), sizeleft;
 | |
| +
 | |
| +			sizeleft = entry->size;
 | |
|  		  	for(k=0; k<entry->components; k++) {
 | |
| +				if (sizeleft < 2)
 | |
| +					break;
 | |
|  				vs = exif_get_short (data, entry->order);
 | |
|  				snprintf (val+len, maxlen-len, "%i ", vs);
 | |
|  				len = strlen(val);
 | |
|  				data += 2;
 | |
| +				sizeleft -= 2;
 | |
|  			}
 | |
|  		  }
 | |
|  		  break;
 | |
|  		case EXIF_FORMAT_LONG:
 | |
|  		  {
 | |
|  			const unsigned char *data = entry->data;
 | |
| -		  	size_t k, len = strlen(val);
 | |
| +		  	size_t k, len = strlen(val), sizeleft;
 | |
| +
 | |
| +			sizeleft = entry->size;
 | |
|  		  	for(k=0; k<entry->components; k++) {
 | |
| +				if (sizeleft < 4)
 | |
| +					break;
 | |
|  				vl = exif_get_long (data, entry->order);
 | |
|  				snprintf (val+len, maxlen-len, "%li", (long int) vl);
 | |
|  				len = strlen(val);
 | |
|  				data += 4;
 | |
| +				sizeleft -= 4;
 | |
|  			}
 | |
|  		  }
 | |
|  		  break;
 | |
| @@ -455,5 +465,5 @@ mnote_pentax_entry_get_value (MnotePentaxEntry *entry,
 | |
|  		break;
 | |
|  	}
 | |
|  
 | |
| -	return (val);
 | |
| +	return val;
 | |
|  }
 | |
| -- 
 | |
| 2.16.0
 | |
| 
 |