基于libcurl示例封装函数时内存分配引发段错误求助

问题背景 #

参考libcurl官方示例实现的独立代码运行正常，封装为可复用的curlGet函数后出现内存异常：
函数声明：

extern int curlGet(char *addr, char *response);

通过memcpy(response,chunk.memory,chunk.size+1);将curl响应返回给调用方。

主函数调用代码：

int main(void){
    char *tmp=malloc(1);
    curlGet("www.google.com",tmp);
    printf("%s\n",tmp);
    //free(tmp);
    return 0;
};

出现的问题：

• addr参数正常，但tmp引发段错误（core dumped）
• 取消注释free(tmp)后，触发堆错误：

  free(): invalid next size (fast)
  Aborted (core dumped)
  

补充的curlGet部分实现代码：

if(res!=CURLE_OK){
    fprintf(stderr, "curl_easy_perform() failed: %s\n",
        curl_easy_strerror(res));
    return -1;
}
curl_easy_cleanup(curl_handle);
memcpy(response,chunk.memory,chunk.size+1);
free(chunk.string);
return 0;

问题根源 #

1. 内存空间严重不足：你仅为tmp分配了1字节内存，但目标网址的响应内容远大于这个尺寸，memcpy会直接越界写入，破坏堆内存的结构，进而触发段错误或free时的堆完整性校验失败。
2. 接口设计不合理：当前curlGet要求调用方预先分配足够内存，但调用方无法提前知晓响应的实际大小，这种设计从根本上就存在缺陷。

修正方案 #

最优方案是修改curlGet的接口，让函数内部负责分配响应内存，并将内存指针和大小交付给调用方，由调用方在使用完成后自行释放。

修改后的函数声明 #

extern int curlGet(const char *addr, char **response, size_t *response_size);

核心实现调整 #

// 省略libcurl初始化、设置write_callback等前置代码
if(res != CURLE_OK){
    fprintf(stderr, "curl_easy_perform() failed: %s\n", curl_easy_strerror(res));
    // 请求失败时清理已分配的chunk内存
    if(chunk.memory) free(chunk.memory);
    return -1;
}

// 将chunk中的内存所有权转移给调用方
*response = chunk.memory;
*response_size = chunk.size;
chunk.memory = NULL; // 避免后续代码误释放该内存

curl_easy_cleanup(curl_handle);
return 0;

主函数调用示例 #

int main(void){
    char *tmp = NULL;
    size_t tmp_size = 0;
    if(curlGet("www.google.com", &tmp, &tmp_size) == 0){
        printf("响应长度：%zu\n内容：%s\n", tmp_size, tmp);
        free(tmp); // 此时可以安全释放内存
    }
    return 0;
};

额外说明 #

如果一定要保留原接口，必须先通过HEAD请求获取响应的Content-Length，再根据该值分配内存，但这种方式并不可靠——很多服务不会返回Content-Length，或使用分块编码传输响应，无法提前获取准确的响应大小。

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