log: avoid buffer overflow in vlog
`vlog()` can not handle log messages longer than 4096 bytes. However, the message generated in `store_restart_layout()` is likely to exceed this as it contains a long JSON string. This has caused a few SEGFAULTS during restarts for me when running with `-d all`. Fix this by truncating the message to 4096 bytes and punching in a newline at the end.
This commit is contained in:
parent
f3d898be4b
commit
717422c12c
|
@ -246,6 +246,15 @@ static void vlog(const bool print, const char *fmt, va_list args) {
|
||||||
len += vsnprintf(message + len, sizeof(message) - len, fmt, args);
|
len += vsnprintf(message + len, sizeof(message) - len, fmt, args);
|
||||||
if (len >= sizeof(message)) {
|
if (len >= sizeof(message)) {
|
||||||
fprintf(stderr, "BUG: single log message > 4k\n");
|
fprintf(stderr, "BUG: single log message > 4k\n");
|
||||||
|
|
||||||
|
/* vsnprintf returns the number of bytes that *would have been written*,
|
||||||
|
* not the actual amount written. Thus, limit len to sizeof(message) to avoid
|
||||||
|
* memory corruption and outputting garbage later. */
|
||||||
|
len = sizeof(message);
|
||||||
|
|
||||||
|
/* Punch in a newline so the next log message is not dangling at
|
||||||
|
* the end of the truncated message. */
|
||||||
|
message[len - 2] = '\n';
|
||||||
}
|
}
|
||||||
|
|
||||||
/* If there is no space for the current message in the ringbuffer, we
|
/* If there is no space for the current message in the ringbuffer, we
|
||||||
|
|
Loading…
Reference in New Issue