Test class JSSample breaks memory ownership rules

CLOSED INVALID

Status

Core Graveyard
Java to XPCOM Bridge
P3
normal
CLOSED INVALID
18 years ago
3 years ago

People

(Reporter: Frank Mitchell, Assigned: Igor Kushnirskiy)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

18 years ago
JSSample::SetSomeName() should copy the char* it receives, not assign it to its
own instance variable.

The code was written before the XPCOM rules were formalized, so there may be
other violations.
(Reporter)

Updated

18 years ago
Status: NEW → ASSIGNED
(Reporter)

Comment 1

18 years ago
Actually, it doesn't, but it does use new/delete instead of nsAllocator.  I've
got a fix in my workarea, pending review.

Comment 2

18 years ago
Frank Mitchell no longer works at Sun and his email is bouncing. Reassigning
his 21 bugs to george for reassignment.
Assignee: frankm → drapeau
Status: ASSIGNED → NEW
(Assignee)

Comment 3

18 years ago
This sources are obsolete.
Assignee: drapeau → idk
(Assignee)

Comment 4

18 years ago
closing as invalid
Status: NEW → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → INVALID

Comment 5

16 years ago
VERIFIED invalid.
Must be closed.
Status: RESOLVED → VERIFIED
QA Contact: leila.garin → avm

Comment 6

16 years ago
Closing as obsolete
Status: VERIFIED → CLOSED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.