[KSECDD] Improve GetSecurityUserInfo stub (#2438)

- Fix UNIMPLEMENTED_DBGBREAK seen using FileMon
- Add SAL2 Annotations and remove unneeded comments
- Return NULL pointer

CORE-15186

drivers/crypto/ksecdd/ksecdd.spec

@@ -12,7 +12,7 @@
 @ stdcall FreeCredentialsHandle(ptr)
 @ stdcall GenerateDirEfs()
 @ stdcall GenerateSessionKey()
-@ stdcall GetSecurityUserInfo()
+@ stdcall GetSecurityUserInfo(ptr long ptr)
 @ stdcall ImpersonateSecurityContext(ptr)
 @ stdcall ImportSecurityContextW(ptr ptr ptr ptr)
 @ stdcall InitSecurityInterfaceW()

drivers/crypto/ksecdd/stubs.c

@@ -13,6 +13,7 @@
 #define NDEBUG
 #include <debug.h>
 
+typedef PVOID PSECURITY_PACKAGE_OPTIONS, PSecurityUserData;
 
 /* FUNCTIONS ******************************************************************/
 
@@ -160,11 +161,16 @@ GenerateSessionKey(VOID)
     UNIMPLEMENTED_DBGBREAK();
 }
 
-VOID
+SECURITY_STATUS
 SEC_ENTRY
-GetSecurityUserInfo(VOID)
+GetSecurityUserInfo(
+    _In_opt_    PLUID LogonId,
+    _In_        ULONG Flags,
+    _Outptr_    PSecurityUserData *UserInformation)
 {
-    UNIMPLEMENTED_DBGBREAK();
+    UNIMPLEMENTED;
+    *UserInformation = NULL;
+    return STATUS_UNSUCCESSFUL;
 }
 
 SECURITY_STATUS