Don't calculate the directory size and the caller of EFI_FILE protocol can calculate that for better UI.

The enhancement algorithm to calculate directory size (Edk r2583) is incorrect because SetPosition() does not allow other arguments other than 0, causing current file cursor is lost. 

git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@9713 6f19259b-4bc3-4df7-8a09-765794883524
This commit is contained in:
qhuang8 2010-01-12 10:32:47 +00:00
parent 6bacae7cdc
commit 49b3b3ab26
1 changed files with 1 additions and 45 deletions

View File

@ -1,6 +1,6 @@
/**@file /**@file
Copyright (c) 2006 - 2009, Intel Corporation Copyright (c) 2006 - 2010, Intel Corporation
All rights reserved. This program and the accompanying materials All rights reserved. This program and the accompanying materials
are licensed and made available under the terms and conditions of the BSD License are licensed and made available under the terms and conditions of the BSD License
which accompanies this distribution. The full text of the license may be found at which accompanies this distribution. The full text of the license may be found at
@ -1972,11 +1972,6 @@ Returns:
SYSTEMTIME SystemTime; SYSTEMTIME SystemTime;
CHAR16 *RealFileName; CHAR16 *RealFileName;
CHAR16 *TempPointer; CHAR16 *TempPointer;
EFI_FILE_INFO *DirInfo;
UINTN ReadSize;
UINT64 Location;
EFI_STATUS DirStatus;
Size = SIZE_OF_EFI_FILE_INFO; Size = SIZE_OF_EFI_FILE_INFO;
NameSize = StrSize (PrivateFile->FileName); NameSize = StrSize (PrivateFile->FileName);
@ -2061,45 +2056,6 @@ Returns:
} else { } else {
CopyMem ((CHAR8 *) Buffer + Size, RealFileName, NameSize); CopyMem ((CHAR8 *) Buffer + Size, RealFileName, NameSize);
} }
if (Info->Attribute & EFI_FILE_DIRECTORY) {
//
// The GetFileInformationByHandle.nFileSizeLow is bogus for dir so we
// need to do the same thing the caller would do to get the right value
//
ASSERT (PrivateFile->EfiFile.Read != NULL);
DirStatus = PrivateFile->EfiFile.GetPosition (&PrivateFile->EfiFile, &Location);
if (EFI_ERROR (DirStatus)) {
Location = 0;
}
PrivateFile->EfiFile.SetPosition (&PrivateFile->EfiFile, 0);
Info->FileSize = 0;
do {
ReadSize = 0;
DirInfo = NULL;
DirStatus = PrivateFile->EfiFile.Read (&PrivateFile->EfiFile, &ReadSize, DirInfo);
if (DirStatus == EFI_BUFFER_TOO_SMALL) {
DirInfo = AllocatePool (ReadSize);
if (DirInfo != NULL) {
//
// Read each dir entry to figure out how big the directory is
//
DirStatus = PrivateFile->EfiFile.Read (&PrivateFile->EfiFile, &ReadSize, DirInfo);
if (!EFI_ERROR (DirStatus) && (ReadSize != 0)) {
Info->FileSize += ReadSize;
}
FreePool (DirInfo);
}
}
} while (!EFI_ERROR (DirStatus) && (ReadSize != 0));
//
// reset the file possition back to the previous location
//
PrivateFile->EfiFile.SetPosition (&PrivateFile->EfiFile, Location);
}
} }
*BufferSize = ResultSize; *BufferSize = ResultSize;