Dmitry Timoshkov : secur32: Print contents of LSA_STRING in traces.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: d2c22ab58e47c80c529268748709b03032b635ce
URL:    https://source.winehq.org/git/wine.git/?a=commit;h=d2c22ab58e47c80c529268748...

Author: Dmitry Timoshkov 
Date:   Fri Feb  2 18:22:29 2018 +0800

secur32: Print contents of LSA_STRING in traces.

Signed-off-by: Dmitry Timoshkov 
Signed-off-by: Hans Leidekker 
Signed-off-by: Alexandre Julliard

---

dlls/secur32/lsa.c | 17 ++++++++++++-----
 1 file changed, 12 insertions(+), 5 deletions(-)

diff --git a/dlls/secur32/lsa.c b/dlls/secur32/lsa.c
index 3a99135..511e24a 100644
--- a/dlls/secur32/lsa.c
+++ b/dlls/secur32/lsa.c
@@ -58,6 +58,12 @@ struct lsa_connection
     DWORD magic;
 };

+static const char *debugstr_as(const LSA_STRING *str)
+{
+    if (!str) return "<null>";
+    return debugstr_an(str->Buffer, str->Length);
+}
+
 NTSTATUS WINAPI LsaCallAuthenticationPackage(HANDLE lsa_handle, ULONG package_id,
         PVOID in_buffer, ULONG in_buffer_length,
         PVOID *out_buffer, PULONG out_buffer_length, PNTSTATUS status)
@@ -135,8 +141,8 @@ NTSTATUS WINAPI LsaLogonUser(HANDLE LsaHandle, PLSA_STRING OriginName,
         PVOID* ProfileBuffer, PULONG ProfileBufferLength, PLUID LogonId,
         PHANDLE Token, PQUOTA_LIMITS Quotas, PNTSTATUS SubStatus)
 {
-    FIXME("%p %p %d %d %p %d %p %p %p %p %p %p %p %p stub\n", LsaHandle,
-            OriginName, LogonType, AuthenticationPackage,
+    FIXME("%p %s %d %d %p %d %p %p %p %p %p %p %p %p stub\n", LsaHandle,
+            debugstr_as(OriginName), LogonType, AuthenticationPackage,
             AuthenticationInformation, AuthenticationInformationLength,
             LocalGroups, SourceContext, ProfileBuffer, ProfileBufferLength,
             LogonId, Token, Quotas, SubStatus);
@@ -158,7 +164,8 @@ static NTSTATUS NTAPI lsa_DeleteLogonSession(LUID *logon_id)
 static NTSTATUS NTAPI lsa_AddCredential(LUID *logon_id, ULONG package_id,
     LSA_STRING *primary_key, LSA_STRING *credentials)
 {
-    FIXME("%p,%u,%p,%p: stub\n", logon_id, package_id, primary_key, credentials);
+    FIXME("%p,%u,%s,%s: stub\n", logon_id, package_id,
+        debugstr_as(primary_key), debugstr_as(credentials));
     return STATUS_NOT_IMPLEMENTED;
 }

@@ -172,7 +179,7 @@ static NTSTATUS NTAPI lsa_GetCredentials(LUID *logon_id, ULONG package_id, ULONG

static NTSTATUS NTAPI lsa_DeleteCredential(LUID *logon_id, ULONG package_id, LSA_STRING *primary_key)
 {
-    FIXME("%p,%#x,%p: stub\n", logon_id, package_id, primary_key);
+    FIXME("%p,%#x,%s: stub\n", logon_id, package_id, debugstr_as(primary_key));
     return STATUS_NOT_IMPLEMENTED;
 }

@@ -859,7 +866,7 @@ NTSTATUS WINAPI LsaLookupAuthenticationPackage(HANDLE lsa_handle,
 {
     ULONG i;

-    TRACE("%p %p %p\n", lsa_handle, package_name, package_id);
+    TRACE("%p %s %p\n", lsa_handle, debugstr_as(package_name), package_id);

for (i = 0; i < loaded_packages_count; i++)
     {