Merge pull request #624 from besser82/topic/besser82/json-c-0.14/rdrand_fixes

json-c-0.14: Detect broken RDRAND during initialization.

Detect broken RDRAND during initialization.

Some CPUs advertise RDRAND in CPUID, but return
0xFFFFFFFF unconditionally.  To avoid locking up
later, test RDRAND during initialization, and if
it returns 0xFFFFFFFF, mark it as nonexistent.

Also fix a possible segmentation fault in CPUID check.

This commit is a squashed backport of the following
commits on the master branch:

  * 0e5bbcaa162ac7850eb4fcd8f91391837d0efb50
  * 4d36b0287d3ab0912ba8a4790340ca099960b2b0
  * 80863140263be5f2dc630938ed8f0066f8a1ab43

Merge pull request #608 from besser82/topic/besser82/json-c-0.14/CVE-2020-12762

json-c-0.14: Fix CVE-2020-12762 - json-c through 0.14 has an integer overflow and out-of-bounds write ...

Fix CVE-2020-12762.

This commit is a squashed backport of the following commits
on the master branch:

  * 099016b7e8d70a6d5dd814e788bba08d33d48426
  * 77d935b7ae7871a1940cd827e850e6063044ec45
  * d07b91014986900a3a75f306d302e13e005e9d67
  * 519dfe1591d85432986f9762d41d1a883198c157
  * a59d5acfab4485d5133114df61785b1fc633e0c6
  * 26f080997d41cfdb17beab65e90c82217d0ac43b

Issue #585: don't install config.h

(cherry picked from commit 8b511c402b73d1d8b195991891c8d44859cb57ec)

Generate docs for the 0.14 release.

Bump version to 0.14 and shared library version to 5.0.0

Fill in the ChangeLog for the upcoming 0.14 release.

Fill in a few more pre-release steps to match what we've done for the 0.14 release.
Move the list of contributors to the AUTHORS file.

Rewrite the issued_closed_for_* files so they display a bit nicer.

Explicitly mark several things in json_tokener deprecated.

Re-do clang-format.

Make json_abort() internal to json_object.c

Make json_tokener_validate_utf8() internal to json_tokener.c, and improve the docs a bit.

Merge pull request #582 from dota17/list_closed_issue_pr

add the list of issues& prs closed

update

manual adjustment

add the list of issues& prs closed

Merge pull request #524 from dota17/addTestCase_obj_token

Increase coverage

Merge pull request #581 from Jehan/wip/Jehan/no-strict-prototypes-windows

CMakeLists: do not enforce strict prototypes on Windows.

remove unsuitable case

CMakeLists: do not enforce strict prototypes on Windows.

On Windows, or at least when cross-built with Mingw-w64, build fails
because strict prototype fails on an included file (thus nothing we can
do about in json-c code):

> from /home/jehan/dev/src/json-c/json_util.c:44:
> /home/jehan/.local/share/crossroad/roads/w64/json-c/include/minwindef.h:196:3: error: function declaration isn't a prototype [-Werror=strict-prototypes]
>   196 |   typedef INT_PTR (WINAPI *FARPROC) ();
>       |   ^~~~~~~
> /home/jehan/.local/share/crossroad/roads/w64/json-c/include/minwindef.h:197:3: error: function declaration isn't a prototype [-Werror=strict-prototypes]
>   197 |   typedef INT_PTR (WINAPI *NEARPROC) ();
>       |   ^~~~~~~
> /home/jehan/.local/share/crossroad/roads/w64/json-c/include/minwindef.h:198:3: error: function declaration isn't a prototype [-Werror=strict-prototypes]
>   198 |   typedef INT_PTR (WINAPI *PROC) ();
>       |   ^~~~~~~

Let's just disable the errors for Windows build.

Merge pull request #580 from besser82/topic/besser82/fix_cmake_tests

Fix CMake tests for enforced strict prototypes.

Fix CMake tests for enforced strict prototypes.

Merge pull request #527 from dota17/arraylist_test

Arraylist testcase

Merge pull request #579 from besser82/topic/besser82/strict-prototypes

Enforce strict prototypes.

clang-format two test_util_file.c and test_object_iterator.c

Increased the test coverage of json_util.c from 76% to 90.3%.

Increased the test coverage of json_object_iterator.c from 0% to 100%

clang-format test1.c

fix valgrind errors

testcase for array_list

Enforce strict prototypes.

Merge pull request #578 from besser82/topic/besser82/pkgconfig_install

CMake: Install pkgconfig file in proper location by default

CMake: Install pkgconfig file in proper location by default

The default location for pkconfig files on most systems is:
  ${CMAKE_INSTALL_LIBDIR}/pkgconfig

Thus the file should get installed in there by default.

Merge pull request #577 from besser82/topic/besser82/json_c_version_test_verbose_on_fail

Be verbose on failing json_c_version test.

Be verbose on failing json_c_version test.

Merge pull request #576 from besser82/topic/besser82/increase_coverage

Test coverage for json_c_version.

Merge pull request #575 from besser82/topic/besser82/cosmetics

Small cosmetics.

Test coverage for json_c_version.

Remove multiple trailing newlines at EOF.

gitignore: Add build folder.

Clean trailing white-space.

Fix line terminators to be UNIX.

Update README.md to remove autoconf instructions in favor of cmake.

Merge pull request #572 from besser82/topic/besser82/cmake_fixes

Small fixes to CMakeLists

Merge pull request #573 from besser82/topic/besser82/coverage

Fix coveralls submission.

Fix coveralls submission.

