mg_printf - return -1 if strlen(fmt) > MG_BUF_LEN

[cesanta]

mg_printf - return -1 if strlen(fmt) > MG_BUF_LEN

[jmucchiello]

That code is not right. You should not call mg_write with l_fly_size. You should call it with the return value from _vsnprintf!

...
va_start(ap, fmt);
int actual_size;
while ((actual_size = _vsnprintf(l_fly_buf, l_fly_size, fmt, ap)) == -1) {
...
}
va_end(ap);
len = mg_write(conn, l_fly_buff, (size_t)actual_size);
...

[pavel-pimenov]

thanks!

[cpq]

That is fixed in b589e0c
I've added alloc_vprintf() function. It looks like if the buffer is 0, windows snprintf() behaves correctly.
I assume this is the case for all versions of msvcrt.dll