Re: [PATCH v4 0/1] MR766: advapi32/test: Add tests for RegLoadAppKey.

[Huw Davies (＠huw)]

Huw Davies (@huw) commented about dlls/advapi32/tests/registry.c:
> +                      FILE_ATTRIBUTE_NORMAL, NULL);
> +    size1 = GetFileSize(hivefile, NULL);
> +    CloseHandle(hivefile);
> +
> +    ret = RegLoadAppKeyA(hivefilepath, &key1, KEY_READ | KEY_WRITE, 0, 0);
> +    todo_wine ok(ret == ERROR_SUCCESS, "expected ERROR_SUCCESS, got %ld\n", ret);
> +    todo_wine ok(key1 != NULL, "got a null key\n");
> +
> +    ret = RegCreateKeyA(key1, "testkey", &childkey);
> +    RegCloseKey(key1);
> +
> +    hivefile = CreateFileA(hivefilepath, GENERIC_READ, FILE_SHARE_READ | FILE_SHARE_WRITE, NULL, OPEN_EXISTING,
> +                      FILE_ATTRIBUTE_NORMAL, NULL);
> +    size2 = GetFileSize(hivefile, NULL);
> +    todo_wine ok(size2 > size1, "Expected file to increase size. %ld <= %ld\n", size2, size1);
> +    CloseHandle(hivefile);
> You're leaking `childkey`.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/766#note_7779