xz: Properly clean up resources on error path

If vtable is not set to NULL, close function would be called during
shutdown. Since data is already freed and set to NULL, this would lead
to a NULL pointer dereference later on.

The called library functions should never fail though, so this is a
purely defensive measure against future lzma changes.

Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org>

diff --git a/libarchive/archive_read_support_filter_xz.c b/libarchive/archive_read_support_filter_xz.c
index 1fff30ab3c343108b75b19c0e7dd97714e4fd14a..501a4772b4feaf435efa07838001a9bdbe9cebaa 100644 (file)
--- a/libarchive/archive_read_support_filter_xz.c
+++ b/libarchive/archive_read_support_filter_xz.c
@@ -524,6 +524,7 @@ xz_lzma_bidder_init(struct archive_read_filter *self)
        free(state->out_block);
        free(state);
        self->data = NULL;
+       self->vtable = NULL;
        return (ARCHIVE_FATAL);
 }