From 18f3bc28a0173c62d0c1650fdca70a904e83ad31 Mon Sep 17 00:00:00 2001
From: Timo Kreuzer <timo.kreuzer@reactos.org>
Date: Wed, 8 Aug 2007 00:28:08 +0000
Subject: [PATCH] make NtGdiGetRandomRgn behave more like on XP, 33/34 tests
 now passing in w32knapi test

svn path=/trunk/; revision=28233
---
 .../subsystems/win32/win32k/objects/region.c  | 33 ++++++++++---------
 1 file changed, 18 insertions(+), 15 deletions(-)

diff --git a/reactos/subsystems/win32/win32k/objects/region.c b/reactos/subsystems/win32/win32k/objects/region.c
index da2dd3045a7..6b5648347c6 100644
--- a/reactos/subsystems/win32/win32k/objects/region.c
+++ b/reactos/subsystems/win32/win32k/objects/region.c
@@ -2349,22 +2349,15 @@ UnsafeIntGetRgnBox(PROSRGNDATA  Rgn,
 INT STDCALL
 NtGdiGetRandomRgn(HDC hDC, HRGN hDest, INT iCode)
 {
+    INT ret = 0;
     PDC pDC;
     HRGN hSrc = NULL;
     POINT org;
 
-    if ((hDC == NULL) || (hDest == NULL))
-    {
-        return -1;
-    }
-    if ((iCode<1 ) || (iCode>4 ))
-    {
-        return -1;
-    }
-
     pDC = DC_LockDc(hDC);
     if (pDC == NULL)
     {
+        SetLastWin32Error(ERROR_INVALID_HANDLE);
         return -1;
     }
 
@@ -2375,12 +2368,12 @@ NtGdiGetRandomRgn(HDC hDC, HRGN hDest, INT iCode)
             break;
         case 2:
             //hSrc = dc->hMetaRgn;
-            DPRINT1("hMetaRgn not implement\n");
+            DPRINT1("hMetaRgn not implemented\n");
             DC_UnlockDc(pDC);
             return -1;
             break;
         case 3:
-            DPRINT1("waring : hMetaRgn not implement\n");
+            DPRINT1("hMetaRgn not implemented\n");
             //hSrc = dc->hMetaClipRgn;
             if(!hSrc)
             {
@@ -2390,20 +2383,30 @@ NtGdiGetRandomRgn(HDC hDC, HRGN hDest, INT iCode)
             break;
         case 4:
             hSrc = pDC->w.hVisRgn;
+            break;
+        default:
+            hSrc = 0;
     }
     if (hSrc)
     {
-        NtGdiCombineRgn(hDest, hSrc, 0, RGN_COPY);
+        if(NtGdiCombineRgn(hDest, hSrc, 0, RGN_COPY) == ERROR)
+        {
+        	ret = -1;
+        }
+        else
+        {
+        	ret = 1;
+        }
     }
-    if (iCode ==  SYSRGN)
+    if (iCode == SYSRGN)
     {
         IntGdiGetDCOrgEx(pDC, &org);
         NtGdiOffsetRgn(hDest, org.x, org.y );
     }
 
-   DC_UnlockDc(pDC);
+    DC_UnlockDc(pDC);
 
-    return (hSrc != 0);
+    return ret;
 }
 
 INT STDCALL