[MLK] JavaPackage_resolve() leaks a C string

VERIFIED FIXED in mozilla0.9

Status

Core Graveyard
Java: Live Connect
P4
normal
VERIFIED FIXED
18 years ago
7 years ago

People

(Reporter: Patrick C. Beard, Assigned: Patrick C. Beard)

Tracking

({memory-leak})

Trunk
mozilla0.9
memory-leak

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(2 attachments)

(Assignee)

Description

18 years ago
If jsj_EnterJava(cx, &jEnv) returns a NULL jEnv, the C string newPath is leaked.
(Assignee)

Comment 1

18 years ago
Created attachment 12977 [details]
leak report

Updated

18 years ago
Keywords: mlk
(Assignee)

Comment 2

18 years ago
This is a very noticeable leak, as it happens whenever a new JS context/global 
object is initialized in the DOM.
Keywords: nsbeta3
(Assignee)

Comment 3

18 years ago
Low priority leak, only occurs when Java isn't installed.
Status: NEW → ASSIGNED
Priority: P3 → P4
Whiteboard: [nsbeta3+]

Comment 4

18 years ago
PDT is downgrading to [nsbeta3-], looking for big bloat.  How many javascript 
context would a page have?  Seems like low priority to fix.
Whiteboard: [nsbeta3+] → [nsbeta3-][minus]

Comment 5

18 years ago
asking for reconsideration. java isn't installed in some of our embedding cases.
Whiteboard: [nsbeta3-][minus] → [minus]
(Assignee)

Updated

18 years ago
Whiteboard: [minus] → [nsbeta3-] [minus]
(Assignee)

Comment 6

18 years ago
Here's a trivial patch to fix. Given the simplicity of this fix, it is very low 
risk and thus I'd nominate for RTM.
Keywords: rtm
Whiteboard: [nsbeta3-] [minus] → [nsbeta3-] [minus] [rtm+]
(Assignee)

Comment 7

18 years ago
Created attachment 16276 [details] [diff] [review]
patch to fix memory leak

Comment 8

18 years ago
PDT marking [rtm-] because small leaks with little user impact aren't going to
move the needle on Seamonkey stability. If appropriate, please check in on the
trunk.
Whiteboard: [nsbeta3-] [minus] [rtm+] → [nsbeta3-] [minus] [rtm-]
(Assignee)

Comment 9

17 years ago
Moving to 0.9.
Keywords: nsbeta3, rtm
Whiteboard: [nsbeta3-] [minus] [rtm-]
Target Milestone: --- → mozilla0.9
(Assignee)

Comment 10

17 years ago
Fix checked in.
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED

Comment 11

17 years ago
Marking Verified - 
Status: RESOLVED → VERIFIED

Updated

7 years ago
Component: Java: Live Connect → Java: Live Connect
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.