C语言HTTP服务器图片无法显示问题求助（附相关代码）

你遇到的图片无法渲染问题，核心是HTTP响应格式错误和代码中的几处细节问题，下面是具体排查和修复方案：

核心错误点分析 #

1. HTTP响应头部与正文分隔符缺失 #

HTTP协议要求响应头部和正文之间必须用两个连续的\r\n分隔。当前图片处理的代码中，生成的HTTP头部末尾只有一个\r\n，导致浏览器无法正确识别正文起始位置，解析图片数据出错：

// 错误写法：末尾只有一个\r\n
snprintf(http_header,256,"HTTP/1.1 200 OK\r\nContent-Type: image/jpeg\r\nContent-Length: %llu\r\n",fdata->length);

2. 内存泄漏问题 #

read_file函数返回的file_data结构体没有释放，长期运行会导致内存耗尽：

// 原代码没有释放fdata
fdata = read_file("image.jpg");
// ... 使用后没有free

3. 无效且危险的调试代码 #

这段未初始化的buff写入文件的代码完全多余，会写入随机垃圾数据，毫无意义，应该删除：

uint8_t buff[fdata->length];
FILE *wbf = fopen("test.jpg","wb");
fwrite(buff,fdata->length,1,wbf);
fclose(wbf);

4. Content-Length格式化不匹配 #

fdata->length是size_t类型，代码中用%llu（无符号长long）格式化，会导致长度值输出错误，浏览器接收的长度和实际数据不匹配：

// 错误的格式符
snprintf(http_header,256,"...Content-Length: %llu\r\n",fdata->length);

5. requestpath函数的潜在问题 #

requestpath中read返回的字节数没有处理，也没有给buffer添加终止符，可能导致后续strstr解析请求路径时出错：

char *requestpath(int sockfd) {
    char *buffer = (char *)malloc(BUFFSIZE);
    int bytes = read(sockfd, buffer, BUFFSIZE);
    // 缺少：buffer[bytes] = '\0';
    return buffer;
};

修复后的代码片段 #

main.c中图片处理部分修复 #

else if(strcmp(filepath,"/image.jpg") == 0){
    char http_header[256];
    char *date = get_date_for_server();

    struct file_data *fdata = read_file("image.jpg");
    if(fdata == NULL){
        fprintf(stderr,"Error opening binary file");
        shutdown(newfd, SHUT_WR);
        free(date);
        free(filepath); // 释放filepath，避免内存泄漏
        continue;
    }
   
    // 修复：添加两个\r\n分隔头部和正文，用%zu格式化size_t类型长度
    snprintf(http_header,256,"HTTP/1.1 200 OK\r\nContent-Type: image/jpeg\r\nContent-Length: %zu\r\n\r\n",fdata->length);
    send(newfd,http_header,strlen(http_header),0);
    send(newfd,fdata->data,fdata->length,0);
    printf("length is %zu\n",fdata->length);

    // 修复：释放内存
    free(fdata->data);
    free(fdata);
    free(date);
    shutdown(newfd, SHUT_WR);
}

requestpath函数修复 #

char *requestpath(int sockfd) {
    char *buffer = (char *)malloc(BUFFSIZE);
    if (!buffer) {
        perror("malloc failed");
        return NULL;
    }
    ssize_t bytes = read(sockfd, buffer, BUFFSIZE - 1); // 留一个位置放终止符
    if (bytes > 0) {
        buffer[bytes] = '\0'; // 添加字符串终止符
    } else {
        buffer[0] = '\0';
    }
    return buffer;
};

get_file_path函数修复（修复缩进，避免逻辑错误） #

char *get_file_path(const char *request) {
    char *path = NULL;
    if (!request) return NULL;

    char *start = strstr(request, "GET ");
    if (start != NULL) {    
        start += 4; // 跳过"GET "
        char *end = strstr(start, " ");    
        if (end != NULL) {
            int path_length = end - start;
            path = (char *)malloc(path_length + 1);
            if (path) {
                strncpy(path, start, path_length);
                path[path_length] = '\0';
            }
        }
    }
    return path;
};

额外注意事项 #

• 原代码中filepath在所有分支都没有释放，会导致内存泄漏，需要在每个分支结束前执行free(filepath);
• send函数可能不会一次性发送所有数据，生产环境需要循环发送直到所有字节都发送完毕，避免数据截断
• read_file函数中用feof作为循环条件可能会多读一次（当文件长度刚好是BUFFSIZE整数倍时），可以改为检查read_length > 0：

// 修复read_file的循环条件
while((read_length = fread(buffer,1,BUFFSIZE,handler)) > 0){
    fdata->data = realloc(fdata->data,fdata->length + read_length);
    if(fdata->data == NULL){
        fclose(handler);
        free(fdata);
        return NULL;
    }
    memcpy(fdata->data +fdata->length, buffer, read_length);
    fdata->length += read_length;
}

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