Closed Bug 621848 Opened 9 years ago Closed 9 years ago

memory leak in nsNSSComponent::IdentityInfoInit

Categories

(Core :: Security: PSM, defect)

x86
All
defect
Not set

Tracking

()

VERIFIED FIXED
mozilla2.0b10

People

(Reporter: eagle.lu, Assigned: eagle.lu)

Details

(Keywords: memory-leak)

Attachments

(1 file, 1 obsolete file)

Using libumem.so provided by OpenSolaris and run following test case, I find a memory leak in nsNSSComponent::IdentityInfoInit

Test case:
1. Run Thunderbird trunk buid
2. Click 'Cancel' button in the password dialog
3. Quit from Thunderbird

The libumem.so shows a memory leak and the following is the trace stack

> 0c4e1908::bufctl_audit
            ADDR          BUFADDR        TIMESTAMP           THREAD
                            CACHE          LASTLOG         CONTENTS
         c4e1908          c4e26a0     b578ebb7cf89               14
                          807c510                0                0
                 libumem.so.1`umem_cache_alloc_debug+0x158
                 libumem.so.1`umem_cache_alloc+0x157
                 libumem.so.1`umem_alloc+0xd0
                 libumem.so.1`malloc+0x2d
                 libnspr4.so`PR_Malloc+0x63
                 libnssutil3.so`PORT_Alloc_Util+0x46
                 libnssutil3.so`SECITEM_AllocItem_Util+0xfb
                 libnssutil3.so`NSSBase64_DecodeBuffer_Util+0x99
                 libnssutil3.so`ATOB_ConvertAsciiToItem_Util+0x7a
                 libnss3.so`ATOB_ConvertAsciiToItem+0x2e
                 libxul.so`PRStatus nsNSSComponent::IdentityInfoInit+0x84
                 libnspr4.so`PR_CallOnce+0x5e
                 
                 libxul.so`unsigned nsNSSComponent::EnsureIdentityInfoLoaded+0x3
                 7
                 libxul.so`unsigned nsNSSCertificate::hasValidEVOidTag+0x102
                 libxul.so`unsigned nsNSSCertificate::getValidEVOidTag+0x75
Attached patch patch (obsolete) — Splinter Review
Attachment #500173 - Flags: review?(wtc)
Assignee: nobody → nobody
Component: Libraries → Security: PSM
Product: NSS → Core
QA Contact: libraries → psm
Version: trunk → unspecified
Attachment #500173 - Flags: review?(wtc) → review?(honzab.moz)
Comment on attachment 500173 [details] [diff] [review]
patch

The same should be made in function loadTestEVInfos (the same file).

r=honzab
Attachment #500173 - Flags: review?(honzab.moz) → review+
Assignee: nobody → brian.lu
Attachment #500173 - Attachment is obsolete: true
Keywords: checkin-needed
Comment on attachment 501585 [details] [diff] [review]
add same codes in loadTestEVInfos

Can we get approval2.0 for this?  This is a low-risk mlk fix.
Attachment #501585 - Flags: approval2.0?
This needs approval2.0+ before it can be checkin-needed.
Keywords: checkin-needed
Attachment #501585 - Flags: approval2.0? → approval2.0+
http://hg.mozilla.org/mozilla-central/rev/70ca6a1b9d29
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b10
Attachment #501585 - Flags: review+
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.