From b85c4c3da0a560cb98d13902c308e54fdeb52d70 Mon Sep 17 00:00:00 2001 From: Martin Matuska Date: Fri, 28 Jul 2017 23:51:08 +0000 Subject: [PATCH] Update vendor/libarchive to git de20494ba2a4fcff8b56010faa75467ad8d5a40b Relevant vendor changes: PR #926: ensure ar strtab is null terminated --- libarchive/archive_cryptor_private.h | 2 +- libarchive/archive_write_set_format_ar.c | 5 +++-- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/libarchive/archive_cryptor_private.h b/libarchive/archive_cryptor_private.h index 0ca544b57971..b9759220df69 100644 --- a/libarchive/archive_cryptor_private.h +++ b/libarchive/archive_cryptor_private.h @@ -64,7 +64,7 @@ typedef struct { } archive_crypto_ctx; #elif defined(_WIN32) && !defined(__CYGWIN__) && defined(HAVE_BCRYPT_H) -#include +#include /* Common in other bcrypt implementations, but missing from VS2008. */ #ifndef BCRYPT_SUCCESS diff --git a/libarchive/archive_write_set_format_ar.c b/libarchive/archive_write_set_format_ar.c index c9771d81a128..441119d5d714 100644 --- a/libarchive/archive_write_set_format_ar.c +++ b/libarchive/archive_write_set_format_ar.c @@ -374,13 +374,14 @@ archive_write_ar_data(struct archive_write *a, const void *buff, size_t s) return (ARCHIVE_WARN); } - ar->strtab = (char *)malloc(s); + ar->strtab = (char *)malloc(s + 1); if (ar->strtab == NULL) { archive_set_error(&a->archive, ENOMEM, "Can't allocate strtab buffer"); return (ARCHIVE_FATAL); } - strncpy(ar->strtab, buff, s); + memcpy(ar->strtab, buff, s); + ar->strtab[s] = '\0'; ar->has_strtab = 1; }