From 41e4912f2932985647927229118179d1f5eb0283 Mon Sep 17 00:00:00 2001 From: eric_tian Date: Wed, 26 Dec 2007 09:31:03 +0000 Subject: [PATCH] fix DataHubGetNextRecord bug git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@4441 6f19259b-4bc3-4df7-8a09-765794883524 --- .../Universal/DataHubDxe/DataHub.c | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/IntelFrameworkModulePkg/Universal/DataHubDxe/DataHub.c b/IntelFrameworkModulePkg/Universal/DataHubDxe/DataHub.c index 75c1addb09..386c31c921 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