Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Null pointer fixes #183

Merged
merged 23 commits into from
Jun 28, 2017
Merged

Null pointer fixes #183

merged 23 commits into from
Jun 28, 2017

Conversation

FSMaxB
Copy link
Collaborator

@FSMaxB FSMaxB commented Jun 27, 2017

This fixes a lot of potential null pointer dereferences and some memory leaks.

My search for NULL pointer dereferences was triggered by #182.

I wrote a test bench to test almost all functions with NULL pointers as inputs, but stupid me accidentally deleted it, so I'm gonna have to do that from scratch ...

  • test cJSON functions with NULL parameters
  • fix NULL pointer dereferences in cJSON
  • test cJSON_Utils functions with NULL parameters
  • fix NULL pointer dereferences in cJSON_Utils

@FSMaxB FSMaxB force-pushed the null-pointer-fixes branch from 2118c3d to bdd5ff7 Compare June 28, 2017 12:37
@FSMaxB FSMaxB merged commit 48eaecd into DaveGamble:master Jun 28, 2017
@FSMaxB FSMaxB changed the title WIP: Null pointer fixes Null pointer fixes Jun 28, 2017
@FSMaxB FSMaxB deleted the null-pointer-fixes branch February 1, 2018 00:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant