[ros-bugs] [Bug 6109] PATCH: fixes for desktop.c and winsta.c

[ReactOS.Bugzilla at reactos.org]

http://www.reactos.org/bugzilla/show_bug.cgi?id=6109





--- Comment #12 from ThePhysicist <timo.kreuzer at web.de>  2011-04-16 10:32:24 CET ---
You are comparing the session Id with the process session handle:
> if(PsGetCurrentProcess()->Session != DesktopObject->rpwinstaParent->SessionId)
This is by chance working on our current hacked implementation (the SessionId
is the Session handle casted to ULONG) but it is not correct.
The session check is also not neccessary, since win32k is (once we support
this) loaded on a per session basis. So all handles are session wide. If the
handle is valid in the current session, then the object belongs to the current
session.
Can you explain why there would be crash when the new desktop equals the old
one?
Btw, please don't mix tabs and spaces for indentation, use spaces only.


-- 
Configure bugmail: http://www.reactos.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.