Revision: 17445 http://sourceforge.net/p/edk2/code/17445 Author: ydong10 Date: 2015-05-14 02:46:11 +0000 (Thu, 14 May 2015) Log Message: ----------- MdeModulePkg: ConfigKeywordHandler: fix pointer target signedness.
SVN r17428 ("MdeModulePkg: Implement UEFI25 HII Config keyword handler protocol") introduced code that triggers -Werror=pointer-sign under gcc ("warn for pointer argument passing or assignment with different signedness"). This patch fix up those locations. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Laszlo Ersek <ler...@redhat.com> Reviewed-by: Eric Dong <eric.d...@intel.com> Revision Links: -------------- http://sourceforge.net/p/edk2/code/17428 Modified Paths: -------------- trunk/edk2/MdeModulePkg/Universal/HiiDatabaseDxe/ConfigKeywordHandler.c Modified: trunk/edk2/MdeModulePkg/Universal/HiiDatabaseDxe/ConfigKeywordHandler.c =================================================================== --- trunk/edk2/MdeModulePkg/Universal/HiiDatabaseDxe/ConfigKeywordHandler.c 2015-05-14 02:09:59 UTC (rev 17444) +++ trunk/edk2/MdeModulePkg/Universal/HiiDatabaseDxe/ConfigKeywordHandler.c 2015-05-14 02:46:11 UTC (rev 17445) @@ -806,7 +806,7 @@ Offset = sizeof (EFI_HII_STRING_BLOCK); StringTextPtr = BlockHdr + Offset; BlockSize += Offset + AsciiStrSize ((CHAR8 *) StringTextPtr); - if (AsciiStrCmp(AsciiKeywordValue, StringTextPtr) == 0) { + if (AsciiStrCmp(AsciiKeywordValue, (CHAR8 *) StringTextPtr) == 0) { *StringId = CurrentStringId; goto Done; } @@ -816,7 +816,7 @@ case EFI_HII_SIBT_STRING_SCSU_FONT: Offset = sizeof (EFI_HII_SIBT_STRING_SCSU_FONT_BLOCK) - sizeof (UINT8); StringTextPtr = BlockHdr + Offset; - if (AsciiStrCmp(AsciiKeywordValue, StringTextPtr) == 0) { + if (AsciiStrCmp(AsciiKeywordValue, (CHAR8 *) StringTextPtr) == 0) { *StringId = CurrentStringId; goto Done; } @@ -831,7 +831,7 @@ for (Index = 0; Index < StringCount; Index++) { BlockSize += AsciiStrSize ((CHAR8 *) StringTextPtr); - if (AsciiStrCmp(AsciiKeywordValue, StringTextPtr) == 0) { + if (AsciiStrCmp(AsciiKeywordValue, (CHAR8 *) StringTextPtr) == 0) { *StringId = CurrentStringId; goto Done; } @@ -851,7 +851,7 @@ for (Index = 0; Index < StringCount; Index++) { BlockSize += AsciiStrSize ((CHAR8 *) StringTextPtr); - if (AsciiStrCmp(AsciiKeywordValue, StringTextPtr) == 0) { + if (AsciiStrCmp(AsciiKeywordValue, (CHAR8 *) StringTextPtr) == 0) { *StringId = CurrentStringId; goto Done; } @@ -1069,7 +1069,7 @@ if (*KeywordValue == NULL) { return 0; } - AsciiStrToUnicodeStr(StringTextPtr, *KeywordValue); + AsciiStrToUnicodeStr ((CHAR8 *) StringTextPtr, *KeywordValue); return CurrentStringId; } else if (CurrentStringId == StringId) { FindString = TRUE; @@ -1088,7 +1088,7 @@ if (*KeywordValue == NULL) { return 0; } - AsciiStrToUnicodeStr(StringTextPtr, *KeywordValue); + AsciiStrToUnicodeStr ((CHAR8 *) StringTextPtr, *KeywordValue); return CurrentStringId; } else if (CurrentStringId == StringId) { FindString = TRUE; @@ -1109,7 +1109,7 @@ if (*KeywordValue == NULL) { return 0; } - AsciiStrToUnicodeStr(StringTextPtr, *KeywordValue); + AsciiStrToUnicodeStr ((CHAR8 *) StringTextPtr, *KeywordValue); return CurrentStringId; } else if (CurrentStringId == StringId) { FindString = TRUE; @@ -1136,7 +1136,7 @@ if (*KeywordValue == NULL) { return 0; } - AsciiStrToUnicodeStr(StringTextPtr, *KeywordValue); + AsciiStrToUnicodeStr ((CHAR8 *) StringTextPtr, *KeywordValue); return CurrentStringId; } else if (CurrentStringId == StringId) { FindString = TRUE; @@ -1674,7 +1674,7 @@ switch (((EFI_IFR_OP_HEADER *)OpCodeData)->OpCode) { case EFI_IFR_VARSTORE_OP: Guid = (EFI_GUID *)(UINTN *)&((EFI_IFR_VARSTORE *) OpCodeData)->Guid; - AsciiName = ((EFI_IFR_VARSTORE *) OpCodeData)->Name; + AsciiName = (CHAR8 *) ((EFI_IFR_VARSTORE *) OpCodeData)->Name; break; case EFI_IFR_VARSTORE_NAME_VALUE_OP: @@ -1684,7 +1684,7 @@ case EFI_IFR_VARSTORE_EFI_OP: Guid = (EFI_GUID *)(UINTN *)&((EFI_IFR_VARSTORE_EFI *) OpCodeData)->Guid; - AsciiName = ((EFI_IFR_VARSTORE_EFI *) OpCodeData)->Name; + AsciiName = (CHAR8 *) ((EFI_IFR_VARSTORE_EFI *) OpCodeData)->Name; break; default: @@ -2571,7 +2571,7 @@ CHAR8 *LocalNameSpace; EFI_STRING_ID NextStringId; EFI_STATUS Status; - CHAR8 *OpCode; + UINT8 *OpCode; CHAR16 *ConfigRequest; CHAR16 *ValueElement; CHAR16 *KeywordResp; ------------------------------------------------------------------------------ One dashboard for servers and applications across Physical-Virtual-Cloud Widest out-of-the-box monitoring support with 50+ applications Performance metrics, stats and reports that give you Actionable Insights Deep dive visibility with transaction tracing using APM Insight. http://ad.doubleclick.net/ddm/clk/290420510;117567292;y _______________________________________________ edk2-commits mailing list edk2-commits@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/edk2-commits