Merge pull request #325 from rouault/fix_stack_overflow_in_json_object_double_to_json_string_format

Fix stack buffer overflow in json_object_double_to_json_string_format()

Fix stack buffer overflow in json_object_double_to_json_string_format()

Issue originally found in the json-c 0.11 internal copy in GDAL but also found
in latest git version.

If doing things like
json_object* obj = json_object_new_double(1e300);
json_object_set_serializer(obj, json_object_double_to_json_string, "%f", NULL);
json_object_to_json_string(obj)

    size = snprintf(buf, sizeof(buf),
        format ? format :
          (modf(jso->o.c_double, &dummy) == 0) ? "%.17g.0" : "%.17g",
          jso->o.c_double);
will return a value greater than 128 since at least 300 characters are needed.
This value is then passed to printbuf_memappend(pb, buf, size); that tries to
read size bytes in buf.

So we should clamp size to sizeof(buf). And on Windows, _snprintf() returns -1
in that situation, so deal also with this case.

Fixes https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=1682
Credit to OSS-Fuzz

Merge pull request #311 from jj1118/master

Fix error C3688 when compiling on Visual Studio 2015

Fix error C3688 when compiling on Visual Studio 2015: invalid literal suffix 'PRId64'; literal operator or literal operator template 'operator ""PRId64' not found

Merge pull request #310 from marxin/fix-gcc7-warnings

Add FALLTHRU comment to handle GCC7 warnings.

Add FALLTHRU comment to handle GCC7 warnings.

Merge pull request #305 from alexanderst/master

Fix compilation without C-99 option

Fix compilation without C-99 option

Add a few more cases to test_parse to provide some examples of how parsing works; should help address the questions raised in Issue #302.

Merge pull request #303 from yogo1212/tokener_h

fix doc in tokener header file

fix doc in tokener header file

Issue #300: Cast size_t values to int instead of unsigned long long to work around broken behavior of MinGW.

Issue #300: Some platforms, such as MinGW, don't have ETXTBSY and ENOTBLK, so skip those there.

Issue #300: Use %p instead of %lx in test_util_file since some platforms' pointers are larger than "unsigned long".
Also, there's no need to worry about output consistency here, since it'll be
 different anyway due to different pointer values.

Add cast to int in calls to isdigit() since some platforms complain about char being used as an array index.

Merge pull request #299 from qlyoung/perf-improvements

Improve json_object -> string performance

Restore sprintbuf(), add macro for string literals

Hawciz pointed out that the previous commit modifies the public
interface of printbuf. Per his suggestion, sprintbuf() was restored
and a new pair of macros was added that wraps printbuf_memappend().

Using a wrapper macro instead of modifying sprintbuf() also reduces
function call overhead, bringing total performance gains to
approximately 400%.

Use strdup instead of strndup in test1.c, there's no need for the latter because json_object_to_json_string_length() will always return a properly sized string.

Improve json_object -> string performance

Removes variadic prints for ~3x performance improvement.

Issue #295: also check if size_t is the size of long long, to help support 64-bit Windows platforms.

Ignore tests/test_float

Change a memcpy that should be a memmove within json_pointer_get, and fix memory leaks in of one the json_pointer tests.

Note some minimum versions needed for autoconf, etc...

Issue#292: bump the required version of configure listed in configure.ac so a more useful error message is generated, rather than failing later because the extra macros from ./autoconf-archive/m4/ didn't get loaded.

Merge pull request #289 from jgerhards/i-278

bugfix: floating point representaion without fractional part

testbench: add test for floating point representation

bugfix: floating point representaion without fractional part

closes https://github.com/json-c/json-c/issues/278

Merge pull request #287 from commodo/json_pointer_va_args

json_pointer: extend setter & getter with printf() style arguments

vasprintf_compat.h: spin-off this compat header ; use math_compat.h as template

Signed-off-by: Alexandru Ardelean <ardeleanalex@gmail.com>

strdup_compat.h: re-spin this compat header ; use math_compat.h as template

Signed-off-by: Alexandru Ardelean <ardeleanalex@gmail.com>

test_json_pointer: update test with a few printf variants

Signed-off-by: Alexandru Ardelean <ardeleanalex@gmail.com>

json_pointer: add json_pointer_getf/setf() function variants

These include support for printf() style args for path.

Adds support for calling with 'json_pointer_getf(obj, &res, "/foo/%d/%s", 0, bar)'
style args.

Makes it easier for doing more dynamic stuff/magic, without
needing to use vasprintf() externally.

Signed-off-by: Alexandru Ardelean <ardeleanalex@gmail.com>

json_pointer.c: fix whitespace

Signed-off-by: Alexandru Ardelean <ardeleanalex@gmail.com>

Merge pull request #285 from ebassi/revert-strdup-compat

Revert "compat/strdup.h: move common compat check for strdup() to own…

Revert "compat/strdup.h: move common compat check for strdup() to own file"

This reverts commit aaba8c1080533133329086e47f4f45981dfdd1ee.

This commit breaks builddir != srcdir build, but, more importantly, it
also adds a dependency on a header, "config.h", which is not installed
and it's supposed to be private — since it's generated at configuration
time and it's not meant to be used by projects compiling against a
library.

Merge pull request #283 from commodo/json_pointer

json_pointer: add first revision based on RFC 6901

tests: add test_json_pointer test

Signed-off-by: Alexandru Ardelean <ardeleanalex@gmail.com>

