UMR: Uninitialized memory read in nsMsgSearchTerm::~nsMsgSearchTerm(void)

VERIFIED FIXED in mozilla1.2alpha

Status

SeaMonkey
MailNews: Message Display
P3
trivial
VERIFIED FIXED
17 years ago
10 years ago

People

(Reporter: stephend@netscape.com (gone - use stephen.donner@gmail.com instead), Assigned: Bienvenu)

Tracking

Trunk
mozilla1.2alpha
x86
Windows 2000

Firefox Tracking Flags

(Not tracked)

Details

Windows 2000, latest trunk, I typed "re" into the quick search widget, on an 
IMAP account.

    [W] UMR: Uninitialized memory read in nsMsgSearchTerm::~nsMsgSearchTerm
(void) {1 occurrence}
        Reading 4 bytes from 0x08ca2e74 (4 bytes at 0x08ca2e74 uninitialized)
        Address 0x08ca2e74 is 12 bytes into a 72 byte block at 0x08ca2e68
    Address 0x08ca2e74 points to a C++ new block in heap 0x02920000
    Thread ID: 0x38c
    Error location
        nsMsgSearchTerm::~nsMsgSearchTerm(void) [nsMsgSearchTerm.cpp:333]
        ???            [ip=0x08ca2e68]
        nsMsgSearchTerm::`vector deleting destructor'(UINT) [msgbase.dll]
        nsMsgServiceProviderService::Release(void) [nsMsgServiceProvider.cpp:69]
        XPCJSRuntime::GCCallback(JSContext *,JSGCStatus) [xpcjsruntime.cpp:529]
        DOMGCCallback  [nsJSEnvironment.cpp:1550]
        js_ForceGC     [jsgc.c:961]
        DocumentViewerImpl::Init(nsIWidget *,nsIDeviceContext *,nsRect const&) 
[nsDocumentViewer.cpp:1001]
        nsDocShell::SetupNewViewer(nsIContentViewer *) [nsDocShell.cpp:3778]
        nsWebShell::SetupNewViewer(nsIContentViewer *) [nsWebShell.cpp:319]
    Allocation location
        new(UINT)      [msvcrt.DLL]
        nsMsgSearchSession::CreateTerm(nsIMsgSearchTerm * *) 
[nsMsgSearchSession.cpp:117]
        XPTC_InvokeByIndex [xptcinvoke.cpp:152]
        XPCWrappedNative::CallMethod
(XPCCallContext&,CallMode::XPCWrappedNative) [xpcwrappednative.cpp:2009]
        XPC_WN_CallMethod(JSContext *,JSObject *,UINT,long *,long *) 
[xpcwrappednativejsops.cpp:1266]
        js_Invoke      [jsinterp.c:832]
        js_Interpret   [jsinterp.c:2791]
        js_Execute     [jsinterp.c:1012]
        JS_EvaluateUCScriptForPrincipals [jsapi.c:3368]
        nsJSContext::EvaluateString(nsAString const&,void *,nsIPrincipal *,char 
const*,UINT,char const*,nsAString&,int *) [nsJSEnvironment.cpp:653]
        GlobalWindowImpl::RunTimeout(nsTimeoutImpl *) [nsGlobalWindow.cpp:3779]
        GlobalWindowImpl::TimerCallback(nsITimer *,void *) 
[nsGlobalWindow.cpp:4103]
        FireTimeout(HWND__ *,UINT,UINT,DWORD) [nsTimer.cpp:91]
        ScrollDC       [user32.dll]
        ScrollDC       [user32.dll]
        DispatchMessageA [user32.dll]
        DispatchMessageA [USER32.DLL]
        nsAppShell::Run(void) [nsAppShell.cpp:121]
        nsAppShellService::Run(void) [nsAppShellService.cpp:301]
        main1          [nsAppRunner.cpp:1304]
QA Contact: laurel → stephend

Comment 1

17 years ago
marking nsbeta1+ in case this causes a problem. If you don't think it's a big
deal then you can move it out.
Status: NEW → ASSIGNED
Keywords: nsbeta1+
Priority: -- → P3
Target Milestone: --- → mozilla0.9.9

Comment 2

17 years ago
moving to 1.0.1
Target Milestone: mozilla0.9.9 → mozilla1.0.1

Updated

17 years ago
Blocks: 122274
Keywords: nsbeta1+ → nsbeta1-
Target Milestone: mozilla1.0.1 → mozilla1.2

Comment 3

17 years ago
Curious: what's the rationale for rating this bug as "major"? It's hardly
visible to the user in any way.

Comment 4

17 years ago
alecf: you own current blame, bienvenu: you own past blame.
(Assignee)

Comment 5

17 years ago
taking - the search js front end is going through code that should only be gone
through during filter file parsing, for some reason. It's completely harmless,
but it should be trivial to fix.
Assignee: naving → bienvenu
Severity: major → trivial
Status: ASSIGNED → NEW
(Assignee)

Comment 6

16 years ago
this was fixed a while ago, by adding the following line to the default
constructor, nsMsgSearchTerm::nsMsgSearchTerm():

    m_attribute = nsMsgSearchAttrib::Default;                                   
Status: NEW → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED
Verified FIXED on the trunk with Purify running under Windows 2000.
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey

Updated

10 years ago
Component: MailNews: Search → MailNews: Message Display
QA Contact: stephend → search
You need to log in before you can comment on or make changes to this bug.