summaryrefslogtreecommitdiff
path: root/ShellPkg
diff options
context:
space:
mode:
authorjcarsey <jcarsey@6f19259b-4bc3-4df7-8a09-765794883524>2011-03-28 21:49:17 +0000
committerjcarsey <jcarsey@6f19259b-4bc3-4df7-8a09-765794883524>2011-03-28 21:49:17 +0000
commite35b53179454119ba0d8591ccdfee9f0efe5fd77 (patch)
treefa12df56752026e64c2cb461b755ef9946bf366f /ShellPkg
parent590c3cb14ad6547528ea13805817676e32c204aa (diff)
downloadedk2-e35b53179454119ba0d8591ccdfee9f0efe5fd77.zip
edk2-e35b53179454119ba0d8591ccdfee9f0efe5fd77.tar.gz
edk2-e35b53179454119ba0d8591ccdfee9f0efe5fd77.tar.bz2
prevents "" from being the result of trying to open the root of a drive.
more input validation on vol command. git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@11444 6f19259b-4bc3-4df7-8a09-765794883524
Diffstat (limited to 'ShellPkg')
-rw-r--r--ShellPkg/Application/Shell/ShellProtocol.c6
-rw-r--r--ShellPkg/Library/UefiShellLevel2CommandsLib/Vol.c26
2 files changed, 27 insertions, 5 deletions
diff --git a/ShellPkg/Application/Shell/ShellProtocol.c b/ShellPkg/Application/Shell/ShellProtocol.c
index e2da5d3..2d587ea 100644
--- a/ShellPkg/Application/Shell/ShellProtocol.c
+++ b/ShellPkg/Application/Shell/ShellProtocol.c
@@ -585,7 +585,11 @@ EfiShellGetDevicePathFromFilePath(
//
// build the full device path
//
- DevicePathForReturn = FileDevicePath(Handle, Path+StrLen(MapName)+1);
+ if (*(Path+StrLen(MapName)+1) == CHAR_NULL) {
+ DevicePathForReturn = FileDevicePath(Handle, L"\\");
+ } else {
+ DevicePathForReturn = FileDevicePath(Handle, Path+StrLen(MapName)+1);
+ }
FreePool(MapName);
if (DevicePathCopyForFree != NULL) {
diff --git a/ShellPkg/Library/UefiShellLevel2CommandsLib/Vol.c b/ShellPkg/Library/UefiShellLevel2CommandsLib/Vol.c
index 3688a50..8757ff1 100644
--- a/ShellPkg/Library/UefiShellLevel2CommandsLib/Vol.c
+++ b/ShellPkg/Library/UefiShellLevel2CommandsLib/Vol.c
@@ -108,13 +108,24 @@ HandleVol(
}
}
- SysInfo = AllocateZeroPool(SysInfoSize);
+ SysInfoSize = 0;
+ SysInfo = NULL;
+
Status = EfiFpHandle->GetInfo(
EfiFpHandle,
&gEfiFileSystemInfoGuid,
&SysInfoSize,
SysInfo);
+ if (Status == EFI_BUFFER_TOO_SMALL) {
+ SysInfo = AllocateZeroPool(SysInfoSize);
+ Status = EfiFpHandle->GetInfo(
+ EfiFpHandle,
+ &gEfiFileSystemInfoGuid,
+ &SysInfoSize,
+ SysInfo);
+ }
+
gEfiShellProtocol->CloseFile(ShellFileHandle);
//
@@ -164,6 +175,7 @@ ShellCommandRunVol (
CONST CHAR16 *CurDir;
BOOLEAN DeleteMode;
CHAR16 *FullPath;
+ CHAR16 *TempSpot;
UINTN Length;
Length = 0;
@@ -220,10 +232,16 @@ ShellCommandRunVol (
}
}
if (PathName != NULL) {
- StrnCatGrow(&FullPath, &Length, PathName, StrStr(PathName, L"\\")==NULL?0:StrStr(PathName, L"\\")-PathName+1);
- if (StrStr(FullPath, L":\\") == NULL) {
- StrnCatGrow(&FullPath, &Length, L":\\", 0);
+ TempSpot = StrStr(PathName, L":");
+ if (TempSpot != NULL) {
+ *TempSpot = CHAR_NULL;
+ }
+ TempSpot = StrStr(PathName, L"\\");
+ if (TempSpot != NULL) {
+ *TempSpot = CHAR_NULL;
}
+ StrnCatGrow(&FullPath, &Length, PathName, 0);
+ StrnCatGrow(&FullPath, &Length, L":\\", 0);
DeleteMode = ShellCommandLineGetFlag(Package, L"-d");
if (DeleteMode && ShellCommandLineGetFlag(Package, L"-n")) {
ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_GEN_PARAM_CON), gShellLevel2HiiHandle);