Libarchive 3.4.3

Merge pull request #1378 from stefan-baranoff/issue-1377-passphrase-memory-leak

Fix memory leak from passphrase callback

Fix memory leak from passphrase callback

There is a bug in the linked list implementation for passphrases. The insert to
head function does not account for the tail==head case and causes a leak. The
first entry into the list is lost when the second entry is added. The second
and beyond entries are are released properly, but the first is lost entirely.

Merge pull request #1376 from AdamKorcz/master

Moved oss-fuzz files from oss-fuzz to Libarchives repository

Moved oss-fuzz files from oss-fuzz to Libarchives repository

Add test for #1357 (detection of pzstd compressed files)

Merge pull request #1357 from mako2580/master

zstd: Add support for files created with pzstd

Merge pull request #1360 from EricAustinYu/master

Remove unreachable code in RAR reader

Fix retrieving incorrect member from struct statfs

Fixes #1355

tar reader: update handling of RHT.security.selinux xattr

Add test for PR #1348
Rename test_read_pax_schily_xattr to test_read_pax_xattr_schily

Merge pull request #1372 from evelikov/single-child

Cleanup child handling

Merge pull request #1371 from evelikov/digest-ifdef

Remove #ifdef chain in __archive_digest init

Merge pull request #1365 from AdamKorcz/master

Added seed corpus

