如何安全地调用 vsnprintf() ？

我正在将一些非常古老（> 10 年）的 C 代码移植到现代 Linux。我在自定义编写的 vsnprintf() 包装器中遇到分段错误（显然它的任务是检测重复的输出字符串并实习它们）：

char* strVPrintf(const String fmt, va_list ap)
{
  /* Guess we need no more than 50 bytes. */
  int n, size = 50;
  char* p = (char*)memMalloc(size), q;

  while (1) {
    /* Try to print in the allocated space. */
    n = vsnprintf(p, size, fmt, ap);
    /* If that worked, return the string. */
    if (n > -1 && n < size) {
      break;
    }
    /* Else try again with more space. */
    if (n > -1)                /* glibc 2.1 */
      size = n + 1;            /* precisely what is needed */
    else                   /* glibc 2.0 */
      size *= 2;               /* twice the old size */
    p = memRealloc(p, size);
  }

  q =  strRegister(p);
  memFree(p);
  return q;
}

作者似乎假设标准vsnprintf()函数返回写入的字符数，如果没有收到足够的空间来格式化所有参数，则简单地返回一个哨兵值。这意味着您可以猜测缓冲区大小并在必要时增加它。

但在我的系统（Ubuntu 14.04、glibc 2.19）上，当调用提供的空间的参数过多时，vnprintf 会导致分段错误。的语义是否snprintf()与此同时，家庭发生了巨大的变化？确保您提供足够缓冲空间的现代方法是什么？

这才是正确的使用方法snprintf and vsnprintf除了 SunOS 4（它已经过时了 20 年）之外的每个操作系统上都有，所以你的问题出在其他地方。

我会做一个纯粹的猜测，并说我几乎可以肯定你的问题是你传递了 va_listap into vsnprintf它会消耗它，然后您希望它在下次调用时重置。这是不正确的，并且许多年前已经在 gcc 中停止工作（因为它只在某些体系结构上工作）。

Change:

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

To:

va_list apc;
va_copy(apc, ap);
n = vsnprintf(p, size, fmt, apc);
va_end(apc);

看看是否有帮助。

这是一个简单的测试，看看发生了什么：

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

void
foo(const char *fmt, va_list ap)
{
#ifdef BAD
    vprintf(fmt, ap);
#else
    va_list apc;
    va_copy(apc, ap);
    vprintf(fmt, apc);
    va_end(apc);
#endif
    vprintf(fmt, ap);
}

void
bar(const char *fmt, ...)
{
    va_list ap;
    va_start(ap, fmt);
    foo(fmt, ap);
    va_end(ap);
}

int
main(int argc, char **argv)
{
    bar("foo %s\n", "bar");
    return 0;
}

运行时我得到这个：

$ cc -o foo foo.c && ./foo
foo bar
foo bar
$ cc -DBAD -o foo foo.c && ./foo
foo bar
foo ����