64720 - 'CR CR NL' end of line => truncated paste (& possible crash)

Status: RESOLVED DUPLICATE of bug 63563

(Core :: XUL, defect)

Description

[John Morrison]

Overview Description & Steps to Reproduce: 

Open the 'master.ini' file (from .\components directory), and copy the
text from the beginning of the file, to the end of the line that says
"ManifestVersion = 1". This a total of 8 lines. 

The unusual thing about this file is that the lines end with the sequence
'CR CR NL' (a.k.a \015\015\012) [this probably happened from being moved
back and forth from unix to win32].

At any rate, when you paste this copied text into an HTML textarea:

1) The pasted text ends with "...anifestVers", which is missing the last 7
characters. (Not coincidentally, 7 is also the number of preceding lines 
in the region).

2) An extra line break occurs between each line (although it could be argued
that this is the correct behaviour). 

3) There is a random crash that will occur following this paste. I can't hit 
it everytime, but after doing a number of pastes, I will either crash on a 
paste action, or if I switch windows, I crash on a blur, or when I click on 
a link in the page (a bugzilla page) I will crash. The crashes all occur in 
win32 libraries (e.g., ntdll.dll on win2k). Likely on some memory corruption.


Build Date & Platform Bug Found: 
 This bug actually was in Netscape 6 RTM, and is in the branch, and on the 
 trunk (20000108 win2k). 

Additional Builds and Platforms Tested On: 
 I don't know what the equivalent test would be on Mac and Linux since they 
 have different interpretations of line endings. I could try all the reasonable
 permutations, but it seems that the win32 case is a good starting point to 
 look at on the code side.

Comment 1

[Mike Pinkerton (not reading bugmail)]

uh oh simon, more problems.

Comment 2

[Mike Pinkerton (not reading bugmail)]

vidur changed line endings out of the dom on 12/12, the day that another crash
started happening. i think this is a dupe of that bug.

*** This bug has been marked as a duplicate of 63563 ***

Comment 3

[John Morrison]

Actually, this behaviour was noted in Netscape 6 RTM (~Nov 11 builds), a 
month before vidur's changes went into the trunk. 

It may be that the proposed fix will address this anyways, though.