Crash in win32k.sys

[Maarten Bosma]

After calling "NtGdiDdCreateDirectDrawObject" (ntddraw/ddraw.c) I get a 
    bsod. The problem seams to be this line:
> HANDLE hDirectDraw = GDIOBJ_AllocObj(GDI_OBJECT_TYPE_DIRECTDRAW);

The debug output says:
> (objects/gdiobj.c:179) Can't find lookaside list for object type 0x00600000

[Filip Navara]

Maarten Bosma wrote:
> After calling "NtGdiDdCreateDirectDrawObject" (ntddraw/ddraw.c) I get 
> a    bsod. The problem seams to be this line:
> > HANDLE hDirectDraw = GDIOBJ_AllocObj(GDI_OBJECT_TYPE_DIRECTDRAW);
>
> The debug output says:
> > (objects/gdiobj.c:179) Can't find lookaside list for object type 
> > 0x00600000

Uncomment these lines in subsys/win32k/objects/gdiobj.c:
/*{GDI_OBJECT_TYPE_DIRECTDRAW,  sizeof(DD_DIRECTDRAW), DD_Cleanup},
  {GDI_OBJECT_TYPE_DD_SURFACE,  sizeof(DD_SURFACE),    DDSURF_Cleanup},*/

- Filip

[Maarten Bosma]

Filip Navara wrote:
> Uncomment these lines in subsys/win32k/objects/gdiobj.c:
> /*{GDI_OBJECT_TYPE_DIRECTDRAW,  sizeof(DD_DIRECTDRAW), DD_Cleanup},
>  {GDI_OBJECT_TYPE_DD_SURFACE,  sizeof(DD_SURFACE),    DDSURF_Cleanup},*/

If I do so, I get a lot of errors. I guess "win32k/ntddraw.h" has to be 
included somewhere.

Maarten Bosma

[Maarten Bosma]

I wrote:
> > Uncomment these lines in subsys/win32k/objects/gdiobj.c:
> > /*{GDI_OBJECT_TYPE_DIRECTDRAW,  sizeof(DD_DIRECTDRAW), DD_Cleanup},
> >  {GDI_OBJECT_TYPE_DD_SURFACE,  sizeof(DD_SURFACE),    DDSURF_Cleanup},*/
>
> If I do so, I get a lot of errors. I guess "win32k/ntddraw.h" has to be 
> included somewhere.

If I include "intddraw.h", where GDI_OBJECT_TYPE_DIRECTDRAW is defined, 
I get a lot of errors. I think something in the header system has to be 
changed.

[Filip Navara]

Maarten Bosma wrote:
> I wrote:
> > > Uncomment these lines in subsys/win32k/objects/gdiobj.c:
> > > /*{GDI_OBJECT_TYPE_DIRECTDRAW,  sizeof(DD_DIRECTDRAW), DD_Cleanup},
> > >  {GDI_OBJECT_TYPE_DD_SURFACE,  sizeof(DD_SURFACE),    
> > > DDSURF_Cleanup},*/
> >
> > If I do so, I get a lot of errors. I guess "win32k/ntddraw.h" has to 
> > be included somewhere.
>
> If I include "intddraw.h", where GDI_OBJECT_TYPE_DIRECTDRAW is 
> defined, I get a lot of errors. I think something in the header system 
> has to be changed.

See attached patch.

Index: include/win32k/ntddraw.h
===================================================================
RCS file: /CVS/ReactOS/reactos/include/win32k/ntddraw.h,v
retrieving revision 1.2
diff -u -r1.2 ntddraw.h
--- include/win32k/ntddraw.h	28 Oct 2003 19:30:08 -0000	1.2
+++ include/win32k/ntddraw.h	30 Dec 2004 13:35:30 -0000
@@ -2,8 +2,6 @@
 #ifndef __WIN32K_NTDDRAW_H
 #define __WIN32K_NTDDRAW_H

-#include <windows.h>
-#include <ddk/ddrawint.h>
 #include <ddk/d3dnthal.h>

#ifdef __cplusplus
Index: subsys/win32k/w32k.h
===================================================================
RCS file: /CVS/ReactOS/reactos/subsys/win32k/w32k.h,v
retrieving revision 1.7
diff -u -r1.7 w32k.h
--- subsys/win32k/w32k.h	12 Dec 2004 01:40:37 -0000	1.7
+++ subsys/win32k/w32k.h	30 Dec 2004 13:37:58 -0000
@@ -8,6 +8,8 @@
 #include <roscfg.h>
 #include <roskrnl.h>

+#include <win32k/ntddraw.h>
+
 #include <ddk/winddi.h>
 #include <ddk/ntddk.h>
 #include <ddk/ntddmou.h>
@@ -49,6 +51,7 @@
 #include <include/input.h>
 #include <include/inteng.h>
 #include <include/intgdi.h>
