使用 strcat 的分段错误

Question

这是我的代码：

char *name, name_log="log-";

------从用户那里得到“名字”-----

strcat(name_log, name);
char ext[] = ".log";
strcat(name_log, ext);

我最终需要的是 name_log = "log-'name'.log" 但我得到一个分段错误错误:((。我做错了什么，我该如何解决它？谢谢

Answer 1

name_log指向一个静态的地方：“log-”，表示不能修改，而strcat()的第一个参数必须是可修改的。

尝试将 name_log 的类型 char* 更改为 char[]，例如

char[20] name_log = "log-";

Answer 2

一个完全不同的解决方案是这样的：

const char *prefix = "log-";
const char *suffix = ".log";
// There's a "char *name" somewhere
int size_needed;
char *result;

size_needed = snprintf(NULL, 0, "%s%s%s", prefix, name, suffix);
result = malloc(size_needed + 1);
snprintf(result, size_needed + 1, "%s%s%s", prefix, name, suffix);

// "result" now contains the desired string.

snprintf 的好处在于它返回如果有足够的空间它将写入的字符数。这可以通过预先测量要分配多少内存来使用，这使得复杂且容易出错的计算变得不必要。

如果你碰巧在一个带有asprintf 的系统上，那就更简单了：

char *result = NULL /* in case asprintf fails */;
asprintf(&result, "log-%s.log", name);
// "result" must be released with "free"

Answer 3

字符串文字被分配了固定数量的内存，通常在只读部分中，您需要使用缓冲区。

char buffer[64] = "log-";
strncat(buffer,".log",32);

附带说明，strcat 通常是不安全的，您需要使用一些东西来检查它使用的缓冲区的大小或限制它可以连接的内容，例如 strncat。

Answer 4

首先，如果 this 是您的代码：

char *name, name_log="log-";

那么name_log是一个char，不是一个char指针。

假设这是一个错字，你不能像这样附加到字符串文字。对字符串文字的修改是未定义的行为。

对于可变大小的字符串，就像user 看起来那样，可能最安全的选择是分配另一个足够大的字符串来保存结果，例如：

char *name, *name_log = "log-", *ext = ".log";
// Do something to allocate and populate name
char *buffer = malloc (strlen (name_log) + strlen (name) + strlen (ext) + 1);
if (buffer == NULL) {
    // Out of memory.
} else {
    strcpy (buffer, name_log);
    strcat (buffer, name);
    strcat (buffer, ext);
    // Do something with buffer.
    free (buffer);
}

malloc 确保您有足够的空间来安全地执行所有字符串操作、三个组件的足够字符以及一个空终止符。

Answer 5

你需要分配内存。您不能以这种方式添加到字符串，因为添加的字符串会进入尚未分配的内存。

你可以的

char[20] strarray;

strcat(strarray, "log-");
strcat(strarray, "abcd");