Closed Bug 103005 Opened 23 years ago Closed 23 years ago

leaking the m_mdbAllFoldersTable in nsMsgFolderCache

Categories

(MailNews Core :: Backend, defect)

Product:
MailNews Core
Component:
Backend
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
VERIFIED FIXED

People

(Reporter: Bienvenu, Assigned: Bienvenu)

Details

(Keywords: memory-leak)

Attachments

(1 file)

proposed fix
693 bytes, patch
sspitzer
: superreview+
Details | Diff | Splinter Review 
we're not releasing the m_mdbAllFoldersTable from nsMsgFolderCache, which will
generally show up as a leak with nsMsgFolderCache::InitExistingDB somewhere at
the top of the stack and a bunch of mork stuff beneath it.
Attached patch proposed fix — — Splinter Review 

    
    

    
  
fix is to release the table.
Status: NEW → ASSIGNED

    
    

    
  
Comment on attachment 51959 [details] [diff] [review]
proposed fix

sr=sspitzer

        Attachment #51959 -
        Flags: superreview+

    
    

    
  
fix checked in.
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Keywords: mlk
QA Contact: esther → stephend

    
    

    
  
I didn't see this with the following operations:

* loading a folder
* dragging a message into a folder
* deleting a message

Verified FIXED using Win2K and Purify (latest opt build with symbols).
Status: RESOLVED → VERIFIED
Product: MailNews → Core
Product: Core → MailNews Core

          You need to log in
          before you can comment on or make changes to this bug.
        






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: