Compare commits
52 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
afd5d186b7 | ||
|
|
7456637793 | ||
|
|
d92754cd30 | ||
|
|
e4980b65b9 | ||
|
|
629c354390 | ||
|
|
1b2236a9a6 | ||
|
|
1925d1bbe5 | ||
|
|
76b705576f | ||
|
|
88d66c5da9 | ||
|
|
954d61e5e7 | ||
|
|
ecdff7837c | ||
|
|
a3c2eba991 | ||
|
|
d3bc571a38 | ||
|
|
48eaecd172 | ||
|
|
18ad8a8770 | ||
|
|
93227319f0 | ||
|
|
f0c1b896ba | ||
|
|
2d252ae595 | ||
|
|
c46c4d1559 | ||
|
|
1af74c8cc1 | ||
|
|
9bdf19fde1 | ||
|
|
bdd5ff7ad6 | ||
|
|
24ea388dcf | ||
|
|
39745c9c75 | ||
|
|
9585c38d5a | ||
|
|
c268e77b21 | ||
|
|
9f745a2251 | ||
|
|
010e31f2f2 | ||
|
|
8ea37fce01 | ||
|
|
b2fe02712d | ||
|
|
c179509b31 | ||
|
|
46c4f55c94 | ||
|
|
90ff72c8bb | ||
|
|
e9d1de24cf | ||
|
|
56f2bc6f3e | ||
|
|
cdc35ebf88 | ||
|
|
90a46eaccd | ||
|
|
2a3a313f83 | ||
|
|
a2a2411b12 | ||
|
|
03ba72faec | ||
|
|
569aa060c6 | ||
|
|
b9cc911831 | ||
|
|
9abe75e072 | ||
|
|
9189b3322a | ||
|
|
bfbd8fe0d8 | ||
|
|
82295f9e4f | ||
|
|
38b44a298d | ||
|
|
00d5e225a6 | ||
|
|
9ecc96878f | ||
|
|
3efee9fda8 | ||
|
|
71a7b64860 | ||
|
|
ddf268b074 |
45
CHANGELOG.md
45
CHANGELOG.md
@@ -1,3 +1,48 @@
|
||||
1.5.9
|
||||
=====
|
||||
* Set the global error pointer even if `return_parse_end` is passed to `cJSON_ParseWithOpts`. See #200, thanks @rmallins
|
||||
|
||||
1.5.8
|
||||
=====
|
||||
* Fix `make test` in the Makefile, thanks @YanhaoMo for reporting this (#195)
|
||||
|
||||
1.5.7
|
||||
=====
|
||||
Fixes:
|
||||
------
|
||||
* Fix a bug where realloc failing would return a pointer to an invalid memory address. This is a security issue as it could potentially be used by an attacker to write to arbitrary memory addresses. (see #189), fixed in (954d61e5e7cb9dc6c480fc28ac1cdceca07dd5bd), big thanks @timothyjohncarney for reporting this issue
|
||||
* Fix a spelling mistake in the AFL fuzzer dictionary (#185), thanks @jwilk
|
||||
|
||||
1.5.6
|
||||
=====
|
||||
Fixes:
|
||||
------
|
||||
* Make cJSON a lot more tolerant about passing NULL pointers to its functions, it should now fail safely instead of dereferencing the pointer. (#183) Thanks @msichal for reporting #182
|
||||
|
||||
1.5.5
|
||||
=====
|
||||
Fixes:
|
||||
------
|
||||
* Fix pointers to nested arrays in cJSON_Utils (9abe75e072050f34732a7169740989a082b65134)
|
||||
* Fix an error with case sensitivity handling in cJSON_Utils (b9cc911831b0b3e1bb72f142389428e59f882b38)
|
||||
* Fix cJSON_Compare for arrays that are prefixes of the other and objects that are a subset of the other (03ba72faec115160d1f3aea5582d9b6af5d3e473) See #180, thanks @zhengqb for reporting
|
||||
|
||||
1.5.4
|
||||
=====
|
||||
Fixes:
|
||||
------
|
||||
* Fix build with GCC 7.1.1 and optimization level `-O2` (bfbd8fe0d85f1dd21e508748fc10fc4c27cc51be)
|
||||
|
||||
Other Changes:
|
||||
--------------
|
||||
* Update [Unity](https://github.com/ThrowTheSwitch/Unity) to 3b69beaa58efc41bbbef70a32a46893cae02719d
|
||||
|
||||
1.5.3
|
||||
=====
|
||||
Fixes:
|
||||
------
|
||||
* Fix `cJSON_ReplaceItemInObject` not keeping the name of an item (#174)
|
||||
|
||||
1.5.2
|
||||
=====
|
||||
Fixes:
|
||||
|
||||
@@ -7,7 +7,7 @@ project(cJSON C)
|
||||
|
||||
set(PROJECT_VERSION_MAJOR 1)
|
||||
set(PROJECT_VERSION_MINOR 5)
|
||||
set(PROJECT_VERSION_PATCH 2)
|
||||
set(PROJECT_VERSION_PATCH 9)
|
||||
set(CJSON_VERSION_SO 1)
|
||||
set(CJSON_UTILS_VERSION_SO 1)
|
||||
set(PROJECT_VERSION "${PROJECT_VERSION_MAJOR}.${PROJECT_VERSION_MINOR}.${PROJECT_VERSION_PATCH}")
|
||||
@@ -41,6 +41,7 @@ if (ENABLE_CUSTOM_COMPILER_FLAGS)
|
||||
-Wcomma
|
||||
-Wdouble-promotion
|
||||
-Wparentheses
|
||||
-Wformat-overflow
|
||||
)
|
||||
endif()
|
||||
|
||||
|
||||
@@ -8,13 +8,14 @@ Contributors
|
||||
* [Dave Gamble](https://github.com/DaveGamble)
|
||||
* [Debora Grosse](https://github.com/DeboraG)
|
||||
* [dieyushi](https://github.com/dieyushi)
|
||||
* [Dongwen Huang (黄东文)](https://github.com/DongwenHuang)
|
||||
* [Dōngwén Huáng (黄东文)](https://github.com/DongwenHuang)
|
||||
* Eswar Yaganti
|
||||
* [Evan Todd](https://github.com/etodd)
|
||||
* [Fabrice Fontaine](https://github.com/ffontaine)
|
||||
* Ian Mobley
|
||||
* Irwan Djadjadi
|
||||
* [IvanVoid](https://github.com/npi3pak)
|
||||
* [Jakub Wilk](https://github.com/jwilk)
|
||||
* [Jiri Zouhar](https://github.com/loigu)
|
||||
* [Jonathan Fether](https://github.com/jfether)
|
||||
* [Julián Vásquez](https://github.com/juvasquezg)
|
||||
@@ -29,6 +30,7 @@ Contributors
|
||||
* [Pawel Winogrodzki](https://github.com/PawelWMS)
|
||||
* [prefetchnta](https://github.com/prefetchnta)
|
||||
* [Rafael Leal Dias](https://github.com/rafaeldias)
|
||||
* [Robin Mallinson](https://github.com/rmallins)
|
||||
* [Rod Vagg](https://github.com/rvagg)
|
||||
* [Roland Meertens](https://github.com/rmeertens)
|
||||
* [Romain Porte](https://github.com/MicroJoe)
|
||||
|
||||
7
Makefile
7
Makefile
@@ -8,7 +8,7 @@ CJSON_TEST_SRC = cJSON.c test.c
|
||||
|
||||
LDLIBS = -lm
|
||||
|
||||
LIBVERSION = 1.5.2
|
||||
LIBVERSION = 1.5.9
|
||||
CJSON_SOVERSION = 1
|
||||
UTILS_SOVERSION = 1
|
||||
|
||||
@@ -66,11 +66,10 @@ shared: $(CJSON_SHARED) $(UTILS_SHARED)
|
||||
|
||||
static: $(CJSON_STATIC) $(UTILS_STATIC)
|
||||
|
||||
tests: $(CJSON_TEST) $(UTILS_TEST)
|
||||
tests: $(CJSON_TEST)
|
||||
|
||||
test: tests
|
||||
./$(CJSON_TEST)
|
||||
./$(UTILS_TEST)
|
||||
|
||||
.c.o:
|
||||
$(CC) -c $(R_CFLAGS) $<
|
||||
@@ -150,4 +149,4 @@ clean:
|
||||
$(RM) $(CJSON_OBJ) $(UTILS_OBJ) #delete object files
|
||||
$(RM) $(CJSON_SHARED) $(CJSON_SHARED_VERSION) $(CJSON_SHARED_SO) $(CJSON_STATIC) #delete cJSON
|
||||
$(RM) $(UTILS_SHARED) $(UTILS_SHARED_VERSION) $(UTILS_SHARED_SO) $(UTILS_STATIC) #delete cJSON_Utils
|
||||
$(RM) $(CJSON_TEST) $(UTILS_TEST) #delete tests
|
||||
$(RM) $(CJSON_TEST) #delete test
|
||||
|
||||
@@ -383,17 +383,17 @@ cJSON doesn't support strings that contain the zero character `'\0'` or `\u0000`
|
||||
|
||||
#### Character Encoding
|
||||
|
||||
cJSON only supports UTF-8 encoded input and will always produce UTF-8 as output (If the input contained invalid UTF-8, it will most likely propagate it through to the output, thereby making the output non-valid UTF-8).
|
||||
cJSON only supports UTF-8 encoded input. In most cases it doesn't reject invalid UTF-8 as input though, it just propagates it through as is. As long as the input doesn't contain invalid UTF-8, the output will always be valid UTF-8.
|
||||
|
||||
#### C Standard
|
||||
|
||||
cJSON is written in ANSI C (or C89, C90). If your compiler or C library doesn't follow this standard, correct behavior is not guaranteed.
|
||||
|
||||
NOTE: ANSI C is not C++ therefore it shouldn't be compiled by a C++ compiler. You can compile it with a C compiler and link it with your C++ code however. Although compiling with a C++ compiler might work, correct behavior is not guaranteed.
|
||||
NOTE: ANSI C is not C++ therefore it shouldn't be compiled with a C++ compiler. You can compile it with a C compiler and link it with your C++ code however. Although compiling with a C++ compiler might work, correct behavior is not guaranteed.
|
||||
|
||||
#### Floating Point Numbers
|
||||
|
||||
cJSON does not officially support any `double` implementations other than IEE754 double precision floating point numbers. It might still work with other implementations but bugs with these will be considered invalid.
|
||||
cJSON does not officially support any `double` implementations other than IEEE754 double precision floating point numbers. It might still work with other implementations but bugs with these will be considered invalid.
|
||||
|
||||
The maximum length of a floating point literal that cJSON supports is currently 63 characters.
|
||||
|
||||
@@ -417,3 +417,4 @@ When cJSON was originally created, it didn't follow the JSON standard and didn't
|
||||
# Enjoy cJSON!
|
||||
|
||||
- Dave Gamble, Aug 2009
|
||||
- [cJSON contributors](CONTRIBUTORS.md)
|
||||
|
||||
141
cJSON.c
141
cJSON.c
@@ -58,7 +58,7 @@ CJSON_PUBLIC(const char *) cJSON_GetErrorPtr(void)
|
||||
}
|
||||
|
||||
/* This is a safeguard to prevent copy-pasters from using incompatible C and header files */
|
||||
#if (CJSON_VERSION_MAJOR != 1) || (CJSON_VERSION_MINOR != 5) || (CJSON_VERSION_PATCH != 2)
|
||||
#if (CJSON_VERSION_MAJOR != 1) || (CJSON_VERSION_MINOR != 5) || (CJSON_VERSION_PATCH != 9)
|
||||
#error cJSON.h and cJSON.c have different versions. Make sure that both have the same.
|
||||
#endif
|
||||
|
||||
@@ -377,6 +377,14 @@ static unsigned char* ensure(printbuffer * const p, size_t needed)
|
||||
{
|
||||
/* reallocate with realloc if available */
|
||||
newbuffer = (unsigned char*)p->hooks.reallocate(p->buffer, newsize);
|
||||
if (newbuffer == NULL)
|
||||
{
|
||||
p->hooks.deallocate(p->buffer);
|
||||
p->length = 0;
|
||||
p->buffer = NULL;
|
||||
|
||||
return NULL;
|
||||
}
|
||||
}
|
||||
else
|
||||
{
|
||||
@@ -998,10 +1006,8 @@ fail:
|
||||
{
|
||||
*return_parse_end = (const char*)local_error.json + local_error.position;
|
||||
}
|
||||
else
|
||||
{
|
||||
global_error = local_error;
|
||||
}
|
||||
|
||||
global_error = local_error;
|
||||
}
|
||||
|
||||
return NULL;
|
||||
@@ -1111,6 +1117,7 @@ CJSON_PUBLIC(char *) cJSON_PrintBuffered(const cJSON *item, int prebuffer, cJSON
|
||||
|
||||
if (!print_value(item, &p))
|
||||
{
|
||||
global_hooks.deallocate(p.buffer);
|
||||
return NULL;
|
||||
}
|
||||
|
||||
@@ -1121,7 +1128,7 @@ CJSON_PUBLIC(cJSON_bool) cJSON_PrintPreallocated(cJSON *item, char *buf, const i
|
||||
{
|
||||
printbuffer p = { 0, 0, 0, 0, 0, 0, { 0, 0, 0 } };
|
||||
|
||||
if (len < 0)
|
||||
if ((len < 0) || (buf == NULL))
|
||||
{
|
||||
return false;
|
||||
}
|
||||
@@ -1652,17 +1659,25 @@ static cJSON_bool print_object(const cJSON * const item, printbuffer * const out
|
||||
/* Get Array size/item / object item. */
|
||||
CJSON_PUBLIC(int) cJSON_GetArraySize(const cJSON *array)
|
||||
{
|
||||
cJSON *c = array->child;
|
||||
size_t i = 0;
|
||||
while(c)
|
||||
cJSON *child = NULL;
|
||||
size_t size = 0;
|
||||
|
||||
if (array == NULL)
|
||||
{
|
||||
i++;
|
||||
c = c->next;
|
||||
return 0;
|
||||
}
|
||||
|
||||
child = array->child;
|
||||
|
||||
while(child != NULL)
|
||||
{
|
||||
size++;
|
||||
child = child->next;
|
||||
}
|
||||
|
||||
/* FIXME: Can overflow here. Cannot be fixed without breaking the API */
|
||||
|
||||
return (int)i;
|
||||
return (int)size;
|
||||
}
|
||||
|
||||
static cJSON* get_array_item(const cJSON *array, size_t index)
|
||||
@@ -1747,16 +1762,23 @@ static void suffix_object(cJSON *prev, cJSON *item)
|
||||
/* Utility for handling references. */
|
||||
static cJSON *create_reference(const cJSON *item, const internal_hooks * const hooks)
|
||||
{
|
||||
cJSON *ref = cJSON_New_Item(hooks);
|
||||
if (!ref)
|
||||
cJSON *reference = NULL;
|
||||
if (item == NULL)
|
||||
{
|
||||
return NULL;
|
||||
}
|
||||
memcpy(ref, item, sizeof(cJSON));
|
||||
ref->string = NULL;
|
||||
ref->type |= cJSON_IsReference;
|
||||
ref->next = ref->prev = NULL;
|
||||
return ref;
|
||||
|
||||
reference = cJSON_New_Item(hooks);
|
||||
if (reference == NULL)
|
||||
{
|
||||
return NULL;
|
||||
}
|
||||
|
||||
memcpy(reference, item, sizeof(cJSON));
|
||||
reference->string = NULL;
|
||||
reference->type |= cJSON_IsReference;
|
||||
reference->next = reference->prev = NULL;
|
||||
return reference;
|
||||
}
|
||||
|
||||
/* Add item to array/object. */
|
||||
@@ -1789,6 +1811,11 @@ CJSON_PUBLIC(void) cJSON_AddItemToArray(cJSON *array, cJSON *item)
|
||||
|
||||
CJSON_PUBLIC(void) cJSON_AddItemToObject(cJSON *object, const char *string, cJSON *item)
|
||||
{
|
||||
if (item == NULL)
|
||||
{
|
||||
return;
|
||||
}
|
||||
|
||||
/* call cJSON_AddItemToObjectCS for code reuse */
|
||||
cJSON_AddItemToObjectCS(object, (char*)cJSON_strdup((const unsigned char*)string, &global_hooks), item);
|
||||
/* remove cJSON_StringIsConst flag */
|
||||
@@ -1805,7 +1832,7 @@ CJSON_PUBLIC(void) cJSON_AddItemToObject(cJSON *object, const char *string, cJSO
|
||||
/* Add an item to an object with constant string as key */
|
||||
CJSON_PUBLIC(void) cJSON_AddItemToObjectCS(cJSON *object, const char *string, cJSON *item)
|
||||
{
|
||||
if (!item)
|
||||
if ((item == NULL) || (string == NULL))
|
||||
{
|
||||
return;
|
||||
}
|
||||
@@ -1823,11 +1850,21 @@ CJSON_PUBLIC(void) cJSON_AddItemToObjectCS(cJSON *object, const char *string, cJ
|
||||
|
||||
CJSON_PUBLIC(void) cJSON_AddItemReferenceToArray(cJSON *array, cJSON *item)
|
||||
{
|
||||
if (array == NULL)
|
||||
{
|
||||
return;
|
||||
}
|
||||
|
||||
cJSON_AddItemToArray(array, create_reference(item, &global_hooks));
|
||||
}
|
||||
|
||||
CJSON_PUBLIC(void) cJSON_AddItemReferenceToObject(cJSON *object, const char *string, cJSON *item)
|
||||
{
|
||||
if ((object == NULL) || (string == NULL))
|
||||
{
|
||||
return;
|
||||
}
|
||||
|
||||
cJSON_AddItemToObject(object, string, create_reference(item, &global_hooks));
|
||||
}
|
||||
|
||||
@@ -1932,7 +1969,7 @@ CJSON_PUBLIC(void) cJSON_InsertItemInArray(cJSON *array, int which, cJSON *newit
|
||||
|
||||
CJSON_PUBLIC(cJSON_bool) cJSON_ReplaceItemViaPointer(cJSON * const parent, cJSON * const item, cJSON * replacement)
|
||||
{
|
||||
if ((parent == NULL) || (replacement == NULL))
|
||||
if ((parent == NULL) || (replacement == NULL) || (item == NULL))
|
||||
{
|
||||
return false;
|
||||
}
|
||||
@@ -1975,14 +2012,34 @@ CJSON_PUBLIC(void) cJSON_ReplaceItemInArray(cJSON *array, int which, cJSON *newi
|
||||
cJSON_ReplaceItemViaPointer(array, get_array_item(array, (size_t)which), newitem);
|
||||
}
|
||||
|
||||
static cJSON_bool replace_item_in_object(cJSON *object, const char *string, cJSON *replacement, cJSON_bool case_sensitive)
|
||||
{
|
||||
if ((replacement == NULL) || (string == NULL))
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
/* replace the name in the replacement */
|
||||
if (!(replacement->type & cJSON_StringIsConst) && (replacement->string != NULL))
|
||||
{
|
||||
cJSON_free(replacement->string);
|
||||
}
|
||||
replacement->string = (char*)cJSON_strdup((const unsigned char*)string, &global_hooks);
|
||||
replacement->type &= ~cJSON_StringIsConst;
|
||||
|
||||
cJSON_ReplaceItemViaPointer(object, get_object_item(object, string, case_sensitive), replacement);
|
||||
|
||||
return true;
|
||||
}
|
||||
|
||||
CJSON_PUBLIC(void) cJSON_ReplaceItemInObject(cJSON *object, const char *string, cJSON *newitem)
|
||||
{
|
||||
cJSON_ReplaceItemViaPointer(object, cJSON_GetObjectItem(object, string), newitem);
|
||||
replace_item_in_object(object, string, newitem, false);
|
||||
}
|
||||
|
||||
CJSON_PUBLIC(void) cJSON_ReplaceItemInObjectCaseSensitive(cJSON *object, const char *string, cJSON *newitem)
|
||||
{
|
||||
cJSON_ReplaceItemViaPointer(object, cJSON_GetObjectItemCaseSensitive(object, string), newitem);
|
||||
replace_item_in_object(object, string, newitem, true);
|
||||
}
|
||||
|
||||
/* Create basic types: */
|
||||
@@ -2120,7 +2177,7 @@ CJSON_PUBLIC(cJSON *) cJSON_CreateIntArray(const int *numbers, int count)
|
||||
cJSON *p = NULL;
|
||||
cJSON *a = NULL;
|
||||
|
||||
if (count < 0)
|
||||
if ((count < 0) || (numbers == NULL))
|
||||
{
|
||||
return NULL;
|
||||
}
|
||||
@@ -2155,7 +2212,7 @@ CJSON_PUBLIC(cJSON *) cJSON_CreateFloatArray(const float *numbers, int count)
|
||||
cJSON *p = NULL;
|
||||
cJSON *a = NULL;
|
||||
|
||||
if (count < 0)
|
||||
if ((count < 0) || (numbers == NULL))
|
||||
{
|
||||
return NULL;
|
||||
}
|
||||
@@ -2191,7 +2248,7 @@ CJSON_PUBLIC(cJSON *) cJSON_CreateDoubleArray(const double *numbers, int count)
|
||||
cJSON *p = NULL;
|
||||
cJSON *a = NULL;
|
||||
|
||||
if (count < 0)
|
||||
if ((count < 0) || (numbers == NULL))
|
||||
{
|
||||
return NULL;
|
||||
}
|
||||
@@ -2227,7 +2284,7 @@ CJSON_PUBLIC(cJSON *) cJSON_CreateStringArray(const char **strings, int count)
|
||||
cJSON *p = NULL;
|
||||
cJSON *a = NULL;
|
||||
|
||||
if (count < 0)
|
||||
if ((count < 0) || (strings == NULL))
|
||||
{
|
||||
return NULL;
|
||||
}
|
||||
@@ -2339,6 +2396,12 @@ fail:
|
||||
CJSON_PUBLIC(void) cJSON_Minify(char *json)
|
||||
{
|
||||
unsigned char *into = (unsigned char*)json;
|
||||
|
||||
if (json == NULL)
|
||||
{
|
||||
return;
|
||||
}
|
||||
|
||||
while (*json)
|
||||
{
|
||||
if (*json == ' ')
|
||||
@@ -2574,16 +2637,22 @@ CJSON_PUBLIC(cJSON_bool) cJSON_Compare(const cJSON * const a, const cJSON * cons
|
||||
b_element = b_element->next;
|
||||
}
|
||||
|
||||
/* one of the arrays is longer than the other */
|
||||
if (a_element != b_element) {
|
||||
return false;
|
||||
}
|
||||
|
||||
return true;
|
||||
}
|
||||
|
||||
case cJSON_Object:
|
||||
{
|
||||
cJSON *a_element = NULL;
|
||||
cJSON *b_element = NULL;
|
||||
cJSON_ArrayForEach(a_element, a)
|
||||
{
|
||||
/* TODO This has O(n^2) runtime, which is horrible! */
|
||||
cJSON *b_element = get_object_item(b, a_element->string, case_sensitive);
|
||||
b_element = get_object_item(b, a_element->string, case_sensitive);
|
||||
if (b_element == NULL)
|
||||
{
|
||||
return false;
|
||||
@@ -2595,6 +2664,22 @@ CJSON_PUBLIC(cJSON_bool) cJSON_Compare(const cJSON * const a, const cJSON * cons
|
||||
}
|
||||
}
|
||||
|
||||
/* doing this twice, once on a and b to prevent true comparison if a subset of b
|
||||
* TODO: Do this the proper way, this is just a fix for now */
|
||||
cJSON_ArrayForEach(b_element, b)
|
||||
{
|
||||
a_element = get_object_item(a, b_element->string, case_sensitive);
|
||||
if (a_element == NULL)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
if (!cJSON_Compare(b_element, a_element, case_sensitive))
|
||||
{
|
||||
return false;
|
||||
}
|
||||
}
|
||||
|
||||
return true;
|
||||
}
|
||||
|
||||
|
||||
12
cJSON.h
12
cJSON.h
@@ -31,7 +31,7 @@ extern "C"
|
||||
/* project version */
|
||||
#define CJSON_VERSION_MAJOR 1
|
||||
#define CJSON_VERSION_MINOR 5
|
||||
#define CJSON_VERSION_PATCH 2
|
||||
#define CJSON_VERSION_PATCH 9
|
||||
|
||||
#include <stddef.h>
|
||||
|
||||
@@ -138,6 +138,10 @@ CJSON_PUBLIC(void) cJSON_InitHooks(cJSON_Hooks* hooks);
|
||||
/* Memory Management: the caller is always responsible to free the results from all variants of cJSON_Parse (with cJSON_Delete) and cJSON_Print (with stdlib free, cJSON_Hooks.free_fn, or cJSON_free as appropriate). The exception is cJSON_PrintPreallocated, where the caller has full responsibility of the buffer. */
|
||||
/* Supply a block of JSON, and this returns a cJSON object you can interrogate. */
|
||||
CJSON_PUBLIC(cJSON *) cJSON_Parse(const char *value);
|
||||
/* ParseWithOpts allows you to require (and check) that the JSON is null terminated, and to retrieve the pointer to the final byte parsed. */
|
||||
/* If you supply a ptr in return_parse_end and parsing fails, then return_parse_end will contain a pointer to the error so will match cJSON_GetErrorPtr(). */
|
||||
CJSON_PUBLIC(cJSON *) cJSON_ParseWithOpts(const char *value, const char **return_parse_end, cJSON_bool require_null_terminated);
|
||||
|
||||
/* Render a cJSON entity to text for transfer/storage. */
|
||||
CJSON_PUBLIC(char *) cJSON_Print(const cJSON *item);
|
||||
/* Render a cJSON entity to text for transfer/storage without any formatting. */
|
||||
@@ -228,10 +232,6 @@ The item->next and ->prev pointers are always zero on return from Duplicate. */
|
||||
CJSON_PUBLIC(cJSON_bool) cJSON_Compare(const cJSON * const a, const cJSON * const b, const cJSON_bool case_sensitive);
|
||||
|
||||
|
||||
/* ParseWithOpts allows you to require (and check) that the JSON is null terminated, and to retrieve the pointer to the final byte parsed. */
|
||||
/* If you supply a ptr in return_parse_end and parsing fails, then return_parse_end will contain a pointer to the error. If not, then cJSON_GetErrorPtr() does the job. */
|
||||
CJSON_PUBLIC(cJSON *) cJSON_ParseWithOpts(const char *value, const char **return_parse_end, cJSON_bool require_null_terminated);
|
||||
|
||||
CJSON_PUBLIC(void) cJSON_Minify(char *json);
|
||||
|
||||
/* Macros for creating things quickly. */
|
||||
@@ -249,7 +249,7 @@ CJSON_PUBLIC(void) cJSON_Minify(char *json);
|
||||
CJSON_PUBLIC(double) cJSON_SetNumberHelper(cJSON *object, double number);
|
||||
#define cJSON_SetNumberValue(object, number) ((object != NULL) ? cJSON_SetNumberHelper(object, (double)number) : (number))
|
||||
|
||||
/* Macro for iterating over an array */
|
||||
/* Macro for iterating over an array or object */
|
||||
#define cJSON_ArrayForEach(element, array) for(element = (array != NULL) ? (array)->child : NULL; element != NULL; element = element->next)
|
||||
|
||||
/* malloc/free objects using the malloc/free functions that have been set with cJSON_InitHooks */
|
||||
|
||||
@@ -162,6 +162,11 @@ CJSON_PUBLIC(char *) cJSONUtils_FindPointerFromObjectTo(const cJSON * const obje
|
||||
size_t child_index = 0;
|
||||
cJSON *current_child = 0;
|
||||
|
||||
if ((object == NULL) || (target == NULL))
|
||||
{
|
||||
return NULL;
|
||||
}
|
||||
|
||||
if (object == target)
|
||||
{
|
||||
/* found */
|
||||
@@ -257,6 +262,12 @@ static cJSON_bool decode_array_index_from_pointer(const unsigned char * const po
|
||||
static cJSON *get_item_from_pointer(cJSON * const object, const char * pointer, const cJSON_bool case_sensitive)
|
||||
{
|
||||
cJSON *current_element = object;
|
||||
|
||||
if (pointer == NULL)
|
||||
{
|
||||
return NULL;
|
||||
}
|
||||
|
||||
/* follow path of the pointer */
|
||||
while ((pointer[0] == '/') && (current_element != NULL))
|
||||
{
|
||||
@@ -279,16 +290,17 @@ static cJSON *get_item_from_pointer(cJSON * const object, const char * pointer,
|
||||
{
|
||||
current_element = current_element->next;
|
||||
}
|
||||
/* skip to the next path token or end of string */
|
||||
while ((pointer[0] != '\0') && (pointer[0] != '/'))
|
||||
{
|
||||
pointer++;
|
||||
}
|
||||
}
|
||||
else
|
||||
{
|
||||
return NULL;
|
||||
}
|
||||
|
||||
/* skip to the next path token or end of string */
|
||||
while ((pointer[0] != '\0') && (pointer[0] != '/'))
|
||||
{
|
||||
pointer++;
|
||||
}
|
||||
}
|
||||
|
||||
return current_element;
|
||||
@@ -538,6 +550,10 @@ static cJSON *sort_list(cJSON *list, const cJSON_bool case_sensitive)
|
||||
|
||||
static void sort_object(cJSON * const object, const cJSON_bool case_sensitive)
|
||||
{
|
||||
if (object == NULL)
|
||||
{
|
||||
return;
|
||||
}
|
||||
object->child = sort_list(object->child, case_sensitive);
|
||||
}
|
||||
|
||||
@@ -942,7 +958,14 @@ static int apply_patch(cJSON *object, const cJSON *patch, const cJSON_bool case_
|
||||
}
|
||||
else if (cJSON_IsObject(parent))
|
||||
{
|
||||
cJSON_DeleteItemFromObject(parent, (char*)child_pointer);
|
||||
if (case_sensitive)
|
||||
{
|
||||
cJSON_DeleteItemFromObjectCaseSensitive(parent, (char*)child_pointer);
|
||||
}
|
||||
else
|
||||
{
|
||||
cJSON_DeleteItemFromObject(parent, (char*)child_pointer);
|
||||
}
|
||||
cJSON_AddItemToObject(parent, (char*)child_pointer, value);
|
||||
value = NULL;
|
||||
}
|
||||
@@ -1020,7 +1043,14 @@ CJSON_PUBLIC(int) cJSONUtils_ApplyPatchesCaseSensitive(cJSON * const object, con
|
||||
|
||||
static void compose_patch(cJSON * const patches, const unsigned char * const operation, const unsigned char * const path, const unsigned char *suffix, const cJSON * const value)
|
||||
{
|
||||
cJSON *patch = cJSON_CreateObject();
|
||||
cJSON *patch = NULL;
|
||||
|
||||
if ((patches == NULL) || (operation == NULL) || (path == NULL))
|
||||
{
|
||||
return;
|
||||
}
|
||||
|
||||
patch = cJSON_CreateObject();
|
||||
if (patch == NULL)
|
||||
{
|
||||
return;
|
||||
@@ -1198,7 +1228,14 @@ static void create_patches(cJSON * const patches, const unsigned char * const pa
|
||||
|
||||
CJSON_PUBLIC(cJSON *) cJSONUtils_GeneratePatches(cJSON * const from, cJSON * const to)
|
||||
{
|
||||
cJSON *patches = cJSON_CreateArray();
|
||||
cJSON *patches = NULL;
|
||||
|
||||
if ((from == NULL) || (to == NULL))
|
||||
{
|
||||
return NULL;
|
||||
}
|
||||
|
||||
patches = cJSON_CreateArray();
|
||||
create_patches(patches, (const unsigned char*)"", from, to, false);
|
||||
|
||||
return patches;
|
||||
@@ -1206,7 +1243,14 @@ CJSON_PUBLIC(cJSON *) cJSONUtils_GeneratePatches(cJSON * const from, cJSON * con
|
||||
|
||||
CJSON_PUBLIC(cJSON *) cJSONUtils_GeneratePatchesCaseSensitive(cJSON * const from, cJSON * const to)
|
||||
{
|
||||
cJSON *patches = cJSON_CreateArray();
|
||||
cJSON *patches = NULL;
|
||||
|
||||
if ((from == NULL) || (to == NULL))
|
||||
{
|
||||
return NULL;
|
||||
}
|
||||
|
||||
patches = cJSON_CreateArray();
|
||||
create_patches(patches, (const unsigned char*)"", from, to, true);
|
||||
|
||||
return patches;
|
||||
|
||||
@@ -25,8 +25,8 @@ escape_sequence_r="\\r"
|
||||
escape_sequence_t="\\t"
|
||||
escape_sequence_quote="\\\""
|
||||
escape_sequence_backslash="\\\\"
|
||||
escapce_sequence_slash="\\/"
|
||||
escpae_sequence_utf16_base="\\u"
|
||||
escape_sequence_slash="\\/"
|
||||
escape_sequence_utf16_base="\\u"
|
||||
escape_sequence_utf16="\\u12ab"
|
||||
|
||||
number_integer="1"
|
||||
|
||||
@@ -58,7 +58,7 @@ if(ENABLE_CJSON_TEST)
|
||||
message(WARNING "Valgrind couldn't be found.")
|
||||
unset(MEMORYCHECK_COMMAND)
|
||||
else()
|
||||
set(MEMORYCHECK_COMMAND_OPTIONS --trace-children=yes --leak-check=full --error-exitcode=1)
|
||||
set(MEMORYCHECK_COMMAND_OPTIONS --trace-children=yes --leak-check=full --error-exitcode=1 --suppressions=${CMAKE_CURRENT_SOURCE_DIR}/../valgrind.supp)
|
||||
endif()
|
||||
endif()
|
||||
|
||||
@@ -84,7 +84,8 @@ if(ENABLE_CJSON_TEST)
|
||||
|
||||
set (cjson_utils_tests
|
||||
json_patch_tests
|
||||
old_utils_tests)
|
||||
old_utils_tests
|
||||
misc_utils_tests)
|
||||
|
||||
foreach (cjson_utils_test ${cjson_utils_tests})
|
||||
add_executable("${cjson_utils_test}" "${cjson_utils_test}.c")
|
||||
|
||||
@@ -148,6 +148,10 @@ static void cjson_compare_should_compare_arrays(void)
|
||||
|
||||
TEST_ASSERT_FALSE(compare_from_string("[true,null,42,\"string\",[],{}]", "[false, true, null, 42, \"string\", [], {}]", true));
|
||||
TEST_ASSERT_FALSE(compare_from_string("[true,null,42,\"string\",[],{}]", "[false, true, null, 42, \"string\", [], {}]", false));
|
||||
|
||||
/* Arrays that are a prefix of another array */
|
||||
TEST_ASSERT_FALSE(compare_from_string("[1,2,3]", "[1,2]", true));
|
||||
TEST_ASSERT_FALSE(compare_from_string("[1,2,3]", "[1,2]", false));
|
||||
}
|
||||
|
||||
static void cjson_compare_should_compare_objects(void)
|
||||
@@ -171,6 +175,15 @@ static void cjson_compare_should_compare_objects(void)
|
||||
"{\"Flse\": false, \"true\": true, \"null\": null, \"number\": 42, \"string\": \"string\", \"array\": [], \"object\": {}}",
|
||||
"{\"true\": true, \"false\": false, \"null\": null, \"number\": 42, \"string\": \"string\", \"array\": [], \"object\": {}}",
|
||||
false));
|
||||
/* test objects that are a subset of each other */
|
||||
TEST_ASSERT_FALSE(compare_from_string(
|
||||
"{\"one\": 1, \"two\": 2}",
|
||||
"{\"one\": 1, \"two\": 2, \"three\": 3}",
|
||||
true))
|
||||
TEST_ASSERT_FALSE(compare_from_string(
|
||||
"{\"one\": 1, \"two\": 2}",
|
||||
"{\"one\": 1, \"two\": 2, \"three\": 3}",
|
||||
false))
|
||||
}
|
||||
|
||||
int main(void)
|
||||
|
||||
@@ -304,6 +304,128 @@ static void cjson_replace_item_via_pointer_should_replace_items(void)
|
||||
cJSON_free(array);
|
||||
}
|
||||
|
||||
static void cjson_replace_item_in_object_should_preserve_name(void)
|
||||
{
|
||||
cJSON root[1] = {{ NULL, NULL, NULL, 0, NULL, 0, 0, NULL }};
|
||||
cJSON *child = NULL;
|
||||
cJSON *replacement = NULL;
|
||||
|
||||
child = cJSON_CreateNumber(1);
|
||||
TEST_ASSERT_NOT_NULL(child);
|
||||
replacement = cJSON_CreateNumber(2);
|
||||
TEST_ASSERT_NOT_NULL(replacement);
|
||||
|
||||
cJSON_AddItemToObject(root, "child", child);
|
||||
cJSON_ReplaceItemInObject(root, "child", replacement);
|
||||
|
||||
TEST_ASSERT_TRUE(root->child == replacement);
|
||||
TEST_ASSERT_EQUAL_STRING("child", replacement->string);
|
||||
|
||||
cJSON_Delete(replacement);
|
||||
}
|
||||
|
||||
static void cjson_functions_shouldnt_crash_with_null_pointers(void)
|
||||
{
|
||||
char buffer[10];
|
||||
cJSON *item = cJSON_CreateString("item");
|
||||
|
||||
cJSON_InitHooks(NULL);
|
||||
TEST_ASSERT_NULL(cJSON_Parse(NULL));
|
||||
TEST_ASSERT_NULL(cJSON_ParseWithOpts(NULL, NULL, true));
|
||||
TEST_ASSERT_NULL(cJSON_Print(NULL));
|
||||
TEST_ASSERT_NULL(cJSON_PrintUnformatted(NULL));
|
||||
TEST_ASSERT_NULL(cJSON_PrintBuffered(NULL, 10, true));
|
||||
TEST_ASSERT_FALSE(cJSON_PrintPreallocated(NULL, buffer, sizeof(buffer), true));
|
||||
TEST_ASSERT_FALSE(cJSON_PrintPreallocated(item, NULL, 1, true));
|
||||
cJSON_Delete(NULL);
|
||||
cJSON_GetArraySize(NULL);
|
||||
TEST_ASSERT_NULL(cJSON_GetArrayItem(NULL, 0));
|
||||
TEST_ASSERT_NULL(cJSON_GetObjectItem(NULL, "item"));
|
||||
TEST_ASSERT_NULL(cJSON_GetObjectItem(item, NULL));
|
||||
TEST_ASSERT_NULL(cJSON_GetObjectItemCaseSensitive(NULL, "item"));
|
||||
TEST_ASSERT_NULL(cJSON_GetObjectItemCaseSensitive(item, NULL));
|
||||
TEST_ASSERT_FALSE(cJSON_HasObjectItem(NULL, "item"));
|
||||
TEST_ASSERT_FALSE(cJSON_HasObjectItem(item, NULL));
|
||||
TEST_ASSERT_FALSE(cJSON_IsInvalid(NULL));
|
||||
TEST_ASSERT_FALSE(cJSON_IsFalse(NULL));
|
||||
TEST_ASSERT_FALSE(cJSON_IsTrue(NULL));
|
||||
TEST_ASSERT_FALSE(cJSON_IsBool(NULL));
|
||||
TEST_ASSERT_FALSE(cJSON_IsNull(NULL));
|
||||
TEST_ASSERT_FALSE(cJSON_IsNumber(NULL));
|
||||
TEST_ASSERT_FALSE(cJSON_IsString(NULL));
|
||||
TEST_ASSERT_FALSE(cJSON_IsArray(NULL));
|
||||
TEST_ASSERT_FALSE(cJSON_IsObject(NULL));
|
||||
TEST_ASSERT_FALSE(cJSON_IsRaw(NULL));
|
||||
TEST_ASSERT_NULL(cJSON_CreateString(NULL));
|
||||
TEST_ASSERT_NULL(cJSON_CreateRaw(NULL));
|
||||
TEST_ASSERT_NULL(cJSON_CreateIntArray(NULL, 10));
|
||||
TEST_ASSERT_NULL(cJSON_CreateFloatArray(NULL, 10));
|
||||
TEST_ASSERT_NULL(cJSON_CreateDoubleArray(NULL, 10));
|
||||
TEST_ASSERT_NULL(cJSON_CreateStringArray(NULL, 10));
|
||||
cJSON_AddItemToArray(NULL, item);
|
||||
cJSON_AddItemToArray(item, NULL);
|
||||
cJSON_AddItemToObject(item, "item", NULL);
|
||||
cJSON_AddItemToObject(item, NULL, item);
|
||||
cJSON_AddItemToObject(NULL, "item", item);
|
||||
cJSON_AddItemToObjectCS(item, "item", NULL);
|
||||
cJSON_AddItemToObjectCS(item, NULL, item);
|
||||
cJSON_AddItemToObjectCS(NULL, "item", item);
|
||||
cJSON_AddItemReferenceToArray(NULL, item);
|
||||
cJSON_AddItemReferenceToArray(item, NULL);
|
||||
cJSON_AddItemReferenceToObject(item, "item", NULL);
|
||||
cJSON_AddItemReferenceToObject(item, NULL, item);
|
||||
cJSON_AddItemReferenceToObject(NULL, "item", item);
|
||||
TEST_ASSERT_NULL(cJSON_DetachItemViaPointer(NULL, item));
|
||||
TEST_ASSERT_NULL(cJSON_DetachItemViaPointer(item, NULL));
|
||||
TEST_ASSERT_NULL(cJSON_DetachItemFromArray(NULL, 0));
|
||||
cJSON_DeleteItemFromArray(NULL, 0);
|
||||
TEST_ASSERT_NULL(cJSON_DetachItemFromObject(NULL, "item"));
|
||||
TEST_ASSERT_NULL(cJSON_DetachItemFromObject(item, NULL));
|
||||
TEST_ASSERT_NULL(cJSON_DetachItemFromObjectCaseSensitive(NULL, "item"));
|
||||
TEST_ASSERT_NULL(cJSON_DetachItemFromObjectCaseSensitive(item, NULL));
|
||||
cJSON_DeleteItemFromObject(NULL, "item");
|
||||
cJSON_DeleteItemFromObject(item, NULL);
|
||||
cJSON_DeleteItemFromObjectCaseSensitive(NULL, "item");
|
||||
cJSON_DeleteItemFromObjectCaseSensitive(item, NULL);
|
||||
cJSON_InsertItemInArray(NULL, 0, item);
|
||||
cJSON_InsertItemInArray(item, 0, NULL);
|
||||
TEST_ASSERT_FALSE(cJSON_ReplaceItemViaPointer(NULL, item, item));
|
||||
TEST_ASSERT_FALSE(cJSON_ReplaceItemViaPointer(item, NULL, item));
|
||||
TEST_ASSERT_FALSE(cJSON_ReplaceItemViaPointer(item, item, NULL));
|
||||
cJSON_ReplaceItemInArray(item, 0, NULL);
|
||||
cJSON_ReplaceItemInArray(NULL, 0, item);
|
||||
cJSON_ReplaceItemInObject(NULL, "item", item);
|
||||
cJSON_ReplaceItemInObject(item, NULL, item);
|
||||
cJSON_ReplaceItemInObject(item, "item", NULL);
|
||||
cJSON_ReplaceItemInObjectCaseSensitive(NULL, "item", item);
|
||||
cJSON_ReplaceItemInObjectCaseSensitive(item, NULL, item);
|
||||
cJSON_ReplaceItemInObjectCaseSensitive(item, "item", NULL);
|
||||
TEST_ASSERT_NULL(cJSON_Duplicate(NULL, true));
|
||||
TEST_ASSERT_FALSE(cJSON_Compare(item, NULL, false));
|
||||
TEST_ASSERT_FALSE(cJSON_Compare(NULL, item, false));
|
||||
cJSON_Minify(NULL);
|
||||
/* skipped because it is only used via a macro that checks for NULL */
|
||||
/* cJSON_SetNumberHelper(NULL, 0); */
|
||||
|
||||
cJSON_Delete(item);
|
||||
}
|
||||
|
||||
static void *failing_realloc(void *pointer, size_t size)
|
||||
{
|
||||
(void)size;
|
||||
(void)pointer;
|
||||
return NULL;
|
||||
}
|
||||
|
||||
static void ensure_should_fail_on_failed_realloc(void)
|
||||
{
|
||||
printbuffer buffer = {NULL, 10, 0, 0, false, false, {&malloc, &free, &failing_realloc}};
|
||||
buffer.buffer = (unsigned char*)malloc(100);
|
||||
TEST_ASSERT_NOT_NULL(buffer.buffer);
|
||||
|
||||
TEST_ASSERT_NULL_MESSAGE(ensure(&buffer, 200), "Ensure didn't fail with failing realloc.");
|
||||
}
|
||||
|
||||
int main(void)
|
||||
{
|
||||
UNITY_BEGIN();
|
||||
@@ -317,6 +439,8 @@ int main(void)
|
||||
RUN_TEST(cjson_set_number_value_should_set_numbers);
|
||||
RUN_TEST(cjson_detach_item_via_pointer_should_detach_items);
|
||||
RUN_TEST(cjson_replace_item_via_pointer_should_replace_items);
|
||||
|
||||
RUN_TEST(cjson_replace_item_in_object_should_preserve_name);
|
||||
RUN_TEST(cjson_functions_shouldnt_crash_with_null_pointers);
|
||||
RUN_TEST(ensure_should_fail_on_failed_realloc);
|
||||
return UNITY_END();
|
||||
}
|
||||
|
||||
80
tests/misc_utils_tests.c
Normal file
80
tests/misc_utils_tests.c
Normal file
@@ -0,0 +1,80 @@
|
||||
/*
|
||||
Copyright (c) 2009-2017 Dave Gamble and cJSON contributors
|
||||
|
||||
Permission is hereby granted, free of charge, to any person obtaining a copy
|
||||
of this software and associated documentation files (the "Software"), to deal
|
||||
in the Software without restriction, including without limitation the rights
|
||||
to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
|
||||
copies of the Software, and to permit persons to whom the Software is
|
||||
furnished to do so, subject to the following conditions:
|
||||
|
||||
The above copyright notice and this permission notice shall be included in
|
||||
all copies or substantial portions of the Software.
|
||||
|
||||
THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
|
||||
IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
|
||||
FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
|
||||
AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
|
||||
LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
|
||||
OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
|
||||
THE SOFTWARE.
|
||||
*/
|
||||
|
||||
#include <stdio.h>
|
||||
#include <stdlib.h>
|
||||
#include <string.h>
|
||||
|
||||
#include "unity/examples/unity_config.h"
|
||||
#include "unity/src/unity.h"
|
||||
#include "common.h"
|
||||
#include "../cJSON_Utils.h"
|
||||
|
||||
static void cjson_utils_functions_shouldnt_crash_with_null_pointers(void)
|
||||
{
|
||||
cJSON *item = cJSON_CreateString("item");
|
||||
TEST_ASSERT_NOT_NULL(item);
|
||||
|
||||
TEST_ASSERT_NULL(cJSONUtils_GetPointer(item, NULL));
|
||||
TEST_ASSERT_NULL(cJSONUtils_GetPointer(NULL, "pointer"));
|
||||
TEST_ASSERT_NULL(cJSONUtils_GetPointerCaseSensitive(NULL, "pointer"));
|
||||
TEST_ASSERT_NULL(cJSONUtils_GetPointerCaseSensitive(item, NULL));
|
||||
TEST_ASSERT_NULL(cJSONUtils_GeneratePatches(item, NULL));
|
||||
TEST_ASSERT_NULL(cJSONUtils_GeneratePatches(NULL, item));
|
||||
TEST_ASSERT_NULL(cJSONUtils_GeneratePatchesCaseSensitive(item, NULL));
|
||||
TEST_ASSERT_NULL(cJSONUtils_GeneratePatchesCaseSensitive(NULL, item));
|
||||
cJSONUtils_AddPatchToArray(item, "path", "add", NULL);
|
||||
cJSONUtils_AddPatchToArray(item, "path", NULL, item);
|
||||
cJSONUtils_AddPatchToArray(item, NULL, "add", item);
|
||||
cJSONUtils_AddPatchToArray(NULL, "path", "add", item);
|
||||
cJSONUtils_ApplyPatches(item, NULL);
|
||||
cJSONUtils_ApplyPatches(NULL, item);
|
||||
cJSONUtils_ApplyPatchesCaseSensitive(item, NULL);
|
||||
cJSONUtils_ApplyPatchesCaseSensitive(NULL, item);
|
||||
TEST_ASSERT_NULL(cJSONUtils_MergePatch(item, NULL));
|
||||
item = cJSON_CreateString("item");
|
||||
TEST_ASSERT_NULL(cJSONUtils_MergePatchCaseSensitive(item, NULL));
|
||||
item = cJSON_CreateString("item");
|
||||
/* these calls are actually valid */
|
||||
/* cJSONUtils_MergePatch(NULL, item); */
|
||||
/* cJSONUtils_MergePatchCaseSensitive(NULL, item);*/
|
||||
/* cJSONUtils_GenerateMergePatch(item, NULL); */
|
||||
/* cJSONUtils_GenerateMergePatch(NULL, item); */
|
||||
/* cJSONUtils_GenerateMergePatchCaseSensitive(item, NULL); */
|
||||
/* cJSONUtils_GenerateMergePatchCaseSensitive(NULL, item); */
|
||||
|
||||
TEST_ASSERT_NULL(cJSONUtils_FindPointerFromObjectTo(item, NULL));
|
||||
TEST_ASSERT_NULL(cJSONUtils_FindPointerFromObjectTo(NULL, item));
|
||||
cJSONUtils_SortObject(NULL);
|
||||
cJSONUtils_SortObjectCaseSensitive(NULL);
|
||||
|
||||
cJSON_Delete(item);
|
||||
}
|
||||
|
||||
int main(void)
|
||||
{
|
||||
UNITY_BEGIN();
|
||||
|
||||
RUN_TEST(cjson_utils_functions_shouldnt_crash_with_null_pointers);
|
||||
|
||||
return UNITY_END();
|
||||
}
|
||||
@@ -142,7 +142,7 @@ static void file_test6_should_not_be_parsed(void)
|
||||
tree = cJSON_Parse(test6);
|
||||
TEST_ASSERT_NULL_MESSAGE(tree, "Should fail to parse what is not JSON.");
|
||||
|
||||
TEST_ASSERT_EQUAL_STRING_MESSAGE(test6, cJSON_GetErrorPtr(), "Error pointer is incorrect.");
|
||||
TEST_ASSERT_EQUAL_PTR_MESSAGE(test6, cJSON_GetErrorPtr(), "Error pointer is incorrect.");
|
||||
|
||||
if (test6 != NULL)
|
||||
{
|
||||
@@ -179,6 +179,22 @@ static void file_test11_should_be_parsed_and_printed(void)
|
||||
do_test("test11");
|
||||
}
|
||||
|
||||
static void test12_should_not_be_parsed(void)
|
||||
{
|
||||
const char *test12 = "{ \"name\": ";
|
||||
cJSON *tree = NULL;
|
||||
|
||||
tree = cJSON_Parse(test12);
|
||||
TEST_ASSERT_NULL_MESSAGE(tree, "Should fail to parse incomplete JSON.");
|
||||
|
||||
TEST_ASSERT_EQUAL_PTR_MESSAGE(test12 + strlen(test12), cJSON_GetErrorPtr(), "Error pointer is incorrect.");
|
||||
|
||||
if (tree != NULL)
|
||||
{
|
||||
cJSON_Delete(tree);
|
||||
}
|
||||
}
|
||||
|
||||
int main(void)
|
||||
{
|
||||
UNITY_BEGIN();
|
||||
@@ -193,5 +209,6 @@ int main(void)
|
||||
RUN_TEST(file_test9_should_be_parsed_and_printed);
|
||||
RUN_TEST(file_test10_should_be_parsed_and_printed);
|
||||
RUN_TEST(file_test11_should_be_parsed_and_printed);
|
||||
RUN_TEST(test12_should_not_be_parsed);
|
||||
return UNITY_END();
|
||||
}
|
||||
|
||||
@@ -31,8 +31,8 @@
|
||||
static void parse_hex4_should_parse_all_combinations(void)
|
||||
{
|
||||
unsigned int number = 0;
|
||||
unsigned char digits_lower[5];
|
||||
unsigned char digits_upper[5];
|
||||
unsigned char digits_lower[6];
|
||||
unsigned char digits_upper[6];
|
||||
/* test all combinations */
|
||||
for (number = 0; number <= 0xFFFF; number++)
|
||||
{
|
||||
|
||||
@@ -40,11 +40,23 @@ static void parse_with_opts_should_handle_empty_strings(void)
|
||||
{
|
||||
const char empty_string[] = "";
|
||||
const char *error_pointer = NULL;
|
||||
|
||||
TEST_ASSERT_NULL(cJSON_ParseWithOpts(empty_string, NULL, false));
|
||||
error_pointer = cJSON_GetErrorPtr();
|
||||
TEST_ASSERT_EQUAL_INT(0, error_pointer - empty_string);
|
||||
TEST_ASSERT_EQUAL_PTR(empty_string, cJSON_GetErrorPtr());
|
||||
|
||||
TEST_ASSERT_NULL(cJSON_ParseWithOpts(empty_string, &error_pointer, false));
|
||||
TEST_ASSERT_EQUAL_INT(0, error_pointer - empty_string);
|
||||
TEST_ASSERT_EQUAL_PTR(empty_string, error_pointer);
|
||||
TEST_ASSERT_EQUAL_PTR(empty_string, cJSON_GetErrorPtr());
|
||||
}
|
||||
|
||||
static void parse_with_opts_should_handle_incomplete_json(void)
|
||||
{
|
||||
const char json[] = "{ \"name\": ";
|
||||
const char *parse_end = NULL;
|
||||
|
||||
TEST_ASSERT_NULL(cJSON_ParseWithOpts(json, &parse_end, false));
|
||||
TEST_ASSERT_EQUAL_PTR(json + strlen(json), parse_end);
|
||||
TEST_ASSERT_EQUAL_PTR(json + strlen(json), cJSON_GetErrorPtr());
|
||||
}
|
||||
|
||||
static void parse_with_opts_should_require_null_if_requested(void)
|
||||
@@ -65,7 +77,7 @@ static void parse_with_opts_should_return_parse_end(void)
|
||||
|
||||
cJSON *item = cJSON_ParseWithOpts(json, &parse_end, false);
|
||||
TEST_ASSERT_NOT_NULL(item);
|
||||
TEST_ASSERT_EQUAL_INT(2, parse_end - json);
|
||||
TEST_ASSERT_EQUAL_PTR(json + 2, parse_end);
|
||||
cJSON_Delete(item);
|
||||
}
|
||||
|
||||
@@ -75,6 +87,7 @@ int main(void)
|
||||
|
||||
RUN_TEST(parse_with_opts_should_handle_null);
|
||||
RUN_TEST(parse_with_opts_should_handle_empty_strings);
|
||||
RUN_TEST(parse_with_opts_should_handle_incomplete_json);
|
||||
RUN_TEST(parse_with_opts_should_require_null_if_requested);
|
||||
RUN_TEST(parse_with_opts_should_return_parse_end);
|
||||
|
||||
|
||||
@@ -704,9 +704,9 @@ int UnityTestMatches(void);
|
||||
#define UNITY_TEST_ASSERT_EQUAL_INT64_ARRAY(expected, actual, num_elements, line, message) UnityAssertEqualIntArray((UNITY_INTERNAL_PTR)(expected), (UNITY_INTERNAL_PTR)(actual), (UNITY_UINT32)(num_elements), (message), (UNITY_LINE_TYPE)(line), UNITY_DISPLAY_STYLE_INT64, UNITY_ARRAY_TO_ARRAY)
|
||||
#define UNITY_TEST_ASSERT_EQUAL_UINT64_ARRAY(expected, actual, num_elements, line, message) UnityAssertEqualIntArray((UNITY_INTERNAL_PTR)(expected), (UNITY_INTERNAL_PTR)(actual), (UNITY_UINT32)(num_elements), (message), (UNITY_LINE_TYPE)(line), UNITY_DISPLAY_STYLE_UINT64, UNITY_ARRAY_TO_ARRAY)
|
||||
#define UNITY_TEST_ASSERT_EQUAL_HEX64_ARRAY(expected, actual, num_elements, line, message) UnityAssertEqualIntArray((UNITY_INTERNAL_PTR)(expected), (UNITY_INTERNAL_PTR)(actual), (UNITY_UINT32)(num_elements), (message), (UNITY_LINE_TYPE)(line), UNITY_DISPLAY_STYLE_HEX64, UNITY_ARRAY_TO_ARRAY)
|
||||
#define UNITY_TEST_ASSERT_EACH_EQUAL_INT64(expected, actual, num_elements, line, message) UnityAssertEqualIntArray(UnityNumToPtr((UNITY_INT)(UNITY_INT64)expected, 8), (UNITY_INTERNAL_PTR)(actual), (UNITY_UINT32)(num_elements), (message), (UNITY_LINE_TYPE)(line), UNITY_DISPLAY_STYLE_INT64, UNITY_ARRAY_TO_ARRAY)
|
||||
#define UNITY_TEST_ASSERT_EACH_EQUAL_UINT64(expected, actual, num_elements, line, message) UnityAssertEqualIntArray(UnityNumToPtr((UNITY_INT)(UNITY_INT64)expected, 8), (UNITY_INTERNAL_PTR)(actual), (UNITY_UINT32)(num_elements), (message), (UNITY_LINE_TYPE)(line), UNITY_DISPLAY_STYLE_UINT64, UNITY_ARRAY_TO_ARRAY)
|
||||
#define UNITY_TEST_ASSERT_EACH_EQUAL_HEX64(expected, actual, num_elements, line, message) UnityAssertEqualIntArray(UnityNumToPtr((UNITY_INT)(UNITY_INT64)expected, 8), (UNITY_INTERNAL_PTR)(actual), (UNITY_UINT32)(num_elements), (message), (UNITY_LINE_TYPE)(line), UNITY_DISPLAY_STYLE_HEX64, UNITY_ARRAY_TO_ARRAY)
|
||||
#define UNITY_TEST_ASSERT_EACH_EQUAL_INT64(expected, actual, num_elements, line, message) UnityAssertEqualIntArray(UnityNumToPtr((UNITY_INT)(UNITY_INT64)expected, 8), (UNITY_INTERNAL_PTR)(actual), (UNITY_UINT32)(num_elements), (message), (UNITY_LINE_TYPE)(line), UNITY_DISPLAY_STYLE_INT64, UNITY_ARRAY_TO_VAL)
|
||||
#define UNITY_TEST_ASSERT_EACH_EQUAL_UINT64(expected, actual, num_elements, line, message) UnityAssertEqualIntArray(UnityNumToPtr((UNITY_INT)(UNITY_UINT64)expected, 8), (UNITY_INTERNAL_PTR)(actual), (UNITY_UINT32)(num_elements), (message), (UNITY_LINE_TYPE)(line), UNITY_DISPLAY_STYLE_UINT64, UNITY_ARRAY_TO_VAL)
|
||||
#define UNITY_TEST_ASSERT_EACH_EQUAL_HEX64(expected, actual, num_elements, line, message) UnityAssertEqualIntArray(UnityNumToPtr((UNITY_INT)(UNITY_INT64)expected, 8), (UNITY_INTERNAL_PTR)(actual), (UNITY_UINT32)(num_elements), (message), (UNITY_LINE_TYPE)(line), UNITY_DISPLAY_STYLE_HEX64, UNITY_ARRAY_TO_VAL)
|
||||
#define UNITY_TEST_ASSERT_INT64_WITHIN(delta, expected, actual, line, message) UnityAssertNumbersWithin((delta), (UNITY_INT)(expected), (UNITY_INT)(actual), (message), (UNITY_LINE_TYPE)(line), UNITY_DISPLAY_STYLE_INT64)
|
||||
#define UNITY_TEST_ASSERT_UINT64_WITHIN(delta, expected, actual, line, message) UnityAssertNumbersWithin((delta), (UNITY_INT)(expected), (UNITY_INT)(actual), (message), (UNITY_LINE_TYPE)(line), UNITY_DISPLAY_STYLE_UINT64)
|
||||
#define UNITY_TEST_ASSERT_HEX64_WITHIN(delta, expected, actual, line, message) UnityAssertNumbersWithin((delta), (UNITY_INT)(expected), (UNITY_INT)(actual), (message), (UNITY_LINE_TYPE)(line), UNITY_DISPLAY_STYLE_HEX64)
|
||||
|
||||
6
valgrind.supp
Normal file
6
valgrind.supp
Normal file
@@ -0,0 +1,6 @@
|
||||
{
|
||||
suppress_ld_on_armv7
|
||||
Memcheck:Cond
|
||||
...
|
||||
obj:*/ld-*.so
|
||||
}
|
||||
Reference in New Issue
Block a user