From 98c1f51f769841d99d879099f9075ff60d51ee4a Mon Sep 17 00:00:00 2001 From: Martin Matuska Date: Mon, 6 Jan 2020 13:13:58 +0000 Subject: Update vendor/libarchive/dist to git 3f1bad815d02160ab27f7063257aed4b25dbaebe Relevant vendor changes: Issue #1302: Re-do fix for archive_write_client_open() --- libarchive/archive_cryptor.c | 1 + libarchive/archive_hmac.c | 1 + libarchive/archive_write.c | 44 +++++++++++++++++--------------------------- 3 files changed, 19 insertions(+), 27 deletions(-) diff --git a/libarchive/archive_cryptor.c b/libarchive/archive_cryptor.c index 769756d0b5ab..8ab2b0979676 100644 --- a/libarchive/archive_cryptor.c +++ b/libarchive/archive_cryptor.c @@ -299,6 +299,7 @@ aes_ctr_release(archive_crypto_ctx *ctx) } #elif defined(HAVE_LIBMBEDCRYPTO) && defined(HAVE_MBEDTLS_AES_H) + static int aes_ctr_init(archive_crypto_ctx *ctx, const uint8_t *key, size_t key_len) { diff --git a/libarchive/archive_hmac.c b/libarchive/archive_hmac.c index 68f2bb55dc04..2a9d04c8d8f1 100644 --- a/libarchive/archive_hmac.c +++ b/libarchive/archive_hmac.c @@ -151,6 +151,7 @@ __hmac_sha1_cleanup(archive_hmac_sha1_ctx *ctx) } #elif defined(HAVE_LIBMBEDCRYPTO) && defined(HAVE_MBEDTLS_MD_H) + static int __hmac_sha1_init(archive_hmac_sha1_ctx *ctx, const uint8_t *key, size_t key_len) { diff --git a/libarchive/archive_write.c b/libarchive/archive_write.c index afa26b8ea0ed..98a55fb2aa00 100644 --- a/libarchive/archive_write.c +++ b/libarchive/archive_write.c @@ -339,6 +339,7 @@ archive_write_client_open(struct archive_write_filter *f) struct archive_none *state; void *buffer; size_t buffer_size; + int ret; f->bytes_per_block = archive_write_get_bytes_per_block(f->archive); f->bytes_in_last_block = @@ -363,7 +364,13 @@ archive_write_client_open(struct archive_write_filter *f) if (a->client_opener == NULL) return (ARCHIVE_OK); - return (a->client_opener(f->archive, a->client_data)); + ret = a->client_opener(f->archive, a->client_data); + if (ret != ARCHIVE_OK) { + free(state->buffer); + free(state); + f->data = NULL; + } + return (ret); } static int @@ -448,30 +455,6 @@ archive_write_client_write(struct archive_write_filter *f, return (ARCHIVE_OK); } -static int -archive_write_client_free(struct archive_write_filter *f) -{ - struct archive_write *a = (struct archive_write *)f->archive; - struct archive_none *state = (struct archive_none *)f->data; - - if (state != NULL) { - free(state->buffer); - free(state); - state = NULL; - } - - a->client_data = NULL; - /* Clear passphrase. */ - if (a->passphrase != NULL) { - memset(a->passphrase, 0, strlen(a->passphrase)); - free(a->passphrase); - a->passphrase = NULL; - } - - return (ARCHIVE_OK); -} - - static int archive_write_client_close(struct archive_write_filter *f) { @@ -508,7 +491,15 @@ archive_write_client_close(struct archive_write_filter *f) } if (a->client_closer) (*a->client_closer)(&a->archive, a->client_data); - + free(state->buffer); + free(state); + a->client_data = NULL; + /* Clear passphrase. */ + if (a->passphrase != NULL) { + memset(a->passphrase, 0, strlen(a->passphrase)); + free(a->passphrase); + a->passphrase = NULL; + } /* Clear the close handler myself not to be called again. */ f->state = ARCHIVE_WRITE_FILTER_STATE_CLOSED; return (ret); @@ -539,7 +530,6 @@ archive_write_open(struct archive *_a, void *client_data, client_filter->open = archive_write_client_open; client_filter->write = archive_write_client_write; client_filter->close = archive_write_client_close; - client_filter->free = archive_write_client_free; ret = __archive_write_filters_open(a); if (ret < ARCHIVE_WARN) { -- cgit v1.2.3