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

RESOLVED FIXED

Status

()

RESOLVED FIXED
15 years ago
15 years ago

People

(Reporter: jst, Assigned: jst)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

15 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

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

Updated

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

Comment 2

15 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

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