about summary refs log tree commit diff
path: root/posix/execvp.c
diff options
context:
space:
mode:
Diffstat (limited to 'posix/execvp.c')
-rw-r--r--posix/execvp.c43
1 files changed, 30 insertions, 13 deletions
diff --git a/posix/execvp.c b/posix/execvp.c
index 76a9ba62a3..a1fcb791fa 100644
--- a/posix/execvp.c
+++ b/posix/execvp.c
@@ -30,6 +30,8 @@ execvp (file, argv)
      const char *file;
      char *const argv[];
 {
+  int got_eacces = 0;
+
   void execute (const char *file, char *const argv[])
     {
       execv (file, argv);
@@ -39,19 +41,25 @@ execvp (file, argv)
 	  /* The file is accessible but it is not an executable file.
 	     Invoke the shell to interpret it as a script.  */
 
-	  int argc;
-	  char **new_argv;
-
 	  /* Count the arguments.  */
-	  for (argc = 0; argv[argc++];);
+	  int argc = 0;
+	  while (argv[argc++])
+	    ;
 
 	  /* Construct an argument list for the shell.  */
-	  new_argv = __alloca ((argc + 1) * sizeof (char *));
-	  for (new_argv[0] = _PATH_BSHELL; argc > 0; --argc)
-	    new_argv[argc] = argv[argc - 1];
-
-	  /* Execute the shell.  */
-	  execv (new_argv[0], new_argv);
+	  {
+	    char *new_argv[argc + 1];
+	    new_argv[0] = (char *) _PATH_BSHELL;
+	    new_argv[1] = (char *) file;
+	    while (argc > 1)
+	      {
+		new_argv[argc] = argv[argc - 1];
+		--argc;
+	      }
+
+	    /* Execute the shell.  */
+	    execv (new_argv[0], new_argv);
+	  }
 	}
     }
 
@@ -102,8 +110,12 @@ execvp (file, argv)
 
 	  switch (errno)
 	    {
-	    case ENOENT:
 	    case EACCES:
+	      /* Record the we got a `Permission denied' error.  If we end
+		 up finding no executable we can use, we want to diagnose
+		 that we did find one but were denied access.  */
+	      got_eacces = 1;
+	    case ENOENT:
 	      /* Those errors indicate the file is missing or not executable
 		 by us, in which case we want to just try the next path
 		 directory.  */
@@ -119,7 +131,12 @@ execvp (file, argv)
       while (*p++ != '\0');
     }
 
-  /* We tried every element and none of them worked.
-     Return the error from the last attempt (probably ENOENT).  */
+  /* We tried every element and none of them worked.  */
+
+  if (got_eacces)
+    /* At least one failure was due to permissions, so report that error.  */
+    errno = EACCES;
+
+  /* Return the error from the last attempt (probably ENOENT).  */
   return -1;
 }