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

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

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: 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

Update archive_read_support_format_rar5.c

A patch to make this compile in older Visual Studio.
Moving variable declaration up for C89 standard.

CI: raise error if MinGW tests fail

Fix error when overwriting files on Windows

Introduced in 467d193dd61ca7669a5ba8d38e09836babf64e58

Fix 7zip_packinfo_digests test when LZMA is not supported

contrib/archivetest: fix compilation under MinGW

7zip reader tests: fixes when LZMA is unsupported

bsdtar.1: --exclude-vcs sort version control systems and add git

Fixes #1314

Add test_option_safe_writes to tar tests

Implement ARCHIVE_EXTRACT_SAFE_WRITES on Windows

As Windows does not support atomic rename/_wrename
we have to unlink the file right before calling _wrename
leaving a short unsafe window where a different file with the
same name can be created and make the rename operation fail.

Introduce archive_write_disk(3) flag ARCHIVE_EXTRACT_SAFE_WRITES

This flag changes the way that regular files are extracted:

Instead of removing existing files first and re-creating them in
order to replace their contents, a temporary file is created and
when writing to the temporary file is completed, the file is
rename(2)d to the final destination name.

This has the effect of presenting a consistent view of the file to
the system (either the file with the new contents or the file with
the old contents). Removing and overwriting the file has the
undesired side effect that the the system can either not see the
file at all (from the time it is being removed till the time it is
being re-created), or worse it can see partial file contents. This
is problematic when extracting system files (for example shared
libraries).

If the existing file that is going to be overwritten is a hard link,
for now we unlink it before calling rename(2). This can be done
correctly by creating a hardlink to a tmpnam(3) generated file
and then use rename(2), but that is fairly intrusive and requires
refactoring.

Fixes #1289

RAR5 reader: fix unsafe sign check of a bitwise operation.

This has no but code correctness effect as cdeque_init() is static
and max_capacity_power_of_2 in our code is always 8192.

Found by LGTM.com code analysis

archive_read.c: remove unnecessary comparsion (iindex is unsigned int)

Found by LGTM.com code analysis

Merge pull request #1316 from zoulasc/tests

Various test improvements

ANSI-C prototypes

Add missing const

Add missing const

Add missing const

- Add the NetBSD-specific config.h
- Use --help instead of -V when -V exits with non-zero code
- Double the string size to prevent truncation

RAR5 and WARC readers: move unnecessary comparsion into a comment
RAR5 reader: comment out unreachable code

Found by LGTM.com code analysis

archive_ppmd7.c: avoid possible integer overflow (security)

Found by LGTM.com code analysis

Multiple code fixes and optimizations

archive_read_support_format_rar5.c:
  Bitfield int -> signed int

Archive_write_set_format_iso9660.c:
  Bitfield int -> signed int

archive_write_set_format_xar.c:
  Bitfield int -> signed int

archive_write_set_format_7zip.c:
  Bitfield int -> signed int

archive_read_support_format_xar.c
  Remove useless comparsion

archive_read_support_format_rar.c:
  Fix invalid nested loop break.
  Comment out dead code sections.
  Simplify size comparsions of lensymbol and offsymbol.

archive_read_support_filter_uu.c:
  Remove useless comparsions

archive_read_disk_posix.c:
  Remove useless do-while-zero

Found by LGTM.com code analysis