snprintf手册页示例内存泄漏？

Question

snprintf(3)的Linux手册页给出了以下示例:

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

char *
make_message(const char *fmt, ...)
{
    int n;
    int size = 100;     /* Guess we need no more than 100 bytes */
    char *p, *np;
    va_list ap;

    if ((p = malloc(size)) == NULL)
        return NULL;

    while (1) {

        /* Try to print in the allocated space */

        va_start(ap, fmt);
        n = vsnprintf(p, size, fmt, ap);
        va_end(ap);

        /* Check error code */

        if (n < 0)
            return NULL;

        /* If that worked, return the string */

        if (n < size)
            return p;

        /* Else try again with more space */

        size = n + 1;       /* Precisely what is needed */

        if ((np = realloc (p, size)) == NULL) {
            free(p);
            return NULL;
        } else {
            p = np;
        }
    }
}

后/* check error code */如果不是这种:

        if (n < 0) {
            free(p);
            return NULL;
        }

为了避免内存泄漏？

我不能发布这个,因为单词代码比率不正确,所以我必须在最后添加一些更多的文本.请忽略这一段,因为上述内容已经完成且重点突出.我希望这是足够的文字可以接受.

顺便说一句:我喜欢最后一行 p = np;

Answer 1

是的，这段代码是有漏洞的。

vsnprintf 出错时可以返回负数。在 VC++ 中，当目标缓冲区太小时，vsnprintf 返回 -1，这会破坏此代码中的逻辑...请参阅此处： MSDN VC 实现不符合 C 标准...

vsnprintf 失败的其他原因是发送 NULL“格式”缓冲区或格式缓冲区中的错误编码。