From 4519685bb857a0867f8ce870c27ed5f0c34004de Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?J=C3=A9r=C3=B4me=20Gardou?= <jerome.gardou@reactos.org>
Date: Mon, 2 Aug 2010 16:49:51 +0000
Subject: [PATCH] [WIN32K]   - NtGdiGetBitmapBits : Bits we're given are 16
 bits aligned.   - NtGdiGetDIBitsInternal : we must not stretch, for this
 there is NtGdiStretchDIBitsInternal. Use lower level functions to do the
 work, there is no need to create HDCs etc... Now icons are back.

svn path=/branches/reactos-yarotows/; revision=48416
---
 subsystems/win32/win32k/objects/bitmaps.c | 33 +++++++-
 subsystems/win32/win32k/objects/dibobj.c  | 94 ++++++++---------------
 2 files changed, 64 insertions(+), 63 deletions(-)

diff --git a/subsystems/win32/win32k/objects/bitmaps.c b/subsystems/win32/win32k/objects/bitmaps.c
index c47afa58c12..163fda614a4 100644
--- a/subsystems/win32/win32k/objects/bitmaps.c
+++ b/subsystems/win32/win32k/objects/bitmaps.c
@@ -581,6 +581,36 @@ IntGetBitmapBits(
     return ret;
 }
 
