Merge pull request #899 from Seagate/meson_support

Meson support

Merge branch 'json-c:master' into meson_support

Merge pull request #898 from Seagate/fix_cmake_clangcl

bug: Fixing Cmake build when using clang-cl

Merge pull request #895 from nim65s/patch-1

CMake: sync minimum version

Removing werror option as meson has a built-in option for this

Signed-off-by: Tyler Erickson <tyler.erickson@seagate.com>

bug: Fixing runtime issue with test_util_file.c in Windows

Windows was failing tests due to how the Windows API works with some of the calls used.
When opening and reading a file O_BINARY is needed otherwise it fails for size checks. This is due to how Windows handles newlines and counts between text and binary mode file reads.

Also fixed is the test for /dev/null.
In Windows, this fails, but crashes due to a missing return statement when it cannot open this file.
I also tried telling windows to open the special file NUL, however that leads to a CRT crash later in the test that cannot be stopped and will always fail. Rather than fail a test that Windows will always fail, it has been disabled in that specific case.

Signed-off-by: Tyler Erickson <tyler.erickson@seagate.com>

make: Adding support for building json-c with meson

Adding meson build files for json-c that work similarly to the cmake build files.
Where it made sense, I reused existing cmake .h.in files or generated entirely from meson.
All tests were done with GCC and Clang in ubuntu 24.04, Windows using MSVC 2022 and Clang-cl from llvm's repo using version 21.1.3

Signed-off-by: Tyler Erickson <tyler.erickson@seagate.com>

bug: Fixing Cmake build when using clang-cl

Clang-cl will fail to build and produce warnings about redefining existing symbols, mostly for float.h and math.h compatibility.

To resolve this, this moves the clang-cl detection earlier in the CMakeLists.txt so that CLANG_CL can be checked properly where there is an existing MSVC workaround for these symbols.
This resolves the build using the latest clang-cl from the LLVM clang repo as well as clang-cl that can be installed with MSVC 2022.

Signed-off-by: Tyler Erickson <tyler.erickson@seagate.com>

Merge pull request #893 from sffc/supplemental-code-point-bug

Fix bug involving supplemental code points that look like high surrogates

CMake: sync minimum version

Fix code and update tests

Add tests with current behavior

Make json_parse a bit more useful by adding -u (validate UTF8) and -P (specify arbitrary tokener parse flags), and read from stdin if no filename is provided.

Issue #867 - also disallow control characters in keys

Merge pull request #884 from careles/mingw_win64-WIN32_LEAN_AND_MEAN-redefined

fix macro WIN32_LEAN_AND_MEAN redefined

fix macro WIN32_LEAN_AND_MEAN redefined

json-c as add_subdirectory and project cmakelists.txt has add_definitions(-DWIN32_LEAN_AND_MEAN)

Merge pull request #872 from GitMensch/patch-2

fix linkhash breaking -std=c89

Merge pull request #847 from sacredbanana/amiga

Add support for Commodore Amiga

Merge pull request #882 from simonresch/handle-nan-to-int-conversion

Explicitly handle NaN values when converting to integer

Add support for MorphOS

Remove linking to libamiga and libc

Do not use duplocale if building for libnix because it isnt supported yet

Clean up CMakeLists.txt and remove Amiga specific locale.h guard

Fix Readme

Get working ror libnix and clib2

Add support for Commodore Amiga

Merge pull request #879 from janotomko/null

Handle NULL gracefully in json_tokener_free

Set error codes for truncating int conversions

json_object_get_int/int64/uint64() now sets errno to ERANGE when the source value
can't be represented in the target type.

Explicitly handle NaN values when converting to int

Json objects of type double with the value NaN could cause undefined
behavior when casting double to int in `json_object_get_int`.

Merge pull request #878 from 0xfadead/readme-fix-underline

Fix underline bug in README

Merge pull request #883 from simonresch/add-oss-fuzz-tests

Add fuzz tests for json_object/point/array apis

Add fuzz tests for json_object/point/array apis

Extends the coverage of fuzz tests executed in OSS-Fuzz.

Issue #881: don't allow json_tokener_new_ex() with a depth < 1

Fix issue #875: cast to unsigned char so bytes above 0x7f aren't interpreted as negative, which was causing the strict-mode control characters check to incorrectly trigger.

Handle NULL gracefully in json_tokener_free

Similarly to glibc's free, make json_tokener_free(NULL)
a no-op, to simplify cleanup paths.

Signed-off-by: Ján Tomko <jtomko@redhat.com>

Fix the apps/json_parse "-s" (strict) option so it actually does something, and default to non-strict.

Fix underline bug in README

Merge pull request #873 from esiaero/master

Correct year in changelog for 0.18 release

Correct year in changelog for 0.18 release

fix linkhash breaking -std=c89

making the definition of _LH_INLINE depending on __STDC_VERSION__, not on a specific compiler

Update openssl command to work for newer versions of openssl.

Update the master branch to version 0.18.99

Update abi-check.sh to compare 0.16 and 0.18

Fix the expected output for test_parse

Update ChangeLog, AUTHORS and issues_closed_for_0.18.md for an upcoming 0.18 release.

Merge pull request #871 from json-c/disallow-control-chars

Issue #867: disallow control characters in strict mode.

Issue #867: disallow control characters in strict mode.

Merge pull request #864 from pavelzw/patch-1

Build pkg-config for msvc as well

Update CMakeLists.txt

Build pkg-config for msvc as well

