changeset 9168:9cfcb8b155ed

7006339: [parfait] potential null pointer dereference in splashscreen Reviewed-by: anthony, art
author pchelko
date Wed, 19 Feb 2014 15:08:52 +0400
parents d7656263ed31
children 5cf31af3c37c
files src/share/native/sun/awt/splashscreen/splashscreen_gfx_impl.h
diffstat 1 files changed, 6 insertions(+), 2 deletions(-) [+]
line wrap: on
line diff
--- a/src/share/native/sun/awt/splashscreen/splashscreen_gfx_impl.h	Tue Feb 18 14:44:29 2014 +0400
+++ b/src/share/native/sun/awt/splashscreen/splashscreen_gfx_impl.h	Wed Feb 19 15:08:52 2014 +0400
@@ -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);