+VOID
+FASTCALL
+UnsafeGetBitmapBits(
+    PSURFACE psurf,
+	DWORD Bytes,
+	OUT PBYTE pvBits)
+{
+	PUCHAR pjDst, pjSrc;
+    LONG lDeltaDst, lDeltaSrc;
+    ULONG nWidth, nHeight, cBitsPixel;
+
+    nWidth = psurf->SurfObj.sizlBitmap.cx;
+    nHeight = psurf->SurfObj.sizlBitmap.cy;
+    cBitsPixel = BitsPerFormat(psurf->SurfObj.iBitmapFormat);
+
+    /* Get pointers */
+    pjSrc = psurf->SurfObj.pvScan0;
+    pjDst = pvBits;
+    lDeltaSrc = psurf->SurfObj.lDelta;
+    lDeltaDst = BITMAP_GetWidthBytes(nWidth, cBitsPixel);
+
+    while (nHeight--)
+    {
+        /* Copy one line */
+        RtlCopyMemory(pjDst, pjSrc, lDeltaDst);
+        pjSrc += lDeltaSrc;
+        pjDst += lDeltaDst;
+    }
+}
+
 LONG APIENTRY
 NtGdiGetBitmapBits(
     HBITMAP hBitmap,
@@ -620,7 +650,8 @@ NtGdiGetBitmapBits(
     _SEH2_TRY
     {
         ProbeForWrite(pUnsafeBits, Bytes, 1);
-        ret = IntGetBitmapBits(psurf, Bytes, pUnsafeBits);
+        UnsafeGetBitmapBits(psurf, Bytes, pUnsafeBits);
+		ret = Bytes;
     }
     _SEH2_EXCEPT(EXCEPTION_EXECUTE_HANDLER)
     {
diff --git a/subsystems/win32/win32k/objects/dibobj.c b/subsystems/win32/win32k/objects/dibobj.c
index 74931f8a54f..36b4e9c64ff 100644
--- a/subsystems/win32/win32k/objects/dibobj.c
+++ b/subsystems/win32/win32k/objects/dibobj.c
@@ -695,7 +695,7 @@ NtGdiGetDIBitsInternal(
         Info->bmiHeader.biYPelsPerMeter = 0;
         Info->bmiHeader.biClrUsed = 0;
         Info->bmiHeader.biClrImportant = 0;
-		ScanLines = psurf->SurfObj.sizlBitmap.cy;
+		ScanLines = 1;
 		/* Get Complete info now */
 		goto done;
 
@@ -897,8 +897,11 @@ NtGdiGetDIBitsInternal(
 	{
 		/* Create a DIBSECTION, blt it, profit */
 		PVOID pDIBits ;
-		HBITMAP hBmpDest, hOldDest = NULL, hOldSrc = NULL;
-		HDC hdcDest = NULL, hdcSrc;
+		HBITMAP hBmpDest;
+		PSURFACE psurfDest;
+		EXLATEOBJ exlo;
+		RECT rcDest;
+		POINTL srcPoint;
 		BOOL ret ;
 
 		if (StartScan > psurf->SurfObj.sizlBitmap.cy)
@@ -911,7 +914,15 @@ NtGdiGetDIBitsInternal(
             ScanLines = min(ScanLines, psurf->SurfObj.sizlBitmap.cy - StartScan);
 		}
 
+		/* Fixup values */
+		Info->bmiHeader.biWidth = psurf->SurfObj.sizlBitmap.cx;
+		Info->bmiHeader.biHeight = height < 0 ?
+			-ScanLines : ScanLines;
+		/* Create the DIB */
 		hBmpDest = DIB_CreateDIBSection(pDC, Info, Usage, &pDIBits, NULL, 0, 0);
+		/* Restore them */
+		Info->bmiHeader.biWidth = width;
+		Info->bmiHeader.biHeight = height;
 		
 		if(!hBmpDest)
 		{
@@ -921,56 +932,25 @@ NtGdiGetDIBitsInternal(
 			goto done ;
 		}
 
-		if(psurf->hdc)
-			hdcSrc = psurf->hdc;
-		else
-		{
-			hdcSrc = NtGdiCreateCompatibleDC(0);
-			if(!hdcSrc)
-			{
-				DPRINT1("Error: could not create HDC!\n");
-				ScanLines = 0;
-				goto cleanup_blt;
-			}
-			hOldSrc = NtGdiSelectBitmap(hdcSrc, hBitmap);
-			if(!hOldSrc)
-			{
-				DPRINT1("Error : Could not Select bitmap\n");
-				ScanLines = 0;
-				goto cleanup_blt;
-			}
-		}
+		psurfDest = SURFACE_LockSurface(hBmpDest);
 
-		hdcDest = NtGdiCreateCompatibleDC(0);
-		if(!hdcDest)
-		{
-			DPRINT1("Error: could not create HDC!\n");
-			ScanLines = 0;
-			goto cleanup_blt;
-		}
-		hOldDest = NtGdiSelectBitmap(hdcDest, hBmpDest);
-		if(!hOldDest)
-		{
-			DPRINT1("Error : Could not Select bitmap\n");
-			ScanLines = 0;
-			goto cleanup_blt;
-		}
+		rcDest.left = 0;
+		rcDest.top = 0;
+		rcDest.bottom = ScanLines;
+		rcDest.right = psurf->SurfObj.sizlBitmap.cx;
 
-		ret = GreStretchBltMask(hdcDest,
-								0,
-								0,
-								width, 
-								height,
-								hdcSrc,
-								0,
-								StartScan,
-								psurf->SurfObj.sizlBitmap.cx,
-								ScanLines,
-								SRCCOPY,
-								0,
-								NULL,
-								0,
-								0);
+		srcPoint.x = 0;
+		srcPoint.y = height < 0 ?
+			psurf->SurfObj.sizlBitmap.cy - (StartScan + ScanLines) : StartScan;
+
+		EXLATEOBJ_vInitialize(&exlo, psurf->ppal, psurfDest->ppal, 0, 0, 0);
+
+		ret = IntEngCopyBits(&psurfDest->SurfObj,
+							 &psurf->SurfObj,
+							 NULL,
+							 &exlo.xlo,
+							 &rcDest,
+							 &srcPoint);
 
 		if(!ret)
 			ScanLines = 0;
@@ -994,18 +974,8 @@ NtGdiGetDIBitsInternal(
 			}
 		}
 
-	cleanup_blt:
-		if(hdcSrc && (hdcSrc != psurf->hdc))
-		{
-			if(hOldSrc) NtGdiSelectBitmap(hdcSrc, hOldSrc);
-			NtGdiDeleteObjectApp(hdcSrc);
-		}
-		if(hdcSrc)
-		{
-			if(hOldDest) NtGdiSelectBitmap(hdcDest, hOldDest);
-			NtGdiDeleteObjectApp(hdcDest);
-		}
 		GreDeleteObject(hBmpDest);
+		EXLATEOBJ_vCleanup(&exlo);
 	}
 
 done: