Closed
Bug 59617
Opened 24 years ago
Closed 24 years ago
mail crashes on second never read folder
Categories
(MailNews Core :: Backend, defect, P3)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: axel, Assigned: sspitzer)
References
Details
Attachments
(1 file)
(deleted),
patch
|
Details | Diff | Splinter Review |
There is a small bug in nsLocalStringService::GetBundle that produces a crasher
on all but the first call of this function.
Well, it doesn't return the stringbundle, so not checking the return will crash.
nsMsgMailboxParser::UpdateStatusText does so.
In
http://lxr.mozilla.org/seamonkey/source/mailnews/local/src/nsLocalStringBundle.cpp#70
nsresult rv;
should read
nsresult rv = NS_OK;
as for a uninitialized rv, the NS_ENSURE_SUCCESS(rv,rv) failes otherwise,
and we don't return the cached mLocalStringBundle.
Axel
Comment 1•24 years ago
|
||
Assigning to kin, who knows about this problem. Thanks!
Assignee: sspitzer → kin
Assignee | ||
Comment 2•24 years ago
|
||
I've got this fix for this, and another one in the msg compose string bundle code.
I'll land them as soon as the tree opens.
taking from kin.
Assignee: kin → sspitzer
Comment 3•24 years ago
|
||
Seth, could you add an attachment to this bug with the diffs please?
We (Sun) will want to apply them to our RTM/FCS release as well. Thanks.
Assignee | ||
Comment 5•24 years ago
|
||
Assignee | ||
Comment 6•24 years ago
|
||
fix checked in.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Comment 7•24 years ago
|
||
The same fix has been checked in into the OEM branch: Netscape_6_0_OEM_BRANCH.
a=jdunn@netscape.com.
Comment 10•24 years ago
|
||
Verified as fixed on win32, linux, and macos using the following builds:
win32 commercial seamonkey build 2000-121809-mtrunk installed on P500 Win98
linux commercial seamonkey build 2000-121808-mtrunk installed on P200 RedHat
6.2
macos commercial seamonkey build 2000-121808-mtrunk installed on G3/400 OS 9.04
Status: RESOLVED → VERIFIED
Comment 11•24 years ago
|
||
*** Bug 61752 has been marked as a duplicate of this bug. ***
Updated•20 years ago
|
Product: MailNews → Core
Updated•16 years ago
|
Product: Core → MailNews Core
You need to log in
before you can comment on or make changes to this bug.
Description
•