From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wm0-x22d.google.com (mail-wm0-x22d.google.com [IPv6:2a00:1450:400c:c09::22d]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 205F41A1E3E for ; Mon, 24 Oct 2016 10:42:18 -0700 (PDT) Received: by mail-wm0-x22d.google.com with SMTP id c78so128222385wme.1 for ; Mon, 24 Oct 2016 10:42:18 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=tK3V+PBCd8LUbn8lU9kyO0aZ2x4Yp9QanHhGd4+bX5I=; b=h7jHOSWgrYXfeHBnmMSJMyVdmJf8sJRWOZi8fl6Bl3xsT4zSNLIQtGGy+s0U5TLipy n0Xk2HrVp0ZYUXroSwpwi69TrxI2yc81yy880N6TXEATRkpgpLLFgXhuMQ6d1bXPy9UN nIvEdN1sQlrXU1Pvy0fyg4KI0wFRNUKa5ALJk= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=tK3V+PBCd8LUbn8lU9kyO0aZ2x4Yp9QanHhGd4+bX5I=; b=byPkvP61xSWvb20cgnEFh+AO0Ip/AtM4qGCYVkqNXMbG10sKAro9Zo75/He2xjTAgK mQYy/mc0rLKDJArkrGMHoEbAsOoLlylLr9Ia/XcsmQNqgCGNSSjSlRR7f5TrP8KfsUhs 5sEgFCskJnqblaByLhT/+vlueEOdsfeW/OTT6hD9VIiNEHg62BHdENDPpbpooFxHS2MG J4kak9s7nhdof32BNIYbZJeY2xEQokS+zj98ddZ6dOiBGjaR/aqVfs882PuzPQAyw4v3 jyBDIHsQvY4O2p7vVYiVenOhIe7U1GnfR8cTtouWW8JJvklXmhf7ALA1SlFBi6809pz/ ZJ/Q== X-Gm-Message-State: AA6/9Rk6gaAuS1X7I94b7LXj+r0ygK98MjIiQSTpQ14cvLHFDN+TcRShK7SNalMoSsTi/BWI X-Received: by 10.28.46.15 with SMTP id u15mr24052297wmu.61.1477330935272; Mon, 24 Oct 2016 10:42:15 -0700 (PDT) Received: from localhost.localdomain ([105.146.229.54]) by smtp.gmail.com with ESMTPSA id 194sm15894873wmj.20.2016.10.24.10.42.13 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Mon, 24 Oct 2016 10:42:14 -0700 (PDT) From: Ard Biesheuvel To: edk2-devel@lists.01.org, leif.lindholm@linaro.org, lersek@redhat.com Cc: Ard Biesheuvel Date: Mon, 24 Oct 2016 18:41:44 +0100 Message-Id: <1477330907-13733-7-git-send-email-ard.biesheuvel@linaro.org> X-Mailer: git-send-email 2.7.4 In-Reply-To: <1477330907-13733-1-git-send-email-ard.biesheuvel@linaro.org> References: <1477330907-13733-1-git-send-email-ard.biesheuvel@linaro.org> Subject: [PATCH 6/9] EmbeddedPkg/Ebl: eliminate deprecated string function calls X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 24 Oct 2016 17:42:18 -0000 Get rid of calls to unsafe string functions. These are deprecated and may be removed in the future. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel --- EmbeddedPkg/Ebl/Command.c | 2 +- EmbeddedPkg/Ebl/Dir.c | 4 ++-- EmbeddedPkg/Ebl/EfiDevice.c | 11 ++++++----- EmbeddedPkg/Ebl/Main.c | 8 ++++---- EmbeddedPkg/Ebl/Variable.c | 17 +++++++++++------ 5 files changed, 24 insertions(+), 18 deletions(-) diff --git a/EmbeddedPkg/Ebl/Command.c b/EmbeddedPkg/Ebl/Command.c index e75c6a2e5c32..4bc1f4df0ca0 100644 --- a/EmbeddedPkg/Ebl/Command.c +++ b/EmbeddedPkg/Ebl/Command.c @@ -614,7 +614,7 @@ OutputData ( UINTN Spaces = 0; CHAR8 Blanks[80]; - AsciiStrCpy (Blanks, mBlanks); + AsciiStrCpyS (Blanks, sizeof Blanks, mBlanks); for (EndAddress = Address + Length; Address < EndAddress; Offset += Line) { AsciiPrint ("%08x: ", Offset); for (Line = 0; (Line < 0x10) && (Address < EndAddress);) { diff --git a/EmbeddedPkg/Ebl/Dir.c b/EmbeddedPkg/Ebl/Dir.c index 36095b633019..8dd9d48ff6ac 100644 --- a/EmbeddedPkg/Ebl/Dir.c +++ b/EmbeddedPkg/Ebl/Dir.c @@ -116,7 +116,7 @@ EblDirCmd ( UnicodeFileName[0] = '\0'; MatchSubString = &UnicodeFileName[0]; if (Argc > 2) { - AsciiStrToUnicodeStr (Argv[2], UnicodeFileName); + AsciiStrToUnicodeStrS (Argv[2], UnicodeFileName, MAX_CMD_LINE); if (UnicodeFileName[0] == '*') { // Handle *Name substring matching MatchSubString = &UnicodeFileName[1]; @@ -231,7 +231,7 @@ EblDirCmd ( MatchSubString = NULL; UnicodeFileName[0] = '\0'; if (Argc > 2) { - AsciiStrToUnicodeStr (Argv[2], UnicodeFileName); + AsciiStrToUnicodeStrS (Argv[2], UnicodeFileName, MAX_CMD_LINE); if (UnicodeFileName[0] == '*') { MatchSubString = &UnicodeFileName[1]; } diff --git a/EmbeddedPkg/Ebl/EfiDevice.c b/EmbeddedPkg/Ebl/EfiDevice.c index ec9c331b7004..f6969e7b2b05 100644 --- a/EmbeddedPkg/Ebl/EfiDevice.c +++ b/EmbeddedPkg/Ebl/EfiDevice.c @@ -343,7 +343,7 @@ EblStartCmd ( ImageInfo->LoadOptionsSize = (UINT32)AsciiStrSize (Argv[2]); ImageInfo->LoadOptions = AllocatePool (ImageInfo->LoadOptionsSize); - AsciiStrCpy (ImageInfo->LoadOptions, Argv[2]); + AsciiStrCpyS (ImageInfo->LoadOptions, ImageInfo->LoadOptionsSize, Argv[2]); } // Transfer control to the EFI image we loaded with LoadImage() @@ -741,7 +741,7 @@ EblFileCopyCmd ( UINTN Size; UINTN Offset; UINTN Chunk = FILE_COPY_CHUNK; - UINTN FileNameLen; + UINTN FileNameLen, DestFileNameLen; CHAR8* DestFileName; CHAR8* SrcFileName; CHAR8* SrcPtr; @@ -786,9 +786,10 @@ EblFileCopyCmd ( } // Construct the destination filepath - DestFileName = (CHAR8*)AllocatePool (FileNameLen + AsciiStrLen (SrcFileName) + 1); - AsciiStrCpy (DestFileName, Argv[2]); - AsciiStrCat (DestFileName, SrcFileName); + DestFileNameLen = FileNameLen + AsciiStrLen (SrcFileName) + 1; + DestFileName = (CHAR8*)AllocatePool (DestFileNameLen); + AsciiStrCpyS (DestFileName, DestFileNameLen, Argv[2]); + AsciiStrCatS (DestFileName, DestFileNameLen, SrcFileName); } Source = EfiOpen(Argv[1], EFI_FILE_MODE_READ, 0); diff --git a/EmbeddedPkg/Ebl/Main.c b/EmbeddedPkg/Ebl/Main.c index 18b2878f69a1..4f04ae4afd33 100644 --- a/EmbeddedPkg/Ebl/Main.c +++ b/EmbeddedPkg/Ebl/Main.c @@ -88,7 +88,7 @@ SetCmdHistory ( } // Copy the new command line into the ring buffer - AsciiStrnCpy(&mCmdHistory[mCmdHistoryStart][0], Cmd, MAX_CMD_LINE); + AsciiStrnCpyS (&mCmdHistory[mCmdHistoryStart][0], MAX_CMD_LINE, Cmd, MAX_CMD_LINE); } // Reset the command history for the next up arrow press @@ -432,7 +432,7 @@ GetCmd ( } AsciiPrint (History); Index = AsciiStrLen (History); - AsciiStrnCpy (Cmd, History, CmdMaxSize); + AsciiStrnCpyS (Cmd, CmdMaxSize, History, CmdMaxSize); } else { Cmd[Index++] = Char; if (FixedPcdGetBool(PcdEmbeddedShellCharacterEcho) == TRUE) { @@ -644,14 +644,14 @@ EdkBootLoaderEntry ( Status = gRT->GetVariable(CommandLineVariableName, &VendorGuid, NULL, &CommandLineVariableSize, CommandLineVariable); if (!EFI_ERROR(Status)) { - UnicodeStrToAsciiStr(CommandLineVariable, CmdLine); + UnicodeStrToAsciiStrS (CommandLineVariable, CmdLine, CommandLineVariableSize); } FreePool(CommandLineVariable); } if (EFI_ERROR(Status)) { - AsciiStrCpy (CmdLine, (CHAR8 *)PcdGetPtr (PcdEmbeddedAutomaticBootCommand)); + AsciiStrCpyS (CmdLine, MAX_CMD_LINE, (CHAR8 *)PcdGetPtr (PcdEmbeddedAutomaticBootCommand)); } for (;;) { diff --git a/EmbeddedPkg/Ebl/Variable.c b/EmbeddedPkg/Ebl/Variable.c index f440c48f16dd..b94531300d07 100644 --- a/EmbeddedPkg/Ebl/Variable.c +++ b/EmbeddedPkg/Ebl/Variable.c @@ -29,6 +29,7 @@ EblGetCmd ( VOID* Value; CHAR8* AsciiVariableName = NULL; CHAR16* VariableName; + UINTN VariableNameLen; UINT32 Index; if (Argc == 1) { @@ -48,8 +49,9 @@ EblGetCmd ( AsciiPrint("Variable name is missing.\n"); return Status; } else { - VariableName = AllocatePool((AsciiStrLen (AsciiVariableName) + 1) * sizeof (CHAR16)); - AsciiStrToUnicodeStr (AsciiVariableName,VariableName); + VariableNameLen = (AsciiStrLen (AsciiVariableName) + 1) * sizeof (CHAR16); + VariableName = AllocatePool(VariableNameLen); + AsciiStrToUnicodeStrS (AsciiVariableName, VariableName, VariableNameLen); } // Try to get the variable size. @@ -93,6 +95,7 @@ EblSetCmd ( CHAR8* AsciiValue; UINT32 AsciiValueLength; CHAR16* VariableName; + UINTN VariableNameLen; UINT32 Index; UINT32 EscapedQuotes = 0; BOOLEAN Volatile = FALSE; @@ -125,8 +128,9 @@ EblSetCmd ( // // Convert VariableName into Unicode - VariableName = AllocatePool((AsciiStrLen (AsciiVariableSetting) + 1) * sizeof (CHAR16)); - AsciiStrToUnicodeStr (AsciiVariableSetting,VariableName); + VariableNameLen = (AsciiStrLen (AsciiVariableSetting) + 1) * sizeof (CHAR16); + VariableName = AllocatePool(VariableNameLen); + AsciiStrToUnicodeStrS (AsciiVariableSetting, VariableName, VariableNameLen); Status = gRT->SetVariable ( VariableName, @@ -170,8 +174,9 @@ EblSetCmd ( } // Convert VariableName into Unicode - VariableName = AllocatePool((AsciiStrLen (AsciiVariableName) + 1) * sizeof (CHAR16)); - AsciiStrToUnicodeStr (AsciiVariableName,VariableName); + VariableNameLen = (AsciiStrLen (AsciiVariableName) + 1) * sizeof (CHAR16); + VariableName = AllocatePool(VariableNameLen); + AsciiStrToUnicodeStrS (AsciiVariableName, VariableName, VariableNameLen); Status = gRT->SetVariable ( VariableName, -- 2.7.4