diff options
author | Nick Mathewson <nickm@torproject.org> | 2019-10-22 08:36:32 -0400 |
---|---|---|
committer | Nick Mathewson <nickm@torproject.org> | 2019-10-22 08:36:32 -0400 |
commit | 2d013bbe5cccaccf9ace0beec5a987fb80e850ad (patch) | |
tree | 39110f4cdfd13bc2479ea3a9d18105104a2a7f5d /src | |
parent | a1d53413742840f4dab499531cb29a9d74c51b8a (diff) | |
parent | 8162859ef727619493cfd9368214d3a769b338cb (diff) | |
download | tor-2d013bbe5cccaccf9ace0beec5a987fb80e850ad.tar.gz tor-2d013bbe5cccaccf9ace0beec5a987fb80e850ad.zip |
Merge remote-tracking branch 'tor-github/pr/1417'
Diffstat (limited to 'src')
-rw-r--r-- | src/feature/control/control_events.c | 5 | ||||
-rw-r--r-- | src/lib/buf/buffers.c | 1 | ||||
-rw-r--r-- | src/lib/process/process.c | 1 |
3 files changed, 6 insertions, 1 deletions
diff --git a/src/feature/control/control_events.c b/src/feature/control/control_events.c index 82ea943999..00659c0477 100644 --- a/src/feature/control/control_events.c +++ b/src/feature/control/control_events.c @@ -1653,7 +1653,10 @@ control_event_status(int type, int severity, const char *format, va_list args) log_warn(LD_BUG, "Format string too long."); return -1; } - tor_vasprintf(&user_buf, format, args); + if (tor_vasprintf(&user_buf, format, args)<0) { + log_warn(LD_BUG, "Failed to create user buffer."); + return -1; + } send_control_event(type, "%s %s\r\n", format_buf, user_buf); tor_free(user_buf); diff --git a/src/lib/buf/buffers.c b/src/lib/buf/buffers.c index 4d026bd37d..452bf74376 100644 --- a/src/lib/buf/buffers.c +++ b/src/lib/buf/buffers.c @@ -578,6 +578,7 @@ buf_add_vprintf(buf_t *buf, const char *format, va_list args) /* XXXX Faster implementations are easy enough, but let's optimize later */ char *tmp; tor_vasprintf(&tmp, format, args); + tor_assert(tmp != NULL); buf_add(buf, tmp, strlen(tmp)); tor_free(tmp); } diff --git a/src/lib/process/process.c b/src/lib/process/process.c index 2194a603ff..b01c99992c 100644 --- a/src/lib/process/process.c +++ b/src/lib/process/process.c @@ -550,6 +550,7 @@ process_vprintf(process_t *process, char *data; size = tor_vasprintf(&data, format, args); + tor_assert(data != NULL); process_write(process, (uint8_t *)data, size); tor_free(data); } |