Merge pull request #537 from stoeckmann/vsnprintf
Handle vsnprintf corner cases.
This commit is contained in:
commit
52dfc3dd4a
@ -797,16 +797,18 @@ json_t *json_vsprintf(const char *fmt, va_list ap) {
|
|||||||
va_copy(aq, ap);
|
va_copy(aq, ap);
|
||||||
|
|
||||||
length = vsnprintf(NULL, 0, fmt, ap);
|
length = vsnprintf(NULL, 0, fmt, ap);
|
||||||
|
if (length < 0)
|
||||||
|
goto out;
|
||||||
if (length == 0) {
|
if (length == 0) {
|
||||||
json = json_string("");
|
json = json_string("");
|
||||||
goto out;
|
goto out;
|
||||||
}
|
}
|
||||||
|
|
||||||
buf = jsonp_malloc(length + 1);
|
buf = jsonp_malloc((size_t)length + 1);
|
||||||
if (!buf)
|
if (!buf)
|
||||||
goto out;
|
goto out;
|
||||||
|
|
||||||
vsnprintf(buf, length + 1, fmt, aq);
|
vsnprintf(buf, (size_t)length + 1, fmt, aq);
|
||||||
if (!utf8_check_string(buf, length)) {
|
if (!utf8_check_string(buf, length)) {
|
||||||
jsonp_free(buf);
|
jsonp_free(buf);
|
||||||
goto out;
|
goto out;
|
||||||
|
Loading…
Reference in New Issue
Block a user