asserting (in nsAReadableString) when reply to a message

VERIFIED FIXED

Status

MailNews Core
Composition
P3
normal
VERIFIED FIXED
18 years ago
9 years ago

People

(Reporter: (not reading, please use seth@sspitzer.org instead), Assigned: Nisheeth Ranjan)

Tracking

Trunk
x86
Windows NT

Firefox Tracking Flags

(Not tracked)

Details

here's the assertion:

NTDLL! 77f7629c()
nsDebug::Assertion(const char * 0x01e380c8 
??_C@_0BG@EIKL@?$HMCharAt?$HM?5out?9of?9range?$AA@, const char * 0x01e380e0 
??_C@_0BA@OBAB@aIndex?$DMLength?$CI?$CJ?$AA@, const char * 0x01e3191c 
??_C@_0CK@NPGP@?4?4?2?4?4?2?4?4?2dist?2include?2nsAReadabl@, int 544) line 242 + 
13 bytes
basic_nsAReadableString<unsigned short>::CharAt(unsigned int 0) line 544 + 42 
bytes
basic_nsAReadableString<unsigned short>::First() line 564
nsHTMLFragmentContentSink::GetAttributeValueAt(const nsIParserNode & {...}, int 
0, nsString & {...}) line 779 + 8 bytes
nsHTMLFragmentContentSink::AddAttributes(const nsIParserNode & {...}, nsIContent 
* 0x04023798) line 927
nsHTMLFragmentContentSink::OpenContainer(nsHTMLFragmentContentSink * const 
0x03fccc00, const nsIParserNode & {...}) line 465 + 19 bytes
CNavDTD::OpenContainer(const nsIParserNode * 0x02f388e0, nsHTMLTag eHTMLTag_pre, 
int 1, nsEntryStack * 0x00000000) line 3187 + 31 bytes
CNavDTD::HandleDefaultStartToken(CToken * 0x02740fa0, nsHTMLTag eHTMLTag_pre, 
nsIParserNode * 0x02f388e0) line 1138 + 20 bytes
CNavDTD::HandleStartToken(CToken * 0x02740fa0) line 1562 + 22 bytes
CNavDTD::HandleToken(CNavDTD * const 0x04013210, CToken * 0x02740fa0, nsIParser 
* 0x03fb4150) line 770 + 12 bytes
CNavDTD::BuildModel(CNavDTD * const 0x04013210, nsIParser * 0x03fb4150, 
nsITokenizer * 0x04023b00, nsITokenObserver * 0x00000000, nsIContentSink * 
0x03fccc00) line 499 + 20 bytes
nsParser::BuildModel() line 1646 + 34 bytes
nsParser::ResumeParse(int 0, int 0) line 1527 + 11 bytes
nsParser::Parse(const nsString & {...}, void * 0x0012e90c, const nsString & 
{...}, int 0, int 1, nsDTDMode eDTDMode_autodetect) line 1337 + 15 bytes
nsParser::ParseFragment(const nsString & {...}, void * 0x00000000, nsITagStack & 
{...}, unsigned int 0, const nsString & {...}, nsDTDMode eDTDMode_autodetect) 
line 1466 + 41 bytes
nsRange::CreateContextualFragment(nsRange * const 0x03f7ca34, const nsString & 
{...}, nsIDOMDocumentFragment * * 0x0012eea8) line 1932 + 52 bytes
nsHTMLEditor::InsertHTMLWithCharset(nsHTMLEditor * const 0x03e3c4ac, const 
nsString & {...}, const nsString & {...}) line 2474 + 72 bytes
nsHTMLEditor::InsertAsCitedQuotation(nsHTMLEditor * const 0x03e3c4b0, const 
nsString & {...}, const nsString & {...}, int 1, const nsString & {...}, 
nsIDOMNode * * 0x0012f574) line 5240 + 24 bytes
nsHTMLEditorLog::InsertAsCitedQuotation(nsHTMLEditorLog * const 0x03e3c4b0, 
const nsString & {...}, const nsString & {...}, int 1, const nsString & {...}, 
nsIDOMNode * * 0x0012f574) line 465 + 29 bytes
nsEditorShell::InsertAsCitedQuotation(nsEditorShell * const 0x04081a40, const 
unsigned short * 0x03fb62c0, const unsigned short * 0x03fb5330, int 1, const 
unsigned short * 0x0012f4d8, nsIDOMNode * * 0x0012f574) line 2443 + 64 bytes
nsMsgCompose::ConvertAndLoadComposeWindow(nsIEditorShell * 0x04081a40, nsString 
& {...}, nsString & {...}, nsString & {...}, int 1, int 1) line 253 + 93 bytes
QuotingOutputStreamListener::OnStopRequest(QuotingOutputStreamListener * const 
0x03e41bc0, nsIChannel * 0x03e576e0, nsISupports * 0x03e55f40, unsigned int 0, 
nsISupports * 0x03e55f40) line 1455
nsStreamConverter::OnStopRequest(nsStreamConverter * const 0x03e57960, 
nsIChannel * 0x03e56834, nsISupports * 0x03e55f40, unsigned int 0, const 
unsigned short * 0x00000000) line 869
nsMsgProtocol::OnStopRequest(nsMsgProtocol * const 0x03e56830, nsIChannel * 
0x03e5af70, nsISupports * 0x03e55f40, unsigned int 0, const unsigned short * 
0x00000000) line 227 + 92 bytes
nsMailboxProtocol::OnStopRequest(nsMailboxProtocol * const 0x03e56830, 
nsIChannel * 0x03e5af70, nsISupports * 0x03e55f40, unsigned int 0, const 
unsigned short * 0x00000000) line 200
nsFileChannel::OnStopRequest(nsFileChannel * const 0x03e5af78, nsIChannel * 
0x03e5add0, nsISupports * 0x03e55f40, unsigned int 0, const unsigned short * 
0x00000000) line 635 + 45 bytes
nsOnStopRequestEvent::HandleEvent(nsOnStopRequestEvent * const 0x03e5ab20) line 
307
nsStreamListenerEvent::HandlePLEvent(PLEvent * 0x03e5a870) line 97 + 12 bytes
PL_HandleEvent(PLEvent * 0x03e5a870) line 575 + 10 bytes
PL_ProcessPendingEvents(PLEventQueue * 0x0107cfd0) line 520 + 9 bytes
_md_EventReceiverProc(HWND__ * 0x000601fe, unsigned int 49363, unsigned int 0, 
long 17289168) line 1032 + 9 bytes
USER32! 77e71820()
0107cfd0()

