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

Memory leak bug in apps/s_server.c: 3330 #7061

Closed
gangfan opened this issue Aug 27, 2018 · 1 comment
Closed

Memory leak bug in apps/s_server.c: 3330 #7061

gangfan opened this issue Aug 27, 2018 · 1 comment

Comments

@gangfan
Copy link

gangfan commented Aug 27, 2018 •
edited

Hi,

We find a memory leak bug in apps/s_server.c 3294 (3330 in the most recent version). The variable con is not released when taking the "err" goto statement at line 3305.

image

GF
Sourcebrella
@gangfan gangfan changed the title Memory leak bug in mem.c 3294. Memory leak bug in apps/s_server.c:3424 Aug 27, 2018
@gangfan gangfan changed the title Memory leak bug in apps/s_server.c:3424 Memory leak bug in apps/s_server.c: 3330 Aug 27, 2018
@mattcaswell
Copy link
Member

Yes, looks like we need to free con in both www_body and rev_body

mattcaswell added a commit to mattcaswell/openssl that referenced this issue Aug 27, 2018
@mattcaswell
Free SSL object on an error path
4b2cb93 
Thanks to @fangang190 for reporting this

Fixes openssl#7061
@mattcaswell mattcaswell mentioned this issue Aug 27, 2018
Closed
InfoHunter added a commit to InfoHunter/openssl that referenced this issue Aug 27, 2018
@InfoHunter
Fix memory leak issues
05988ec 
Reported by issue openssl#7060 and openssl#7061
@InfoHunter InfoHunter mentioned this issue Aug 27, 2018
Closed
levitte pushed a commit that referenced this issue Aug 30, 2018
@mattcaswell @InfoHunter
Free SSL object on an error path
eeee521 
Thanks to @fangang190 for reporting this

Fixes #7061

Reviewed-by: Paul Yang <yang.yang@baishancloud.com>
(Merged from #7065)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@gangfan @mattcaswell