Merge pull request #861 from bhaible/handle-another-oom-condition

Handle yet another out-of-memory condition.

Although CMAKE_C_COMPILER_FRONTEND_VARIANT is defined, it's not always set to some value.  Fix builds for cases when it's blank.

Handle yet another out-of-memory condition.

duplocale() can return NULL, with errno set to ENOMEM.
In this case, bail out and set the current error code to
json_tokener_error_memory.

Issue #857: fix a few places where json_tokener should have been returning json_tokener_error_memory but wasn't.

Merge pull request #858 from alurm/master

json_tokener.h: json_tokener_parse_verbose: fix typo

json_tokener.h: json_tokener_parse_verbose: fix typo

Issue #829: attempt to detect clang-cl.exe and pass MSVC-compatile command line arguments.

Issue #850: fill in & adjust the "@returns" docs for several functions.

doxygen can't handle html on heading, split that to separate lines
https://github.com/doxygen/doxygen/issues/10466

Fix issue #854: Set error=json_tokener_error_memory in json_tokener_parser_verbose() when allocating the tokener fails.

Merge pull request #852 from fedefrancescon/docs/micro-typo-fix

docs: micro typo in json_object_put doc

docs: micro typo in json_object_get doc

Merge pull request #831 from neheb/1

rename WIN32 to _WIN32

rename WIN32 to _WIN32

The latter is the proper macro defined by Windows headers.

Fixes compilation under at least clang-cl which mandates function
declarations.

Signed-off-by: Rosen Penev <rosenp@gmail.com>

Merge pull request #849 from rouault/random_seed_toctou

random_seed.c: add a Coverity Scan suppression

random_seed.c: add a Coverity Scan suppression

Coverity Scan warns about the use of open() after stat() being a
potential TOCTOU (Time of check time of use) issue. But here there is no
such issue.

Merge pull request #846 from evo-i/CMP0153

Use execute_process instead of exec_program

Update CMakeLists.txt

Replace with `execute_process` available on v3.0+

Update CMakeLists.txt

Now saved to variable.

Fix ECHO_OUTPUT_VARIABLE

[CMP0153](https://cmake.org/cmake/help/latest/policy/CMP0153.html)

Issue #842 - fix one particular sign conversion warning.
There are many others that show up if we were to add -Wsign-conversions,
but this is the only one using a literal constant.

Merge pull request #839 from ffontaine/master

json_pointer.c: initialize idx

json_pointer.c: initialize idx

Fix the following build failure with gcc 5:

/home/thomas/autobuild/instance-2/output-1/build/json-c-0.17/json_pointer.c: In function 'json_pointer_result_get_recursive':
/home/thomas/autobuild/instance-2/output-1/build/json-c-0.17/json_pointer.c:193:25: error: 'idx' may be used uninitialized in this function [-Werror=maybe-uninitialized]
    res->index_in_parent = idx;
                         ^

Fixes:
 - http://autobuild.buildroot.org/results/523b35a979d59121fe4e18c38171792b06233940/

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>

Merge pull request #837 from HansolChoe/fix-tokener-parse-ex-fuzzer

Add type checking for parsed objects in tokener_parse_ex_fuzzer

tokener_parse_ex_fuzzer: add type checking for parsed object

Take 2 fixing the placement of json_tokener_error_memory in the enum.  (json_tokener_error_size is an actual error, *not* a measure of the size of the enum!)

Merge pull request #827 from robert-pang/master

Resolve build issue in tokener_parse_ex_fuzzer.c

Merge pull request #826 from emlix/cmake-policies

CMake: remove needless policy setting

Fix variable and API names in tokener_parse_ex_fuzzer.cc to resolve build issue.

CMake: request all policies up to CMake 3.12 being set to NEW

CMake: request consistent minimum version

CMake: remove needless policy setting

These policies were all introduced before CMake 3.9, so they will automatically
be initialized to the new behavior when requesting a minimum version of 3.9.

Update the master branch to version 0.17.99

Add an abi-check.sh script that shows how to run the abi-compliance-checker, update release creation steps slightly.

Also fix messages returned from json_tokener_error_desc() (broke due to the ordering change of enum json_tokener_error)

Move the json_tokener_error_memory entry to the end of enum json_tokener_error to restore binary compatibility with 0.16

Update the AUTHORS file and add issues_closed_for_0.17.md for the upcoming 0.17 release

Update the ChangeLog for 0.17 based on notable commits since 0.16

Fix issue #823: add back json_number_chars, but only because it's part of the public API.

Merge pull request #679 from GitMensch/patch-1

Let json-c be used with obsolete compilers

PR #679: add workaround for old compilers w/o stdint.h (i.e. VS2008 and earlier)

Merge pull request #822 from SSharshunov/master

Added option to disable app build

Added option to disable app build

Merge pull request #702 from commodo/json_patch

json_patch: add first implementation only with patch application

Don't export json_pointer_get_internal, move json_object_array_insert_idx and json_patch_apply to the JSONC_0.17 section in json-c.sym

Work around a somewhat misleading warning about "a function declaration without a prototype is deprecated in all versions of C" in test1.c

Fix json_patch_apply handling of removing the whole document (i.e. "path":"").
Enable all disabled tests, add a few more including some with null documents.

Adjust the behavior of the args passed to json_patch_apply to make it easier to do in place modifications, and add a struct json_patch_error to report more details on failures.

Fix an uninitialized memory access in json_pointer.
Add comments describing when the fields of the internal struct json_pointer_get_result are valid.