Changeset 70601 in vbox for trunk/src/VBox/HostServices/SharedOpenGL/unpacker

Timestamp:

Jan 16, 2018 4:26:58 PM (7 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

120311

Message:

3D: SharedOpenGL code cleanup, bugref:9043

Location:

trunk/src/VBox/HostServices/SharedOpenGL/unpacker

Files:

• unpack_stipple.c (modified) (1 diff)
• unpack_texture.c (modified) (1 diff)

trunk/src/VBox/HostServices/SharedOpenGL/unpacker/unpack_stipple.c

@@ -10 +10 @@
 {
     int nodata = READ_DATA(0, int);
-    GLubyte *mask;
 
     if (nodata)
-        mask = (void*) (uintptr_t) READ_DATA(4, GLint);
+    {
+        crError("crUnpackPolygonStipple: GL_PIXEL_UNPACK_BUFFER is not supported");
+        INCR_DATA_PTR(8);
+    }
     else
-        mask = DATA_POINTER( 4, GLubyte );
+    {
+        GLubyte *mask;
 
-    cr_unpackDispatch.PolygonStipple(mask);
-
-    if (nodata)
-        INCR_DATA_PTR(8);
-    else
-        INCR_DATA_PTR(4 + 32*32/8);
+        mask = DATA_POINTER(4, GLubyte);
+        cr_unpackDispatch.PolygonStipple(mask);
+        // Stipple mask consists of 32 * 32 bits
+        INCR_DATA_PTR(4 + 32 * 32 / 8);
+    }
 }

trunk/src/VBox/HostServices/SharedOpenGL/unpacker/unpack_texture.c

@@ -290 +290 @@
     unsigned int n_param = READ_DATA( 0, int ) - ( sizeof(int) + 8 );
 
-    if ( n_param > sizeof(params) )
-        crError( "crUnpackTexGendv: n_param=%d, expected <= %d\n", n_param,
-                 (unsigned int)sizeof(params) );
+    if (n_param > sizeof(params))
+    {
+        crError("crUnpackTexGendv: n_param=%d, expected <= %d\n", n_param,
+            (unsigned int)sizeof(params));
+        return;
+    }
+
     crMemcpy( params, DATA_POINTER( sizeof( int ) + 8, GLdouble ), n_param );