Hello, please review this small change . I noticed that at a few places in java.desktop checking malloc/calloc return values is not done. This should be changed .
Additionally I think we miss here a free in an early return, changed it too : http://cr.openjdk.java.net/~mbaesken/webrevs/8230480.0/src/java.desktop/unix/native/common/awt/X11Color.c.frames.html 567 if (depth > 12) { 568 free (awt_data->awtImage); 569 return 1; 570 } Build/webrev : https://bugs.openjdk.java.net/browse/JDK-8230480 http://cr.openjdk.java.net/~mbaesken/webrevs/8230480.0/ Thanks, Matthias