diff --git a/src/modulefinder/sentry_modulefinder_apple.c b/src/modulefinder/sentry_modulefinder_apple.c index 96dc1a70d..e42fae009 100644 --- a/src/modulefinder/sentry_modulefinder_apple.c +++ b/src/modulefinder/sentry_modulefinder_apple.c @@ -105,7 +105,7 @@ remove_image(const struct mach_header *mh, intptr_t UNUSED(vmaddr_slide)) goto done; } - char ref_addr[100]; + char ref_addr[32]; snprintf(ref_addr, sizeof(ref_addr), "0x%llx", (long long)info.dli_fbase); sentry_value_t new_modules = sentry_value_new_list(); diff --git a/src/modulefinder/sentry_modulefinder_linux.c b/src/modulefinder/sentry_modulefinder_linux.c index 513c22165..de22db6a2 100644 --- a/src/modulefinder/sentry_modulefinder_linux.c +++ b/src/modulefinder/sentry_modulefinder_linux.c @@ -435,25 +435,31 @@ load_modules(sentry_value_t modules) return; } - // just read the whole map at once, maybe do it line-by-line as a followup… - char buf[4096]; + // Read the whole map at once. Doing it line-by-line would be a good + // followup. sentry_stringbuilder_t sb; sentry__stringbuilder_init(&sb); while (true) { + char *buf = sentry__stringbuilder_reserve(&sb, 4096); + if (!buf) { + sentry__stringbuilder_cleanup(&sb); + close(fd); + return; + } ssize_t n = read(fd, buf, 4096); if (n < 0 && (errno == EAGAIN || errno == EINTR)) { continue; } else if (n <= 0) { break; } - if (sentry__stringbuilder_append_buf(&sb, buf, n)) { - sentry__stringbuilder_cleanup(&sb); - close(fd); - return; - } + sentry__stringbuilder_set_len(&sb, sentry__stringbuilder_len(&sb) + n); } close(fd); + if (sentry__stringbuilder_append_char(&sb, "\0")) { + sentry__stringbuilder_cleanup(&sb); + return; + } char *contents = sentry__stringbuilder_into_string(&sb); if (!contents) { return; diff --git a/src/sentry_json.c b/src/sentry_json.c index 378725e45..316868256 100644 --- a/src/sentry_json.c +++ b/src/sentry_json.c @@ -194,7 +194,7 @@ void sentry__jsonwriter_write_double(sentry_jsonwriter_t *jw, double val) { if (can_write_item(jw)) { - char buf[50]; + char buf[24]; // The MAX_SAFE_INTEGER is 9007199254740991, which has 16 digits int written = sentry__snprintf_c(buf, sizeof(buf), "%.16g", val); // print `null` if we have printf issues or a non-finite double, which diff --git a/src/sentry_os.c b/src/sentry_os.c index c4479bc65..977261eba 100644 --- a/src/sentry_os.c +++ b/src/sentry_os.c @@ -34,7 +34,7 @@ sentry__get_os_context(void) } ffi->dwFileFlags &= ffi->dwFileFlagsMask; - char buf[100]; + char buf[32]; snprintf(buf, sizeof(buf), "%u.%u.%u", ffi->dwFileVersionMS >> 16, ffi->dwFileVersionMS & 0xffff, ffi->dwFileVersionLS >> 16); @@ -71,7 +71,7 @@ sentry__get_os_context(void) sentry_value_set_by_key(os, "name", sentry_value_new_string("macOS")); - char buf[100]; + char buf[32]; size_t buf_len = sizeof(buf); if (sysctlbyname("kern.osproductversion", buf, &buf_len, NULL, 0) != 0) { diff --git a/src/sentry_string.c b/src/sentry_string.c index a13eacc96..a1581c752 100644 --- a/src/sentry_string.c +++ b/src/sentry_string.c @@ -13,10 +13,10 @@ sentry__stringbuilder_init(sentry_stringbuilder_t *sb) sb->len = 0; } -static int -append(sentry_stringbuilder_t *sb, const char *s, size_t len) +char * +sentry__stringbuilder_reserve(sentry_stringbuilder_t *sb, size_t len) { - size_t needed = sb->len + len + 1; + size_t needed = sb->len + len; if (!sb->buf || needed > sb->allocated) { size_t new_alloc_size = sb->allocated; if (new_alloc_size == 0) { @@ -27,7 +27,7 @@ append(sentry_stringbuilder_t *sb, const char *s, size_t len) } char *new_buf = sentry_malloc(new_alloc_size); if (!new_buf) { - return 1; + return NULL; } if (sb->buf) { memcpy(new_buf, sb->buf, sb->allocated); @@ -36,7 +36,17 @@ append(sentry_stringbuilder_t *sb, const char *s, size_t len) sb->buf = new_buf; sb->allocated = new_alloc_size; } - memcpy(sb->buf + sb->len, s, len); + return &sb->buf[sb->len]; +} + +static int +append(sentry_stringbuilder_t *sb, const char *s, size_t len) +{ + char *buf = sentry__stringbuilder_reserve(sb, len + 1); + if (!buf) { + return 1; + } + memcpy(buf, s, len); sb->len += len; // make sure we're always zero terminated @@ -105,6 +115,12 @@ sentry__stringbuilder_len(const sentry_stringbuilder_t *sb) return sb->len; } +void +sentry__stringbuilder_set_len(sentry_stringbuilder_t *sb, size_t len) +{ + sb->len = len; +} + char * sentry__string_clone(const char *str) { diff --git a/src/sentry_string.h b/src/sentry_string.h index c537b5e3e..feeff3678 100644 --- a/src/sentry_string.h +++ b/src/sentry_string.h @@ -73,6 +73,18 @@ void sentry__stringbuilder_cleanup(sentry_stringbuilder_t *sb); */ size_t sentry__stringbuilder_len(const sentry_stringbuilder_t *sb); +/** + * Resizes the stringbuilder buffer to make sure there is at least `len` bytes + * available at the end, and returns a pointer *to the reservation*. + */ +char *sentry__stringbuilder_reserve(sentry_stringbuilder_t *sb, size_t len); + +/** + * Sets the number of used bytes in the string builder, to be used together with + * `sentry__stringbuilder_reserve` to avoid copying from an intermediate buffer. + */ +void sentry__stringbuilder_set_len(sentry_stringbuilder_t *sb, size_t len); + /** * Duplicates a zero terminated string. */ diff --git a/src/sentry_utils.c b/src/sentry_utils.c index 88b1b6a30..8e3300871 100644 --- a/src/sentry_utils.c +++ b/src/sentry_utils.c @@ -358,7 +358,7 @@ sentry__dsn_get_minidump_url(const sentry_dsn_t *dsn) char * sentry__msec_time_to_iso8601(uint64_t time) { - char buf[255]; + char buf[64]; size_t buf_len = sizeof(buf); time_t secs = time / 1000; struct tm *tm; diff --git a/src/sentry_value.c b/src/sentry_value.c index 7a14a980f..668f0dc1e 100644 --- a/src/sentry_value.c +++ b/src/sentry_value.c @@ -544,7 +544,7 @@ sentry__value_stringify(sentry_value_t value) case SENTRY_VALUE_TYPE_STRING: return sentry__string_clone(sentry_value_as_string(value)); default: { - char buf[50]; + char buf[24]; size_t written = (size_t)sentry__snprintf_c( buf, sizeof(buf), "%g", sentry_value_as_double(value)); if (written >= sizeof(buf)) { @@ -934,7 +934,7 @@ sentry__value_new_string_from_wstr(const wchar_t *s) sentry_value_t sentry__value_new_addr(uint64_t addr) { - char buf[100]; + char buf[32]; size_t written = (size_t)snprintf( buf, sizeof(buf), "0x%llx", (unsigned long long)addr); if (written >= sizeof(buf)) {