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

null in ReadStrinZ()...

VERIFIED INVALID

Status

()

Core
Networking
P3
normal
VERIFIED INVALID
18 years ago
17 years ago

People

(Reporter: rickg, Assigned: Scott Collins)

Tracking

Trunk
x86
Windows NT
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

(Reporter)

Description

18 years ago
When loading winfiles.com, the nsString debug assertion that tests for embedded 
nulls is getting tripped. It may be legal, but you should review the code.

Comment 1

18 years ago
=> scc
Assignee: warren → scc
(Assignee)

Updated

18 years ago
Status: NEW → ASSIGNED
Target Milestone: --- → M16
(Assignee)

Comment 2

18 years ago
It's a reasonable warning in the presence of embedded NULLs given rickgs 
implementation.  Removing for this good case would stop it from catching other 
peoples bad cases.  I think this warning needs to stay in the code until the new 
concrete string classes land and replace this use.  Sorry.
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → INVALID

Comment 3

17 years ago
vrfy invalid
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.