cJSON_AddItemToObject() causes segmentation fault if id == NULL

[tranzistoren]

First, the function:

cJSON_AddItemToObject(cJSON *object, const char *string, cJSON *item)
{
    cJSON_AddItemToObjectCS(object, (char*)cJSON_strdup((const unsigned char*)string, &global_hooks), item)
    item->type &= ~cJSON_StringIsConst;
}

In cJSON_AddItemToObjectCS we see:

if (!item)
{
    return;
}

So that would be fine, but then, cJSON_AddItemToObject references a field from item, which, as we know, is a NULL.

[FSMaxB]

Yes, cJSON_AddItemToObject should also check if the item is a NULL pointer.

[FSMaxB]

Fixed in version 1.5.6