WAIT 不等待孩子

Question

#include <stdio.h>
#include <sys/types.h>
#include <unistd.h>
#include <assert.h>
#include <stdlib.h>
#include <sys/wait.h>
#include <string.h>

char *getdir()  /*find working directory*/
{
  char *buffer;/*buffer is going to be used in getcwd function to get the current directory*/
  char *path_buffer;//path_buffer is going to contain the directory//
  long maxsize = pathconf(".", _PC_PATH_MAX);/* we set maxsize as the maximum pathname length*/
  if((buffer=(char*) malloc((size_t)maxsize))!=NULL)
    {
      path_buffer=getcwd(buffer,maxsize); /*get the current directory*/
      printf("\nMy working directory = %s",path_buffer);
      return path_buffer;
    }
  else{
    exit(-1);
  }
}

char * getcmline() /*get command from stdin by the user*/
{
  int bytes_read;
  int nchars=200;/*max possible number for the input of the user*/
  int nbytes=(sizeof(char))*nchars; /*size of chars in bytes*/
  char *line=(char*) malloc(nbytes+1);
  bytes_read=getline(line,&nbytes,stdin);/*read line from stdin*/
  if(bytes_read == -1){
    printf("Read line error");
    exit(-1);
  } /*error handling for bytes_read*/
  else{
    if(line[strlen(line)-1]=='\n')
      {
        line[strlen(line)-1]='\0'; /*change new line character in the end of the line of stdin*/
      }
  }
  return line;
}

int main(void)
{
  pid_t pid,child_pid;
  int rv=0;
  char* exit_string="exit";
  char *path_buffer=NULL;
  int nchars=200;
  int nbytes=(sizeof(char))*nchars;
  char *line=malloc(nbytes+1);
  char *commands[2];
  while(1){
    switch(pid = fork())
      {
      case -1:
        perror("fork"); /* something went wrong */
        exit(1);
      case 0:
        printf(" CHILD: This is the child process!\n");
        child_pid=getpid();
        printf(" CHILD: My PID is %d\n", child_pid);
        path_buffer=getdir();/*get the directory path*/
        line=getcmline();/*get a command by the user*/
        if(strcmp(line,exit_string)==0)
          {
            rv=3;
            exit(rv);
          }
        commands[0]=line;
        commands[1]=NULL;
        execvp(commands[0],commands);
        perror("Execution error");
        exit(-1);



      default:
        waitpid(-1, &rv, 0);
        if(WIFEXITED(rv)){
          printf("Child exited normally and child's exit status is: %d\n", WEXITSTATUS(rv));
          if((WEXITSTATUS(rv))==3){
            exit(1);
          }
        }
      }
  }
  return 0;
}

我在getline、rv 中进行了更改，并为execvp 创建了一个适当的变量。但是现在发生的错误是在我输入例如“ls”之后。它说：

执行错误：没有这样的文件或目录。

再次感谢您的帮助和我缺乏的知识。

错误似乎在行变量中，但我无法理解问题所在！

Char Array and getline in C

这里给出了解决方案。它应该是 line[(strlen)-1] 而不是 line[(strlen-1)]

Answer 1

父母永远不会得到孩子的 pid，因此 waitpid 不会等待它。 waitpid(-1,...) 将等待所有子进程完成后再继续。

我还建议缩进 case 并以 break 结束每个 case 语句；

Answer 2

所以，对于警告：你知道这些数字指的是什么行，我不得不猜测：你可以让这更容易！

int rv=NULL;

初始化从没有强制转换的指针生成整数 [默认启用] 第 42 行

NULL 是一个指针值，rv 是一个整数。如果您是这个意思，请将其设置为 0。

execvp(line[0],line);

传递“execv”的参数 1 使指针从没有强制转换的整数 [默认启用] 第 64 行 从不兼容的指针类型[默认启用]第 64 行传递“execv”的参数 2

如果您调用execv，则您正在编译的代码不是您发布的。然而...line 是char *，所以line[0] 是char。 execv 或 execvp 的第一个参数是 const char *。

execv 试图完成什么？它需要一个程序名来运行（第一个参数，const char *）和一个参数数组（这些将形成argv[]数组传递给新程序的main函数，数组中的最后一项必须为NULL） .

char *line=(char*) malloc(nbytes+1);
bytes_read=getline(&line,&nbytes,stdin);/*read line from stdin*/

从不兼容的指针类型[默认启用]第 29 行传递“getline”的参数 2

getline 需要缓冲区的地址才能将字符放入... line 是这样的地址。但是，&line 是 指针的地址。只需取出&。

---

您的代码中存在更多问题，但这些警告都有一些共同点：

• 您似乎不了解函数需要什么类型，
• 你的变量是什么类型，
• 您实际传递的是什么类型，
• 或者当编译器抱怨你的类型不匹配时是什么意思

您需要在开始尝试编写多进程程序之前了解这一点。用一些简单的东西来学习它，学习阅读文档，并理解编译器警告。

Answer 3

default:
waitpid(child_pid, &status, 0);
//wait(&rv);
printf("PARENT:My child's exit status is: %d\n", WEXITSTATUS(rv));
if((WEXITSTATUS(rv))==3){exit(1);}

您将未初始化的变量child_pid 传递给waitpid，该变量仅在子进程中设置。你应该通过pid。使用未初始化的变量会导致未定义的行为，因此无法预测会发生什么。你也可以用另一个未初始化的变量rv 调用WEXITSTATUS，你应该在那里传递status。

孩子在这里什么都不做，因为

path_buffer=getcwd(buffer,sizeof(buffer));

传递给getcwd 的size 参数是char* 的大小，通常是4 或8 个字节，对于大多数目录的绝对路径来说太短了，path_buffer is set toNULLand the child immediately exits. You should pass the size of the buffer,maxsize` 那里。