Date: Thu, 29 Oct 2020 00:00:42 +0000 (UTC) From: Kyle Evans <kevans@FreeBSD.org> To: ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-branches@freebsd.org Subject: svn commit: r553571 - branches/2020Q4/devel/llvm10/files Message-ID: <202010290000.09T00gUZ042519@repo.freebsd.org>
next in thread | raw e-mail | index | archive | help
Author: kevans Date: Thu Oct 29 00:00:42 2020 New Revision: 553571 URL: https://svnweb.freebsd.org/changeset/ports/553571 Log: MFH: r553570 devel/llvm10: really fix the AT_EXECPATH fallback =( It turns out that I made not one but *two* mistakes in applying this patch from the version that I had sent upstream. This one is not an extra semicolon, but rather not dereferencing p... p may never hit AT_NULL, but *p certainly will. The problem was isolated and the fix identified + tested by Andrew Gierth. Reported by: Andrew Gierth <andrew_tao173 riddles org uk> Approved by: ports-secteam (blanket: runtime fix) Modified: branches/2020Q4/devel/llvm10/files/patch-lib_Support_Unix_Path.inc Directory Properties: branches/2020Q4/ (props changed) Modified: branches/2020Q4/devel/llvm10/files/patch-lib_Support_Unix_Path.inc ============================================================================== --- branches/2020Q4/devel/llvm10/files/patch-lib_Support_Unix_Path.inc Wed Oct 28 23:58:56 2020 (r553570) +++ branches/2020Q4/devel/llvm10/files/patch-lib_Support_Unix_Path.inc Thu Oct 29 00:00:42 2020 (r553571) @@ -7,7 +7,7 @@ - for (;;) { - switch (*(uintptr_t *)p++) { - case AT_EXECPATH: -+ for (; (uintptr_t *)p != AT_NULL; p++) { ++ for (; *(uintptr_t *)p != AT_NULL; p++) { + if (*(uintptr_t *)p++ == AT_EXECPATH) return *p; - case AT_NULL:
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202010290000.09T00gUZ042519>