`execvp()` 似乎没有完成路径搜索

Question

TL;DR -- 什么样的事情可能会导致 execvp 调用无法完全发挥作用/无法正确搜索路径？

我正在构建一个带有一些生活质量功能的基本外壳，我随着时间的推移添加了一些功能，例如。历史，别名和完成。我在一个功能性外壳之上构建了这些功能，该外壳具有有效的 $PATH 搜索执行，例如输入“ls -la”会产生所需的行为。正如您可能想象的那样，我只是使用execvp 完成了这项工作。 （如果不清楚，这是用 C 写的）

我没有更改任何标记化逻辑，并确保文件名正确；特别是，在我将这些功能添加到我的 REPL 之前，execvp 正在产生所需的行为。 echo "hello" 仍然产生一个标记化的char **xyz 并且第一个标记确实是echo，以空终止，等等。也就是说，我的调用仍然看起来像，填充了变量，... execvp("echo", argv);，然后我调用perror，它应该只在出现问题时触发。但是，每次我只运行上述命令时，由于我已经添加了这些功能，它会返回 no such file or directory 的失败 --- 在我添加这些功能之前，行为是符合要求的。不过，我会注意到，运行 /bin/echo "hello" 会按预期运行。例如 WLOG。

我什至不确定应该从哪里开始查找错误，而且我的 Google-fu 几乎没有结果：有什么建议吗？

我最初会省略代码，因为它总共有几百行，而且 MWE 不会特别小，除了我希望保持这个通用性而不是对我的代码非常特殊，虽然我不确定什么是造成这种情况。我的存储库是公开的并且是最新的，我很高兴在这里发布任何代码。

编辑： 我知道我没有明确地编辑 PATH 变量等，但是这段代码是问题所在：

  // Grab $PATH from env
  char *pathvar = getenv("PATH");

  if (pathvar) {
    char *path;
    int i;

    // tokenize on colon to get paths
    // then use that immediately to 
    // scandir, and add everything in 
    // there to the completions system

    path = strtok(pathvar, ":");
    while (path) {
      struct dirent **fListTemp;
      int num_files = scandir(path, &fListTemp, NULL, alphasort);
      // only adding the names that are completely composed of 
      // lower case letters; completions are done using a naive
      // Trie Node structure that only supports lowercase letters 
      // for now... e.g. g++ does not work, and the '+' leads to 
      // a seg-fault. Same holds for . and ..
      for (i = 0; i < num_files; i++) {
        char *curr = fListTemp[i]->d_name;
        if (strcmp(curr, ".")==0 || strcmp(curr, "..")==0){
          continue;
        } else if (notalpha(curr)) {
          continue;
        } else {
          str_tolower(curr);
          tn_insert(completions, curr);
        }
      }
      for (i = 0; i < num_files; i++) {
        free(fListTemp[i]);
      }
      free (fListTemp);
      path = strtok(NULL, ":");
    }
  } else {
    fprintf(stderr, "{wsh @ init} -- $PATH variable could not be found?");
  }

Answer 1

注意

   The getenv() function returns a pointer to the value in the
   environment, or NULL if there is no match.

所以我的原始代码确实篡改了 PATH 变量。我很快想出的解决方案就是创建该字符串的副本并使用它来解析 PATH：

  // Grab $PATH from env
  char *pathvar = getenv("PATH");
  char *pathvar_cpy = strcpy(pathvar_cpy, pathvar);

  if (pathvar_cpy) {
    char *path;
    int i;
    path = strtok(pathvar_cpy, ":");
    while (path) {
      // Scan directory
      struct dirent **fListTemp;
      int num_files = scandir(path, &fListTemp, NULL, alphasort);

      for (i = 0; i < num_files; i++) {
        char *curr = fListTemp[i]->d_name;
        if (strcmp(curr, ".")==0 || strcmp(curr, "..")==0){
          continue;
        } else if (notalpha(curr)) {
          continue;
        } else {
          str_tolower(curr);
          tn_insert(completions, curr);
        }
      }
      for (i = 0; i < num_files; i++) {
        free(fListTemp[i]);
      }
      free (fListTemp);
      path = strtok(NULL, ":");
    }
  } else {
    fprintf(stderr, "{wsh @ init} -- $PATH variable could not be found?");
  }