commit 00afc8f82061677fedc86cb05e3b8c75a3c986ff
author Dandan Bi <dandan.bi@intel.com> Tue Aug 23 10:29:08 2016 +0800
committer Ruiyu Ni <ruiyu.ni@intel.com> Thu Sep 01 16:38:07 2016 +0800
tree b011a0cbccd31661e00d0b703928cc4691a56b2a
parent c6fc823413861c6bcafbb21bae9aab66b8ee5a24

ShellPkg: Fix the incorrect return status in function FindFiles()

According to the latest shell spec, in function FindFiles(),
when no files were found, it should return EFI_NOT_FOUND.
But current codes don't follow the spec.
This patch is to fix this issue.

Cc: Ruiyu Ni <ruiyu.ni@intel.com>
Cc: Jaben Carsey <jaben.carsey@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
Reviewed-by: Ruiyu Ni <ruiyu.ni@intel.com>

ShellPkg/Application/Shell/ShellProtocol.c

diff --git a/ShellPkg/Application/Shell/ShellProtocol.c b/ShellPkg/Application/Shell/ShellProtocol.c
index 0e5d954..6f29250 100644
--- a/ShellPkg/Application/Shell/ShellProtocol.c
+++ b/ShellPkg/Application/Shell/ShellProtocol.c
@@ -2509,6 +2509,10 @@
     }

   }

 

+  if (*FileList == NULL || (*FileList != NULL && IsListEmpty(&(*FileList)->Link))) {

+    Status = EFI_NOT_FOUND;

+  }

+

   FreePool(CurrentFilePattern);

   return (Status);

 }