Question

我正在尝试创建一个简单的程序，该程序在作为命令参数输入的文件位置上使用system（）调用cat。但是每次调用文件时我都会遇到分段错误（核心转储）。你能否检查一下为什么（我在程序中的任何地方都看不到我正在做内存以获得此错误！）。

#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;
}

Answer 1

您无法修改字符串文字，例如"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);

Answer 2

在strcat调用中，您尝试修改字符串文字“cat”，这是未定义的行为。 strcat的第一个参数应该是一个可以写入的缓冲区，而不是字符串文字。

Answer 3

您正在错误地使用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]);

简单命令行程序中的分段错误

3 个答案: