ntdll: Set the DataLength field of the information field in NtQueryValueKey to…

ntdll: Set the DataLength field of the information field in NtQueryValueKey to how much data should be returned, not what was actually returned.

ntdll: Set the DataLength field of the information field in NtQueryValueKey to…
ec3448ff · Rob Shearman · Alexandre Julliard · 05fbee5e · ec3448ff · ec3448ff
Commit ec3448ff authored Dec 29, 2007 by Rob Shearman Committed by Alexandre Julliard Dec 31, 2007
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 5 deletions

reg.c dlls/ntdll/reg.c +1 -1

reg.c dlls/ntdll/tests/reg.c +0 -4

No files found.
--- a/dlls/ntdll/reg.c
+++ b/dlls/ntdll/reg.c
@@ -505,7 +505,7 @@ NTSTATUS WINAPI NtQueryValueKey( HANDLE handle, const UNICODE_STRING *name,
        if (!(ret = wine_server_call( req )))
        {
            copy_key_value_info( info_class, info, length, reply->type,
-                                 0, wine_server_reply_size(reply) );
+                                 name->Length, reply->total );
            *result_len = fixed_size + reply->total;
            if (length < *result_len) ret = STATUS_BUFFER_OVERFLOW;
        }

--- a/dlls/ntdll/tests/reg.c
+++ b/dlls/ntdll/tests/reg.c
@@ -480,9 +480,7 @@ static void test_NtQueryValueKey(void)
    ok(status == STATUS_BUFFER_OVERFLOW, "NtQueryValueKey should have returned STATUS_BUFFER_OVERFLOW instead of 0x%08x\n", status);
    ok(partial_info->TitleIndex == 0, "NtQueryValueKey returned wrong TitleIndex %d\n", partial_info->Type);
    ok(partial_info->Type == REG_DWORD, "NtQueryValueKey returned wrong Type %d\n", partial_info->Type);
-    todo_wine
    ok(partial_info->DataLength == 4, "NtQueryValueKey returned wrong DataLength %d\n", partial_info->DataLength);
-    todo_wine
    ok(len == FIELD_OFFSET(KEY_VALUE_PARTIAL_INFORMATION, Data[partial_info->DataLength]), "NtQueryValueKey returned wrong len %d\n", len);
    HeapFree(GetProcessHeap(), 0, partial_info);

@@ -492,9 +490,7 @@ static void test_NtQueryValueKey(void)
    ok(status == STATUS_BUFFER_OVERFLOW, "NtQueryValueKey should have returned STATUS_BUFFER_OVERFLOW instead of 0x%08x\n", status);
    ok(full_info->TitleIndex == 0, "NtQueryValueKey returned wrong TitleIndex %d\n", full_info->Type);
    ok(full_info->Type == REG_DWORD, "NtQueryValueKey returned wrong Type %d\n", full_info->Type);
-    todo_wine
    ok(full_info->DataLength == 4, "NtQueryValueKey returned wrong DataLength %d\n", full_info->DataLength);
-    todo_wine
    ok(full_info->NameLength == 20, "NtQueryValueKey returned wrong NameLength %d\n", full_info->NameLength);
    todo_wine
    ok(len == FIELD_OFFSET(KEY_VALUE_FULL_INFORMATION, Name[0]) + full_info->DataLength + full_info->NameLength,