Closed
Bug 637592
Opened 15 years ago
Closed 15 years ago
nsJNIString needs to call ReleaseStringChars if len <= 0
Categories
(Core Graveyard :: Widget: Android, defect)
Tracking
(Not tracked)
RESOLVED
DUPLICATE
of bug 637915
People
(Reporter: jdm, Unassigned)
Details
>450 const jchar* jCharPtr = jni->GetStringChars(jstr, NULL);
>451 jsize len = jni->GetStringLength(jstr);
>452 if (!jCharPtr || len <= 0) {
>453 SetIsVoid(PR_TRUE);
>454 return;
>455 }
If jCharPtr is non-null we need to be calling ReleaseStringChars or there's leak potential, as far as I can tell.
|
Reporter | |
Comment 1•15 years ago
|
||
Although according to bug 629604 this situation can't occur and shouldn't even be checked.
|
|
Reporter | |
Comment 2•15 years ago
|
||
Well, I found this before anyone else did, so I feel good about myself at least.
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → DUPLICATE
|
||
Updated•4 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•