Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
8282887: Potential memory leak in sun.util.locale.provider.HostLocale…
…ProviderAdapterImpl.getNumberPattern() on Windows

Reviewed-by: naoto, alanb
  • Loading branch information
zhengyu123 committed Mar 15, 2022
1 parent 710653c commit 2cddf3f
Showing 1 changed file with 5 additions and 6 deletions.
Expand Up @@ -359,18 +359,17 @@ JNIEXPORT jobjectArray JNICALL Java_sun_util_locale_provider_HostLocaleProviderA
JNIEXPORT jstring JNICALL Java_sun_util_locale_provider_HostLocaleProviderAdapterImpl_getNumberPattern
(JNIEnv *env, jclass cls, jint numberStyle, jstring jlangtag) {
const jchar *langtag;
jstring ret;
jstring ret = NULL;
WCHAR * pattern;

langtag = (*env)->GetStringChars(env, jlangtag, NULL);
CHECK_NULL_RETURN(langtag, NULL);
pattern = getNumberPattern(langtag, numberStyle);
CHECK_NULL_RETURN(pattern, NULL);

if (!IS_NULL(pattern)) {
ret = (*env)->NewString(env, pattern, (jsize)wcslen(pattern));
free(pattern);
}
(*env)->ReleaseStringChars(env, jlangtag, langtag);
ret = (*env)->NewString(env, pattern, (jsize)wcslen(pattern));
free(pattern);

return ret;
}

Expand Down

1 comment on commit 2cddf3f

@openjdk-notifier
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.