List split not initialising fully.

[andysCaplin]

sort_list() isn't initialising the previous pointer at the start of the second list to NULL.

[andysCaplin]

I tried to create a PR and couldn't the fix is in cJSON_Utils.c - just added line 502 below

498 if ((second != NULL) && (second->prev != NULL))
499 {
500 /* Split the lists */
501 second->prev->next = NULL;
502 second->prev = NULL;
503 }

[FSMaxB]

Thanks. I fixed this in 1.7.10

[andysCaplin]

Cheers for implementing the fix.

The prev from that element did end up getting traversed in one of my tests and it was pointing at freed memory - valgrind caught it.

The test did a sort and the element with the bad previous pointer ended up as the 1st element. The problem happened when I did a remove operation on it.