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 leak#12471 #12533

Closed
wants to merge 1 commit into from
Closed

fix memory leak#12471 #12533

wants to merge 1 commit into from

Conversation

@luxinyou
Copy link
Contributor

@luxinyou luxinyou commented Jul 25, 2020

Source leaks conf/conf_def.c
Fixes #12471

Checklist
  • documentation is added or updated
  • tests are added or updated
CLA: trivial
@luxinyou luxinyou force-pushed the luxinyou:master branch to 018c89d Jul 25, 2020
@mattcaswell
Copy link
Member

@mattcaswell mattcaswell commented Jul 27, 2020

I think you meant:

Fixes #12471

@mattcaswell
Copy link
Member

@mattcaswell mattcaswell commented Jul 27, 2020

Travis failures look unrelated to this PR.

Copy link
Member

@mattcaswell mattcaswell left a comment

LGTM. I agree this is trivial.

Copy link
Contributor

@paulidale paulidale left a comment

Agreed, trivial.

@paulidale
Copy link
Contributor

@paulidale paulidale commented Jul 27, 2020

And for 1.1.1.

@openssl-machine
Copy link

@openssl-machine openssl-machine commented Jul 28, 2020

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set. Please review the updates and set the label manually.

@slontis slontis self-assigned this Sep 7, 2020
openssl-machine pushed a commit that referenced this pull request Sep 7, 2020
Fixes #12471
CLA: trivial

Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Paul Dale <paul.dale@oracle.com>
Reviewed-by: Shane Lontis <shane.lontis@oracle.com>
(Merged from #12533)
@slontis
Copy link
Contributor

@slontis slontis commented Sep 7, 2020

Merged to master..

openssl-machine pushed a commit that referenced this pull request Sep 7, 2020
Fixes #12471
CLA: trivial

Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Paul Dale <paul.dale@oracle.com>
Reviewed-by: Shane Lontis <shane.lontis@oracle.com>
(Merged from #12533)

(cherry picked from commit 4348995)
@slontis
Copy link
Contributor

@slontis slontis commented Sep 7, 2020

Thanks for fixing..
Merged also to 111

@slontis slontis closed this Sep 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

6 participants