Paul Vriens : advapi32/tests: Add some ReadEventLogA tests.

Alexandre Julliard julliard at winehq.org
Tue Nov 3 15:37:19 CST 2009


Module: wine
Branch: master
Commit: 531363f6e68c30227c6fcb1e18f39998df05a621
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=531363f6e68c30227c6fcb1e18f39998df05a621

Author: Paul Vriens <Paul.Vriens.Wine at gmail.com>
Date:   Sun Nov  1 16:02:40 2009 +0100

advapi32/tests: Add some ReadEventLogA tests.

---

 dlls/advapi32/tests/eventlog.c |  149 ++++++++++++++++++++++++++++++++++++++++
 include/winnt.h                |   24 +++++++
 2 files changed, 173 insertions(+), 0 deletions(-)

diff --git a/dlls/advapi32/tests/eventlog.c b/dlls/advapi32/tests/eventlog.c
index 9794f30..f6377a6 100644
--- a/dlls/advapi32/tests/eventlog.c
+++ b/dlls/advapi32/tests/eventlog.c
@@ -256,6 +256,154 @@ static void test_backup(void)
     DeleteFileA(backup);
 }
 
+static void test_read(void)
+{
+    HANDLE handle;
+    BOOL ret;
+    DWORD count, toread, read, needed;
+    void *buf;
+
+    SetLastError(0xdeadbeef);
+    ret = ReadEventLogA(NULL, 0, 0, NULL, 0, NULL, NULL);
+    ok(!ret, "Expected failure\n");
+    todo_wine
+    ok(GetLastError() == ERROR_INVALID_PARAMETER, "Expected ERROR_INVALID_PARAMETER, got %d\n", GetLastError());
+
+    read = 0xdeadbeef;
+    SetLastError(0xdeadbeef);
+    ret = ReadEventLogA(NULL, 0, 0, NULL, 0, &read, NULL);
+    ok(!ret, "Expected failure\n");
+    ok(read == 0xdeadbeef, "Expected 'read' parameter to remain unchanged\n");
+    todo_wine
+    ok(GetLastError() == ERROR_INVALID_PARAMETER, "Expected ERROR_INVALID_PARAMETER, got %d\n", GetLastError());
+
+    needed = 0xdeadbeef;
+    SetLastError(0xdeadbeef);
+    ret = ReadEventLogA(NULL, 0, 0, NULL, 0, NULL, &needed);
+    ok(!ret, "Expected failure\n");
+    ok(needed == 0xdeadbeef, "Expected 'needed' parameter to remain unchanged\n");
+    todo_wine
+    ok(GetLastError() == ERROR_INVALID_PARAMETER, "Expected ERROR_INVALID_PARAMETER, got %d\n", GetLastError());
+
+    /* 'read' and 'needed' are only filled when the needed buffer size is passed back or when the call succeeds */
+    SetLastError(0xdeadbeef);
+    ret = ReadEventLogA(NULL, 0, 0, NULL, 0, &read, &needed);
+    ok(!ret, "Expected failure\n");
+    todo_wine
+    ok(GetLastError() == ERROR_INVALID_PARAMETER, "Expected ERROR_INVALID_PARAMETER, got %d\n", GetLastError());
+
+    SetLastError(0xdeadbeef);
+    ret = ReadEventLogA(NULL, EVENTLOG_SEQUENTIAL_READ | EVENTLOG_FORWARDS_READ, 0, NULL, 0, NULL, NULL);
+    ok(!ret, "Expected failure\n");
+    todo_wine
+    ok(GetLastError() == ERROR_INVALID_PARAMETER, "Expected ERROR_INVALID_PARAMETER, got %d\n", GetLastError());
+
+    SetLastError(0xdeadbeef);
+    ret = ReadEventLogA(NULL, EVENTLOG_SEQUENTIAL_READ | EVENTLOG_FORWARDS_READ, 0, NULL, 0, &read, &needed);
+    ok(!ret, "Expected failure\n");
+    todo_wine
+    ok(GetLastError() == ERROR_INVALID_PARAMETER, "Expected ERROR_INVALID_PARAMETER, got %d\n", GetLastError());
+
+    buf = NULL;
+    SetLastError(0xdeadbeef);
+    ret = ReadEventLogA(NULL, EVENTLOG_SEQUENTIAL_READ | EVENTLOG_FORWARDS_READ,
+                        0, buf, sizeof(EVENTLOGRECORD), &read, &needed);
+    ok(!ret, "Expected failure\n");
+    todo_wine
+    ok(GetLastError() == ERROR_INVALID_PARAMETER, "Expected ERROR_INVALID_PARAMETER, got %d\n", GetLastError());
+
+    buf = HeapAlloc(GetProcessHeap(), 0, sizeof(EVENTLOGRECORD));
+    SetLastError(0xdeadbeef);
+    ret = ReadEventLogA(NULL, EVENTLOG_SEQUENTIAL_READ | EVENTLOG_FORWARDS_READ,
+                        0, buf, sizeof(EVENTLOGRECORD), &read, &needed);
+    ok(!ret, "Expected failure\n");
+    todo_wine
+    ok(GetLastError() == ERROR_INVALID_HANDLE, "Expected ERROR_INVALID_HANDLE, got %d\n", GetLastError());
+    HeapFree(GetProcessHeap(), 0, buf);
+
+    handle = OpenEventLogA(NULL, "Application");
+
+    /* Show that we need the proper dwFlags with a (for the rest) proper call */
+    buf = HeapAlloc(GetProcessHeap(), 0, sizeof(EVENTLOGRECORD));
+
+    SetLastError(0xdeadbeef);
+    ret = ReadEventLogA(handle, 0, 0, buf, sizeof(EVENTLOGRECORD), &read, &needed);
+    ok(!ret, "Expected failure\n");
+    todo_wine
+    ok(GetLastError() == ERROR_INVALID_PARAMETER, "Expected ERROR_INVALID_PARAMETER, got %d\n", GetLastError());
+
+    SetLastError(0xdeadbeef);
+    ret = ReadEventLogA(handle, EVENTLOG_SEQUENTIAL_READ, 0, buf, sizeof(EVENTLOGRECORD), &read, &needed);
+    ok(!ret, "Expected failure\n");
+    todo_wine
+    ok(GetLastError() == ERROR_INVALID_PARAMETER, "Expected ERROR_INVALID_PARAMETER, got %d\n", GetLastError());
+
+    SetLastError(0xdeadbeef);
+    ret = ReadEventLogA(handle, EVENTLOG_SEEK_READ, 0, buf, sizeof(EVENTLOGRECORD), &read, &needed);
+    ok(!ret, "Expected failure\n");
+    todo_wine
+    ok(GetLastError() == ERROR_INVALID_PARAMETER, "Expected ERROR_INVALID_PARAMETER, got %d\n", GetLastError());
+
+    SetLastError(0xdeadbeef);
+    ret = ReadEventLogA(handle, EVENTLOG_SEQUENTIAL_READ | EVENTLOG_FORWARDS_READ | EVENTLOG_BACKWARDS_READ,
+                        0, buf, sizeof(EVENTLOGRECORD), &read, &needed);
+    ok(!ret, "Expected failure\n");
+    todo_wine
+    ok(GetLastError() == ERROR_INVALID_PARAMETER, "Expected ERROR_INVALID_PARAMETER, got %d\n", GetLastError());
+
+    SetLastError(0xdeadbeef);
+    ret = ReadEventLogA(handle, EVENTLOG_SEEK_READ | EVENTLOG_FORWARDS_READ | EVENTLOG_BACKWARDS_READ,
+                        0, buf, sizeof(EVENTLOGRECORD), &read, &needed);
+    ok(!ret, "Expected failure\n");
+    todo_wine
+    ok(GetLastError() == ERROR_INVALID_PARAMETER, "Expected ERROR_INVALID_PARAMETER, got %d\n", GetLastError());
+
+    SetLastError(0xdeadbeef);
+    ret = ReadEventLogA(handle, EVENTLOG_SEEK_READ | EVENTLOG_SEQUENTIAL_READ | EVENTLOG_FORWARDS_READ,
+                        0, buf, sizeof(EVENTLOGRECORD), &read, &needed);
+    ok(!ret, "Expected failure\n");
+    todo_wine
+    ok(GetLastError() == ERROR_INVALID_PARAMETER, "Expected ERROR_INVALID_PARAMETER, got %d\n", GetLastError());
+
+    HeapFree(GetProcessHeap(), 0, buf);
+
+    /* First check if there are any records (in practice only on Wine: FIXME) */
+    count = 0;
+    GetNumberOfEventLogRecords(handle, &count);
+    if (!count)
+    {
+        skip("No records in the 'Application' log\n");
+        CloseEventLog(handle);
+        return;
+    }
+
+    /* Get the buffer size for the first record */
+    buf = HeapAlloc(GetProcessHeap(), 0, sizeof(EVENTLOGRECORD));
+    read = needed = 0xdeadbeef;
+    SetLastError(0xdeadbeef);
+    ret = ReadEventLogA(handle, EVENTLOG_SEQUENTIAL_READ | EVENTLOG_FORWARDS_READ,
+                        0, buf, sizeof(EVENTLOGRECORD), &read, &needed);
+    ok(!ret, "Expected failure\n");
+    ok(read == 0, "Expected no bytes read\n");
+    ok(needed > sizeof(EVENTLOGRECORD), "Expected the needed buffersize to be bigger than sizeof(EVENTLOGRECORD)\n");
+    ok(GetLastError() == ERROR_INSUFFICIENT_BUFFER, "Expected ERROR_INVALID_PARAMETER, got %d\n", GetLastError());
+
+    /* Read the first record */
+    toread = needed;
+    buf = HeapReAlloc(GetProcessHeap(), 0, buf, toread);
+    read = needed = 0xdeadbeef;
+    SetLastError(0xdeadbeef);
+    ret = ReadEventLogA(handle, EVENTLOG_SEQUENTIAL_READ | EVENTLOG_FORWARDS_READ, 0, buf, toread, &read, &needed);
+    ok(ret, "Expected succes\n");
+    ok(read == toread ||
+       broken(read < toread), /* NT4 wants a buffer size way bigger than just 1 record */
+       "Expected the requested size to be read\n");
+    ok(needed == 0, "Expected no extra bytes to be read\n");
+    HeapFree(GetProcessHeap(), 0, buf);
+
+    CloseEventLog(handle);
+}
+
 START_TEST(eventlog)
 {
     SetLastError(0xdeadbeef);
@@ -274,4 +422,5 @@ START_TEST(eventlog)
     test_count();
     test_oldest();
     test_backup();
+    test_read();
 }
