我在实现自己的手工外壳时遇到了一些麻烦。我已经能够派生一个进程,并使用waitpid在前驱中运行它,但是当我尝试在后台运行诸如“ sleep 5&”之类的简单进程时,该进程似乎可以永远运行。 checkListJobs将确定该进程是否已完成运行,但永远不会停止。任何帮助将不胜感激。我认为错误出在我的“ foo”函数中。
void insertJob(int pid) {
printf("beginning job %d.\n", pid);
struct job *node = malloc(sizeof(struct job));
node->pid = pid;
node->next = NULL;
if(root == NULL) {
root = node;
} else {
node->next = root;
root = node;
}
}
void checkListJobs(int z) {
curr = root;
while(curr!=NULL) {
if(kill(curr->pid,0) != 0) {
if(prev==NULL) {
prev = curr;
root = curr;
} else {
prev->next = curr->next;
}
} else {
if(!z) printf("%d is still running.\n", curr->pid);
}
prev = curr;
curr = curr->next;
}
}
//code for child forking
void foo(char *cmd, char *argv[], int args) {
int bgFlag;
if(!strcmp(argv[args], "&")){
argv[args] = '\0';
bgFlag = 1;
}
int pid = fork();
int status = 0;
if(pid==0){
if(bgFlag) {
fclose(stdin); // close child's stdin
fopen("/dev/null", "r"); // open a new stdin that is always empty
}
execvp(cmd, argv);
// this should never be reached, unless there is an error
fprintf (stderr, "unknown command: %s\n", cmd);
exit(0);
} else {
if(!bgFlag) {
waitpid(pid, &status, 0);
} else {
insertJob(pid);
}
if (status != 0) {
fprintf (stderr, "error: %s exited with status code %d\n", cmd, status);
} else {
// cmd exec'd successfully
}
}
// this is the parent still, since the child always terminates from exec or exit
// continue being a shell...
}
最佳答案
您将需要为SIGCHLD安装信号处理程序,因为它将在子进程完成时告诉您的程序。收到SIGCHLD后,您应该再调用wait()(或PID值为-1的waitpid(),因为您不知道哪个孩子完成了,只是一个孩子完成了)。
编写处理程序的最安全方法是:
volatile sig_atomic_t sigchld;
int handle_child(int sig)
{
if (sig == SIGCHLD)
sigchld = 1;
}
然后在主循环中检查
sigchld
是否为1。如果是,则子进程结束,然后可以调用waidpid()
(使用-1的PID,因为您不知道哪个子进程结束了) )循环播放(请参见下文),因为可能有多个孩子同时结束。另外,如果有任何系统调用返回错误并且errno
是EINTR
,则它被信号中断,因此可以返回到主循环的顶部,或者检查sigchld
并相应地进行处理(并且不要忘记尽快将sigchld
重置为0)。for(;;)
{
int status;
pid_t child;
child = waitpid(-1,&status,WNOHANG);
if (child == -1)
{
if (errno == ECHILD) break; /* no more children */
/* error, handle how you wish */
}
/* handle the return status of the child */
}
sigchld = 0;
您可以从信号处理程序中调用
waitpid()
(POSIX表示这样做是安全的),但实际上您不应在信号处理程序中执行任何其他操作,因为这可能会导致非常细微的错误(例如,在调用过程中引发了SIGCHLD)到malloc()
---信号处理程序中导致调用malloc()
的任何代码都会导致非常讨厌的问题,这就是为什么我建议在信号处理程序中设置标志的原因---您在信号处理中做的事情越少处理程序)。关于c - 手工 shell 背景问题中的子过程,我们在Stack Overflow上找到一个类似的问题:https://stackoverflow.com/questions/15081658/