Ребенок не заканчивается правильно в вилке - PullRequest
Новая
       25

Ребенок не заканчивается правильно в вилке

0 голосов
/ 17 апреля 2011

Я пишу программу для класса, который представляет собой небольшую оболочку.Пользователь вводит команду, а код выполняет ее, используя функцию exec().

Мне нужно иметь разветвление в процессе, чтобы вся работа выполнялась в дочернем процессе.Единственная проблема заключается в том, что ребенок не завершит работу должным образом и не выполнит команду.Когда я запускаю код без форка, он отлично выполняет команды.

Проблема, похоже, исходит от того, где я создаю строку для использования в вызове execv.Это строка кода, где я звоню strcpy.Если я это прокомментирую, все будет хорошо.Я также попытался изменить его на strncat с той же проблемой.Я понятия не имею, что вызывает это, и приветствую любую помощь. 

#include <sys/wait.h>
#include <vector>
#include <sstream>
#include <cstdlib>
#include <stdio.h>
#include <iostream>
#include <string.h>
#include <unistd.h>

using namespace std;

string *tokenize(string line);
void setCommand(string *ary);

string command;
static int argument_length;

int main() {
    string argument;
    cout << "Please enter a unix command:\n";
    getline(cin, argument);
    string *ary = tokenize(argument);

    //begin fork process
    pid_t pID = fork();
    if (pID == 0) { // child
        setCommand(ary);

        char *full_command[argument_length];
        for (int i = 0; i <= argument_length; i++) {
            if (i == 0) {
                full_command[i] = (char *) command.c_str();
                //  cout<<"full_command " <<i << " = "<<full_command[i]<<endl;
            } else if (i == argument_length) {
                full_command[i] = (char *) 0;
            } else {
                full_command[i] = (char *) ary[i].c_str();
            //  cout<<"full_command " <<i << " = "<<full_command[i]<<endl;
            }
        }    

        char* arg1;
        const char *tmpStr=command.c_str();        
        strcpy(arg1, tmpStr);
        execv((const char*) arg1, full_command);
        cout<<"I'm the child"<<endl;
    } else if (pID < 0) { //error
        cout<<"Could not fork"<<endl;
    } else { //Parent
        int childExitStatus;
        pid_t wpID = waitpid(pID, &childExitStatus, WCONTINUED);
        cout<<"wPID = "<< wpID<<endl;
        if(WIFEXITED(childExitStatus))
            cout<<"Completed "<<ary[0]<<endl;
        else
            cout<<"Could not terminate child properly."<<WEXITSTATUS(childExitStatus)<<endl;
    }

    // cout<<"Command = "<<command<<endl;
    return 0;
}

string *tokenize(string line) //splits lines of text into seperate words
{
    int counter = 0;
    string tmp = "";
    istringstream first_ss(line, istringstream::in);
    istringstream second_ss(line, istringstream::in);

    while (first_ss >> tmp) {
        counter++;
    }

    argument_length = counter;
    string *ary = new string[counter];
    int i = 0;
    while (second_ss >> tmp) {
        ary[i] = tmp;
        i++;
    }

    return ary;
}

void setCommand(string *ary) {
    command = "/bin/" + ary[0];

// codeblock paste stops here

1 Ответ

2 голосов
/ 17 апреля 2011

Вы сказали:

Это строка кода, где я звоню зЬгср.

Вы не выделили памяти для хранения вашей строки. Первым параметром strcpy является указатель назначения, и вы используете неинициализированное значение для этого указателя. Со страницы руководства strcpy:

char * strcpy (char * s1, const char * s2);

Функции stpcpy () и strcpy () копируют строку s2 в s1 (включая завершающий символ `\ 0 ').

Могут быть и другие проблемы, но это первое, на что я обратил внимание.

...