Ensure memory is freed when zrealloc is called with size 0

zrealloc is supposed to handle freeing the old pointer when called, but
when called with size 0, it was not always working as expected.  This
commit fixes the bug.  Thanks to Agostino Sarubbo of Gentoo for providing
a bug report with a reproducible test case.

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

Handle read_data errors correctly

The read_data function returns the number of bytes written, not a boolean,
so fix the error checking so we exit if fewer than expected bytes were
read.  Thanks to Agostino Sarubbo of Gentoo for providing a bug report with
a reproducible test case.

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

Fix read off-by-one bug in compressed int function

A malformed compressed integer would cause unzck to read one byte past the
end of the allocated memory.  This commit fixes this bug.  Thanks to
Agostino Sarubbo of Gentoo for providing a bug report with a reproducible
test case.

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

Bump to 1.3.0

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

Merge pull request #92 from kraj/main

Dont let clang optimize away a meson check

Merge pull request #91 from zchunk/no_write

Add ZCK_NO_WRITE option to disable writing to a file

meson: Use global function in argp check

clang is optimizing away parse_opt() since its marked static and not
used in main(), make life a bit harder for clang :)
removing static now emits the global symbols to be resolved during link
and when libargp is missing the error is thrown rightly

riscv64-yoe-linux-musl-ld: /tmp/a-5ba039.o: in function `.Lpcrel_hi0':
a.c:(.text+0x44): undefined reference to `argp_state_help'
clang-16: error: linker command failed with exit code 1 (use -v to see invocation)

Signed-off-by: Khem Raj <raj.khem@gmail.com>

Add ZCK_NO_WRITE option to disable writing to a file

There are situations where we need to recreate the zchunk header for an
uncompressed file without actually writing out the zchunk file.  This
commit allows setting an `ioption`, `ZCK_NO_WRITE`, that will go through
the full process of creating the zchunk file without actually writing it
to disk, leaving you with a complete zchunk header in the end.

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

Bump version to 1.2.4

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

Update tests to handle zstd 1.5.4

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

Merge pull request #87 from zchunk/update-readme

Clarify what the primary benefit of zchunk is

Update .pre-commit-config.yaml

This fixes some GitHub Action linting issues that we're seeing.

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

Clarify what the primary benefit of zchunk is

zstd now offers rsync-friendly output, so mentioning that zchunk files are
efficient over rsync misses the primary benefit zchunk has, namely that
zchunk files can be downloaded efficiently from normal web servers that
support range requests (which is basically all of them, except for some
rather strange proxies).

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

Bump version to 1.2.3

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

Remove meson deprecation warning

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

Merge pull request #85 from JohanMabille/windows_static_build

Fixed meson for static build on windows

Added Windows static build to the CI

Fix `alpine-edge` build failures

The Alpine Edge container's curl-dev package is built against openssl 3,
so we can't use the openssl1.1-compat package anymore.

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

Fixed meson for static build on windows

Fix missing newline

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

Merge pull request #84 from fossabot/add-license-scan-badge

Add license scan report and status

Add license scan report and status

Signed off by: fossabot <badges@fossa.com>

Merge pull request #80 from jameshilliard/fix-printf-format

test/zck_cmp_uncomp: fix printf format types

Merge pull request #82 from jameshilliard/optional-docs

meson: add option to build without docs

test/zck_cmp_uncomp: fix printf format types

Signed-off-by: James Hilliard <james.hilliard1@gmail.com>

meson: add option to build without docs

Signed-off-by: James Hilliard <james.hilliard1@gmail.com>

Merge pull request #79 from jameshilliard/write-data

zck: declare write_data as static

Merge pull request #81 from jameshilliard/optional-tests

meson: make tests optional

Merge pull request #83 from jameshilliard/update-argp-wrap

argp-standalone.wrap: update to version 1.5.0

argp-standalone.wrap: update to version 1.5.0

Signed-off-by: James Hilliard <james.hilliard1@gmail.com>

meson: make tests optional

Signed-off-by: James Hilliard <james.hilliard1@gmail.com>

zck: declare write_data as static

