[EVENTLOG]
authorSylvain Petreolle <spetreolle@yahoo.fr>
Wed, 3 Sep 2014 23:07:47 +0000 (23:07 +0000)
committerSylvain Petreolle <spetreolle@yahoo.fr>
Wed, 3 Sep 2014 23:07:47 +0000 (23:07 +0000)
Remove the event log handle from the list *before* destroying it.
Fixes failure in advapi32:eventlog double close test.

svn path=/trunk/; revision=64034

reactos/base/services/eventlog/file.c
reactos/base/services/eventlog/rpc.c

index 0a784cf..4781e82 100644 (file)
@@ -1252,7 +1252,7 @@ LogfBackupFile(PLOGFILE LogFile,
                              NULL);
         if (!NT_SUCCESS(Status))
         {
-            DPRINT1("NtWriteFile() failed!\n");
+            DPRINT1("NtWriteFile() failed! (Status: 0x%08lx)\n", Status);
             goto Done;
         }
 
@@ -1311,7 +1311,7 @@ LogfBackupFile(PLOGFILE LogFile,
                          NULL);
     if (!NT_SUCCESS(Status))
     {
-        DPRINT1("NtWriteFile() failed!\n");
+        DPRINT1("NtWriteFile() failed! (Status: 0x%08lx)\n", Status);
     }
 
 Done:
index b33a119..1154f01 100644 (file)
@@ -215,9 +215,9 @@ ElfDeleteEventLogHandle(IELF_HANDLE EventLogHandle)
     if (!ElfGetLogHandleEntryByHandle(lpLogHandle))
         return STATUS_INVALID_HANDLE;
 
+    RemoveEntryList(&lpLogHandle->LogHandleListEntry);
     LogfClose(lpLogHandle->LogFile, FALSE);
 
-    RemoveEntryList(&lpLogHandle->LogHandleListEntry);
     HeapFree(GetProcessHeap(),0,lpLogHandle);
 
     return STATUS_SUCCESS;
@@ -291,7 +291,7 @@ NTSTATUS ElfrNumberOfRecords(
     PLOGHANDLE lpLogHandle;
     PLOGFILE lpLogFile;
 
-    DPRINT("ElfrNumberOfRecords()");
+    DPRINT("ElfrNumberOfRecords()\n");
 
     lpLogHandle = ElfGetLogHandleEntryByHandle(LogHandle);
     if (!lpLogHandle)