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.

Sign up for GitHub

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

Jump to bottom

Don't NULL check before calling DSO_free. #4703

Closed
wants to merge 1 commit into from
Closed

Don't NULL check before calling DSO_free. #4703

wants to merge 1 commit into from

Conversation

richsalz
Copy link
Contributor

@richsalz richsalz commented Nov 8, 2017

Missed during previous free-null-check cleanups.

@richsalz richsalz added branch: master Merge to master branch approval: review pending This pull request needs review by a committer labels Nov 8, 2017
@richsalz richsalz self-assigned this Nov 8, 2017
@bernd-edlinger bernd-edlinger added approval: done This pull request has the required number of approvals and removed approval: review pending This pull request needs review by a committer labels Nov 8, 2017
@richsalz
Copy link
Contributor Author

richsalz commented Nov 8, 2017

thanks!

@richsalz richsalz closed this Nov 8, 2017
@richsalz richsalz deleted the fix-dso-free branch November 8, 2017 20:01
levitte pushed a commit that referenced this pull request Nov 8, 2017
Don't NULL check before calling DSO_free.
1b6fa9f 
Reviewed-by: Bernd Edlinger <bernd.edlinger@hotmail.de>
(Merged from #4703)
@richsalz
Copy link
Contributor Author

richsalz commented Nov 8, 2017

thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bernd-edlinger bernd-edlinger bernd-edlinger approved these changes

Assignees

@richsalz richsalz

Labels
approval: done This pull request has the required number of approvals branch: master Merge to master branch
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@richsalz @mspncp @bernd-edlinger