Comment 1

18 years ago
I've been seeing this a lot too, especially when reading a message.  This
assertion is happening in layout.  reassigning to Nisheeth. I'm not sure who
owns this file anymore so if this is someone else's please reassign.
Assignee: ducarroz → nisheeth

Comment 2

18 years ago
Index: src/nsHTMLFragmentContentSink.cpp
Here is a patch:

===================================================================
RCS file: 
/cvsroot/mozilla/layout/html/document/src/nsHTMLFragmentContentSink.cpp,v
retrieving revision 1.19
diff -r1.19 nsHTMLFragmentContentSink.cpp
779,785c779,789
<   PRUnichar first = aResult.First();
<   if ((first == '\"') || (first == '\'')) {
<     if (aResult.Last() == first) {
<       aResult.Cut(0, 1);
<       PRInt32 pos = aResult.Length() - 1;
<       if (pos >= 0) {
<         aResult.Cut(pos, 1);
---
>   if (aResult.Length() > 0) {
>     PRUnichar first = aResult.First();
>     if ((first == '\"') || (first == '\'')) {
>       if (aResult.Last() == first) {
>         aResult.Cut(0, 1);
>         PRInt32 pos = aResult.Length() - 1;
>         if (pos >= 0) {
>           aResult.Cut(pos, 1);
>         }
>       } else {
>         // Mismatched quotes - leave them in
787,788d790
<     } else {
<       // Mismatched quotes - leave them in
(Assignee)

Comment 3

18 years ago
The patch is checked in.  Thanks, Rod.
Status: NEW → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED
I don't see this on a current debug CVS pull (windows 200).  marking verified.
Status: RESOLVED → VERIFIED
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.