From 3ba5368df0a3d459606fe78bc73776359c8bc005 Mon Sep 17 00:00:00 2001 From: Star Zeng Date: Tue, 12 Mar 2013 01:51:42 +0000 Subject: [PATCH] 1. Expand and add null terminate to end of string like function comments said in EngFatToStr(), it could fix the randomly failure during recovery to find the correct recovery image for name length = 8.3 case. 2. Skip directory entry with FAT_ATTR_DIRECTORY, not just = FAT_ATTR_DIRECTORY in FatReadNextDirectoryEntry(). Signed-off-by: Star Zeng Reviewed-by: Ruiyu Ni (based on FatPkg commit 1cc9454c1223e8177ba3028bdf3fa72b52aacffe) [jordan.l.justen@intel.com: Use script to relicense to 2-clause BSD] Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jordan Justen Acked-by: Mark Doran Acked-by: Laszlo Ersek --- FatPkg/FatPei/FatLiteAccess.c | 4 ++-- FatPkg/FatPei/FatLiteLib.c | 11 ++++++----- 2 files changed, 8 insertions(+), 7 deletions(-) diff --git a/FatPkg/FatPei/FatLiteAccess.c b/FatPkg/FatPei/FatLiteAccess.c index 53f5e3f60c..110634508b 100644 --- a/FatPkg/FatPei/FatLiteAccess.c +++ b/FatPkg/FatPei/FatLiteAccess.c @@ -1,7 +1,7 @@ /** @file FAT file system access routines for FAT recovery PEIM -Copyright (c) 2006 - 2011, Intel Corporation. All rights reserved.
+Copyright (c) 2006 - 2013, Intel Corporation. All rights reserved.
This program and the accompanying materials are licensed and made available under the terms and conditions of the BSD License which accompanies this @@ -475,7 +475,7 @@ FatReadNextDirectoryEntry ( // We only search for *FILE* in root directory // Long file name entry is *NOT* supported // - if ((DirEntry.Attributes == FAT_ATTR_DIRECTORY) || (DirEntry.Attributes == FAT_ATTR_LFN)) { + if (((DirEntry.Attributes & FAT_ATTR_DIRECTORY) == FAT_ATTR_DIRECTORY) || (DirEntry.Attributes == FAT_ATTR_LFN)) { continue; } // diff --git a/FatPkg/FatPei/FatLiteLib.c b/FatPkg/FatPei/FatLiteLib.c index b5191b6706..06f031defe 100644 --- a/FatPkg/FatPei/FatLiteLib.c +++ b/FatPkg/FatPei/FatLiteLib.c @@ -1,7 +1,7 @@ /** @file General purpose supporting routines for FAT recovery PEIM -Copyright (c) 2006 - 2010, Intel Corporation. All rights reserved.
+Copyright (c) 2006 - 2013, Intel Corporation. All rights reserved.
This program and the accompanying materials are licensed and made available under the terms and conditions of the BSD License which accompanies this @@ -317,15 +317,16 @@ EngFatToStr ( // No DBCS issues, just expand and add null terminate to end of string // while (*Fat != 0 && FatSize != 0) { + if (*Fat == ' ') { + break; + } *String = *Fat; String += 1; Fat += 1; FatSize -= 1; - if (*Fat == ' ') { - *String = 0; - return ; - } } + + *String = 0; } -- 2.39.2