tyler.durden
/
audk
mirror of https://github.com/acidanthera/audk.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

ShellPkg: Refine the code logic of mv.c. 

When doing the mv we should check whether source is 'above' dest on file path tree. This patch make the check logic more precise.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Qiu Shumin <shumin.qiu@intel.com>
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com>
Reviewed-by: Samer El-Haj-Mahmoud <elhaj@hp.com>



git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@17750 6f19259b-4bc3-4df7-8a09-765794883524
This commit is contained in:
Qiu Shumin 2015-07-01 01:46:20 +00:00 committed by shenshushi
parent 203beadc12
commit 2ce8ce9b06
1 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
ShellPkg/Library/UefiShellLevel2CommandsLib/Mv.c
View File

@ -128,8 +128,10 @@ IsValidMove(
// //
// If they're the same, or if source is "above" dest on file path tree // If they're the same, or if source is "above" dest on file path tree
// //
if ( StringNoCaseCompare (&DestPathWalker, &SourcePath) == 0 if ( StringNoCaseCompare (&DestPathWalker, &SourcePath) == 0 ||
|| StrStr(DestPathWalker, SourcePath) == DestPathWalker ((StrStr(DestPathWalker, SourcePath) == DestPathWalker) &&
(DestPathWalker[StrLen(SourcePath)] == '\\')
)
) { ) {
ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_MV_INV_SUB), gShellLevel2HiiHandle); ShellPrintHiiEx(-1, -1, NULL, STRING_TOKEN (STR_MV_INV_SUB), gShellLevel2HiiHandle);
FreePool(DestPathCopy); FreePool(DestPathCopy);