Remove unneeded return.

Style.

Properly free and close when exiting write_hierarchy() on failed
archive_read_next_header2().

Fixes #794

Merge pull request #791 from hartzell/bug/disk-variable-masked

Fix scoping error with "disk" variable

Fix scoping error with "disk" variable

The variable named 'disk' inside the loop was hiding the variable named
'disk' at the outer level.  The outer variable was never used, get rid
of it.

Move the call to archive_read_disk_set_standard_lookup(disk) inside the
loop so that it Does What It Should.

Merge pull request #790 from Tarsnap/uninitialized-subst-result

Initialize subst rule->result pointer to NULL

Initialize subst rule->result pointer to NULL

This is not directly useful in libarchive at the present time, but
it might be in the future.  If a project which uses this code adds
an atexit(3) handler to clean up memory, and somebody runs:
    bsdtar somestring -t
then "omestring" will be interpreted as an (invalid) substitution
rule and call lafe_errc().  However, if the atexit() handler then
calls cleanup_substition(), it will try to free(rule->result)
which was not initialized.

Sponsored by: Tarsnap Backup Inc.

Merge pull request #787 from praiskup/fix-umask

tests: fix testsuite for umask issues

Merge pull request #788 from Tarsnap/check-test-mode

Check test mode

Add chmod() after make_{dir,file} in test suites

This is the same change that
    38b3f516df865e2dffd0a1bbc9feb923ab2efc38
made to
    libarchive/test/main.c
, but now applied to:
    cat/test/main.c
    cpio/test/main.c
    tar/test/main.c

Sponsored by: Tarsnap Backup Inc.

Add and use assertion_file_mode() in other tests

This adds the assertion_file_mode() function from
    libarchive/test/main.c
and applies it to
    cat/test/main.c
    cpio/test/main.c
    tar/test/main.c

Sponsored by: Tarsnap Backup Inc.

tests: fix testsuite for umask issues

Reproducible with umask 0077.

Resolves: #782

"bunzip2 -V" does not exit with 0, use "bunzip2 -h" instead

Add READ_DATA dir permission to @owner to make test cleanup work for non-root

Merge pull request #774 from Tarsnap/check-test-mode

chmod and check the mode after creating a file or dir

Issue 774, 782: chmod() after creating a file or dir

If the user's system has a default umask, then
    mkdir(pathname, mode);
will report "success" even if the created dir does not match the specified
mode.  Presumably that is desired in the general case, but when it comes to
testing libarchive this can generate false errors in `make check`.

chmod() is not affected by umask, so we call that after creating the file
or directory.

Sponsored by: Tarsnap Backup Inc.

Issue 761:  Heap overflow reading corrupted 7Zip files

The sample file that demonstrated this had multiple 'EmptyStream'
attributes.  The first one ended up being used to calculate
certain statistics, then was overwritten by the second which
was incompatible with those statistics.

The fix here is to reject any header with multiple EmptyStream
attributes.  While here, also reject headers with multiple
EmptyFile, AntiFile, Name, or Attributes markers.

Merge branch 'master' of github.com:libarchive/libarchive

Issue 747 (and others?):  Avoid OOB read when parsing multiple long lines

The mtree bidder needs to look several lines ahead
in the input.  It does this by extending the read-ahead
and parsing subsequent lines from the same growing buffer.
A bookkeeping error when extending the read-ahead would
sometimes lead it to significantly over-count the
size of the line being read.

Merge pull request #781 from praiskup/build-fix-undeclared-var

ftbfs: declare 'r' variable for non NFSv4 ACLs too

ftbfs: declare 'r' variable for non NFSv4 ACLs too

Merge branch 'master' of github.com:libarchive/libarchive

Use O_NOFOLLOW to avoid traversing symlinks when restoring data-carrying hardlinks.

TODO: Consider using this flag in other situations as well.

Thanks to Pavel Raiskup.

Reject non-regular sparse files

Thanks to Pavel Raiskup

Remove dead comments

Fixes for Issue #745 and Issue #746 from Doran Moppert.

Fix the test cases for Issue #745 and Issue #746

