Closed
Bug 230849
Opened 21 years ago
Closed 21 years ago
LossyCopyUTF16toASCII() and CopyASCIItoUTF16() don't work right when passed null.
Categories
(Core :: XPCOM, defect)
Core
XPCOM
Tracking
()
RESOLVED
FIXED
People
(Reporter: jst, Assigned: jst)
Details
Attachments
(1 file)
|
1.07 KB,
patch
|
jshin1987
:
review+
peterv
:
superreview+
|
Details | Diff | Splinter Review |
If null is passed to either LossyCopyUTF16toASCII() or CopyASCIItoUTF16() the
destination string is not truncated. All other Copy*() functions do this right
(as far as I could see), these should too. Patch coming up.
|
Assignee | |
Comment 1•21 years ago
|
||
|
|
Assignee | |
Updated•21 years ago
|
Attachment #138999 -
Flags: superreview?(peterv)
Attachment #138999 -
Flags: review?(jshin)
|
||
Updated•21 years ago
|
Attachment #138999 -
Flags: superreview?(peterv) → superreview+
|
||
Comment 2•21 years ago
|
||
Comment on attachment 138999 [details] [diff] [review]
Truncate even if nothing to copy.
r=jshin
Attachment #138999 -
Flags: review?(jshin) → review+
|
|
Assignee | |
Comment 3•21 years ago
|
||
Fixed.
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
|
||
Updated•4 years ago
|
Component: String → XPCOM
You need to log in
before you can comment on or make changes to this bug.
Description
•