On Mon, 5 Dec 2022 09:51:42 GMT, Matthias Baesken <[email protected]> wrote:
> The cleanup and error handling in function awt_parseColorModel in
> awt_parseImage.c could be improved.
src/java.desktop/share/native/libawt/awt/image/awt_parseImage.c line 104:
> 102: cmP)) <= 0) {
> 103: awt_freeParsedRaster(&imageP->raster, FALSE);
> 104: if (cmP->nBits != NULL) free(cmP->nBits);
It seems wrong to me that awt_parseColorModel should return a failure code and
not have cleaned up any storage it itself allocated.
In other words this clean up belongs in there.
-------------
PR: https://git.openjdk.org/jdk/pull/11508