在使用execvp和while循环中的fork时发生无限循环

时间:2019-04-20 15:22:34

标签: c fork execvp

我正在尝试编写一个小的Shell程序,我希望它在调用另一个程序(通过execvp())后保持活动状态。我想要fork系统调用来“复制”该流程,从而创建发行(父)流程的几乎相同的副本(子代)。因此,当父级由于excevp()调用而终止时,子级将仍然活着。

在下面附加的代码中,我读取了用户输入(要执行的程序和参数),然后我希望父fork将执行该程序,而子fork将等待用户的下一次输入,但是在第一次插入输入(要执行的程序及其参数)时,该程序陷入无限循环。当子fork控制时,它将执行上一个程序,并且不等待新的输入(似乎在第一个fgets()之后,它不再等待用户输入)。

这是代码中有问题的部分:

#include <stdio.h>
#include <string.h>
#include <stdlib.h>
#include "LineParser.h"
#include <limits.h>
#include <unistd.h>

char buf[PATH_MAX];
char buf_user[2048];
int pid;

void execute(cmdLine* pCmdLine) {
   int pid = fork();
   if(pid != 0) {
     printf("I'm the parent.\n");
       if(execvp(pCmdLine->arguments[0], pCmdLine->arguments) == -1) {
         perror("execute failed");
         _exit(1);
       }
   } else {
     freeCmdLines(pCmdLine);        
   }
}

int main(int argc, char *argv[])
{  
    while(1) {
        getcwd(buf, PATH_MAX);
        printf("%s> ", buf);
        fgets(buf_user, 2048, stdin);
        fflush(stdin);
        if(strcmp(buf_user, "quit\n") == 0) {
            exit(0);
        }
        cmdLine* parsedCmd = parseCmdLines(buf_user);
        execute(parsedCmd);
    }
    return 0;
}

cmdLine结构:

#define MAX_ARGUMENTS 256

typedef struct cmdLine
{
    char *const arguments[MAX_ARGUMENTS]; /* command line arguments (arg 0 is the command) */
    int argCount;       /* number of arguments */
    char const *inputRedirect;  /* input redirection path. NULL if no input redirection */
    char const *outputRedirect; /* output redirection path. NULL if no output redirection */
    char blocking;  /* boolean indicating blocking/non-blocking */
    int idx;                /* index of current command in the chain of cmdLines (0 for the first) */
    struct cmdLine *next;   /* next cmdLine in chain */
} cmdLine;

/* Parses a given string to arguments and other indicators */
/* Returns NULL when there's nothing to parse */ 
/* When successful, returns a pointer to cmdLine (in case of a pipe, this will be the head of a linked list) */
cmdLine *parseCmdLines(const char *strLine);    /* Parse string line */

/* Releases all allocated memory for the chain (linked list) */
void freeCmdLines(cmdLine *pCmdLine);       /* Free parsed line */

/* Replaces arguments[num] with newString */
/* Returns 0 if num is out-of-range, otherwise - returns 1 */
int replaceCmdArg(cmdLine *pCmdLine, int num, const char *newString);

1 个答案:

答案 0 :(得分:1)

您不应在父母中执行execvp,而应在孩子中执行。

您的终端等待父级完成。在没有等待孩子完蛋的情况下完成父母工作,将产生zombie process

修改代码以在子代中执行该命令并在父代中等待其完成时,解决了我在测试时遇到的问题。

我建议您阅读 3个人等待