From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 2012) id 3F58038532F9; Fri, 25 Nov 2022 14:32:10 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 3F58038532F9 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1669386730; bh=eETHRzyFg3tHj9TvVRZrCR41CBeU7NBqM84c1FXmDWQ=; h=From:To:Subject:Date:From; b=iVOyBOa4GzIfS9dqOsMrwko7FB1lCQFJYud/8NUfCs02nLvVf1hWdLUvzgunRi/B/ lOa8ozEPxcBGRPiny5QjNWipvEY1M9sQ//A9c7AkoooJQS+1GnXQDMnINhRJQ35jnr /RVFu3QfbcvcS65uW5NMVPI6VggXRxnLiwSfjf7I= Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: Arjun Shankar To: glibc-cvs@sourceware.org Subject: [glibc/release/2.34/master] elf: Fix wrong fscanf usage on tst-pldd X-Act-Checkin: glibc X-Git-Author: Adhemerval Zanella X-Git-Refname: refs/heads/release/2.34/master X-Git-Oldrev: 42b9d7def8fd7d26f1926cbc801923aa2c8a386a X-Git-Newrev: 405b8ae13540e9fd614df614e3361ebf9abd14cf Message-Id: <20221125143210.3F58038532F9@sourceware.org> Date: Fri, 25 Nov 2022 14:32:10 +0000 (GMT) List-Id: https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=405b8ae13540e9fd614df614e3361ebf9abd14cf commit 405b8ae13540e9fd614df614e3361ebf9abd14cf Author: Adhemerval Zanella Date: Tue Jul 5 12:58:40 2022 -0300 elf: Fix wrong fscanf usage on tst-pldd The fix done b2cd93fce666fdc8c9a5c64af2741a8a6940ac99 does not really work since macro strification does not expand the sizeof nor the arithmetic operation. Checked on x86_64-linux-gnu. (cherry picked from commit c353689e49e72f3aafa1a9e68d4f7a4f33a79cbe) Diff: --- elf/tst-pldd.c | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/elf/tst-pldd.c b/elf/tst-pldd.c index 6e4174cbf9..d3ecb99149 100644 --- a/elf/tst-pldd.c +++ b/elf/tst-pldd.c @@ -108,15 +108,16 @@ do_test (void) loader and libc. */ { pid_t pid; - char buffer[512]; -#define STRINPUT(size) "%" # size "s" +#define BUFFERLEN 511 + char buffer[BUFFERLEN + 1]; +#define STRINPUT(size) XSTRINPUT(size) +#define XSTRINPUT(size) "%" # size "s" FILE *out = fmemopen (pldd.out.buffer, pldd.out.length, "r"); TEST_VERIFY (out != NULL); /* First line is in the form of : */ - TEST_COMPARE (fscanf (out, "%u: " STRINPUT (sizeof (buffer) - 1), &pid, - buffer), 2); + TEST_COMPARE (fscanf (out, "%u: " STRINPUT (BUFFERLEN), &pid, buffer), 2); TEST_COMPARE (pid, *target_pid_ptr); TEST_COMPARE (strcmp (basename (buffer), "tst-pldd"), 0);