File CVE-2016-5844.patch of Package libarchive.13055

Overview Repositories Revisions Requests Users Attributes Meta

File CVE-2016-5844.patch of Package libarchive.13055

commit 3ad08e01b4d253c66ae56414886089684155af22
Author: Tim Kientzle <kientzle@acm.org>
Date:   Sun Jun 19 14:34:37 2016 -0700

Issue 717:  Fix integer overflow when computing location of volume descriptor
    
    The multiplication here defaulted to 'int' but calculations
    of file positions should always use int64_t.  A simple cast
    suffices to fix this since the base location is always 32 bits
    for ISO, so multiplying by the sector size will never overflow
    a 64-bit integer.

Index: libarchive-3.1.2/libarchive/archive_read_support_format_iso9660.c
===================================================================
--- libarchive-3.1.2.orig/libarchive/archive_read_support_format_iso9660.c
+++ libarchive-3.1.2/libarchive/archive_read_support_format_iso9660.c
@@ -1089,7 +1089,7 @@ choose_volume(struct archive_read *a, st
 		/* This condition is unlikely; by way of caution. */
 		vd = &(iso9660->joliet);
 
-	skipsize = LOGICAL_BLOCK_SIZE * vd->location;
+	skipsize = LOGICAL_BLOCK_SIZE * (int64_t)vd->location;
 	skipsize = __archive_read_consume(a, skipsize);
 	if (skipsize < 0)
 		return ((int)skipsize);
@@ -1127,7 +1127,7 @@ choose_volume(struct archive_read *a, st
 	    && iso9660->seenJoliet) {
 		/* Switch reading data from primary to joliet. */
 		vd = &(iso9660->joliet);
-		skipsize = LOGICAL_BLOCK_SIZE * vd->location;
+		skipsize = LOGICAL_BLOCK_SIZE * (int64_t)vd->location;
 		skipsize -= iso9660->current_position;
 		skipsize = __archive_read_consume(a, skipsize);
 		if (skipsize < 0)

Places

File CVE-2016-5844.patch of Package libarchive.13055

Places