If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

LossyCopyUTF16toASCII() and CopyASCIItoUTF16() don't work right when passed null.

RESOLVED FIXED

Status

()

Core
String
RESOLVED FIXED
14 years ago
14 years ago

People

(Reporter: jst, Assigned: jst)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

14 years ago
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

14 years ago
Created attachment 138999 [details] [diff] [review]
Truncate even if nothing to copy.
(Assignee)

Updated

14 years ago
Attachment #138999 - Flags: superreview?(peterv)
Attachment #138999 - Flags: review?(jshin)
Attachment #138999 - Flags: superreview?(peterv) → superreview+

Comment 2

14 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

14 years ago
Fixed.
Status: NEW → RESOLVED
Last Resolved: 14 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.