New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.
Starred by 5 users

Issue metadata

Status: WontFix
Owner:
Last visit > 30 days ago
Closed: Mar 30
Cc:
Components:
NextAction: ----
OS: ----
Pri: 3
Type: Patch



Sign in to add a comment

Memoryleaks Dtls/Srtp by using InitializeSSL and CleanupSSL without doing a call

Reported by berthold...@gmail.com, Feb 21 2014

Issue description

What steps will reproduce the problem?
1.webrtc rev. 5462
2.with new implementation acticated opennss 
3.start of application with "InitializeSSL" and shutdown of application with "CleanupSSL"

What is the expected result?
All allocated memory has to be freed.


What do you see instead?

Visual Leak Detector detected 663 memory leaks (95517 bytes).

What version of the product are you using? On what operating system?
webrtc/trunk revision 5462, Windows 7

Please provide any additional information below.

with the corrections in the attached file the memory leaks is 0.





 
memoryleaks_nss_without_call.txt
6.4 KB View Download
Project Member

Comment 1 by braveyao@webrtc.org, Feb 24 2014

Cc: juberti@webrtc.org braveyao@webrtc.org vikasmarwaha@webrtc.org
Owner: jiayl@webrtc.org
@jiay, PTAL!
Project Member

Comment 2 by jiayl@webrtc.org, Feb 26 2014

Cc: wtc@google.com
wtc,

it seems to be a bug inside nss. Could you take a look?

Project Member

Comment 3 by jiayl@webrtc.org, Feb 27 2014

bertholdherrmann08,

I talked with wtc offline. He offers to fix the problem in NSS upstream. Could you format your change into a patch file to make it easier to port for wtc? 

Comment 4 by wtc@google.com, Feb 27 2014

berthold...:

I am an NSS developer. Thank you for reporting the leaks. I filed
an upstream NSS bug report:
https://bugzilla.mozilla.org/show_bug.cgi?id=977407

Once I have your patch file, I will review it and check it in.
I have copied the patched files to the .zip file. So you can open and copy to your directory. With the nss files I have a problem to generate a .patch file. I hope you can work with this solution.
ISSUE_2956.zip
42.9 KB Download
Project Member

Comment 6 by juberti@webrtc.org, Jun 6 2014

Cc: bemasc@webrtc.org
Wan-Teh, did this get resolved?
Project Member

Comment 7 by juberti@webrtc.org, Dec 16 2014

Labels: Area-Network
ping wtc@
Project Member

Comment 8 by juberti@webrtc.org, Dec 17 2014

Labels: -Type-Bug Type-Patch
Project Member

Comment 9 by pthatcher@webrtc.org, Jan 6 2015

Labels: EngTriaged
Owner: juberti@webrtc.org
Status: Assigned
Justin, please keep ping Wan-Teh.
Project Member

Comment 10 by pthatcher@webrtc.org, Nov 8 2016

Labels: Pri-3
Project Member

Comment 11 by deadbeef@chromium.org, Mar 30

Status: WontFix (was: Assigned)
Obsolete since we moved from NSS to BoringSSL: https://bugs.chromium.org/p/webrtc/issues/detail?id=4497

Sign in to add a comment