svn commit: r361142 - releng/11.4/contrib/llvm-project/llvm/lib/Support/Unix

Ed Maste emaste at FreeBSD.org
Sun May 17 21:29:45 UTC 2020


Author: emaste
Date: Sun May 17 21:29:45 2020
New Revision: 361142
URL: https://svnweb.freebsd.org/changeset/base/361142

Log:
  MF11 r360876: Merge commit 21e5e1724b75 from llvm git:
  
    getMainExecutable: Fix hand-rolled AT_EXECPATH for older FreeBSD
  
    Once we hit AT_NULL, we need to bail out of the loop; not just the
    enclosing switch.  This fixes basic usage (e.g. `cc --version`) when
    AT_EXECPATH isn't present on older branches (e.g. under
    qemu-user-static, at the moment), where we would previously run off
    the end of ::environ.
  
    Patch By: kevans
  
    Reviewed By: arichardson
  
  Approved by:	re (kib)

Modified:
  releng/11.4/contrib/llvm-project/llvm/lib/Support/Unix/Path.inc
Directory Properties:
  releng/11.4/   (props changed)
  releng/11.4/contrib/llvm-project/llvm/   (props changed)

Modified: releng/11.4/contrib/llvm-project/llvm/lib/Support/Unix/Path.inc
==============================================================================
--- releng/11.4/contrib/llvm-project/llvm/lib/Support/Unix/Path.inc	Sun May 17 20:37:15 2020	(r361141)
+++ releng/11.4/contrib/llvm-project/llvm/lib/Support/Unix/Path.inc	Sun May 17 21:29:45 2020	(r361142)
@@ -208,14 +208,9 @@ std::string getMainExecutable(const char *argv0, void 
   while (*p++ != 0)
     ;
   // Iterate through auxiliary vectors for AT_EXECPATH.
-  for (;;) {
-    switch (*(uintptr_t *)p++) {
-    case AT_EXECPATH:
+  for (; *(uintptr_t *)p != AT_NULL; p++) {
+    if (*(uintptr_t *)p++ == AT_EXECPATH)
       return *p;
-    case AT_NULL:
-      break;
-    }
-    p++;
   }
 #endif
   // Fall back to argv[0] if auxiliary vectors are not available.


More information about the svn-src-all mailing list