From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 2153) id E2F713A4781A; Thu, 17 Sep 2020 17:16:18 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org E2F713A4781A DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org; s=default; t=1600362978; bh=CBoxbJbz8IS/6lCBQ0sEKpMf5fpsHkBo5+qfkkyk3Ic=; h=From:To:Subject:Date:From; b=W7NOsrgktmmkNdLgWooalFMTPMHAqIYTLjiAX1fIP+SRSibFT5UD4QWJgcXhgvB7o tU1CJS6rJphd7rhCvWjmaCJRQQsGrPAFj+N1k+434AMG1iO/pAcnrXS9/Yd86kDRTd cIpAK7pENoNC6fdm3vRpq9TJinDimu9iJXV16BAY= Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: Jakub Jelinek To: gcc-cvs@gcc.gnu.org Subject: [gcc(refs/vendors/redhat/heads/gcc-8-branch)] AArch64: Fix hwasan failure in readline. X-Act-Checkin: gcc X-Git-Author: Tamar Christina X-Git-Refname: refs/vendors/redhat/heads/gcc-8-branch X-Git-Oldrev: 6af59c313bdbe515c0a89a1498d18d2ab1cf0e85 X-Git-Newrev: 97073a931a7b5659a51f9988ccb3f15a555f5f0a Message-Id: <20200917171618.E2F713A4781A@sourceware.org> Date: Thu, 17 Sep 2020 17:16:18 +0000 (GMT) X-BeenThere: gcc-cvs@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gcc-cvs mailing list List-Unsubscribe: , List-Archive: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 17 Sep 2020 17:16:19 -0000 https://gcc.gnu.org/g:97073a931a7b5659a51f9988ccb3f15a555f5f0a commit 97073a931a7b5659a51f9988ccb3f15a555f5f0a Author: Tamar Christina Date: Mon Aug 3 12:03:17 2020 +0100 AArch64: Fix hwasan failure in readline. My previous fix added an unchecked call to fgets in the new function readline. fgets can fail when there's an error reading the file in which case it returns NULL. It also returns NULL when the next character is EOF. The EOF case is already covered by the existing code but the error case isn't. This fixes it by returning the empty string on error. Also I now use strnlen instead of strlen to make sure we never read outside the buffer. This was flagged by Matthew Malcomson during his hwasan work. gcc/ChangeLog: * config/aarch64/driver-aarch64.c (readline): Check return value fgets. (cherry picked from commit 341573406b392f4d57e052ce22f80e85a7c479e9) Diff: --- gcc/config/aarch64/driver-aarch64.c | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/gcc/config/aarch64/driver-aarch64.c b/gcc/config/aarch64/driver-aarch64.c index 6dc7aa6d3be..4c1e42edff8 100644 --- a/gcc/config/aarch64/driver-aarch64.c +++ b/gcc/config/aarch64/driver-aarch64.c @@ -194,10 +194,16 @@ readline (FILE *f) size += buf_size; buf = (char*) xrealloc (buf, size); gcc_assert (buf); - fgets (buf + last, buf_size, f); + /* If fgets fails it returns NULL, but if it reaches EOF + with 0 characters read it also returns EOF. However + the condition on the loop would have broken out of the + loop in that case, and if we are in the first iteration + then the empty string is the correct thing to return. */ + if (!fgets (buf + last, buf_size, f)) + return std::string (); /* If we're not at the end of the line then override the \0 added by fgets. */ - last = strlen (buf) - 1; + last = strnlen (buf, size) - 1; } while (!feof (f) && buf[last] != '\n');