c-vsnprintf在realloc之后失败

Question

我得到了下面的代码，这基本上是一个snprintf函数，但有点慢，这意味着它还使用malloc/realloc处理缓冲区大小。

代码的工作方式应该达到realloc更改地址的程度(当字符串随着时间的推移越来越大时(。。。如果发生这种情况，则输出完全错误。

bool write_str(char **str, const char *format, ...)
{
// va arg stuff
va_list argp;
va_start(argp, format);
// remember to have a separate va_list for each v*print function, because v*print consumes the function
// but in this case it actually doesn't need it
size_t len = vsnprintf(0, 0, format, argp);
if(*str)
{
*str = realloc(*str, len + 1);
}
else
{
*str = malloc(len + 1);
**str = 0;
}
if(!*str)
{
return false;
}
vsnprintf(*str, len, format, argp);
va_end(argp);
(*str)[len] = 0;
return true;
}

这就是你使用函数的方式。

char *str = 0;
write_str(&str, "That is how %d use itn", 2);

编辑：注释来自这里每个字符串参数的长度

Answer 1

#include <stdio.h>
#include <string.h>
#include <stdlib.h>
#include <stdarg.h>
int write_str(char **str, const char *format, ...)
{
// va arg stuff
int len;
va_list argp;
va_start(argp, format);
len = vsnprintf(NULL, 0, format, argp);
// Remember to read the comments (or remove them)
// remember to have a separate va_list for each v*print function
// , because v*print consumes the function arguments
// but in this case it actually doesn't *use* them -- IT DOES consume them!!!!!
va_end(argp);
*str = realloc(*str, len + 1);
if(!*str) return -1;
va_start(argp, format);
vsnprintf(*str, len, format, argp);
va_end(argp);
return 0;
}
int main(void)
{
char *str = NULL;
write_str(&str, "That is how %d use itn", 2);
puts(str);
write_str(&str, "That is how again %d use itn", 666);
puts(str);
return 0;
}

---

一些注意事项：

• argp被vsnprintf()消耗；如果使用两次，则需要复制或重新初始化
• vsnprintf()返回一个签名类型(-1也是一个有效的返回值：也检查一下(
• realloc()足够聪明，可以理解第一个调用(当第一个参数为NULL时(，无需单独处理

Answer 2

终于找到了导致我的问题的原因。有点傻——应该发布我真正使用的代码。

我主要使用这个函数将一个字符串重复地附加到它的lef中。但问题是，realloc可以更改字符串的地址，因此传递的地址不再相同，从而导致垃圾输出。

char *str = 0;
write_str(&str, "string");
for(int i = 0; i < 100; i++)
{
write_str(&str, "%s%s", str, "string"); // the right str argument may change address, don't use it like that!
printf("%sn", str);
}

edit：我现在有了一个可以工作的函数。事实证明，您确实需要va_end。

bool append_str(char **str, char *format, ...)
{
if(!str || !format) return false;

bool result = false;
va_list argp;
va_start(argp, format);

// get memory
int len_app = vsnprintf(0, 0, format, argp);
va_end(argp);
int len_str = 0;
if(*str)
{
len_str = strlen(*str);
}
*str = realloc(*str, len_str + len_app + 1);

// append
va_start(argp);
if(*str)
{
if(vsnprintf(&(*str)[len_str], len_app + 1, format, argp) >= 0)
{
// successful
result = true;
}
else
{
// encoding error
free(*str);
*str = 0;
}
}
va_end(argp);

return result;
}

还有一个如何使用它的例子：

char *str = 0;
bool success = true;
success &= append_str(&str, "Hello world!!%dn", 231);
success &= append_str(&str, "Hello world!!%dn", 231);
success &= append_str(&str, "Hello world!!%dn", 231);
success &= append_str(&str, "Hello world!!%dn", 231);
success &= append_str(&str, "Hello world!!%dn", 231);
if(success)
{
printf("%sn", str);
}
else
{
printf("failed to append string.n");
}
return 0;