This needs to be declared static to avoid a symbol conflict:
io.c:(.text+0xe8): multiple definition of `write_data'; src/zck.p/zck.c.o:zck.c:(.text+0x1f4): first defined here

Fixes:
 - http://autobuild.buildroot.net/results/225/22590a7038a40da3700d56c1f82f7dc74225702a

Signed-off-by: James Hilliard <james.hilliard1@gmail.com>

Merge pull request #78 from jameshilliard/argp-wrap

meson: fix argp-standalone wrap and find_library

meson: fix argp-standalone wrap and find_library

Signed-off-by: James Hilliard <james.hilliard1@gmail.com>

Bump to 1.2.2

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

Merge branch 'coverity'

Various fixes to make Coverity happy

The single high severity issue was only triggerable if we were unable to
allocate memory and involved a memory leak.  The other issues all revolve
around issues like making sure we check return values of functions we call.

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

Merge pull request #74 from ppentchev/pp-zck-eol

Turn off EOL handling for the *.zck test files.

Merge pull request #73 from ppentchev/pp-zstd-1.4.10

Let the tests also pass with libzstd-1.4.10.

Turn off EOL handling for the *.zck test files.

Let the tests also pass with libzstd-1.4.10.

Bump to 1.2.1

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

Merge branch 'fix_size_limit'

Fix memory leak

In the index, we were leaking the uncompressed digest, so this commit fixes
that.

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

Fix size limit in zck.c

Up to now zck.c reads the whole file into memory and then sends it to
zck_write to actually compress.  This is highly inefficient, but was done
to simplify dealing with a string separator.  This commit fixes this by
reading the file in chunks, and, if using a string separator, keeps track
of the matching bytes from chunk to chunk.

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

Merge pull request #70 from zchunk/1.2.0

Release zchunk 1.2.0 with the following changes:
 * Now builds for Windows
 * Can set `--uncompressed` flag when creating zchunk files to also store uncompressed digests
 * `zck_read_header` now indicates whether there's a dictionary and what flags are enabled in the zchunk file
 * Minimum meson version required to build zchunk is now 0.53
 * Switched to use GitHub actions for automated testing
 * Added the following automated test environments:
   * `centos-8-stream`
   * `centos-9-stream`
   * `ubuntu-lts`
   * `windows`, both using Mamba and meson-wrap
   * `macos`

Release 1.2.0

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

Remove unnecessary Jenkinsfiles

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

Add support for detached headers

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

Add extra data to zck_read_header

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

Make clear that compressed chunk checksum can be 0s

Also, fix a small bug

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

Don't set full-file checksums when using uncompressed sources

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

Fix types when printing

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

Merge pull request #65 from wolfv/print_only_length_chars

Print only length characters

Merge pull request #68 from zchunk/fix-meson-0.53

Fix meson build problems in Ubuntu

Add CentOS 8 and CentOS 9 tests

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

Fix build for meson 0.53

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

Add tests for Ubuntu LTS

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

print only length characters

Merge pull request #63 from wolfv/pre-commit

pre-commit config & github linter CI

Merge pull request #64 from zchunk/fix-wrap

Add test for meson-wrap

install pre-commit from pip

Add `libzck_dep` so other projects can use this as a subproject

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

Pass zcklib into `pkg_mod` as first argument to simplify things

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

Simplify use of disabler

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

Attempt to fix zstd wrap dependency detection

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

zstd doesn't build in msvc, so use gcc when building with wrapped zstd

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

Make libcurl optional

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

Use meson features to simplify dependency checking

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

Add test for meson-wrap

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

add pre-commit lint environment

Change workflow name

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

Merge pull request #62 from zchunk/github-actions-autotest

Update README.md to point to new CI location

Update README.md to point to new CI location

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

Merge pull request #61 from zchunk/github-actions-autotest

Switch to GitHub actions for container CI

Split different os's into different jobs and add missing containers

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

Switch to using GitHub actions for container CI

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

Merge pull request #60 from zchunk/fix-argp-alpine

Verify that include directory exists before trying to include it

Verify that include directory exists before trying to include it

Note that this also bumps the minimum meson version to 0.53.0

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

Merge pull request #54 from wolfv/win

Support for Windows

check that malloc succeeds and use safe strcpy/strcat

fix file end newlines with pre-commit

unify order of ZCK_PUBLIC_API

fix another newline

no libgen on win

add newlines, fix up zck_dl, other review changes

use LOG_ERROR as logging shorthand in utils and tests

fix win

support older meson, fix macos

escape angle brackets on Windows

add .gitattributes to force LF line endings on Windows for test files

fix copy_chunks

fix shacheck for windows

start running tests on win

add fcntl.h header for win

fix up get_tmp_fd for windows

fix tests

make tests build on Windows

Run CI on Windows

build dll on Windows