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`.