Re: [ros-dev] [ros-diffs] [akhaldi] 64994: [ADVAPI32] * Update ImpersonateNamedPipeClient(). CORE-8540

[Aleksey Bragin]

Sometimes, "If it ain't broke, don't fix it" is a good thing. Or in 
other case, specify what does this "sync" actually fix.
To me (and I'm known fan of Wine code, just look into Arwinss) it 
plainly looks like Winisation of our own, good code (the style, function 
behavior, whatever else matched ReactOS coding style, development 
practices, etc).

Amine - please fix this any way you prefer. Thanks!

Regards,
Aleksey Bragin

On 26.10.2014 0:10, Ged Murphy wrote:
>   Eeww, these are a bit ugly :(
>
>
>  On 25/10/2014 19:30, "akhaldi(a)svn.reactos.org" <akhaldi(a)svn.reactos.org>
>  wrote:
>
> >   Author: akhaldi
> >  Date: Sat Oct 25 18:30:05 2014
> >  New Revision: 64994
> >
> >  URL: http://svn.reactos.org/svn/reactos?rev=64994&view=rev
> >  Log:
> >  [ADVAPI32]
> >  * Update ImpersonateNamedPipeClient().
> >  CORE-8540
> >
> >  Modified:
> >      trunk/reactos/dll/win32/advapi32/wine/security.c
> >
> >  Modified: trunk/reactos/dll/win32/advapi32/wine/security.c
> >  URL:
> >  http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/advapi32/wine/s
> >  ecurity.c?rev=64994&r1=64993&r2=64994&view=diff
> >  ==========================================================================
> >  ====
> >  --- trunk/reactos/dll/win32/advapi32/wine/security.c	[iso-8859-1]
> >  (original)
> >  +++ trunk/reactos/dll/win32/advapi32/wine/security.c	[iso-8859-1] Sat Oct
> >  25 18:30:05 2014
> >  @@ -954,37 +954,14 @@
> >       return TRUE;
> >  }
> >
> >  -/**********************************************************************
> >  - * ImpersonateNamedPipeClient			EXPORTED
> >  - *
> >  - * @implemented
> >  - */
> >  -BOOL
> >  -WINAPI
> >  -ImpersonateNamedPipeClient(HANDLE hNamedPipe)
> >  -{
> >  -    IO_STATUS_BLOCK StatusBlock;
> >  -    NTSTATUS Status;
> >  -
> >  -    TRACE("ImpersonateNamedPipeClient() called\n");
> >  -
> >  -    Status = NtFsControlFile(hNamedPipe,
> >  -                             NULL,
> >  -                             NULL,
> >  -                             NULL,
> >  -                             &StatusBlock,
> >  -                             FSCTL_PIPE_IMPERSONATE,
> >  -                             NULL,
> >  -                             0,
> >  -                             NULL,
> >  -                             0);
> >  -    if (!NT_SUCCESS(Status))
> >  -    {
> >  -        SetLastError(RtlNtStatusToDosError(Status));
> >  -        return FALSE;
> >  -    }
> >  -
> >  -    return TRUE;
> >  +BOOL WINAPI ImpersonateNamedPipeClient( HANDLE hNamedPipe )
> >  +{
> >  +    IO_STATUS_BLOCK io_block;
> >  +
> >  +    TRACE("(%p)\n", hNamedPipe);
> >  +
> >  +    return set_ntstatus( NtFsControlFile(hNamedPipe, NULL, NULL, NULL,
> >  +                         &io_block, FSCTL_PIPE_IMPERSONATE, NULL, 0,
> >  NULL, 0) );
> >  }
> >
> >  /*
> >
> >  
>