[greatlrd] 27244: do not cause a crash in lock, some how lock and unlock does not work as they should

Author: greatlrd
Date: Wed Jun 20 09:23:09 2007
New Revision: 27244

URL: http://svn.reactos.org/svn/reactos?rev=27244&view=rev
Log:
do not cause a crash in lock, some how lock and unlock does not work as they should

Modified:
    trunk/reactos/dll/directx/ddraw/Surface/surface_main.c

Modified: trunk/reactos/dll/directx/ddraw/Surface/surface_main.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/dll/directx/ddraw/Surface/…
==============================================================================
--- trunk/reactos/dll/directx/ddraw/Surface/surface_main.c (original)
+++ trunk/reactos/dll/directx/ddraw/Surface/surface_main.c Wed Jun 20 09:23:09 2007
@@ -196,6 +196,7 @@
 
 	/* FIXME add a check see if lock suport or not */
 
+    DX_STUB_str("test\n");
 	if (prect!=NULL)
     {
         mdLock.bHasRect = TRUE;
@@ -206,8 +207,8 @@
         mdLock.bHasRect = FALSE;		
     }
 	
-	This->lpLcl->lpSurfMore->slist[0]->hDC =
This->lpLcl->lpSurfMore->lpDD_lcl->hDC;
-	 
+	//This->lpLcl->lpSurfMore->slist[0]->hDC =
This->lpLcl->lpSurfMore->lpDD_lcl->hDC;
+	DX_STUB_str("test\n");
     mdLock.ddRVal = DDERR_NOTPALETTIZED;
     mdLock.Lock =
This->lpLcl->lpSurfMore->lpDD_lcl->lpDDCB->HALDDSurface.Lock;
     mdLock.dwFlags = flags;
@@ -215,12 +216,14 @@
 	mdLock.lpDD = This->lpLcl->lpSurfMore->lpDD_lcl->lpGbl;   
     mdLock.lpSurfData = NULL;
      
+    DX_STUB_str("test\n");
 	if (!DdResetVisrgn(This->lpLcl->lpSurfMore->slist[0], NULL)) 
     {
       DX_STUB_str("Here DdResetVisrgn lock");
       return DDERR_UNSUPPORTED;
     }
    
+    DX_STUB_str("test\n");
     if (mdLock.Lock(&mdLock)!= DDHAL_DRIVER_HANDLED)
     {
       DX_STUB_str("Here DDHAL_DRIVER_HANDLED lock");
@@ -228,6 +231,7 @@
     }       
    
     // FIXME ??? is this right ?? 
+    DX_STUB_str("test\n");
     if (pDDSD != NULL)
     {
 		ZeroMemory(pDDSD,sizeof(DDSURFACEDESC2));
@@ -247,17 +251,21 @@
         //    pDDSD->dwSize = sizeof(DDSURFACEDESC);
         //}
 
+        DX_STUB_str("test\n");
         pDDSD->lpSurface = (LPVOID)  mdLock.lpSurfData;	
 		
 		pDDSD->dwHeight =This->lpLcl->lpGbl->wHeight;
 		pDDSD->dwWidth = This->lpLcl->lpGbl->wWidth;
-		pDDSD->ddpfPixelFormat.dwRGBBitCount =
This->lpLcl->lpSurfMore->lpDD_lcl->lpGbl->
-			                                   lpModeInfo->dwBPP;
+
+         DX_STUB_str("test\n");
+
+
+		pDDSD->ddpfPixelFormat.dwRGBBitCount = This->lpLcl->lpGbl->lPitch/ 8;
 		pDDSD->lPitch = This->lpLcl->lpGbl->lPitch;
 		pDDSD->dwFlags = DDSD_WIDTH | DDSD_HEIGHT | DDSD_PITCH;
-
-    }
-      
+        DX_STUB_str("test\n");
+    }
+    DX_STUB_str("test\n");
     return mdLock.ddRVal;
 }