diff --git a/include/winnt.h b/include/winnt.h
index fd267c3..032e9c1 100644
--- a/include/winnt.h
+++ b/include/winnt.h
@@ -4972,6 +4972,30 @@ typedef struct _TAPE_GET_MEDIA_PARAMETERS {
 #define EVENTLOG_AUDIT_SUCCESS          0x0008
 #define EVENTLOG_AUDIT_FAILURE          0x0010
 
+#define EVENTLOG_SEQUENTIAL_READ        0x0001
+#define EVENTLOG_SEEK_READ              0x0002
+#define EVENTLOG_FORWARDS_READ          0x0004
+#define EVENTLOG_BACKWARDS_READ         0x0008
+
+typedef struct _EVENTLOGRECORD {
+    DWORD  Length;
+    DWORD  Reserved;
+    DWORD  RecordNumber;
+    DWORD  TimeGenerated;
+    DWORD  TimeWritten;
+    DWORD  EventID;
+    WORD   EventType;
+    WORD   NumStrings;
+    WORD   EventCategory;
+    WORD   ReservedFlags;
+    DWORD  ClosingRecordNumber;
+    DWORD  StringOffset;
+    DWORD  UserSidLength;
+    DWORD  UserSidOffset;
+    DWORD  DataLength;
+    DWORD  DataOffset;
+} EVENTLOGRECORD, *PEVENTLOGRECORD;
+
 #define SERVICE_BOOT_START   0x00000000
 #define SERVICE_SYSTEM_START 0x00000001
 #define SERVICE_AUTO_START   0x00000002




More information about the wine-cvs mailing list