+#include <include/intddraw.h>
 #include <include/menu.h>
 #include <include/monitor.h>
 #include <include/mouse.h>
Index: subsys/win32k/include/intddraw.h
===================================================================
RCS file: /CVS/ReactOS/reactos/subsys/win32k/include/intddraw.h,v
retrieving revision 1.2
diff -u -r1.2 intddraw.h
--- subsys/win32k/include/intddraw.h	12 Dec 2004 01:40:37 -0000	1.2
+++ subsys/win32k/include/intddraw.h	30 Dec 2004 13:39:10 -0000
@@ -61,7 +61,7 @@
    PDD_SURFCB_UNLOCK               DdUnlockD3DBuffer;
 } DD_DIRECTDRAW, *PDD_DIRECTDRAW;

-BOOL FASTCALL DD_Cleanup(PDD_DIRECTDRAW pDD);
-BOOL FASTCALL DDSURF_Cleanup(PDD_SURFACE pDDSurf);
+BOOL INTERNAL_CALL DD_Cleanup(PVOID ObjectBody);
+BOOL INTERNAL_CALL DDSURF_Cleanup(PVOID ObjectBody);

#endif /* _INT_W32k_DDRAW */
Index: subsys/win32k/ntddraw/ddraw.c
===================================================================
RCS file: /CVS/ReactOS/reactos/subsys/win32k/ntddraw/ddraw.c,v
retrieving revision 1.3
diff -u -r1.3 ddraw.c
--- subsys/win32k/ntddraw/ddraw.c	12 Dec 2004 01:40:37 -0000	1.3
+++ subsys/win32k/ntddraw/ddraw.c	30 Dec 2004 13:42:22 -0000
@@ -9,11 +9,7 @@
  *       25-10-2003  PB  Created
  */

-#include <ddk/ntddk.h>
-#include <win32k/ntddraw.h>
-#include <win32k/win32k.h>
-#include <include/intddraw.h>
-#include <win32k/gdiobj.h>
+#include <w32k.h>

#define NDEBUG
 #include <debug.h>
@@ -22,9 +18,11 @@
 /* DIRECT DRAW OBJECT                                                   */
 /************************************************************************/

-BOOL FASTCALL
-DD_Cleanup(PDD_DIRECTDRAW pDD)
+BOOL INTERNAL_CALL
+DD_Cleanup(PVOID ObjectBody)
 {
+	PDD_DIRECTDRAW pDD = (PDD_DIRECTDRAW)ObjectBody;
+
    pDD->DrvDisableDirectDraw(pDD->Global.dhpdev);
    return TRUE;
 }
@@ -197,9 +195,10 @@
 /* SURFACE OBJECT                                                       */
 /************************************************************************/

-BOOL FASTCALL
-DDSURF_Cleanup(PDD_SURFACE pDDSurf)
+BOOL INTERNAL_CALL
+DDSURF_Cleanup(PVOID ObjectBody)
 {
+	/* PDD_SURFACE pDDSurf = (PDD_SURFACE)ObjectBody; */
    //FIXME: implement
    return TRUE;
 }
Index: subsys/win32k/objects/gdiobj.c
===================================================================
RCS file: /CVS/ReactOS/reactos/subsys/win32k/objects/gdiobj.c,v
retrieving revision 1.83
diff -u -r1.83 gdiobj.c
--- subsys/win32k/objects/gdiobj.c	21 Dec 2004 04:05:18 -0000	1.83
+++ subsys/win32k/objects/gdiobj.c	30 Dec 2004 13:31:28 -0000
@@ -97,8 +97,8 @@
   {GDI_OBJECT_TYPE_PEN,         sizeof(GDIBRUSHOBJ),   GDI_CleanupDummy},
   {GDI_OBJECT_TYPE_FONT,        sizeof(TEXTOBJ),       GDI_CleanupDummy},
   {GDI_OBJECT_TYPE_DCE,         sizeof(DCE),           DCE_Cleanup},
-/*{GDI_OBJECT_TYPE_DIRECTDRAW,  sizeof(DD_DIRECTDRAW), DD_Cleanup},
-  {GDI_OBJECT_TYPE_DD_SURFACE,  sizeof(DD_SURFACE),    DDSURF_Cleanup},*/
+  {GDI_OBJECT_TYPE_DIRECTDRAW,  sizeof(DD_DIRECTDRAW), DD_Cleanup},
+  {GDI_OBJECT_TYPE_DD_SURFACE,  sizeof(DD_SURFACE),    DDSURF_Cleanup},
   {GDI_OBJECT_TYPE_EXTPEN,      0,                     GDI_CleanupDummy},
   {GDI_OBJECT_TYPE_METADC,      0,                     GDI_CleanupDummy},
   {GDI_OBJECT_TYPE_METAFILE,    0,                     GDI_CleanupDummy},