Re: [PATCH v24 0/8] MR3103: server: Create and use a desktop shared mapping for GetCursorPos.

[Jinoh Kang (＠iamahuman)]

Jinoh Kang (@iamahuman) commented about server/winstation.c:
> /* get the thread current desktop */
>  DECL_HANDLER(get_thread_desktop)
>  {
> -
>
> struct desktop *desktop;
> struct thread *thread;
>
> if (!(thread = get_thread_from_id( req->tid ))) return;
> reply->handle = thread->desktop;
> -
>
> reply->index = -1;
> -
> -
>
> if ((desktop = get_thread_desktop( thread, 0 )))

(commits) This function call sets error, which introduces behavioral change (and thus possibility of regression) of NtUserGetThreadDesktop().

The `get_thread_desktop()` call should thus be split into its own commit.