tests/strerror_override.c: fix compilation error

I got this on Mac OS X at least.
Not sure if it shows up in other envs

error:
```
strerror_override.c:53:13: error: incompatible redeclaration of library function 'strerror' [-Werror,-Wincompatible-library-redeclaration]
const char *strerror(int errno_in)
```

Signed-off-by: Alexandru Ardelean <ardeleanalex@gmail.com>

json_pointer: add first revision

Signed-off-by: Alexandru Ardelean <ardeleanalex@gmail.com>

compat/strdup.h: move common compat check for strdup() to own file

Signed-off-by: Alexandru Ardelean <ardeleanalex@gmail.com>

Handle NULL objects in json_object_get_userdata() by returning NULL, but abort in json_object_set_userdata() since we can't actually do anything with the userdata.

Issue #280: add a few assert() calls to ensure objects are of the appropriate type.

Use json_visit() to clear the serializer data set when parsing so the output from test_parse reflects the actual values parsed.

Add a json_c_visit() function to provide a way to iterate over a tree of json-c objects.

Fix broken key-order test in test_compare.

Merge pull request #272 from sgerbino/master

Allows for compilation with CMake on macOS

Merge pull request #277 from ALLTERCO/json_object_set_xxx

Json object set xxx

try restore windows automated builds

fix compiler warning for int sizes

string set and tests

gitignore test; add double value set checks in test

some simple tests

Issue #275: fix out of bounds read when handling unicode surrogate pairs.

some basic set

Merge pull request #273 from konoal/master

fixed wrong object name in json_object_all_values_equal

Merge pull request #274 from pepevel/master

Support for 64 bit pointers on Windows

Support for 64 bit pointers on Windows

fixed wrong object name in json_object_all_values_equal

Changed CMake variables and adding output for debugging

Target C99 standards to get rid of warnings

Modified the execute_process commands

Changed PROJECT_SOURCE_DIR to CMAKE_CURRENT_SOURCE_DIR to fix building when used as external project

Modified CMakeLists.txt to use autotools to generate configuration for Unix systems

Lowered CMake required version so that Travis CI is able to build, moved include folder to include/json-c to match the behavior of autotools

Trailing slash seems necessary in the install command to get contents

Forget DESTINATION in CMake install command

We need to also install the generated configuration files

Fixed typo in CMakeList.txt FILES_MATCHING

Added install instructions for CMake

Resolves json-c/json-c#271 by adding MacOS configuration files and modifies CMakeLists.txt to use them

json_object_set_boolean for upstream style check

Merge pull request #267 from sixlettervariables/patch-1

Removes me as Win32 maintainer, because I'm not.

Removes me as a maintainer, because I'm not.

Updates README-WIN32.html to remove my status as a maintainer (I'm not). Also removes ancient porting notes and adds a reference to json_config.h.win32 which is where most folks should go to start with Windows issues.

Issue #263: add const so the prototype for json_object_object_add_ex() matches the definition.

Add Cmakefile

Fix memory leak and inconsistent output (due to varying fd numbers) in test_util_file.

Issue #260: add a check to prevent trivial loops in the object tree, even though it is up to the callers to avoid doing so in more complex cases.

json_object_object_add_ex() should have shared code with json_object_object_add(), and been changed to return int at the same time the latter was.  Do that now.
Also correct a couple of calls to _to_json_string to check the return value.

Merge branch 'RyDroid-tests'

Merge branch 'tests' of https://github.com/RyDroid/json-c into RyDroid-tests

Conflicts:
tests/test_util_file.c

Merge pull request #251 from RyDroid/editorconfig

Adding a file for EditorConfig

Very minor changes to some tests

Adding a file for EditorConfig

Revert bogus change to json_object_get() made in f40b08d8 that caused it to always return NULL. (Issue #259)
Also undo NULL check in json_tokener_set_flags(): it's not at all valid toi
 try to set flags on a NULL pointer, and doing so should not be silently
 ignored.

Merge pull request #255 from RyDroid/minor-c2

Minor changes in C source code

Merge pull request #252 from RyDroid/minor-no-c

Very minor changes not related to C source code

Merge branch 'jobol-issue-165'

Merge branch 'issue-165' of https://github.com/jobol/json-c into jobol-issue-165

Conflicts:
tests/test_util_file.c

Merge pull request #253 from RyDroid/cppcheck-travis

Adding a test with cppcheck for Travis CI

Merge pull request #249 from RyDroid/readme

Improving README

Add a brief style guide.  Address Issue #257.

Merge pull request #250 from RyDroid/gitignore

Improving .gitignore

Improving .gitignore

Improving README

Adding a test with cppcheck for Travis CI

Minor changes in C source code

Very minor changes not related to C source code

Issue #246: Include xlocale.h too, to get locale_t defined, since not all OSes include that in locale.h

test: add  test of 'json_object_to_json_string_length'

The test is implied in test1.

The idea is to check that the returned lengths and strings
are identical to what is expected to return the already
tested function 'json_object_to_json_string_ext'.

Signed-off-by: José Bollo <jose.bollo@iot.bzh>

Issue #195: Actually call uselocale() in the new locale handling code in json_tokener.
Also, be sure the right locale_t is freed if we fail on the second uselocale.
Finally, fix test_locale so it *doesn't* use json_object_to_json_string as
 that will simple re-emit the original parsed string values.