180927 - nsNNTPNewsgroupList::CleanUp deletes uninitialized m_knownArts.set

Status: RESOLVED FIXED

(MailNews Core :: Backend, defect)

Description

[timeless]

if (m_knownArts.set) {
    delete m_knownArts.set;

+	m_knownArts.set	0xcdcdcdcd

nsNNTPNewsgroupList::CleanUp(nsNNTPNewsgroupList * const 0x03b43380) line 148 +
31 bytes
nsNNTPNewsgroupList::~nsNNTPNewsgroupList() line 109
nsNNTPNewsgroupList::`scalar deleting destructor'(unsigned int 1) + 15 bytes
nsNNTPNewsgroupList::Release(nsNNTPNewsgroupList * const 0x03b43380) line 111 +
179 bytes
XPCWrappedNative::~XPCWrappedNative() line 547 + 18 bytes
XPCWrappedNative::`scalar deleting destructor'(unsigned int 1) + 15 bytes
XPCWrappedNative::Release(XPCWrappedNative * const 0x03b43320) line 777 + 147 bytes
XPCWrappedNative::FlatJSObjectFinalized(JSContext * 0x004f3e70, JSObject *
0x035820f0) line 897
XPC_WN_NoHelper_Finalize(JSContext * 0x004f3e70, JSObject * 0x035820f0) line 632
js_FinalizeObject(JSContext * 0x004f3e70, JSObject * 0x035820f0) line 1840 + 96
bytes
js_GC(JSContext * 0x004f3e70, unsigned int 0) line 1311 + 11 bytes
js_ForceGC(JSContext * 0x004f3e70, unsigned int 0) line 993 + 13 bytes
JS_GC(JSContext * 0x004f3e70) line 1659 + 11 bytes
main(int 1, char * * 0x004a46b4) line 939 + 10 bytes
mainCRTStartup() line 338 + 17 bytes

Comment 1

[timeless]

Attachment: move constructor stuff to the constructor...

Comment 2

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 106809 [details] [diff] [review]
move constructor stuff to the constructor...

sr=bzbarsky if those are in the right order (please check!)

Comment 3

[Dan Mosedale (:dmosedale, :dmose)]

Comment on attachment 106809 [details] [diff] [review]
move constructor stuff to the constructor...

r=dmose

Comment 4

[timeless]

checked in