gen_html: Fix build error in windows

Merge pull request #4079 from elasota/truncated-huff-state-error

Throw error if Huffman weight initial states are truncated

Merge pull request #4068 from DimitriPapadopoulos/codespell

Fix new typos found by codespell

Throw error if Huffman weight initial states are truncated

Fix typos not found by codespell

Fix new typos found by codespell

Merge pull request #4076 from facebook/fix_macos_build

fix macos build

fix macos build

weird: after replacing the UNAME line with an identical one,
it does work properly now(??).
Possibly a case of hidden special character?

Expose size estimation helpers for Linux (#4064)

Merge pull request #4073 from facebook/cygwin_install

added a cygwin install test to check #4067

added cygwin install test

Merge pull request #4067 from QBos07/filterfix

Fix $filter operants and Msys/Cygwin

Merge pull request #4065 from embg/freebsd_ci

Drop support for FreeBSD 13.2 CI

Fix $filter and Msys/Cygwin

- switched the patter and input of $filter into the right places
- added pattern wildcard to MSYS_NT & CYGWIN_NT as they change with windows versions
- correctly identify MSYS2, even in an env like MINGW64

minor:doc: specify decompression behavior in presence of multiple concatenated frames

directly at ZSTD_decompress() level.

Unit test for external sequence producer + static CCtx + streaming (#4063)

Drop FreeBSD 13.2 CI

Merge pull request #4046 from josepho0918/iar

Improve support for IAR compiler with attributes and intrinsics

Merge pull request #4054 from jbajic/fix-missing-newline

[fix] Add newline when file exceeds 128KB

Refactor dictionary matchfinder index safety check (#4039)

[fix] Add newline when file exceeds 128KB

Merge pull request #4050 from Adenilson/fix_legacy_nullptr01

[fix] Add check on failed allocation in legacy/zstd_v06

[fix] Add check on failed allocation in legacy/zstd_v06

As reported by Ben Hawkes in #4026, a failure to allocate a zstd context
would lead to a dereference of a NULL pointer due to a missing check
on the returned result of ZSTDv06_createDCtx().

This patch fix the issue by adding a check for valid returned pointer.

revert FSE_readNCount_body attribute

Improve support for IAR compiler with attributes and intrinsics

Merge pull request #4040 from facebook/dependabot/github_actions/msys2/setup-msys2-2.23.0

Bump msys2/setup-msys2 from 2.22.0 to 2.23.0

Bump msys2/setup-msys2 from 2.22.0 to 2.23.0

Bumps [msys2/setup-msys2](https://github.com/msys2/setup-msys2) from 2.22.0 to 2.23.0.
- [Release notes](https://github.com/msys2/setup-msys2/releases)
- [Changelog](https://github.com/msys2/setup-msys2/blob/main/CHANGELOG.md)
- [Commits](https://github.com/msys2/setup-msys2/compare/cc11e9188b693c2b100158c3322424c4cc1dadea...d0e80f58dffbc64f6a3a1f43527d469b4fc7b6c8)

---
updated-dependencies:
- dependency-name: msys2/setup-msys2
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Increase x-compatibility

Make zstd.h compatible with -Wzero-as-null-pointer-constant

Merge pull request #4029 from facebook/dependabot/github_actions/github/codeql-action-3.25.1

Bump github/codeql-action from 3.24.10 to 3.25.1

Merge pull request #4031 from facebook/docDStream

update documentation of ZSTD_decompressStream()

update documentation of ZSTD_decompressStream()

slightly more precise, by recommending to check the return value.
fix #4030

Bump github/codeql-action from 3.24.10 to 3.25.1

Bumps [github/codeql-action](https://github.com/github/codeql-action) from 3.24.10 to 3.25.1.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](https://github.com/github/codeql-action/compare/4355270be187e1b672a7a1c7c7bae5afdc1ab94a...c7f9125735019aa87cfc361530512d50ea439c71)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

update test name

Merge pull request #4025 from alexlnkp/dev

Fixed all memory leaks and almost all undefined behaviour

Merge pull request #3931 from bgilbert/pthread

meson: don't add `-pthread` to static linking flags on Windows

Merge pull request #4019 from b7f7/fix_cmake_rc_build

fix missing include folder for resource compiler

Merge pull request #4020 from Ansuel/install-mt-pc

Provide variant pkg-config file for multi-threaded static lib

removed freeing of the ptr

init out char array with all members as 0 x2

made initialize out char array with all elements set to 0
fixed where i made it to init with just the first one being set to 0

reverted the addition of free at the gz_init()

fixed ISO C incompatibility
for good this time... probably

fixed ISO C incompatibility

fixed memory leaks and almost all undefined behaviour

Merge pull request #4022 from facebook/dependabot/github_actions/github/codeql-action-3.24.10

Bump github/codeql-action from 3.24.9 to 3.24.10

meson: don't add -pthread to static linking flags on Windows

Meson always returns -pthread in dependency('threads') on non-MSVC
compilers.  On Windows we use Windows threading primitives, so we don't
need this.  Avoid adding -pthread to libzstd's link flags, either as a
Meson subproject or via pkg-config Libs.private, so the application
doesn't inadvertently depend on winpthreads.

Add a Meson MinGW cross-compile CI test that checks for this.  It turns
out that pzstd fails to build in that environment, so have the test
skip building contrib for now.

Merge pull request #4021 from pstef/dev

Fix zlibWrapper build

Provide variant pkg-config file for multi-threaded static lib

Multi-threaded static library require -pthread to correctly link and works.
The pkg-config we provide tho only works with dynamic multi-threaded library
and won't provide the correct libs and cflags values if lib-mt is used.

To handle this, introduce an env variable MT to permit advanced user to
install and generate a correct pkg-config file for lib-mt or detect if
lib-mt target is called.

With MT env set on calling make install-pc, libzstd.pc.in is a
pkg-config file for a multi-threaded static library.

On calling make lib-mt, a libzstd.pc is generated for a multi-threaded
static library as it's what asked by the user by forcing it.

libzstd.pc is changed to PHONY to force regeneration of it on calling
lib targets or install-pc to handle case where the same directory is
used for mixed compilation.

This was notice while migrating from meson to make build system where
meson generates a correct .pc file while make doesn't.

Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>

Bump github/codeql-action from 3.24.9 to 3.24.10

Bumps [github/codeql-action](https://github.com/github/codeql-action) from 3.24.9 to 3.24.10.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](https://github.com/github/codeql-action/compare/1b1aada464948af03b950897e5eb522f92603cc2...4355270be187e1b672a7a1c7c7bae5afdc1ab94a)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Fix zlibWrapper build

Just after a clone I'm getting this:

~/zstd/zlibWrapper$ cc -c zstd_zlibwrapper.o gz*.c -lz -lzstd -DSTDC
gzwrite.c: In function ‘gz_write’:
gzwrite.c:226:43: error: ‘z_uInt’ undeclared (first use in this
                         function); did you mean ‘uInt’?
  226 |             state.state->strm.avail_in = (z_uInt)n;
      |                                           ^~~~~~
      |                                           uInt
gzwrite.c:226:43: note: each undeclared identifier is reported only
                        once for each function it appears in
gzwrite.c:226:50: error: expected ‘;’ before ‘n’
  226 |             state.state->strm.avail_in = (z_uInt)n;
      |                                                  ^
      |                                                  ;

z_uInt is never used directly, zconf.h redefines uInt to z_uInt under
the condition that Z_PREFIX is set. All examples use uInt, and the type
of avail_in is also uInt.

In this commit I modify the cast to refer to the same type as the type
of lvalue.

Arguably, the real fix here is to handle possible overflows, but that's
beyond the scope of this commit.

Merge pull request #3996 from facebook/fix_nodejs_version_warning

fix nodejs deprecation warning

Merge pull request #4012 from elasota/spec-remove-fse-overflow

Remove specification text stating that probability overflow is invalid

Merge pull request #4011 from facebook/fix4005

decompression errors always display the full origin filename

fix missing include folder for resource compiler

Remove text specifying probability overflow as invalid, the variable-size value encoding scheme makes this impossible.

decompression errors always display the full origin filename

instead of the truncated size-limited version.

Fix building on windows-x86 if clang already includes

[D101338](https://reviews.llvm.org/D101338) landed in 2021, so clang16 should have it

Merge pull request #3994 from sunpoet/dev

Use md5sum rather than gmd5sum for FreeBSD

Merge pull request #3997 from facebook/readme_bench_156

updated benchmarks for v1.5.6

updated benchmarks for v1.5.6

Merge pull request #3988 from facebook/dependabot/github_actions/github/codeql-action-3.24.9

Bump github/codeql-action from 3.24.7 to 3.24.9

fix nodejs deprecation warning

by updating msys2 action

Merge pull request #3987 from ManuelBlanc/msbuild_vswhere

Use vswhere to find MSBuild; add VS2022 support

Use md5sum rather than gmd5sum for FreeBSD

Reference: https://man.freebsd.org/cgi/man.cgi?query=md5sum

[fuzz] Turn off -Werror by default

This was causing OSS-Fuzz errors, due to compiler differences.
* Fix the issue
* Also turn off -Werror so we don't fail fuzzer builds for warnings
* Turn on -Werror in our CI

Bump github/codeql-action from 3.24.7 to 3.24.9

Bumps [github/codeql-action](https://github.com/github/codeql-action) from 3.24.7 to 3.24.9.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](https://github.com/github/codeql-action/compare/3ab4101902695724f9365a384f86c1074d94e18c...1b1aada464948af03b950897e5eb522f92603cc2)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Improve MSBuild search; add latest option

Use vswhere to find MSBuild; add VS2022 support

See:
https://github.com/microsoft/vswhere/wiki/Find-MSBuild

Merge pull request #3985 from facebook/scorecard_permission

try to silence some scorecard warnings

try to silence some scorecard warnings

Merge pull request #3977 from facebook/doc_advanced

Doc update

fix -Werror=pointer-arith in fuzzers (#3983)

Merge pull request #3982 from embg/fuzzer_readme

Document the process for adding a new fuzzer

Merge pull request #3975 from terrelln/2024-03-18-cmake-docs

[cmake] Emit warnings for contradictory build settings

Merge pull request #3979 from yoniko/Werror-fuzz

Fail on errors when building fuzzers

Merge pull request #3981 from terrelln/2024-03-19-generate-sequences

Fix & fuzz ZSTD_generateSequences

Fix & fuzz ZSTD_generateSequences

This function was seriously flawed:
* It didn't do output bounds checks
* It produced invalid sequences when an uncompressed or RLE block was emitted
* It produced invalid sequences when the block splitter was enabled
* It produced invalid sequences when ZSTD_c_targetCBlockSize was enabled

I've attempted to fix these issues, but this function is just a bad idea,
so I've marked it as deprecated and unsafe. We should replace it with
`ZSTD_extractSequences()` which operates on a compressed frame.

Fuzzing and bugfixes for magicless-format decoding (#3976)

* fuzzing and bugfixes for magicless format

* reset dctx before each decompression

* do not memcmp empty buffers

* nit: decompressor errata

Add docs on how to add a new fuzzer

Fail on errors when building fuzzers

Fails on errors when building fuzzers with `fuzz.py` (adds `Werror`).
Currently allows `declaration-after-statement`, `c++-compat` and
`deprecated` as they are abundant in code (some fixes to
`declaration-after-statement` are presented in this commit).

Fix bugs in simple decompression fuzzer (#3978)

Fixes 2 issue in `simple_decompress.c`:
1. Wrong type used for storing the results of `ZSTD_findDecompressedSize` resulting in never matching to `ZSTD_CONTENTSIZE_ERROR` or `ZSTD_CONTENTSIZE_UNKNOWN`.

2. Experimental API is used (`ZSTD_findDecompressedSize`) without defining `ZSTD_STATIC_LINKING_ONLY`.

fix ZSTD_TARGETCBLOCKSIZE_MIN test

when requested CBlockSize is too low,
bound it to the minimum
instead of returning an error.

fix typo

updated API manual

add doc on CCtx UB state

add a paragraph on UB DCtx state after error

update targetCBlockSize documentation

[cmake] Emit warnings for contradictory build settings

Document that the `ZSTD_BUILD_{SHARED,STATIC}` take precedence over `BUILD_SHARED_LIBS` when exactly one is ON.

Thanks to @teo-tsirpanis for pointing out the potentially confusing behavior.

pzstd: use c++14 without conditions

Doing this check with a direct c++ snippet is prone to portability problems:

- \043 is not portable between shells: dash expands it to #,
bash does not;

- using # directly works with make 4.3 but does not with make 4.2.

Let's just use the c++ version that covers both the code and the gtest.

Implement one-shot fallback for magicless format (#3971)

[cmake] Fix up PR #3716

* Make a variable `PublicHeaders` for Zstd's public headers
* Add `PublicHeaders` to `Headers`, which was missing
* Only export `${LIBRARY_DIR}` publicly, not `common/`
* Switch the `target_include_directories()` to `INTERFACE` because zstd uses relative includes internally, so doesn't need any include directories to build
* Switch installation to use the `PublicHeaders` variable, and test that the right headers are installed

Makes it possible to use the lib through FetchContent or ExternalProject_Add

Merge pull request #3973 from facebook/dependabot/github_actions/github/codeql-action-3.24.7

Bump github/codeql-action from 3.24.6 to 3.24.7

Merge pull request #3972 from facebook/dependabot/github_actions/actions/cache-4

Bump actions/cache from 3 to 4

Bump github/codeql-action from 3.24.6 to 3.24.7

Bumps [github/codeql-action](https://github.com/github/codeql-action) from 3.24.6 to 3.24.7.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](https://github.com/github/codeql-action/compare/8a470fddafa5cbb6266ee11b37ef4d8aae19c571...3ab4101902695724f9365a384f86c1074d94e18c)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Bump actions/cache from 3 to 4

Bumps [actions/cache](https://github.com/actions/cache) from 3 to 4.
- [Release notes](https://github.com/actions/cache/releases)
- [Changelog](https://github.com/actions/cache/blob/main/RELEASES.md)
- [Commits](https://github.com/actions/cache/compare/v3...v4)

---
updated-dependencies:
- dependency-name: actions/cache
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

Merge pull request #3969 from facebook/v156_prep

bump version number

Merge pull request #3966 from facebook/debug_lineNumber

add line number to debug traces

update CHANGELOG for v1.5.6