ошибка сегментации в простой программе командной строки

Question

ошибка сегментации в простой программе командной строки

0

Я пытаюсь создать простую программу, которая вызывает cat с system() в местоположении файла, введенном в качестве аргумента команды. Но я получаю ошибку сегментации (ядро сбрасывается) каждый раз, когда я вызываю файл. Не могли бы вы проверить, почему (я не вижу нигде в своей программе, я делаю что-то с памятью, чтобы получить эту ошибку!).

#include <stdlib.h>
#include <stdio.h>
#include <string.h>

int main(int argc, char *argv[])
{
    if(argc != 2)
    {
        printf("usage: %s filename", argv[0]);
    }
    else
    {
        printf("commad: %s", strcat("cat ", argv[1]));
        system(strcat("cat ", argv[1]));
    }
    return 0;
}

MedAli 28 фев. 2014, в 15:01

Источник

1

Поведение строкового литерала не определено.
hmjd 28 фев. 2014, в 13:39
0

не могли бы вы объяснить
Mohamed Ali JAMAOUI 28 фев. 2014, в 13:40
0

@MohamedAliJamaoui вам нужно предоставить буфер назначения для strcat
Eamonn McEvoy 28 фев. 2014, в 13:45

Показать ещё 1 комментарий

Теги:

c++

c

segmentation-fault

3 ответа

2

В ваших strcat вы пытаетесь изменить строковый литерал "cat", который является неопределенным поведением. Первым аргументом strcat должен быть буфер, в котором вы можете писать, а не строковый литерал.

Wojtek Surowka 28 фев. 2014, в 12:38

2

Вы используете strcat неправильно. Вам необходимо указать буфер назначения.

#include <stdlib.h>
#include <stdio.h>
#include <string.h>

int main(int argc, char *argv[])
{
        if(argc == 2) {
             char[20] c = "cat";
             strcat(c, argv[1]);
             printf("commad: %s", c);
             system(c);
        }
        else {
            printf("usage: %s filename", argv[0]);
        }

        return 0;
}

ИЛИ не конкатенировать

printf("commad: %s%s", "cat ", argv[1]);

Eamonn McEvoy 28 фев. 2014, в 10:53

0

+1 за обнаружение вашей ошибки
John U 28 фев. 2014, в 13:44

Ещё вопросы

Поведение строкового литерала не определено.
@MohamedAliJamaoui вам нужно предоставить буфер назначения для strcat

mockinterface · Accepted Answer · 2014-02-28T10-44-00.000Z

Вы не можете изменять строковые литералы, такие как "cat " они обычно хранятся в сегменте только для чтения в памяти, когда загружается ваш исполняемый файл, а когда вы пытаетесь его модифицировать, вы получите ошибку сегментации, о которой вы просили объяснить.

Вместо этого используйте std::string, это более идиоматический способ C++:

#include <stdlib.h>
#include <stdio.h>

#include <string>

int main(int argc, char *argv[]) {    
    if(argc != 2) {
        printf("usage: %s filename", argv[0]);
        return 0;
    } else {
        std::string command("cat ");
        command += argv[1];
        printf("command: %s", command.c_str());
        return system(command.c_str());
    }
}

Объект std::string будет распределять память динамически по мере необходимости, чтобы разместить дополнительные символы, которые вы добавляете к нему. Однако, если вы хотите продолжить использование C-строк, вам нужно будет управлять символьным буфером явно:

char *buffer = static_cast<char*>(malloc(5 + strlen(argv[1])));
strcpy(buffer, "cat ");
strcat(buffer, argv[1]);
printf("command: %s", buffer);
// ...
free(buffer);