203758 - Memory leak of 24 bytes from 1 block allocated in CEnumFormatEtc::Clone

Status: VERIFIED DUPLICATE of bug 414482

(Core :: Widget: Win32, defect)

Description

[[e-mail] (gone - use [e-mail] instead)]

Build ID: Latest trunk CVS on Win32 debug running in Purify.

Steps to Reproduce:

1.  mozilla.exe -edit
2.  Click on the HTML Source view
3.  Copy the predefined HTML text
4.  Control-X to cut
5.  Click the close widget on the window and select 'Don't Save' when prompted.

We leak:

    [W] MLK: Memory leak of 24 bytes from 1 block allocated in
CEnumFormatEtc::Clone(IEnumFORMATETC * *)
        Distribution of leaked blocks
        Allocation location
        new(UINT)      [new.cpp:23]
        CEnumFormatEtc::Clone(IEnumFORMATETC * *) [L_Ienumfe.cpp:282]
          *ppEnum=NULL;
        
          //Create the clone
     =>   pNew = new CEnumFormatEtc(mNumFEs, mFEList);
        
          if (NULL==pNew)
              return ResultFromScode(E_OUTOFMEMORY);
        nsDataObj::EnumFormatEtc(DWORD,IEnumFORMATETC * *) [nsDataObj.cpp:291]
        
          switch (dwDir) {
            case DATADIR_GET: {
     =>        m_enumFE->Clone(ppEnum);
            } break;
            case DATADIR_SET:
                *ppEnum=NULL;
        OleSetClipboard [ole32.dll]
        OleSetClipboard [ole32.dll]
        nsClipboard::SetNativeClipboardData(int) [nsClipboard.cpp:241]
        
          IDataObject * dataObj;
          if ( NS_SUCCEEDED(CreateNativeDataObject(mTransferable, &dataObj)) ) {
// this add refs
     =>     ::OleSetClipboard(dataObj);
            dataObj->Release();
          }
        
        nsBaseClipboard::SetData(nsITransferable *,nsIClipboardOwner *,int)
[nsBaseClipboard.cpp:108]
        
          if ( mTransferable ) {
            NS_ADDREF(mTransferable);
     =>     rv = SetNativeClipboardData(aWhichClipboard);
          }
        
          return rv;
        nsCopySupport::HTMLCopy(nsISelection *,nsIDocument *,short)
[nsCopySupport.cpp:281]
        
              // put the transferable on the clipboard
              if (doPutOnClipboard)
     =>         clipboard->SetData(trans, nsnull, aClipboardID);
            }
          }
          return rv;
    PresShell::DoCopy(void) [nsPresShell.cpp:4670]
            return NS_OK;
        
          // call the copy code
     =>   rv = nsCopySupport::HTMLCopy(sel, doc, nsIClipboard::kGlobalClipboard);
          if (NS_FAILED(rv))
            return rv;
        
    nsPlaintextEditor::Copy(void) [nsPlaintextEditor.cpp:1408]
          if (!mPresShellWeak) return NS_ERROR_NOT_INITIALIZED;
          nsCOMPtr<nsIPresShell> ps = do_QueryReferent(mPresShellWeak);
          if (!ps) return NS_ERROR_NOT_INITIALIZED;
     =>   return ps->DoCopy();
        }
        
        NS_IMETHODIMP nsPlaintextEditor::CanCopy(PRBool *aCanCopy)

Comment 1

[Ryan VanderMeulen [:RyanVM]]

Carsten, can you confirm?

Comment 2

[Carsten Book [:Tomcat]]

(In reply to comment #1)
> Carsten, can you confirm?
> 

I guess this Bug report was for the old Mozilla Suite and now Seamonkey, so i can't confirm or this bug, because i have only Firefox Debug Leak Test Builds...maybe someone with a SeaMonkey Debug Build can test this bug.

Comment 4

[Stephen Donner [:stephend] Not actively reading bugmail]

Verified dup