diff options
| author | Hao Wu <hao.a.wu@intel.com> | 2015-07-06 01:41:45 +0000 |
|---|---|---|
| committer | Jordan Justen <jordan.l.justen@intel.com> | 2016-04-06 23:22:43 -0700 |
| commit | e76bc43e5d5827e7f8b20b3f89727b9bc86395e8 (patch) | |
| tree | c37edfc048ffa4d1086fd43c9e810af9a59a8bcb /FatPkg/EnhancedFatDxe/Hash.c | |
| parent | 4819b3e8f01e8d61ba5310d9de6ce71598d5b54c (diff) | |
| download | edk2-e76bc43e5d5827e7f8b20b3f89727b9bc86395e8.tar.gz edk2-e76bc43e5d5827e7f8b20b3f89727b9bc86395e8.tar.bz2 edk2-e76bc43e5d5827e7f8b20b3f89727b9bc86395e8.zip | |
FatPkg EnhancedFatDxe: Use safe string functions
Unsafe string functions are replaced with safe ones.
Safe string functions will assert if DestMax is not greater than
StrnLenS(Source, DestMax). Therefore, additional assert for checking the
size of source and destination buffers can be removed.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Hao Wu <hao.a.wu@intel.com>
Reviewed-by: Ruiyu Ni <ruiyu.ni@intel.com>
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com>
(based on FatPkg commit 2cb92b4f19b096daf133d6501afa13e5a85062c5)
[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 <jordan.l.justen@intel.com>
Acked-by: Mark Doran <mark.doran@intel.com>
Acked-by: Laszlo Ersek <lersek@redhat.com>
Diffstat (limited to 'FatPkg/EnhancedFatDxe/Hash.c')
| -rw-r--r-- | FatPkg/EnhancedFatDxe/Hash.c | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/FatPkg/EnhancedFatDxe/Hash.c b/FatPkg/EnhancedFatDxe/Hash.c index dd67bab313..3d0ffe6fb8 100644 --- a/FatPkg/EnhancedFatDxe/Hash.c +++ b/FatPkg/EnhancedFatDxe/Hash.c @@ -1,6 +1,6 @@ /*++
-Copyright (c) 2005 - 2014, Intel Corporation. All rights reserved.<BR>
+Copyright (c) 2005 - 2015, Intel Corporation. All rights reserved.<BR>
This program and the accompanying materials 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
@@ -47,8 +47,12 @@ Returns: {
UINT32 HashValue;
CHAR16 UpCasedLongFileName[EFI_PATH_STRING_LENGTH];
- StrnCpy (UpCasedLongFileName, LongNameString, EFI_PATH_STRING_LENGTH - 1);
- UpCasedLongFileName[EFI_PATH_STRING_LENGTH - 1] = L'\0';
+ StrnCpyS (
+ UpCasedLongFileName,
+ sizeof (UpCasedLongFileName) / sizeof (UpCasedLongFileName[0]),
+ LongNameString,
+ sizeof (UpCasedLongFileName) / sizeof (UpCasedLongFileName[0]) - 1
+ );
FatStrUpr (UpCasedLongFileName);
gBS->CalculateCrc32 (UpCasedLongFileName, StrSize (UpCasedLongFileName), &HashValue);
return (HashValue & HASH_TABLE_MASK);
|