fix typo in README.md (#1373)

Remove CloseHandle() from __archive_write_program_free

The call to CloseHandle() is asymmetric and wrong.

The handle is created during the __achive_create_child() call in
__archive_write_program_open() aka the filter open callback.

While CloseHandle() is already called in the filter close callback aka
__archive_write_program_close().

Remove the incorrect call to CloseHandle() from the bidder free callback

Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>

Call OpenProcess() from within __archive_create_child()

Move the Windows specific to the Windows version of the helper, making
the existing code shorter and simpler.

This means that the child "handle" type will vary across the two
platforms - something that we've already been doing with waitpid.

Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>

Change __archive_create_child() signature

Change the function to return an int - ARCHIVE_OK or ARCHIVE_FAILED,
taking the child as an output argument.

This will allow us to simplify the existing code and have move platform
specifics in the platform files - posix and windows

Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>

Moved the corpus file to the contrib dir

Remove #ifdef chain in __archive_digest init

Follow the approach set by the crypto and use the exact same name for
the respective functions. This is perfectly fine, since only one
provider can be used (built) at given time.

As a result, we can kill off the excessively long ifdef chain, making
the more more manageable and shorter.

Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>

Merge pull request #1363 from aseprite/dont-test-lzma-if-not-needed

Don't even try HAVE_LZMA_STREAM_ENCODER_MT if ENABLE_LZMA is disabled

Merge pull request #1364 from aseprite/macos-no-deprecated

Disable error using deprecated functions on macOS 10.15 to calculate MD5

Merge pull request #1362 from aseprite/libarchive.pc-in-binary-dir

Create libarchive.pc inside the cmake build dir

Merge pull request #1361 from aseprite/fix-msvc-warning-flags

Use /w1nnnn instead of /wennnn to enable warnings on MSVC Debug mode

Merge pull request #1367 from zoulasc/master

Fix issues in acl tests

Make variable const since it is assigned from a string literal

Add missing breaks

Added seed corpus

Use /w1nnnn instead of /wennnn to enable warnings on MSVC Debug mode

On MSVC /wennnn treats the specific warning number nnnn as an error:
https://docs.microsoft.com/en-us/cpp/build/reference/compiler-option-warning-level

But we want to enable this only when ENABLE_WERROR is defined (so the
general /WX flag is enough to treat all warnings as errors instead of
using /we).

Disable error using deprecated functions on macOS 10.15 to calculate MD5

Don't even try HAVE_LZMA_STREAM_ENCODER_MT if ENABLE_LZMA is disabled

Create libarchive.pc inside the cmake build dir

In case that we don't have write access in the source dir, we have to
write generated files in the build dir.

Merge pull request #1348 from mikhailnov/rh-selinux

Support RHT.security.selinux from GNU tar

Merge pull request #1359 from hjmallon/pr.neg_zstd_2

zstd: Improve version checking

Remove unreachable code in RAR reader

zstd: Improve version checking

Merge pull request #1352 from hjmallon/pr.negative_zstd

zstd: Support negative compression-levels

Merge pull request #1356 from robUx4/svfs-variable

Fix stavfs variable name

zstd: Add support for files created with pzstd

zstd: Support negative compression-levels

consistently use svfs for statvfs variables

So it cannot be misinterpreted as a statfs which uses the sfs variable name.

Merge pull request #1354 from robUx4/local-mode_t-fix

archive_entry: use the proper define for mode_t

archive_entry: use the proper define for mode_t

This is the define set in archive_entry.h to use the right mode_t type per
platform. "mode_t" may not be available at all.

This is also how these functions are declared in archive_entry.h

Merge pull request #1350 from josch/master

libarchive/archive_entry_stat.3: typo nlinks -> nlink

Merge pull request #1342 from hjmallon/pr.fix_high_zstd

Fixes for using zstd binary

CI: build Windows version with xz 5.2.5

libarchive/archive_entry_stat.3: typo nlinks -> nlink

Support RHT.security.selinux from GNU tar

zstd: Fix support for compression-level>=20 when using binary

* Requires '--ultra' argument

zstd: Use --no-check with binary to match library default

Safe writes: improve error handling

Print message if creation of temporary file fails.
When rename of temporary file fails, exit with ARCHIVE_FAILED.
Reword message on failed rename.
Revert d84ec3f and 85b9f66, call archive_set_error()w before unlinking.

Add archive_write_disk_windows.c counterpart to previous commit

Store errno when unlinking after rename on safe write failed

cpio/cpio.c: Fix off-by-one in previous commit

cpio/cpio.c: avoid calling strlen() for destdir twice

Safe writes: delete temporary file if rename fails

Change error message to "Failed to safe write"
Fixes #1341

CI: GitHub Actions: use C:\windows\system32\tar.exe as tar on Windows

bsdcpio_test: from_hex() needs to be an uint32_t

Fixes #1339

XAR reader: initialize file_queue with 0 and memcpy() if allocated only

Fixes #1338

CI: use static CMake build of liblzma

CI: various adjustments

Always bootstrap pkg on FreeBSD
Remove unnecessary dependency on libzip-dev
Remove Cirrus CI Fedora builds for now

archive_read.c: silence UBSan warning about undefined behavior

Fixes #1335

Merge pull request #1331 from cure/patch-1

New ASCII format: fix hard link description

New ASCII format: fix hard link description

The description of how hard links are handled in the 'new ASCII format' is wrong. Cpio stores the *first*, not *last* copy of the file with its entire contents, followed by entries with filesize set to zero for each subsequent entry that is hard linked. While creating the archive, how would it ever know it was processing the last one!

XAR writer: initialize size with 0 in xar_write_data()

Solve warning by excluding more code to _WIN32 in file_open()

Add testcase for ZIPX files with LZMA_STREAM_END marker

Testcase for issue #1257 (PR #1297)

CI: add external binary static xz to Windows build

Constify name in __archive_write_entry_filetype_unsupported()

Libarchive 3.4.3dev

Libarchive 3.4.2

Add script to generate release files with Docker.

Unlink before creating a hardlink on Windows
when using ARCHIVE_EXTRACT_SAFE_WRITES.

Completes #1330 for Windows.

Merge pull request #1330 from zoulasc/master

Fix ARCHIVE_EXTRACT_SAFE_WRITES for existing symlinks

Just like the hard link case, for symlinks we need to remove first
for ARCHIVE_EXTRACT_SAFE_WRITES.

Update NEWS

RAR5 reader: use %zu to sprintf() size_t

CI Windows: download zlib and bzip2 from cloned repositories

CI: generate and publish build artifacts on GitHub actions

Fixes #1313

CI: build and use bzip2 on Windows from GiLab repository

Use static versions of zlib and bzip2 on Windows.
Simplify ci.cmd for GitHub Actions.

Merge pull request #1329 from bradking/string-append-wcs-overflow

Fix possible heap-buffer-overflow in archive_string_append_from_wcs on Windows

RAR5 reader: do not mutate a global variable

Read RAR5 signature with a function instead.
Fixes #1303

CI: exit with error on failure in Visual Studio stages

7z reader: support Delta filter with values 2-256

Fixes #987

Fix possible heap-buffer-overflow in archive_string_append_from_wcs on Windows

Fix `archive_string_append_from_wcs_in_codepage` to account for the
already-used portion of the buffer when computing the size of the
remaining buffer for `WideCharToMultiByte` output.

Merge pull request #1326 from antekone/rar5_ossfuzz_20459

RAR5 reader: reject files that declare invalid header flags

RAR5 reader: reject files that declare invalid header flags

One of the fields in RAR5's base block structure is the size of the
header. Some invalid files declare a 0 header size setting, which can
confuse the unpacker. Minimum header size for RAR5 base blocks is 7
bytes (4 bytes for CRC, and 3 bytes for the rest), so block size of 0
bytes should be rejected at header parsing stage.

The fix adds an error condition if header size of 0 bytes is detected.
In this case, the unpacker will not attempt to unpack the file, as the
header is corrupted.

The commit also adds OSSFuzz #20459 sample to test further regressions
in this area.

Merge pull request #1324 from bradking/solaris-11.4

libarchive: Fix detection of 'major' on Solaris 11.4

Fix use after free in ISO9660 and XAR writer

Remove rbtree nodes before freeing them.
For better code readability import new rbtree macros from NetBSD.

Fixes #1325

Update filter and format options in manual pages.

The list of format and filter options in bsdtar(1) is incomplete.
Add reference to manual pages archive_write_set_options(3) and
archive_read_set_options(3) for a complete list of supported options.

Fixes #1323

mtree reader: initialize checkfs with 0

libarchive: Fix detection of 'major' on Solaris 11.4

In `archive_pack_dev.c` there is code checking the `HAVE_MAJOR` macro,
but it is not computed.  Port the equivalent logic from
`archive_entry.c` to define the macro.

Cmake MSVC: replace warning C4061 with C4062

Fixes #1322

test_write_format_xar: add "none" options to test

PAX writer: fix return-on-error memory leaks in archive_write_pax_header()

PAX writer: fix entry uname or gname longer than 32 characters

Fixes #1319

Unify unsupported entry file type error message in writer

Closes #1320

XAR writer: fix compression output buffer handling

Add "none" as acceptable value to xar:checksum and xar:toc-checksum
Document XAR options in bsdtar(1)

Fixes #1317

CI: make mingw32-make verbose

Add printf attributes to the printflike functions used in tests and fix
the format errors.

Closes #1318

RAR5 reader: refactored parse_tables() a little bit

Some if/continue constructs could be changed to if/elseif/else blocks.
This allows elimination of superfluous 'continue' directives.

Closes #1280