Fix incorrect uses of free in cJSON code

The cJSON library we use provides a specific function (cJSON_Free() to free allocations made by cJSON_Print(), however we do not always correctly use cJSON_Free, instead using raw free(). While strictly incorrect, this currently works as long as cJSON uses malloc to allocate memory.

Given we will moving to cbmalloc instead of system malloc, need to update all uses to be correct.