使用snprintf追加结构体char*成员时遇垃圾数据与指针错误的解决方法

首先得点明你踩的核心坑：你在用snprintf追加字符串时，把目标缓冲区result->value同时当成了输入参数——这属于C标准明确规定的未定义行为。snprintf在写入缓冲区的过程中会读取它，这会导致内存数据被混乱覆盖，出现垃圾值，甚至后续free时触发指针错误。另外你初始用sprintf(result->value,"%s","\0")其实没必要，malloc分配的内存是未初始化的，但直接把第一个字节设为'\0'会更高效。

下面给你几个简洁、安全且无需临时变量的解决方案，方便多处复用：

方案1：用strncat（最适合固定缓冲区场景） #

strncat天生就是为安全追加字符串设计的，它会自动处理末尾的'\0'，而且能严格控制不越界：

// 初始化value为空字符串（比sprintf高效太多）
result->value[0] = '\0';

// 循环内追加pointer指向的内容
strncat(result->value, pointer, VALUE_BUFFER - strlen(result->value) - 1);

解释：第三个参数是最多允许追加的字节数，我们用总缓冲区大小减去当前已用长度再减1（留位置给最后的'\0'），这样绝对不会触发缓冲区溢出，代码也极简。

方案2：snprintf的正确用法（必须避免输入输出重叠） #

如果你一定要用snprintf，那必须先计算当前字符串的长度，从缓冲区末尾开始写入，彻底避免输入输出重叠：

// 初始化
result->value[0] = '\0';

// 循环内追加
size_t current_len = strlen(result->value);
// 剩余可用空间 = 总大小 - 当前已用长度（包含'\0'）
snprintf(result->value + current_len, VALUE_BUFFER - current_len, "%s", pointer);

这里result->value + current_len直接指向当前字符串的末尾，snprintf从这个位置开始写入，输入（pointer）和输出（缓冲区末尾）完全不重叠，彻底规避未定义行为。

方案3：动态扩容（适合不确定最终长度的场景） #

如果固定的VALUE_BUFFER可能不够用，推荐用动态内存分配，每次追加前计算所需空间，自动扩容：

// 初始化时分配初始缓冲区
result->value = malloc(VALUE_BUFFER);
if (!result->value) { /* 务必处理内存分配失败的情况 */ }
result->value[0] = '\0';

// 循环内追加
size_t needed_len = strlen(result->value) + strlen(pointer) + 1;
if (needed_len > VALUE_BUFFER) {
    // 可以选择按需要的大小分配，或者每次翻倍（减少realloc次数）
    char *temp = realloc(result->value, needed_len);
    if (!temp) { /* 处理扩容失败 */ }
    result->value = temp;
    VALUE_BUFFER = needed_len; // 更新缓冲区大小标记
}
strcat(result->value, pointer); // 此时空间足够，strcat完全安全

额外注意事项 #

1. 所有malloc/realloc后一定要检查返回值，避免空指针操作；
2. 你的代码里存在笔误：my_print_func调用的是my_func，但定义的函数是myfunc（少了下划线），编译会报错，记得修正；
3. 用完props结构体后，要按顺序释放成员内存再释放结构体本身，避免内存泄漏：

static void my_print_func() {
    props *result = malloc(sizeof(props));
    if (myfunc(result)) { // 修正函数名
        printf("%d\t%s", result->ID, result->value);
    }
    // 释放内存顺序：先成员，再结构体
    free(result->name);
    free(result->value);
    free(result);
}

内容的提问来源于stack exchange，提问作者Aethalides