C语言替换文件扩展名时复制char**到char*出现段错误求助

核心问题 #

你的代码触发段错误的主要原因是错误修改了malloc分配的内存起始指针，同时循环逻辑存在隐患：

1. 指针越界与非法释放：在replace_ext函数中，直接对*buff执行*buff++操作，导致main里的str2不再指向malloc返回的内存起始地址。后续free(str2)会释放非法地址，触发段错误；printf也无法正确读取字符串内容。
2. 循环逻辑不严谨：使用x != dot_pos作为循环条件，若dot_pos超出字符串长度会引发无限循环；且原逻辑在替换扩展名后没有正确终止流程（当前测试用例未触发，但存在风险）。

修复后的完整代码 #

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

int get_dot_pos(const char* str, int* size)
{
    int npos = -1; // 默认返回-1表示未找到点
    int n = 0;
    while (*str != '\0')
    {
        if (*str == '.')
            npos = n;
        str++;
        n++;
    }
    *size = n;
    return npos;
}

int replace_ext(const char* source, int source_size, int dot_pos, const char* new_ext, char** buff)
{
    // 计算新扩展名长度
    int new_ext_len = 0;
    while (new_ext[new_ext_len] != '\0')
        new_ext_len++;

    // 分配足够内存：原字符串前dot_pos个字符 + 新扩展名 + 终止符
    *buff = malloc(dot_pos + new_ext_len + 1);
    if (*buff == NULL) // 检查内存分配是否成功
        return -1;

    char* temp = *buff; // 使用临时指针操作，避免修改原指针指向
    int x = 0;

    // 复制扩展名之前的字符
    while (x < dot_pos)
    {
        *temp = *source;
        temp++;
        source++;
        x++;
    }

    // 复制新扩展名
    while (*new_ext != '\0')
    {
        *temp = *new_ext;
        temp++;
        new_ext++;
        x++;
    }

    // 添加字符串终止符
    *temp = '\0';
    return x;
}

int main()
{
    const char* str1 = "/bar/foo/buz/rolitas.mp3";
    int size = 0;
    int npos = get_dot_pos(str1, &size);

    // 处理无扩展名的情况
    if (npos == -1)
    {
        printf("原字符串未包含扩展名\n");
        return 1;
    }

    char* str2;
    int result = replace_ext(str1, size, npos + 1, "ogg", &str2);
    if (result == -1)
    {
        printf("内存分配失败\n");
        return 1;
    }

    printf("Before: %s\n", str1);
    printf("After: %s\n", str2);
    free(str2);
    return 0;
}

关键修复点说明 #

• 保护起始指针：新增临时指针temp操作分配的内存，*buff（即main中的str2）始终指向malloc返回的起始地址，确保free操作合法。
• 简化循环逻辑：拆分复制流程为“复制前缀”+“复制新扩展名”，逻辑更清晰，避免无限循环风险。
• 增强健壮性：添加malloc失败检查、无扩展名场景处理，提升程序容错能力。
• 优化索引逻辑：get_dot_pos返回-1表示未找到点，方便后续错误判断。

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