FMM: Freeing mismatched memory in delete(void *) {1 occurrence}

RESOLVED FIXED

Status

()

Core
XPCOM
--
minor
RESOLVED FIXED
17 years ago
17 years ago

People

(Reporter: Chris Waterson, Assigned: Edward Kandrot)

Tracking

Trunk
x86
Windows 2000
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

17 years ago
Purify coughed up this hairball on nsThread.

[E] FMM: Freeing mismatched memory in delete(void *) {1 occurrence}
        Address 0x02bf6850 points into a malloc'd block 
        Location of free attempt
            delete(void *) [dbgdel.cpp:35]
            nsThread::~nsThread(void) [nsThread.cpp:87]
            nsThread::`vector deleting destructor'(UINT) [xpcom.dll]
            nsThread::Release(void) [nsThread.cpp:148]
            nsThread::Shutdown(void) [nsThread.cpp:404]
            NS_ShutdownXPCOM(nsIServiceManager *) [nsXPComInit.cpp:515]
            main           [nsWinMain.cpp:160]
            mainCRTStartup [crtexe.c:338]
        Allocation location
            calloc         [dbgheap.c:445]
            PR_Calloc      [prmem.c:48]
            PR_NewLogModule [prlog.c:299]
            nsThread::nsThread(void) [nsThread.cpp:62]
            nsIThread::GetIThread(PRThread *,nsIThread * *) [nsThread.cpp:360]
            nsIThread::GetCurrent(nsIThread * *) [nsThread.cpp:341]
            nsIThread::SetMainThread(void) [nsThread.cpp:382]
            NS_InitXPCOM2(nsIServiceManager * *,nsIFile
*,nsIDirectoryServiceProvider *) [nsXPComInit.cpp:259]
            NS_InitXPCOM(nsIServiceManager * *,nsIFile *) [nsXPComInit.cpp:244]
            main           [nsWinMain.cpp:152]
(Assignee)

Comment 1

17 years ago
Created attachment 34450 [details] [diff] [review]
Changes delete to PR_DELETE
(Assignee)

Comment 2

17 years ago
Can I hear an r=, sr=?
Status: NEW → ASSIGNED
PR_DELETE sets its macro-parameter to null, so the caller should not.  Other
than that, r/sr=brendan@mozilla.org.

/be
(Assignee)

Comment 4

17 years ago
Created attachment 34463 [details] [diff] [review]
Thanks.  Removed =NULL.  Need r=.
(Reporter)

Comment 5

17 years ago
r=waterson
(Assignee)

Comment 6

17 years ago
Thanks.  I just checked it in.
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.