Thanks to Doran Moppert for pointing out the inconsistencies here.

Merge pull request #775 from mmatuska/glibc_2_24_readdir_r

Glibc 2.24 deprecates readdir_r()

Merge pull request #779 from nacho/fix-769

test_write_format_gnutar_filenames: use AE_IFLNK instead of S_IFLNK

test_write_format_gnutar_filenames: use AE_IFLNK instead of S_IFLNK

Since S_IFLNK will fail on windows.

https://github.com/libarchive/libarchive/issues/769

Merge pull request #778 from mmatuska/acl_error_handling

Handle all possible errors from ACL functions

ACL error handling

Fill in checks and support for acl_get_fd_np

Merge branch 'fix_acls_inside_subdirs' of https://github.com/mmatuska/libarchive into mmatuska-fix_acls_inside_subdirs

Test for PR#777: verify ACL reading during directory walks

This verifies the issue mentioned in Pull Request #777:
When using archive_read_disk to read a directory tree,
libarchive would attempt to read ACLs with the wrong
directory.

Fix FreeBSD build:  prefer struct xvfsconf to struct vfsconf

Multiple bugfixes for setup_acls()

Convert README to markdown

Convert README to markdown

Convert README to markdown

Convert README to markdown

Convert README to markdown

Do not use deprecated readdir_r() on glibc 2.24 and later

Issue #731:  Reject tar entries >= INT64_MAX

Note that the code that parses file sizes returns INT64_MAX on
overflow.

Issue #744 (part of Issue #743): Enforce sandbox with very long pathnames

Because check_symlinks is handled separately from the deep-directory
support, very long pathnames cause problems.  Previously, the code
ignored most failures to lstat() a path component.  In particular,
this led to check_symlinks always passing for very long paths, which
in turn provides a way to evade the symlink checks in the sandboxing
code.

We now fail on unrecognized lstat() failures, which plugs this
hole at the cost of disabling deep directory support when the
user requests sandboxing.

TODO:  This probably cannot be completely fixed without
entirely reimplementing the deep directory support to
integrate the symlink checks.  I want to reimplement the
deep directory hanlding someday anyway; openat() and
related system calls now provide a much cleaner way to
handle deep directories than the chdir approach used by this
code.

Issue #748: Zip decompression failure with highly-compressed data

Previously, we stopped driving the decompressor as soon as
we hit end of input, but in some cases, the decompressor
has internal state that can continue generating output
even when there is no more input.  So we now feed zero-length
blocks into the decompressor until the decompressor tells us
it is finished.

Issue #767:  Buffer overflow printing a filename

The safe_fprintf function attempts to ensure clean output for an
arbitrary sequence of bytes by doing a trial conversion of the
multibyte characters to wide characters -- if the resulting wide
character is printable then we pass through the corresponding bytes
unaltered, otherwise, we convert them to C-style ASCII escapes.

The stack trace in Issue #767 suggest that the 20-byte buffer
was getting overflowed trying to format a non-printable multibyte
character.  This should only happen if there is a valid multibyte
character of more than 5 bytes that was unprintable.  (Each byte
would get expanded to a four-charcter octal-style escape of the form
"\123" resulting in >20 characters for the >5 byte multibyte character.)

I've not been able to reproduce this, but have expanded the conversion
buffer to 128 bytes on the belief that no multibyte character set
has a single character of more than 32 bytes.

Issue #770:  Be more careful about extra_length

Hanno Böck pointed out that the loop here computes
extra_length - 4 without first checking for possible underflow.

In addition to fixing this, I also added a bunch of error
checks so Zip parsing will fail if any Zip extra field
is malformed.  Among other things, this uncovered an
old bug that would skip a trailing extra field with
zero-sized data.

Note that we still simply ignore well-formed
extra fields that we don't understand.

Merge pull request #768 from hannob/master

Remove unused variable to avoid compiler warning.

conditionally declare r on ACL_TYPE_NFS4

Remove unused variable to avoid compiler warning.

Merge pull request #755 from mmatuska/fix_acl_get_flagset_np