coveralls-cpp is dead and does not work anymore.
Fortunately there is coveralls-lcov available.

CMake: Check whether BSYMBOLIC is supported by the linker.

Linking with '-Bsymbolic-functions' is supported for ELF-binaries, only.

CMake: Build Doxygen documentation out-of-tree.

CMake: Fix appending of linker flags.

There was a typo in `CMAKE_SHARED_LINKER_FLAGS`.

List the set of contributors since the 0.13 branch

Fix one more assert("!invalid cint_type") to use json_abort(...) instead.

Issue #568: fix the strtoll and strtoull handing so config.h ends up creating defines for those only when needed, which should exclude mingw environments.

Remove autoconf files from .gitignore.

Update the oss-fuzz build script to try to get it to work with cmake.

Remove autoconf machinery, we're using cmake now.  Update the release checklist to account for that, plus fill in other tasks and clean it up a bit.

osx builds also need -DDISABLE_BSYMBOLIC=ON

I have NFC why osx fails when "set -e" is in effect, but leave it off and clean up the rest of the debuging code in the travis config.

Not all systems (e.g. osx) have malloc.h, include stdlib.h instead.

Try yet again to fix the osx build, this time do set +e

Another attempt at debugging the travis osx build, this time just ignore errors.

Wait a while before exiting travis to allow output to flush.

Apparently cmake is already installed on travis osx builds.  Try another approach to figure out what was failing.

Install cmake on osx travis builds.  s/make check/make test/

Perform the travis build in a subdir, so the distcheck target works.

Switch travis to use cmake instead of autoconf.

Add includes and split off json_types.h to help ensure that headers can be included in any order.

Don't let clang-format rearrange Windows includes, since they need to be in a particular order.

Format random_seed.c with clang-format, with key asm sections excluded.

Update the style guide to mention that we're using clang-format now.

Merge pull request #555 from dota17/chang_format_3

Format json-c with clang-format tool

clang-format the files

add the disabling formatting coments and adjust the partial code manuly

add options

clang-format

Merge pull request #563 from robybeen/master

Changed order of calloc args to match stdlib

Changed order of calloc args to match stdlib (2)

Although it is currently working, it's worth to stick with the stdlib definition to avoid further problems

Changed order of calloc args to match stdlib

Although it is currently working, it's worth to stick with the stdlib definition to avoid further problems

Merge pull request #562 from dota17/testcase_add

Bugfix and testcases supplements

add test cases

Merge pull request #561 from dota17/add_badge

add the badge in README.md and test the coveralls

add the badge in README.md and test the coveralls

Merge pull request #541 from dota17/coveralls_final

add coveralls auto tool to json-c

modify the coveralls configuration file

Merge pull request #558 from dota17/doc_message_modify

modify the doc target message

modify the doc target message

Merge pull request #556 from Jehan/wip/Jehan/fix-broken-mingw-w64

Fixes various Wreturn-type and Wimplicit-fallthrough errors on Mingw-w64

Fixes various Wreturn-type and Wimplicit-fallthrough errors on Mingw-w64

This is a recent regression since commit
6359b798479d379a3202e02c6a938d9b40c0d856 which added various assert(0)
calls (often replacing return-s).
With Ming-W64 compiler, json-c build was failing with various errors of
the sort:

> /home/jehan/dev/src/json-c/json_object.c: In function 'json_object_int_inc':
> /home/jehan/dev/src/json-c/json_object.c:841:1: error: control reaches end of non-void function [-Werror=return-type]
>   841 | }
>       | ^
> In file included from /home/jehan/dev/src/json-c/json_object.c:17:
> /home/jehan/dev/src/json-c/json_object.c: In function 'json_object_get_double':
> /home/jehan/.local/share/crossroad/roads/w64/json-c/include/assert.h:76:4: error: this statement may fall through [-Werror=implicit-fallthrough=]
>    76 |   (_assert(#_Expression,__FILE__,__LINE__),0))
>       |    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> /home/jehan/dev/src/json-c/json_object.c:1070:7: note: in expansion of macro 'assert'
>  1070 |       assert(0);
>       |       ^~~~~~
> /home/jehan/dev/src/json-c/json_object.c:1072:3: note: here
>  1072 |   case json_type_boolean:
>       |   ^~~~

The problem is that Mingw-w64 does not consider assert() as a noreturn
(even assert(0)), because it has to be compatible by Microsoft
libraries. See the discussion here:
https://sourceforge.net/p/mingw-w64/bugs/306/

Instead let's create a new json_abort() function which is basically just
an abort() function with an optional message, for such cases where
abortion was non-conditional (using assert() and using the assertion
condition as a message here was clearly a misuse of the function). And
mark json_abort() as 'noreturn', as well as 'cold' for optimization
purpose (this is code we expect to never run, unless there is a bug,
that is).

Finally let's use this json_abort() instead of previous misused assert()
calls.

Merge pull request #546 from dota17/make_unistall

Add uninstall target in cmake

delete -r in uninstall

Merge pull request #547 from dota17/assert_test

modify json-c default build type, and fix up the assert() errors in t…

modify RELEASE

update testcase and delete debug -O0 build

Follow up the PR#542: improve assert() calls, simplify code in json_object_equal().

Merge pull request #542 from dota17/adduint64_final

add uint64 data to json-c

modify json-c default build type, and fix up the assert() errors in testcase

update json_object.c and testcase, delete json_object_uint_inc()

modify partial functions and testcase, in order to support automatic conversion for int64/uint64

Add uninstall target in cmake