fix DataHubGetNextRecord bug

git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@4441 6f19259b-4bc3-4df7-8a09-765794883524

1 files changed, 9 insertions, 9 deletions

diff --git a/IntelFrameworkModulePkg/Universal/DataHubDxe/DataHub.c b/IntelFrameworkModulePkg/Universal/DataHubDxe/DataHub.c
index 75c1add..386c31c 100644
--- a/IntelFrameworkModulePkg/Universal/DataHubDxe/DataHub.c
+++ b/IntelFrameworkModulePkg/Universal/DataHubDxe/DataHub.c
@@ -140,7 +140,10 @@ Returns:
   CopyMem (&Record->ProducerName, ProducerName, sizeof (EFI_GUID));

   Record->DataRecordClass   = DataRecordClass;

 

-  Record->LogMonotonicCount = Private->GlobalMonotonicCount++;

+  //

+  // Ensure LogMonotonicCount is not zero

+  //

+  Record->LogMonotonicCount = ++Private->GlobalMonotonicCount;

 

   gRT->GetTime (&Record->LogTime, NULL);

 

@@ -258,8 +261,7 @@ Returns:
       if (FilterMonotonicCount != 0) {

         //

         // The GetNextMonotonicCount field remembers the last value from the previous time.

-        // But we already processed this vaule, so we need to find the next one. So if

-        // It is not the first time get the new record entry.

+        // But we already processed this vaule, so we need to find the next one.

         //

         *Record         = GetNextDataRecord (&Private->DataListHead, ClassFilter, &FilterMonotonicCount);

         *MonotonicCount = FilterMonotonicCount;

@@ -286,12 +288,10 @@ Returns:
     // If MonotonicCount is zero No more reacords left.

     //

     if (*MonotonicCount == 0) {

-      if (FilterMonotonicCount != 0) {

-        //

-        // Return the result of our extra GetNextDataRecord.

-        //

-        FilterDriver->GetNextMonotonicCount = FilterMonotonicCount;

-      }

+      //

+      // Save the current Record MonotonicCount.

+      //

+      FilterDriver->GetNextMonotonicCount = (*Record)->LogMonotonicCount;

     } else {

       //

       // Point to next undread record