Re: ntdll: Implement RtlValidRelativeSecurityDescriptor.

[Marcus Meissner]

On Fri, Oct 09, 2015 at 06:20:50PM +0800, Dmitry Timoshkov wrote:
> Signed-off-by: Dmitry Timoshkov
>
> dlls/ntdll/ntdll.spec | 2 +-
>  dlls/ntdll/sec.c      | 9 +++++++++
>  include/winternl.h    | 1 +
>  3 files changed, 11 insertions(+), 1 deletion(-)
>
> diff --git a/dlls/ntdll/ntdll.spec b/dlls/ntdll/ntdll.spec
> index ca3561d..1e68ef3 100644
> --- a/dlls/ntdll/ntdll.spec
> +++ b/dlls/ntdll/ntdll.spec
> @@ -936,7 +936,7 @@
>  @ stub RtlUsageHeap
>  @ cdecl -i386 -norelay RtlUshortByteSwap() NTDLL_RtlUshortByteSwap
>  @ stdcall RtlValidAcl(ptr)
> -# @ stub RtlValidRelativeSecurityDescriptor
> +@ stdcall RtlValidRelativeSecurityDescriptor(ptr long long)
>  @ stdcall RtlValidSecurityDescriptor(ptr)
>  @ stdcall RtlValidSid(ptr)
>  @ stdcall RtlValidateHeap(long long ptr)
> diff --git a/dlls/ntdll/sec.c b/dlls/ntdll/sec.c
> index 360a579..bc5d5a3 100644
> --- a/dlls/ntdll/sec.c
> +++ b/dlls/ntdll/sec.c
> @@ -558,6 +558,15 @@ NTSTATUS WINAPI RtlValidSecurityDescriptor(
>  }
>
> /**************************************************************************
> -
> - RtlValidRelativeSecurityDescriptor		[NTDLL.@]
> - */
>
> +BOOLEAN WINAPI RtlValidRelativeSecurityDescriptor(PSECURITY_DESCRIPTOR descriptor,
> - ULONG length, SECURITY_INFORMATION info)
>
> +{
> - return RtlValidSecurityDescriptor(descriptor) == STATUS_SUCCESS;
>
> +}

Perhaps add a semi-stub FIXME message, in case some app needs the behaviour
triggered by the other arguments?

Ciao, Marcus