[OpenJDK 2D-Dev] RFR: 8261533: Java_sun_font_CFont_getCascadeList leaks memory according to Xcode

Phil Race prr at openjdk.java.net
Fri Feb 12 16:18:38 UTC 2021


On Fri, 12 Feb 2021 04:48:27 GMT, Prasanta Sadhukhan <psadhukhan at openjdk.org> wrote:

>> The various CF*Copy* functions called here need a matching CFRelease
>
> src/java.desktop/macosx/native/libawt_lwawt/font/AWTFont.m line 578:
> 
>> 576:         (*env)->CallBooleanMethod(env, arrayListOfString, addMID, jFontName);
>> 577:         if ((*env)->ExceptionOccurred(env)) {
>> 578:             CFRelease(fds);
> 
> but DeleteLocalRef(jFontName) shouldn't be called here too?

Unnecessary. The VM releases it when we return right here (next line)
The other delete is needed because we are in a loop and we don't know how many we might need
and it could exceed what the VM provisions for.

-------------

PR: https://git.openjdk.java.net/jdk/pull/2532


More information about the 2d-dev mailing list