fix use of acl_get_flagset_np() on FreeBSD

Correct the usage of PATH_MAX as reported in Issue #744.

Merge branch 'master' of github.com:libarchive/libarchive

Test cases for Github Issue #744, #745, and #746.

Merge pull request #757 from Tarsnap/compile-test-zip-high

Add test_..._zip_high_compression.c to automake

Add test_..._zip_high_compression.c to automake

Sponsored by: Tarsnap Backup Inc.

fix use of acl_get_flagset_np() on FreeBSD

Merge pull request #751 from aahancoc/patch-1

Fixed build error in test_read_format_zip_high_compression.c

Fixed build error in test_read_format_zip_high_compression.c

C89 (what this library is compiled in) requires for loop iterators to be initialized outside the for loop.

Issue 748: Regression test

Merge pull request #750 from cpatulea/master

ustar: fix out of bounds read on empty string ("") filename.

ustar: fix out of bounds read on empty string ("") filename.

Issue 720: Allow dashes in third-party version numbers

Issue 695:  Check return value of fwrite/fclose

Merge pull request #739 from ppentchev/roam-cppcheck

Fix two minor complaints by cppcheck.

Merge branch 'master' of github.com:libarchive/libarchive

Fix check for GCC >= 4.8

Merge pull request #737 from ppentchev/roam-typo-candidate

Fix a typographical error in a struct member: candiate -> candidate

Merge pull request #736 from ppentchev/roam-typos

Fix a couple of typographical errors.

Fix two minor complaints by cppcheck.

Fix a typo in a struct member: candiate -> candidate.

Fix a couple of typographical errors.

Merge LZMA MP detection from configure.

Merge struct vfsconf detection from configure for DragonFly.

Merge pull request #733 from tpgxyz/master

Support GNU tar --blocking-factor and GNU cpio --dereference

sort it

add GNU compatible blocking-factor alias, add cpio long alias for dereference

Merge pull request #727 from jcfr/use-cmake-builtin-module-CheckStructHasMember

Use CMake module "CheckStructHasMember" and remove custom "CheckStructMember"

FreeBSD bug 204157:  If filesystem does not support birthtime, then make all birthtime comparisons succeed.

This is a bit of a hack:  We should also have tests that verify
different birthtimes.  When/if that happens, we'll need to add
a real system probe to determine whether birthtime is actually
supported on this particular filesystem and skip tests accordingly.

Issue 686: Keep retrying until buffer is big enough

@duy-github pointed out that commit 9134c3a fixed
one place where this mistake occurred, but left three
other places.

For getvfsbyname on DragonFly, struct vfsconf has to be used and not
struct xvfsconf as on FreeBSD.

Avoid use of C99 for-scope declarations to fix issue #729.

Ignore the MT encoder in XZ 5.2 prerelease versions.

Older Solaris at least has no _PATH_TTY, so provide a fallback.

Use CMake module "CheckStructHasMember" and remove custom "CheckStructMember"

Dummy out copy_acls if sys/acl.h is missing or ACL support is disabled.

Merge pull request #726 from mmatuska/cpio_version

Fix test for cpio version dash fix

SIGRTMAX doesn't exist on all systems, so compute the largest used
signal number. Don't bother with dynamically allocating the array, just
use a static array instead. Fix ctype use.

In the past, OpenSSL was not needed by the main libarchive when the hash
functions were provided by libc, e.g. on NetBSD. With the introduction
of the de-/encryption support, this is no longer the case. As such,
explicitly check for the necessary EVP routines as well to make sure
that -lcrypto is added as necessary.

Many systems have bswap16 in system headers, so undef it first to avoid
compiler warnings.

Fix ctype use. Avoid empty loop bodies.

Fix argument range of ctype functions.

Support maintainer mode

The time stamp checks for configure and Makefile.in are sometimes problematic, so allow disabling them.

Fix test for cpio version dash fix

Merge pull request #725 from mmatuska/cpio_version

Report cpio version string with one dash only

Report cpio version string with one dash only

Libarchive 3.2.1

Add missing reference file to distribution

Fix array overflow in test suite