Check for NULL in cJSON_DetachItemViaPointer

Revert "feat: add tests for #842" to fix test failures

This reverts commit 5b502cdbfb21fbe5f6cf9ffbd2b96e4281a741e6.

Related to #860

Fix spelling errors found by CodeSpell.  See https://github.com/codespell-project/codespell

Fix indentation (should use spaces)

Signed-off-by: DL6ER <dl6er@dl6er.de>

add contributors

Release 1.7.18

feat: add tests for #842

Add some tests for setting NULL to deallocated pointers
releated to #842 and #833

Set free'd pointers to NULL whenever they are not reassigned immediately after

fix: remove misused optimization flag -01

related to #850

Fix heap buffer overflow

Fixes #800

Add test for heap buffer overflow

From #800

Remove non-functional list handling of compiler flags

update comments and add tests for cJSON_SetValuestring

fix: fix incorrect name in security.md

Related to #845

Create SECURITY.md

Add NULL check to cJSON_SetValuestring()

If the valuestring passed to cJSON_SetValuestring is NULL, a null pointer dereference will happen.

This commit adds the NULL check of valuestring before it is dereferenced.

Release 1.7.17

update version to 1.7.17

fix error in null checkings (#810)

fixes #802 and #803

add NULL checkings (#809)

* add NULL checks in cJSON_SetValuestring

Fixes #803(CVE-2023-50472)

* add NULL check in cJSON_InsertItemInArray

Fixes #802(CVE-2023-50471)

* add tests for NULL checks

add tests for NULL checks in cJSON_InsertItemInArray and cJSON_SetValuestring

Release 1.7.16 (#770)

* Update version to 1.7.16

* Update contributors

upgrade clang to fix actions error (#768)

Actions builds are failing because clang-8 is failing to be installed. Upgrade clang-8 to clang-14 to fix this.

Add meson documentation (#761)

Fix a null pointer crash in cJSON_ReplaceItemViaPointer (#726)

If the parent passed in cJSON_ReplaceItemViaPointer has not a child, which means parent->child is null, a null pointer dereference crash will be happened inside cJSON_ReplaceItemViaPointer.

This commit adds the NULL check of `parent->child` beforehand to inform user such incorrect usage.

Signed-off-by: hopper-vul <hopper.vul@gmail.com>

fix: add allocate check for replace_item_in_object (#675)

Signed-off-by: Junbo Zheng <zhengjunbo1@xiaomi.com>

feat: add cJSON_SetBoolValue and test (#639)

* cJSON_SetBoolValue plus test

* cJSON_Invalid insted of just 0

* Update tests/misc_tests.c

* VSCode standard C formater applied

Co-authored-by: Alan Wang <wp_scut@163.com>

docs: Fix README typo (#664)

chore: add CIFuzz integration (#437)

* CIFuzz integration

* Rename main.yml to ci-fuzz.yml

chore: ignore all .dylib files (#628)

This fixes some .dylib files being flagged as added when compiled e.g. `libcjson.dylib.1.7.14`

fix: 'cjson_utils-static' target not exist(#625)

* Update CMakeLists.txt

fix the bug:when build with cmake using option '-DBUILD_SHARED_AND_STATIC_LIBS=ON -DENABLE_CJSON_UTILS=ON', build sucess,
but use cmake comand 'find_package(cjson CONFIG)', 'cjson_utils' target is available,but  'cjson_utils-static' target not exist.

add an option for ENABLE_CJSON_VERSION_SO in CMakeLists.txt (#534)

Co-authored-by: m00209177 <malihu@huawei.com>

fix: print int without decimal places (#630)

chore: ignore *.lst/*.lss file (#623)

build: Bump cmake version and use new version syntax (#587)

Co-authored-by: Alan Wang <wp_scut@163.com>

docs: update comment (#622)

fix: add cmake_policy to CMakeLists.txt (#613)

Use the cmake_policy() command to set CMP0054 to NEW explicitly.

fix: for issue #569, now use the guard to turn it off (#617)

fix: potential memory leak in merge_patch() (#611)

chore: update version and changelog (#610)

fix: remove redundant condition (#605)

Typos found by codespell (#607)

Update .gitattributes (#544)

Fix potential core dumped for strrchr (#546)

Remove always true condition in cJSON.c (#539)

Fix null pointer crash, closes #536 (#538)

docs: fix simple typo, transfering -> transferring (#527)

There is a small typo in tests/readme_examples.c.

Should read `transferring` rather than `transfering`.

fix several null pointer problems on allocation failure (#526)

Merge pull request #519 from Alanscut/issue-516

fix a possible dereference of null pointer

fix a possible dereference of null pointer

Merge pull request #518 from fpnuseis/master

fix: windows build failure about defining nan

fix: windows build failure about defining nan

Merge pull request #505 from Alanscut/release-1.7.14

Release 1.7.14

Update changelog and contributors

Update version to 1.7.14

Merge pull request #503 from Alanscut/issue499

optimize the way to find tail node

optimize the way to find tail node

Merge pull request #502 from Alanscut/nan

remove float-divide-by-zero for supporting NAN

remove float-divide-by-zero for supporting NAN

Merge pull request #501 from Alanscut/actions

add github actions CI

add github actions CI

Merge pull request #484 from sappo/master

Problem: WError error on macosx because NAN is a float

Merge pull request #485 from HuKeping/fix-git-archive

Remove unnecessary files in release tarball

Remove unnecessary files in release tarball

Prior to this patch, we would find '.gitignore', '.travisCI.yml' in
the release tarball.

This patch adds a few entries in .gitattributes to specify files that
should never end up in a distribution tarball.

Signed-off-by: Hu Keping <hukeping@huawei.com>

Problem: WError error on macosx because NAN is a float

Solution: Add explicit cast from NAN to double

fix #376

Merge pull request #472 from Alanscut/parse-array

array's item should be in the list

array's item should be in the list

Merge pull request #456 from miaoerduo/master

fix: some bugs in detach and replace, resolved #467

Merge pull request #469 from moorthy-bs/master

pkgconfig: cjson include dir added

Merge pull request #465 from Summus-git/fix_make_install_config

Make install unwanted config files

pkgconfig: cjson include dir added

fixed #468

Fix make.sh install unwanted config files

fix: errors in replacing the first item when array_size is 1, and replacing the last item

fix: error list head's prev when detach the last item

feat: set list head's prev in parse_array and parse_object

Update changelog

Update version to 1.7.13

fix error C2124 in visual studio

Merge pull request #454 from Alanscut/float-compare

comparing double value with DBL_EPSILON

Merge pull request #453 from Alanscut/add-return-value

add return value for  cJSON_AddItemTo... and cJSON_ReplaceItemxxx

Merge pull request #451 from Alanscut/20200324

add new function of `cJSON_SetValuestring`

Add getNumberValue function

* Add GetNumberValue function and testcase

Co-authored-by: Alan Wang <wp_scut@163.com>

Added cJSON_ParseWithLength (#358)

Co-authored-by: Caglar Ivriz <caglar.ivriz@siemens.com>

update testcase, fixes #433

comparing double value with DBL_EPSILON

update testcase

add return value for cJSON_ReplaceItemxxx

false has been redefined to cJSON_False

add return value for  cJSON_AddItemTo...

update cJSON_SetValuestring and testcase

update README

add testcase for cJSON_SetValuestringToObject

add new function of setValuestringToObject

Improve performance of adding item to array (#448)

* use prev pointer when adding item to array

Co-authored-by: xiaomianhehe <hongshaokai@hotmail.com>
Co-authored-by: Alanscut <scut@163.com>
Date:   Tue Feb 18 11:54:23 2020 +0800

* add testcase for cJSON_DeleteItemFromArray

update contributors

Merge pull request #447 from Alanscut/fix_encode_string_as_pointer

Fix encode_string_as_pointer

fix encode_string_as_pointer

Merge pull request #440 from Square789/patch-1

updating doc

Merge pull request #446 from DaveGamble/revert-430-array_add

Revert "use prev pointer when array do adding"

Revert "use prev pointer when array do adding (#430)"

This reverts commit e8077d01500279a7b45b8cd7a0ae94ea7ad5748a.

Spelling; "it's"->"its"