Re: [ros-dev] [ros-diffs] [ekohl] 59013: [LSASRV] - LsapCheckLogonProcess: Open client process using the PROCESS_DUP_HANDLE access right because we need to duplicate the logon token back into the clients handle table.

Tue, 14 May 2013 02:15:20 -0700 

On 2013-05-13 15:05, [email protected] wrote:
>          Status = NtReplyWaitReceivePort(AuthPortHandle,
>                                          (PVOID*)&LogonContext,
> -                                        &ReplyMsg->h,
> -                                        &RequestMsg.h);
> +                                        (PPORT_MESSAGE)ReplyMsg,
> +                                        (PPORT_MESSAGE)&RequestMsg);

As the Guy Who Officially Hates Casts™, I have to ask: wasn't the
previous version better for type safety? With the cast, an additional
[or missing] & can slip in and the code will compile without complaint.

Best,
Thomas

_______________________________________________
Ros-dev mailing list
[email protected]
http://www.reactos.org/mailman/listinfo/ros-dev

Reply via email to