C - 字符串，va_list

Question

我已经(简单地)编写了这段代码。但它需要两次修复才能确定。

size_t send_header(int fd, int seconds, const char* header1, ...) {
    char *buffer;
    int count = 0;
    const char * format = "%s";
    const char *hdr = header1;
    va_list vl, vl_count;
    va_start(vl, header1);

    va_copy(vl_count, vl);
    while (hdr != NULL) {
        hdr=va_arg(vl_count,const char*);
        ++count;
    }
    va_end(vl_count);
    /* allocate storage for format string: */
    buffer = (char*) malloc(strlen(format) * count + 1);
    buffer[0] = '\0';
    /* generate format string: */
    for (; count > 0; --count) {
        strcat(buffer, format);
    }
    /* print integers: */
    printf(format, header1);
    vprintf(buffer, vl);
    va_end(vl);
    if (BUFSIZ == 0) {
        _myfunc_errorcode(ERROR);
        return ERROR;
    }
    write(fd,buffer, strlen(buffer));
    _myfunc_errorcode(SUCCESS);
    return SUCCESS;
}

1 FIX)此代码第一次在控制台打印

HTTP/1.0 200 OK
Cache-Control: no-cache
Pragma: no-cache
Expires: Thu, 01 Dec 1994 16:00:00 GMT
Connection: close
Content-Type: text/html;charset=utf-8

第二次在控制台打印this

(null)HTTP/1.0 200 OK
Cache-Control: no-cache
Pragma: no-cache
Expires: Thu, 01 Dec 1994 16:00:00 GMT
Connection: close
Content-Type: text/html;charset=utf-8

哪里加(null)，我不是很清楚。

2 FIX) fd,in write(), 是它在其他函数中使用的文件描述符，但我的浏览器只显示 %s%s%s ，为什么？

谢谢，问候

Answer 1

这是非常困惑的代码。

这个:

vprintf(buffer, vl);

不是对 vsprintf() 的调用，因此它不会将字符串格式化为 buffer，它只会使用 buffer 作为格式字符串，从 vl 读取参数，并打印到标准输出。

然而你还是不厌其烦地还发送了buffer中的格式字符串，那一长串%ss，到fd。当然，您会在那里看到一堆 %s。

我不明白为什么这段代码要经历所有的麻烦，只是为了构建一个充满重复 %s 代码的格式化字符串；与让 vprintf() 自己打印每个字符串相比，如此要简单得多......

此外，你泄漏了 buffer 内存，因为你从未 free() 它。