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

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]);