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

Possible string dataloss in xpcconvert.cpp

VERIFIED FIXED

Status

()

Core
XPConnect
VERIFIED FIXED
16 years ago
16 years ago

People

(Reporter: jst, Assigned: David Bradley)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

16 years ago
Jband pointed this out, need to pass the length of the string to
nsAString::Assign() to make things a bit faster, and it will make sure embedded
null characters in the JSString will be passed to the nsAString. Patch coming up...
(Reporter)

Comment 1

16 years ago
Created attachment 55159 [details] [diff] [review]
Proposed fix
(Reporter)

Comment 2

16 years ago
Comment on attachment 55159 [details] [diff] [review]
Proposed fix

r=bz, rs=jband
Attachment #55159 - Flags: superreview+
Attachment #55159 - Flags: review+
(Reporter)

Comment 3

16 years ago
Fixed.
Status: NEW → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED

Comment 4

16 years ago
Marking Verified - 
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.