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

Fix memory leaks identified by cppcheck (IDFGH-5573) #7296

Closed
wants to merge 4 commits into from

Conversation

alexhenrie
Copy link
Contributor

These memory leaks were identified by the cppcheck static analyzer.

@espressif-bot espressif-bot added the Status: Opened Issue is new label Jul 20, 2021
@github-actions github-actions bot changed the title Fix memory leaks identified by cppcheck Fix memory leaks identified by cppcheck (IDFGH-5573) Jul 20, 2021
@Alvin1Zhang
Copy link
Collaborator

Thanks for your contribution.

@espressif-bot espressif-bot added Status: In Progress Work is in progress and removed Status: Opened Issue is new labels Jul 25, 2021
@AdityaHPatwardhan
Copy link
Collaborator

@alexhenrie Thank you for the PR. We are in the process of merging it in our internal code base, it should be available on GitHub in a few days.

@espressif-bot espressif-bot added Status: Done Issue is done internally Resolution: Done Issue is done internally and removed Status: In Progress Work is in progress labels Jul 25, 2021
espressif-bot pushed a commit that referenced this pull request Jul 27, 2021
And remove dead error handling code from unregister_select.

Closes #7296
espressif-bot pushed a commit that referenced this pull request Jul 28, 2021
And remove dead error handling code from unregister_select.

Closes #7296
espressif-bot pushed a commit that referenced this pull request Jul 30, 2021
And remove dead error handling code from unregister_select.

Closes #7296
espressif-bot pushed a commit that referenced this pull request Aug 30, 2021
And remove dead error handling code from unregister_select.

Closes #7296
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Resolution: Done Issue is done internally Status: Done Issue is done internally
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants