commit d54b662879fc956350b80ede4b507821995739f9
author Petr Pchelko <pchelko@openjdk.org> Wed Feb 19 15:08:52 2014 +0400
committer Petr Pchelko <pchelko@openjdk.org> Wed Feb 19 15:08:52 2014 +0400
tree 9d17d71ec133e0e5dabacadb30c507b7d264baad
parent 16aafdc14e3bc78b1f8494a2ffd9ee8e944cb2ad

7006339: [parfait] potential null pointer dereference in splashscreen

Reviewed-by: anthony, art

jdk/src/share/native/sun/awt/splashscreen/splashscreen_gfx_impl.h

diff --git a/jdk/src/share/native/sun/awt/splashscreen/splashscreen_gfx_impl.h b/jdk/src/share/native/sun/awt/splashscreen/splashscreen_gfx_impl.h
index 10574ad..536b935 100644
--- a/jdk/src/share/native/sun/awt/splashscreen/splashscreen_gfx_impl.h
+++ b/jdk/src/share/native/sun/awt/splashscreen/splashscreen_gfx_impl.h
@@ -189,7 +189,11 @@
             PUT(rgbquad_t, ptr, value);
             break;
         case 3:                /* not supported, LSB or MSB should always be specified */
-            *(int *) 0 = 0;    /* crash */
+            PUT(byte_t, ptr, 0xff); /* Put a stub value */
+            INCP(byte_t, ptr);
+            PUT(byte_t, ptr, 0xff);
+            INCP(byte_t, ptr);
+            PUT(byte_t, ptr, 0xff);
             break;
         case 2:
             PUT(word_t, ptr, value);
@@ -253,7 +257,7 @@
             value = GET(rgbquad_t, ptr);
             break;
         case 3:                /* not supported, LSB or MSB should always be specified */
-            *(int *) 0 = 0;
+            value = 0xFFFFFFFF; /*return a stub value */
             break;
         case 2:
             value = (rgbquad_t) GET(word_t, ptr);