Skip to content

Conversation

@maebex
Copy link
Contributor

@maebex maebex commented Mar 30, 2024

Fixes #833

@Alanscut
Copy link
Collaborator

Hi @maebex
Thanks for your contribution.
Maybe we can have some tests along with your fix?

@Alanscut Alanscut merged commit 1ac905d into DaveGamble:develop May 9, 2024
@Alanscut
Copy link
Collaborator

Alanscut commented May 9, 2024

Awww
I didn't notice this PR was pushing to branch develop.

Alanscut added a commit to Alanscut/cJSON that referenced this pull request May 9, 2024
Add some tests for setting NULL to deallocated pointers
releated to DaveGamble#842 and DaveGamble#833
Alanscut added a commit to Alanscut/cJSON that referenced this pull request May 9, 2024
Add some tests for setting NULL to deallocated pointers
releated to DaveGamble#842 and DaveGamble#833
@Alanscut
Copy link
Collaborator

Alanscut commented May 9, 2024

Hi @maebex

I cherry-picked your commit to branch master and added some tests.

Please take a look at #855

Alanscut added a commit that referenced this pull request May 13, 2024
Add some tests for setting NULL to deallocated pointers
releated to #842 and #833
Alanscut added a commit to Alanscut/cJSON that referenced this pull request Jun 19, 2024
Alanscut added a commit that referenced this pull request Jun 19, 2024
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.

2 participants