Задача на exec

@firefox123 · Регистрация: 17.11.2013

Author24 — интернет-сервис помощи студентам

Здравствуйте! Никак не получается отладить программу, после вызова exec ничего не происходит. Прошу помочь.

Вкратце, что она должна делать: пользователь вводит имя программы (quit - выход), затем число аргументов и сами аргументы. После этого запускает заданную программу с заданным набором параметров. После этого все начинается заново.

C

#include <unistd.h>
#include <stdio.h>
#include <sys/systypes.h>
#include <stdlib.h>
 
int main() {
    int i = 0;
    char** argv = malloc(sizeof(char*)*100); // пусть можно задать максимум 100 аргументов
    for (int i = 0; i < 100; ++i) {
        argv[i] = malloc(sizeof(char)*100); // пусть аргумент имеет не более 100 символов
    }
    while(1){
        char * str = malloc(sizeof(char)*100); // пусть имя программы имеет не более 100 символов
        printf("Имя программы: ");
        scanf("%s", str);
        if (strcmp(str, "quit") == 0){
            printf("Пока!");
            break;
        }
        int p = 0;
        printf("Число параметров: ");
        scanf("%d", &p);
        if (p < 0) continue;
        for (int i = 0; i < p; ++i) {
            scanf("%s", argv[i]);
        }
        argv[p] = NULL;
        pid_t pid = fork();
        if (pid < 0){
            perror("fork error");
            continue;
        }
        if (pid == 0){
            execv(str, argv);
            break;
        }
        free(str);
        sleep(2);
    }
    for (int i = 0; i < 100; ++i) {
        free(argv[i]);
    }
    free(argv);
 
 
    return 0;
}

@retmas · 09.10.2015, 00:43

argv должно содержать первым элементом(т.е. argv[0]) путь к программе, а только потом - аргументы.

@firefox123 · 11.10.2015, 16:00 **[ТС]**

В argv[0] вставил str (название программы), в argv[1] ... argv[p] аргументы, в argv[p+1] - NULL. Не помогло.

Добавлено через 24 минуты
Сейчас примерно вот что есть

C

#include <unistd.h>
#include <stdio.h>
#include <sys/systypes.h>
#include <stdlib.h>
 
int main() {
    int i = 0;
    char** argv = malloc(sizeof(char*)*100); // пусть можно задать максимум 100 аргументов
    for (i = 0; i < 100; ++i) 
        argv[i] = malloc(sizeof(char)*100); // пусть аргумент имеет не более 100 символов
    
    while(1) {
        char * str = malloc(sizeof(char)*100); // пусть имя программы имеет не более 100 символов
        printf("Имя программы: ");
        scanf("%s", str);
        if (strcmp(str, "quit") == 0){
            printf("Пока!");
            break;
        }
        int p = 0;
        printf("Число параметров: ");
        scanf("%d", &p);
        if (p < 0) continue;
        argv[0] = str;
        for (i = 1; i <= p; ++i) {
            scanf("%s", argv[i]);
        }
        argv[p+1] = NULL;
        pid_t pid = fork();
        if (pid < 0){
            perror("fork error");
            continue;
        }
        if (pid == 0){
            execv(str, argv);
            break;
        }
        free(str);
        sleep(2);
    }
    for (i = 0; i < 100; ++i) {
        free(argv[i]);
    }
    free(argv); 
 
    return 0;
}

@retmas · 11.10.2015, 16:11

перый аргумент для execv - путь, а не просто имя файла

Добавлено через 59 секунд
либо показывай вывод(ошибки)

Добавлено через 1 минуту
в строке 24 копируй а не присваивай. иначе удалишь память дважды

Добавлено через 8 минут
имеет смысл дожидаться завершения дочернего процесса (wait вместо sleep, например). либо задать флаг SA_NOCLDWAIT для SIGCHLD

@firefox123 · 11.10.2015, 16:16 **[ТС]**

Спасибо, вроде заработало.

//в строке 24 копируй а не присваивай. иначе удалишь память дважды
Си подзабыл, как это здесь делается?

@retmas · 11.10.2015, 16:20

str(n)cpy

@retmas Жарю без масла 867 / 749 / 225 Регистрация: 13.01.2012 Сообщений: 1,702
	09.10.2015, 00:43	2
	argv должно содержать первым элементом(т.е. argv[0]) путь к программе, а только потом - аргументы. 1

@retmas Жарю без масла 867 / 749 / 225 Регистрация: 13.01.2012 Сообщений: 1,702
	11.10.2015, 16:11	4
	перый аргумент для execv - путь, а не просто имя файла Добавлено через 59 секунд либо показывай вывод(ошибки) Добавлено через 1 минуту в строке 24 копируй а не присваивай. иначе удалишь память дважды Добавлено через 8 минут имеет смысл дожидаться завершения дочернего процесса (wait вместо sleep, например). либо задать флаг SA_NOCLDWAIT для SIGCHLD 1

@firefox123 0 / 0 / 0 Регистрация: 17.11.2013 Сообщений: 39
	11.10.2015, 16:16 [ТС]	5
	Спасибо, вроде заработало. //в строке 24 копируй а не присваивай. иначе удалишь память дважды Си подзабыл, как это здесь делается? 0

@retmas Жарю без масла 867 / 749 / 225 Регистрация: 13.01.2012 Сообщений: 1,702
	11.10.2015, 16:20	6
	str(n)cpy 0