From: Rémi Bernon rbernon@codeweavers.com
--- dlls/ntdll/tests/atom.c | 516 +++++++++++++++++----------------------- 1 file changed, 223 insertions(+), 293 deletions(-)
diff --git a/dlls/ntdll/tests/atom.c b/dlls/ntdll/tests/atom.c index a93a76eef2f..3837dfacd3b 100644 --- a/dlls/ntdll/tests/atom.c +++ b/dlls/ntdll/tests/atom.c @@ -38,26 +38,6 @@ #include "wine/test.h" #include "winternl.h"
-#ifndef __WINE_WINTERNL_H -typedef unsigned short RTL_ATOM, *PRTL_ATOM; -typedef struct atom_table *RTL_ATOM_TABLE, **PRTL_ATOM_TABLE; -#endif - -/* Function pointers for ntdll calls */ -static HMODULE hntdll = 0; -static NTSTATUS (WINAPI *pRtlCreateAtomTable)(ULONG,PRTL_ATOM_TABLE); -static NTSTATUS (WINAPI *pRtlDestroyAtomTable)(RTL_ATOM_TABLE); -static NTSTATUS (WINAPI *pRtlEmptyAtomTable)(RTL_ATOM_TABLE,BOOLEAN); -static NTSTATUS (WINAPI *pRtlAddAtomToAtomTable)(RTL_ATOM_TABLE,PCWSTR,PRTL_ATOM); -static NTSTATUS (WINAPI *pRtlDeleteAtomFromAtomTable)(RTL_ATOM_TABLE,RTL_ATOM); -static NTSTATUS (WINAPI *pRtlLookupAtomInAtomTable)(RTL_ATOM_TABLE,PCWSTR,PRTL_ATOM); -static NTSTATUS (WINAPI *pRtlPinAtomInAtomTable)(RTL_ATOM_TABLE,RTL_ATOM); -static NTSTATUS (WINAPI *pRtlQueryAtomInAtomTable)(RTL_ATOM_TABLE,RTL_ATOM,PULONG,PULONG,PWSTR,PULONG); - -static NTSTATUS (WINAPI* pNtAddAtom)(LPCWSTR,ULONG,RTL_ATOM*); -static NTSTATUS (WINAPI* pNtAddAtomNT4)(LPCWSTR,RTL_ATOM*); -static NTSTATUS (WINAPI* pNtQueryInformationAtom)(RTL_ATOM,DWORD,void*,ULONG,PULONG); - static const WCHAR EmptyAtom[] = {0}; static const WCHAR testAtom1[] = {'H','e','l','l','o',' ','W','o','r','l','d',0}; static const WCHAR testAtom2[] = {'H','e','l','l','o',' ','W','o','r','l','d','2',0}; @@ -70,27 +50,6 @@ static const WCHAR testAtomInt[] = {'#','1','3','2',0}; static const WCHAR testAtomIntInv[] = {'#','2','3','4','z',0}; static const WCHAR testAtomOTT[] = {'#','1','2','3',0};
-static void InitFunctionPtr(void) -{ - hntdll = LoadLibraryA("ntdll.dll"); - ok(hntdll != 0, "Unable to load ntdll.dll\n"); - - if (hntdll) - { - pRtlCreateAtomTable = (void *)GetProcAddress(hntdll, "RtlCreateAtomTable"); - pRtlDestroyAtomTable = (void *)GetProcAddress(hntdll, "RtlDestroyAtomTable"); - pRtlEmptyAtomTable = (void *)GetProcAddress(hntdll, "RtlEmptyAtomTable"); - pRtlAddAtomToAtomTable = (void *)GetProcAddress(hntdll, "RtlAddAtomToAtomTable"); - pRtlDeleteAtomFromAtomTable = (void *)GetProcAddress(hntdll, "RtlDeleteAtomFromAtomTable"); - pRtlLookupAtomInAtomTable = (void *)GetProcAddress(hntdll, "RtlLookupAtomInAtomTable"); - pRtlPinAtomInAtomTable = (void *)GetProcAddress(hntdll, "RtlPinAtomInAtomTable"); - pRtlQueryAtomInAtomTable = (void *)GetProcAddress(hntdll, "RtlQueryAtomInAtomTable"); - - pNtAddAtom = (void *)GetProcAddress(hntdll, "NtAddAtom"); - pNtQueryInformationAtom = (void *)GetProcAddress(hntdll, "NtQueryInformationAtom"); - } -} - static DWORD WINAPI RtlAtomTestThread(LPVOID Table) { RTL_ATOM_TABLE AtomTable = *(PRTL_ATOM_TABLE)Table; @@ -99,17 +58,17 @@ static DWORD WINAPI RtlAtomTestThread(LPVOID Table) ULONG RefCount = 0, PinCount = 0, Len = 0; WCHAR Name[64];
- res = pRtlLookupAtomInAtomTable(AtomTable, testAtom1, &Atom); + res = RtlLookupAtomInAtomTable( AtomTable, testAtom1, &Atom ); ok(!res, "Unable to find atom from another thread, retval: %lx\n", res);
- res = pRtlLookupAtomInAtomTable(AtomTable, testAtom2, &Atom); + res = RtlLookupAtomInAtomTable( AtomTable, testAtom2, &Atom ); ok(!res, "Unable to lookup pinned atom in table, retval: %lx\n", res);
- res = pRtlQueryAtomInAtomTable(AtomTable, Atom, &RefCount, &PinCount, Name, &Len); + res = RtlQueryAtomInAtomTable( AtomTable, Atom, &RefCount, &PinCount, Name, &Len ); ok(res == STATUS_BUFFER_TOO_SMALL, "We got wrong retval: %lx\n", res);
Len = 64; - res = pRtlQueryAtomInAtomTable(AtomTable, Atom, &RefCount, &PinCount, Name, &Len); + res = RtlQueryAtomInAtomTable( AtomTable, Atom, &RefCount, &PinCount, Name, &Len ); ok(!res, "Failed with long enough buffer, retval: %lx\n", res); ok(RefCount == 1, "Refcount was not 1 but %lx\n", RefCount); ok(PinCount == 1, "Pincount was not 1 but %lx\n", PinCount); @@ -117,12 +76,12 @@ static DWORD WINAPI RtlAtomTestThread(LPVOID Table) ok((lstrlenW(testAtom2) * sizeof(WCHAR)) == Len, "Returned wrong length %ld\n", Len);
Len = 64; - res = pRtlQueryAtomInAtomTable(AtomTable, Atom, NULL, NULL, Name, &Len); + res = RtlQueryAtomInAtomTable( AtomTable, Atom, NULL, NULL, Name, &Len ); ok(!res, "RtlQueryAtomInAtomTable with optional args invalid failed, retval: %lx\n", res); ok(!lstrcmpW(Name, testAtom2), "Found Wrong atom!\n"); ok((lstrlenW(testAtom2) * sizeof(WCHAR)) == Len, "Returned wrong length %ld\n", Len);
- res = pRtlPinAtomInAtomTable(AtomTable, Atom); + res = RtlPinAtomInAtomTable( AtomTable, Atom ); ok(!res, "Unable to pin atom in atom table, retval: %lx\n", res);
return 0; @@ -139,157 +98,149 @@ static void test_NtAtom(void)
/* If we pass a non-null string to create atom table, then it thinks that we * have passed it an already allocated atom table */ - res = pRtlCreateAtomTable(0, &AtomTable); + res = RtlCreateAtomTable( 0, &AtomTable ); ok(!res, "RtlCreateAtomTable should succeed with an atom table size of 0\n"); - - if (!res) - { - res = pRtlDestroyAtomTable(AtomTable); - ok(!res, "We could create the atom table, but we couldn't destroy it! retval: %lx\n", res); - } + res = RtlDestroyAtomTable( AtomTable ); + ok( !res, "We could create the atom table, but we couldn't destroy it! retval: %lx\n", res );
AtomTable = NULL; - res = pRtlCreateAtomTable(37, &AtomTable); + res = RtlCreateAtomTable( 37, &AtomTable ); ok(!res, "We're unable to create an atom table with a valid table size retval: %lx\n", res); - if (!res) - { - ok( *(DWORD *)AtomTable == 0x6d6f7441, "wrong signature %lx\n", *(DWORD *)AtomTable ); - - res = pRtlAddAtomToAtomTable(AtomTable, testAtom1, &Atom1); - ok(!res, "We were unable to add a simple atom to the atom table, retval: %lx\n", res); - - res = pRtlLookupAtomInAtomTable(AtomTable, testAtom1Cap, &testAtom); - ok(!res, "We were unable to find capital version of the atom, retval: %lx\n", res); - ok(Atom1 == testAtom, "Found wrong atom in table when querying capital atom\n"); - - res = pRtlLookupAtomInAtomTable(AtomTable, testAtom1Low, &testAtom); - ok(!res, "Unable to find lowercase version of the atom, retval: %lx\n", res); - ok(testAtom == Atom1, "Found wrong atom when querying lowercase atom\n"); - - res = pRtlAddAtomToAtomTable(AtomTable, EmptyAtom, &testEAtom); - ok(res == STATUS_OBJECT_NAME_INVALID, "Got wrong retval, retval: %lx\n", res); - - res = pRtlLookupAtomInAtomTable(AtomTable, testAtom1, &testAtom); - ok(!res, "Failed to find totally legitimate atom, retval: %lx\n", res); - ok(testAtom == Atom1, "Found wrong atom!\n"); - - res = pRtlAddAtomToAtomTable(AtomTable, testAtom2, &Atom2); - ok(!res, "Unable to add other legitimate atom to table, retval: %lx\n", res); - - res = pRtlPinAtomInAtomTable(AtomTable, Atom2); - ok(!res, "Unable to pin atom in atom table, retval: %lx\n", res); - - testThread = CreateThread(NULL, 0, RtlAtomTestThread, &AtomTable, 0, NULL); - WaitForSingleObject(testThread, INFINITE); - CloseHandle(testThread); - - Len = 64; - res = pRtlQueryAtomInAtomTable(AtomTable, Atom2, &RefCount, &PinCount, Name, &Len); - ok(!res, "Unable to query atom in atom table, retval: %lx\n", res); - ok(RefCount == 1, "RefCount is not 1 but %lx\n", RefCount); - ok(PinCount == 1, "PinCount is not 1 but %lx\n", PinCount); - ok(!lstrcmpW(Name, testAtom2), "We found wrong atom\n"); - ok((lstrlenW(testAtom2) * sizeof(WCHAR)) == Len, "Returned wrong length %ld\n", Len); - - res = pRtlEmptyAtomTable(AtomTable, FALSE); - ok(!res, "Unable to empty atom table, retval %lx\n", res); - - Len = 64; - res = pRtlQueryAtomInAtomTable(AtomTable, Atom2, &RefCount, &PinCount, Name, &Len); - ok(!res, "It seems RtlEmptyAtomTable deleted our pinned atom eaven though we asked it not to, retval: %lx\n", res); - ok(RefCount == 1, "RefCount is not 1 but %lx\n", RefCount); - ok(PinCount == 1, "PinCount is not 1 but %lx\n", PinCount); - ok(!lstrcmpW(Name, testAtom2), "We found wrong atom\n"); - ok((lstrlenW(testAtom2) * sizeof(WCHAR)) == Len, "Returned wrong length %ld\n", Len); - - Len = 8; - Name[0] = Name[1] = Name[2] = Name[3] = Name[4] = 0x1337; - res = pRtlQueryAtomInAtomTable(AtomTable, Atom2, NULL, NULL, Name, &Len); - ok(!res, "query atom %lx\n", res); - ok(Len == 6, "wrong length %lu\n", Len); - ok(!memcmp(Name, testAtom2, Len), "wrong atom string\n"); - ok(!Name[3], "wrong string termination\n"); - ok(Name[4] == 0x1337, "buffer overwrite\n"); - - Len = lstrlenW(testAtom2) * sizeof(WCHAR); - memset(Name, '.', sizeof(Name)); - res = pRtlQueryAtomInAtomTable( AtomTable, Atom2, NULL, NULL, Name, &Len ); - ok(!res, "query atom %lx\n", res); - ok(Len == (lstrlenW(testAtom2) - 1) * sizeof(WCHAR), "wrong length %lu\n", Len); - ok(!memcmp(testAtom2, Name, (lstrlenW(testAtom2) - 1) * sizeof(WCHAR)), "wrong atom name\n"); - ok(Name[lstrlenW(testAtom2) - 1] == '\0', "wrong char\n"); - ok(Name[lstrlenW(testAtom2)] == ('.' << 8) + '.', "wrong char\n"); - - res = pRtlLookupAtomInAtomTable(AtomTable, testAtom2, &testAtom); - ok(!res, "We can't find our pinned atom!! retval: %lx\n", res); - ok(testAtom == Atom2, "We found wrong atom!!!\n"); - - res = pRtlLookupAtomInAtomTable(AtomTable, testAtom1, &testAtom); - ok(res == STATUS_OBJECT_NAME_NOT_FOUND, "We found the atom in our table eaven though we asked RtlEmptyAtomTable to remove it, retval: %lx\n", res); - - res = pRtlAddAtomToAtomTable(AtomTable, testAtom3, &Atom3); - ok(!res, "Unable to add atom to table, retval: %lx\n", res); - - res = pRtlEmptyAtomTable(AtomTable, TRUE); - ok(!res, "Unable to empty atom table, retval: %lx\n", res); - - res = pRtlLookupAtomInAtomTable(AtomTable, testAtom2, &testAtom); - ok(res == STATUS_OBJECT_NAME_NOT_FOUND, "The pinned atom should be removed, retval: %lx\n", res); - - res = pRtlLookupAtomInAtomTable(AtomTable, testAtom3, &testAtom); - ok(res == STATUS_OBJECT_NAME_NOT_FOUND, "Non pinned atom should also be removed, retval: %lx\n", res); - - res = pRtlDestroyAtomTable(AtomTable); - ok(!res, "Can't destroy atom table, retval: %lx\n", res); - } + ok( *(DWORD *)AtomTable == 0x6d6f7441, "wrong signature %lx\n", *(DWORD *)AtomTable ); + + res = RtlAddAtomToAtomTable( AtomTable, testAtom1, &Atom1 ); + ok( !res, "We were unable to add a simple atom to the atom table, retval: %lx\n", res ); + + res = RtlLookupAtomInAtomTable( AtomTable, testAtom1Cap, &testAtom ); + ok( !res, "We were unable to find capital version of the atom, retval: %lx\n", res ); + ok( Atom1 == testAtom, "Found wrong atom in table when querying capital atom\n" ); + + res = RtlLookupAtomInAtomTable( AtomTable, testAtom1Low, &testAtom ); + ok( !res, "Unable to find lowercase version of the atom, retval: %lx\n", res ); + ok( testAtom == Atom1, "Found wrong atom when querying lowercase atom\n" ); + + res = RtlAddAtomToAtomTable( AtomTable, EmptyAtom, &testEAtom ); + ok( res == STATUS_OBJECT_NAME_INVALID, "Got wrong retval, retval: %lx\n", res ); + + res = RtlLookupAtomInAtomTable( AtomTable, testAtom1, &testAtom ); + ok( !res, "Failed to find totally legitimate atom, retval: %lx\n", res ); + ok( testAtom == Atom1, "Found wrong atom!\n" ); + + res = RtlAddAtomToAtomTable( AtomTable, testAtom2, &Atom2 ); + ok( !res, "Unable to add other legitimate atom to table, retval: %lx\n", res ); + + res = RtlPinAtomInAtomTable( AtomTable, Atom2 ); + ok( !res, "Unable to pin atom in atom table, retval: %lx\n", res ); + + testThread = CreateThread( NULL, 0, RtlAtomTestThread, &AtomTable, 0, NULL ); + WaitForSingleObject( testThread, INFINITE ); + CloseHandle( testThread ); + + Len = 64; + res = RtlQueryAtomInAtomTable( AtomTable, Atom2, &RefCount, &PinCount, Name, &Len ); + ok( !res, "Unable to query atom in atom table, retval: %lx\n", res ); + ok( RefCount == 1, "RefCount is not 1 but %lx\n", RefCount ); + ok( PinCount == 1, "PinCount is not 1 but %lx\n", PinCount ); + ok( !lstrcmpW( Name, testAtom2 ), "We found wrong atom\n" ); + ok( (lstrlenW( testAtom2 ) * sizeof(WCHAR)) == Len, "Returned wrong length %ld\n", Len ); + + res = RtlEmptyAtomTable( AtomTable, FALSE ); + ok( !res, "Unable to empty atom table, retval %lx\n", res ); + + Len = 64; + res = RtlQueryAtomInAtomTable( AtomTable, Atom2, &RefCount, &PinCount, Name, &Len ); + ok( !res, "It seems RtlEmptyAtomTable deleted our pinned atom eaven though we asked it not to, retval: %lx\n", res ); + ok( RefCount == 1, "RefCount is not 1 but %lx\n", RefCount ); + ok( PinCount == 1, "PinCount is not 1 but %lx\n", PinCount ); + ok( !lstrcmpW( Name, testAtom2 ), "We found wrong atom\n" ); + ok( (lstrlenW( testAtom2 ) * sizeof(WCHAR)) == Len, "Returned wrong length %ld\n", Len ); + + Len = 8; + Name[0] = Name[1] = Name[2] = Name[3] = Name[4] = 0x1337; + res = RtlQueryAtomInAtomTable( AtomTable, Atom2, NULL, NULL, Name, &Len ); + ok( !res, "query atom %lx\n", res ); + ok( Len == 6, "wrong length %lu\n", Len ); + ok( !memcmp( Name, testAtom2, Len ), "wrong atom string\n" ); + ok( !Name[3], "wrong string termination\n" ); + ok( Name[4] == 0x1337, "buffer overwrite\n" ); + + Len = lstrlenW( testAtom2 ) * sizeof(WCHAR); + memset( Name, '.', sizeof(Name) ); + res = RtlQueryAtomInAtomTable( AtomTable, Atom2, NULL, NULL, Name, &Len ); + ok( !res, "query atom %lx\n", res ); + ok( Len == (lstrlenW( testAtom2 ) - 1) * sizeof(WCHAR), "wrong length %lu\n", Len ); + ok( !memcmp( testAtom2, Name, (lstrlenW( testAtom2 ) - 1) * sizeof(WCHAR) ), + "wrong atom name\n" ); + ok( Name[lstrlenW( testAtom2 ) - 1] == '\0', "wrong char\n" ); + ok( Name[lstrlenW( testAtom2 )] == ('.' << 8) + '.', "wrong char\n" ); + + res = RtlLookupAtomInAtomTable( AtomTable, testAtom2, &testAtom ); + ok( !res, "We can't find our pinned atom!! retval: %lx\n", res ); + ok( testAtom == Atom2, "We found wrong atom!!!\n" ); + + res = RtlLookupAtomInAtomTable( AtomTable, testAtom1, &testAtom ); + ok( res == STATUS_OBJECT_NAME_NOT_FOUND, "We found the atom in our table eaven though we asked RtlEmptyAtomTable to remove it, retval: %lx\n", + res ); + + res = RtlAddAtomToAtomTable( AtomTable, testAtom3, &Atom3 ); + ok( !res, "Unable to add atom to table, retval: %lx\n", res ); + + res = RtlEmptyAtomTable( AtomTable, TRUE ); + ok( !res, "Unable to empty atom table, retval: %lx\n", res ); + + res = RtlLookupAtomInAtomTable( AtomTable, testAtom2, &testAtom ); + ok( res == STATUS_OBJECT_NAME_NOT_FOUND, "The pinned atom should be removed, retval: %lx\n", res ); + + res = RtlLookupAtomInAtomTable( AtomTable, testAtom3, &testAtom ); + ok( res == STATUS_OBJECT_NAME_NOT_FOUND, "Non pinned atom should also be removed, retval: %lx\n", res ); + + res = RtlDestroyAtomTable( AtomTable ); + ok( !res, "Can't destroy atom table, retval: %lx\n", res );
AtomTable = NULL; - res = pRtlCreateAtomTable(37, &AtomTable); + res = RtlCreateAtomTable( 37, &AtomTable ); ok(!res, "Unable to create atom table, retval: %lx\n", res);
- if (!res) - { - res = pRtlLookupAtomInAtomTable(AtomTable, testAtom1, &testAtom); - ok(res == STATUS_OBJECT_NAME_NOT_FOUND, "Didn't get expected retval with querying an empty atom table, retval: %lx\n", res); + res = RtlLookupAtomInAtomTable( AtomTable, testAtom1, &testAtom ); + ok( res == STATUS_OBJECT_NAME_NOT_FOUND, + "Didn't get expected retval with querying an empty atom table, retval: %lx\n", res );
- res = pRtlAddAtomToAtomTable(AtomTable, testAtom1, &Atom1); - ok(!res, "Unable to add atom to atom table, retval %lx\n", res); + res = RtlAddAtomToAtomTable( AtomTable, testAtom1, &Atom1 ); + ok( !res, "Unable to add atom to atom table, retval %lx\n", res );
- res = pRtlLookupAtomInAtomTable(AtomTable, testAtom1, &testAtom); - ok(!res, "Can't find previously added atom in table, retval: %lx\n", res); - ok(testAtom == Atom1, "Found wrong atom! retval: %lx\n", res); + res = RtlLookupAtomInAtomTable( AtomTable, testAtom1, &testAtom ); + ok( !res, "Can't find previously added atom in table, retval: %lx\n", res ); + ok( testAtom == Atom1, "Found wrong atom! retval: %lx\n", res );
- res = pRtlDeleteAtomFromAtomTable(AtomTable, Atom1); - ok(!res, "Unable to delete atom from table, retval: %lx\n", res); + res = RtlDeleteAtomFromAtomTable( AtomTable, Atom1 ); + ok( !res, "Unable to delete atom from table, retval: %lx\n", res );
- res = pRtlLookupAtomInAtomTable(AtomTable, testAtom1, &testAtom); - ok(res == STATUS_OBJECT_NAME_NOT_FOUND, "Able to find previously deleted atom in table, retval: %lx\n", res); + res = RtlLookupAtomInAtomTable( AtomTable, testAtom1, &testAtom ); + ok( res == STATUS_OBJECT_NAME_NOT_FOUND, "Able to find previously deleted atom in table, retval: %lx\n", res );
- res = pRtlAddAtomToAtomTable(AtomTable, testAtom1, &Atom1); - ok(!res, "Unable to add atom to atom table, retval: %lx\n", res); + res = RtlAddAtomToAtomTable( AtomTable, testAtom1, &Atom1 ); + ok( !res, "Unable to add atom to atom table, retval: %lx\n", res );
- Len = 0; - res = pRtlQueryAtomInAtomTable(AtomTable, Atom1, NULL, NULL, Name, &Len); - ok(res == STATUS_BUFFER_TOO_SMALL, "Got wrong retval, retval: %lx\n", res); - ok((lstrlenW(testAtom1) * sizeof(WCHAR)) == Len || broken(!Len) /* nt4 */, "Got wrong length %lx\n", Len); - if (!Len) pNtAddAtomNT4 = (void *)pNtAddAtom; + Len = 0; + res = RtlQueryAtomInAtomTable( AtomTable, Atom1, NULL, NULL, Name, &Len ); + ok( res == STATUS_BUFFER_TOO_SMALL, "Got wrong retval, retval: %lx\n", res ); + ok( lstrlenW( testAtom1 ) * sizeof(WCHAR) == Len, "Got wrong length %lx\n", Len );
- res = pRtlPinAtomInAtomTable(AtomTable, Atom1); - ok(!res, "Unable to pin atom in atom table, retval: %lx\n", res); + res = RtlPinAtomInAtomTable( AtomTable, Atom1 ); + ok( !res, "Unable to pin atom in atom table, retval: %lx\n", res );
- res = pRtlLookupAtomInAtomTable(AtomTable, testAtom1, &testAtom); - ok(!res, "Unable to find atom in atom table, retval: %lx\n", res); - ok(testAtom == Atom1, "Wrong atom found\n"); + res = RtlLookupAtomInAtomTable( AtomTable, testAtom1, &testAtom ); + ok( !res, "Unable to find atom in atom table, retval: %lx\n", res ); + ok( testAtom == Atom1, "Wrong atom found\n" );
- res = pRtlDeleteAtomFromAtomTable(AtomTable, Atom1); - ok(res == STATUS_WAS_LOCKED, "Unable to delete atom from table, retval: %lx\n", res); + res = RtlDeleteAtomFromAtomTable( AtomTable, Atom1 ); + ok( res == STATUS_WAS_LOCKED, "Unable to delete atom from table, retval: %lx\n", res );
- res = pRtlLookupAtomInAtomTable(AtomTable, testAtom1, &testAtom); - ok(!res, "Able to find deleted atom in table\n"); + res = RtlLookupAtomInAtomTable( AtomTable, testAtom1, &testAtom ); + ok( !res, "Able to find deleted atom in table\n" );
- res = pRtlDestroyAtomTable(AtomTable); - ok(!res, "Unable to destroy atom table\n"); - } + res = RtlDestroyAtomTable( AtomTable ); + ok( !res, "Unable to destroy atom table\n" ); }
/* Test Adding integer atoms to atom table */ @@ -304,84 +255,76 @@ static void test_NtIntAtom(void) ULONG Len;
AtomTable = NULL; - res = pRtlCreateAtomTable(37, &AtomTable); + res = RtlCreateAtomTable( 37, &AtomTable ); ok(!res, "Unable to create atom table, %lx\n", res);
- if (!res) + /* According to the kernel32 functions, integer atoms are only allowed from + * 0x0001 to 0xbfff and not 0xc000 to 0xffff, which is correct */ + res = RtlAddAtomToAtomTable( AtomTable, NULL, &testAtom ); + ok( res == STATUS_INVALID_PARAMETER, "Didn't get expected result from adding 0 int atom, retval: %lx\n", res ); + for (i = 1; i <= 0xbfff; i++) { - /* According to the kernel32 functions, integer atoms are only allowed from - * 0x0001 to 0xbfff and not 0xc000 to 0xffff, which is correct */ - res = pRtlAddAtomToAtomTable(AtomTable, NULL, &testAtom); - ok(res == STATUS_INVALID_PARAMETER, "Didn't get expected result from adding 0 int atom, retval: %lx\n", res); - for (i = 1; i <= 0xbfff; i++) - { - res = pRtlAddAtomToAtomTable(AtomTable, (LPWSTR)i, &testAtom); - ok(!res, "Unable to add valid integer atom %Ii, retval: %lx\n", i, res); - } - - for (i = 1; i <= 0xbfff; i++) - { - res = pRtlLookupAtomInAtomTable(AtomTable, (LPWSTR)i, &testAtom); - ok(!res, "Unable to find int atom %Ii, retval: %lx\n", i, res); - if (!res) - { - res = pRtlPinAtomInAtomTable(AtomTable, testAtom); - ok(!res, "Unable to pin int atom %Ii, retval: %lx\n", i, res); - } - } - - for (i = 0xc000; i <= 0xffff; i++) - { - res = pRtlAddAtomToAtomTable(AtomTable, (LPWSTR)i, &testAtom); - ok(res, "Able to illeageal integer atom %Ii, retval: %lx\n", i, res); - } - - res = pRtlDestroyAtomTable(AtomTable); - ok(!res, "Unable to destroy atom table, retval: %lx\n", res); + res = RtlAddAtomToAtomTable( AtomTable, (LPWSTR)i, &testAtom ); + ok( !res, "Unable to add valid integer atom %Ii, retval: %lx\n", i, res ); }
- AtomTable = NULL; - res = pRtlCreateAtomTable(37, &AtomTable); - ok(!res, "Unable to create atom table, %lx\n", res); - if (!res) + for (i = 1; i <= 0xbfff; i++) { - res = pRtlLookupAtomInAtomTable(AtomTable, (PWSTR)123, &testAtom); - ok(!res, "Unable to query atom in atom table, retval: %lx\n", res); + res = RtlLookupAtomInAtomTable( AtomTable, (LPWSTR)i, &testAtom ); + ok( !res, "Unable to find int atom %Ii, retval: %lx\n", i, res ); + res = RtlPinAtomInAtomTable( AtomTable, testAtom ); + ok( !res, "Unable to pin int atom %Ii, retval: %lx\n", i, res ); + }
- res = pRtlAddAtomToAtomTable(AtomTable, testAtomInt, &testAtom); - ok(!res, "Unable to add int atom to table, retval: %lx\n", res); + for (i = 0xc000; i <= 0xffff; i++) + { + res = RtlAddAtomToAtomTable( AtomTable, (LPWSTR)i, &testAtom ); + ok( res, "Able to illeageal integer atom %Ii, retval: %lx\n", i, res ); + }
- res = pRtlAddAtomToAtomTable(AtomTable, testAtomIntInv, &testAtom); - ok(!res, "Unable to add int atom to table, retval: %lx\n", res); + res = RtlDestroyAtomTable( AtomTable ); + ok( !res, "Unable to destroy atom table, retval: %lx\n", res );
- res = pRtlAddAtomToAtomTable(AtomTable, (PWSTR)123, &testAtom); - ok(!res, "Unable to add int atom to table, retval: %lx\n", res); + AtomTable = NULL; + res = RtlCreateAtomTable( 37, &AtomTable ); + ok( !res, "Unable to create atom table, %lx\n", res );
- res = pRtlAddAtomToAtomTable(AtomTable, (PWSTR)123, &testAtom); - ok(!res, "Unable to re-add int atom to table, retval: %lx\n", res); + res = RtlLookupAtomInAtomTable( AtomTable, (PWSTR)123, &testAtom ); + ok( !res, "Unable to query atom in atom table, retval: %lx\n", res );
- Len = 64; - res = pRtlQueryAtomInAtomTable(AtomTable, testAtom, &RefCount, &PinCount, Name, &Len); - ok(!res, "Unable to query atom in atom table, retval: %lx\n", res); - ok(PinCount == 1, "Expected pincount 1 but got %lx\n", PinCount); - ok(RefCount == 1, "Expected refcount 1 but got %lx\n", RefCount); - ok(!lstrcmpW(testAtomOTT, Name), "Got wrong atom name\n"); - ok((lstrlenW(testAtomOTT) * sizeof(WCHAR)) == Len, "Got wrong len %ld\n", Len); + res = RtlAddAtomToAtomTable( AtomTable, testAtomInt, &testAtom ); + ok( !res, "Unable to add int atom to table, retval: %lx\n", res );
- res = pRtlPinAtomInAtomTable(AtomTable, testAtom); - ok(!res, "Unable to pin int atom, retval: %lx\n", res); + res = RtlAddAtomToAtomTable( AtomTable, testAtomIntInv, &testAtom ); + ok( !res, "Unable to add int atom to table, retval: %lx\n", res );
- res = pRtlPinAtomInAtomTable(AtomTable, testAtom); - ok(!res, "Unable to pin int atom, retval: %lx\n", res); + res = RtlAddAtomToAtomTable( AtomTable, (PWSTR)123, &testAtom ); + ok( !res, "Unable to add int atom to table, retval: %lx\n", res );
- res = pRtlQueryAtomInAtomTable(AtomTable, testAtom, &RefCount, &PinCount, NULL, NULL); - ok(!res, "Unable to query atom in atom table, retval: %lx\n", res); - ok(PinCount == 1, "Expected pincount 1 but got %lx\n", PinCount); - ok(RefCount == 1, "Expected refcount 1 but got %lx\n", RefCount); + res = RtlAddAtomToAtomTable( AtomTable, (PWSTR)123, &testAtom ); + ok( !res, "Unable to re-add int atom to table, retval: %lx\n", res );
- res = pRtlDestroyAtomTable(AtomTable); - ok(!res, "Unable to destroy atom table, retval: %lx\n", res); - } + Len = 64; + res = RtlQueryAtomInAtomTable( AtomTable, testAtom, &RefCount, &PinCount, Name, &Len ); + ok( !res, "Unable to query atom in atom table, retval: %lx\n", res ); + ok( PinCount == 1, "Expected pincount 1 but got %lx\n", PinCount ); + ok( RefCount == 1, "Expected refcount 1 but got %lx\n", RefCount ); + ok( !lstrcmpW( testAtomOTT, Name ), "Got wrong atom name\n" ); + ok( (lstrlenW( testAtomOTT ) * sizeof(WCHAR)) == Len, "Got wrong len %ld\n", Len ); + + res = RtlPinAtomInAtomTable( AtomTable, testAtom ); + ok( !res, "Unable to pin int atom, retval: %lx\n", res ); + + res = RtlPinAtomInAtomTable( AtomTable, testAtom ); + ok( !res, "Unable to pin int atom, retval: %lx\n", res ); + + res = RtlQueryAtomInAtomTable( AtomTable, testAtom, &RefCount, &PinCount, NULL, NULL ); + ok( !res, "Unable to query atom in atom table, retval: %lx\n", res ); + ok( PinCount == 1, "Expected pincount 1 but got %lx\n", PinCount ); + ok( RefCount == 1, "Expected refcount 1 but got %lx\n", RefCount ); + + res = RtlDestroyAtomTable( AtomTable ); + ok( !res, "Unable to destroy atom table, retval: %lx\n", res ); }
/* Tests to see how the pincount and refcount actually works */ @@ -393,42 +336,39 @@ static void test_NtRefPinAtom(void) NTSTATUS res;
AtomTable = NULL; - res = pRtlCreateAtomTable(37, &AtomTable); + res = RtlCreateAtomTable( 37, &AtomTable ); ok(!res, "Unable to create atom table, %lx\n", res);
- if (!res) - { - res = pRtlAddAtomToAtomTable(AtomTable, testAtom1, &Atom); - ok(!res, "Unable to add our atom to the atom table, retval: %lx\n", res); + res = RtlAddAtomToAtomTable( AtomTable, testAtom1, &Atom ); + ok( !res, "Unable to add our atom to the atom table, retval: %lx\n", res );
- res = pRtlAddAtomToAtomTable(AtomTable, testAtom1, &Atom); - ok(!res, "Unable to add our atom to the atom table, retval: %lx\n", res); + res = RtlAddAtomToAtomTable( AtomTable, testAtom1, &Atom ); + ok( !res, "Unable to add our atom to the atom table, retval: %lx\n", res );
- res = pRtlAddAtomToAtomTable(AtomTable, testAtom1, &Atom); - ok(!res, "Unable to add our atom to the atom table, retval: %lx\n", res); + res = RtlAddAtomToAtomTable( AtomTable, testAtom1, &Atom ); + ok( !res, "Unable to add our atom to the atom table, retval: %lx\n", res );
- res = pRtlQueryAtomInAtomTable(AtomTable, Atom, &RefCount, &PinCount, NULL, NULL); - ok(!res, "Unable to query atom in atom table, retval: %lx\n", res); - ok(PinCount == 0, "Expected pincount 0 but got %lx\n", PinCount); - ok(RefCount == 3, "Expected refcount 3 but got %lx\n", RefCount); + res = RtlQueryAtomInAtomTable( AtomTable, Atom, &RefCount, &PinCount, NULL, NULL ); + ok( !res, "Unable to query atom in atom table, retval: %lx\n", res ); + ok( PinCount == 0, "Expected pincount 0 but got %lx\n", PinCount ); + ok( RefCount == 3, "Expected refcount 3 but got %lx\n", RefCount );
- res = pRtlPinAtomInAtomTable(AtomTable, Atom); - ok(!res, "Unable to pin atom in atom table, retval: %lx\n", res); + res = RtlPinAtomInAtomTable( AtomTable, Atom ); + ok( !res, "Unable to pin atom in atom table, retval: %lx\n", res );
- res = pRtlPinAtomInAtomTable(AtomTable, Atom); - ok(!res, "Unable to pin atom in atom table, retval: %lx\n", res); + res = RtlPinAtomInAtomTable( AtomTable, Atom ); + ok( !res, "Unable to pin atom in atom table, retval: %lx\n", res );
- res = pRtlPinAtomInAtomTable(AtomTable, Atom); - ok(!res, "Unable to pin atom in atom table, retval: %lx\n", res); + res = RtlPinAtomInAtomTable( AtomTable, Atom ); + ok( !res, "Unable to pin atom in atom table, retval: %lx\n", res );
- res = pRtlQueryAtomInAtomTable(AtomTable, Atom, &RefCount, &PinCount, NULL, NULL); - ok(!res, "Unable to query atom in atom table, retval: %lx\n", res); - ok(PinCount == 1, "Expected pincount 1 but got %lx\n", PinCount); - ok(RefCount == 3, "Expected refcount 3 but got %lx\n", RefCount); + res = RtlQueryAtomInAtomTable( AtomTable, Atom, &RefCount, &PinCount, NULL, NULL ); + ok( !res, "Unable to query atom in atom table, retval: %lx\n", res ); + ok( PinCount == 1, "Expected pincount 1 but got %lx\n", PinCount ); + ok( RefCount == 3, "Expected refcount 3 but got %lx\n", RefCount );
- res = pRtlDestroyAtomTable(AtomTable); - ok(!res, "Unable to destroy atom table, retval: %lx\n", res); - } + res = RtlDestroyAtomTable( AtomTable ); + ok( !res, "Unable to destroy atom table, retval: %lx\n", res ); }
static void test_Global(void) @@ -439,15 +379,11 @@ static void test_Global(void) ATOM_BASIC_INFORMATION* abi = (ATOM_BASIC_INFORMATION*)ptr; ULONG ptr_size = sizeof(ptr);
- if (pNtAddAtomNT4) - res = pNtAddAtomNT4(testAtom1, &atom); - else - res = pNtAddAtom(testAtom1, lstrlenW(testAtom1) * sizeof(WCHAR), &atom); - + res = NtAddAtom( testAtom1, lstrlenW( testAtom1 ) * sizeof(WCHAR), &atom ); ok(!res, "Added atom (%lx)\n", res);
memset( ptr, 0xcc, sizeof(ptr) ); - res = pNtQueryInformationAtom( atom, AtomBasicInformation, (void*)ptr, ptr_size, NULL ); + res = NtQueryInformationAtom( atom, AtomBasicInformation, (void *)ptr, ptr_size, NULL ); ok(!res, "atom lookup\n"); ok(!lstrcmpW(abi->Name, testAtom1), "ok strings\n"); ok(abi->NameLength == lstrlenW(testAtom1) * sizeof(WCHAR), "wrong string length\n"); @@ -455,14 +391,14 @@ static void test_Global(void) ok(abi->Name[lstrlenW(testAtom1) + 1] == 0xcccc, "buffer overwrite %x\n", abi->Name[lstrlenW(testAtom1) + 1]);
ptr_size = sizeof(ATOM_BASIC_INFORMATION); - res = pNtQueryInformationAtom( atom, AtomBasicInformation, (void*)ptr, ptr_size, NULL ); + res = NtQueryInformationAtom( atom, AtomBasicInformation, (void *)ptr, ptr_size, NULL ); ok(res == STATUS_BUFFER_TOO_SMALL, "wrong return status (%lx)\n", res); ok(abi->NameLength == lstrlenW(testAtom1) * sizeof(WCHAR) || broken(abi->NameLength == sizeof(WCHAR)), /* nt4 */ "string length %u\n",abi->NameLength);
memset( ptr, 0xcc, sizeof(ptr) ); ptr_size = sizeof(ATOM_BASIC_INFORMATION) + lstrlenW(testAtom1) * sizeof(WCHAR); - res = pNtQueryInformationAtom( atom, AtomBasicInformation, (void*)ptr, ptr_size, NULL ); + res = NtQueryInformationAtom( atom, AtomBasicInformation, (void *)ptr, ptr_size, NULL ); ok(!res, "atom lookup %lx\n", res); ok(!lstrcmpW(abi->Name, testAtom1), "strings don't match\n"); ok(abi->NameLength == lstrlenW(testAtom1) * sizeof(WCHAR), "wrong string length\n"); @@ -471,28 +407,22 @@ static void test_Global(void)
memset( ptr, 0xcc, sizeof(ptr) ); ptr_size = sizeof(ATOM_BASIC_INFORMATION) + 4 * sizeof(WCHAR); - res = pNtQueryInformationAtom( atom, AtomBasicInformation, (void*)ptr, ptr_size, NULL ); + res = NtQueryInformationAtom( atom, AtomBasicInformation, (void *)ptr, ptr_size, NULL ); ok(!res, "couldn't find atom\n"); ok(abi->NameLength == 8, "wrong string length %u\n", abi->NameLength); ok(!memcmp(abi->Name, testAtom1, 8), "strings don't match\n"); + + NtDeleteAtom( atom ); }
START_TEST(atom) { - InitFunctionPtr(); - if (pRtlCreateAtomTable) - { - /* Global atom table seems to be available to GUI apps only in - Win7, so let's turn this app into a GUI app */ - GetDesktopWindow(); - - test_NtAtom(); - test_NtIntAtom(); - test_NtRefPinAtom(); - test_Global(); - } - else - win_skip("Needed atom functions are not available\n"); - - FreeLibrary(hntdll); + /* Global atom table seems to be available to GUI apps only in + Win7, so let's turn this app into a GUI app */ + GetDesktopWindow(); + + test_NtAtom(); + test_NtIntAtom(); + test_NtRefPinAtom(); + test_Global(); }