diff options
| author | Nick Mathewson <nickm@torproject.org> | 2006-12-15 05:12:42 +0000 |
|---|---|---|
| committer | Nick Mathewson <nickm@torproject.org> | 2006-12-15 05:12:42 +0000 |
| commit | fdb10ff0b59eaa60b66be5eed6077551d17ec331 (patch) | |
| tree | c859e31808c36bf98d5541c2227e95afb21d3113 /src/or/hibernate.c | |
| parent | c44dd3870e4fa46ad983632b873bcf7e4f973ff8 (diff) | |
| download | tor-fdb10ff0b59eaa60b66be5eed6077551d17ec331.tar tor-fdb10ff0b59eaa60b66be5eed6077551d17ec331.tar.gz | |
r11580@Kushana: nickm | 2006-12-15 00:09:46 -0500
Resolve bug 369: Check for integer underflow when printing "bytes left" accounting numbers. Also fix a copyright date that I noticed while reading the bug. Also make a buffer big enough that strings will not get truncated. All are backport candidates.
svn:r9115
Diffstat (limited to 'src/or/hibernate.c')
| -rw-r--r-- | src/or/hibernate.c | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/src/or/hibernate.c b/src/or/hibernate.c index c0e6cf1b5..5a48138a2 100644 --- a/src/or/hibernate.c +++ b/src/or/hibernate.c @@ -949,10 +949,14 @@ getinfo_helper_accounting(control_connection_t *conn, U64_PRINTF_ARG(n_bytes_written_in_interval)); } else if (!strcmp(question, "accounting/bytes-left")) { uint64_t limit = get_options()->AccountingMax; - *answer = tor_malloc(32); - tor_snprintf(*answer, 32, U64_FORMAT" "U64_FORMAT, - U64_PRINTF_ARG(limit - n_bytes_read_in_interval), - U64_PRINTF_ARG(limit - n_bytes_written_in_interval)); + uint64_t read_left = 0, write_left = 0; + if (n_bytes_read_in_interval < limit) + read_left = limit - n_bytes_read_in_interval; + if (n_bytes_written_in_interval < limit) + write_left = limit - n_bytes_written_in_interval; + *answer = tor_malloc(64); + tor_snprintf(*answer, 64, U64_FORMAT" "U64_FORMAT, + U64_PRINTF_ARG(read_left), U64_PRINTF_ARG(write_left)); } else if (!strcmp(question, "accounting/interval-start")) { *answer = tor_malloc(ISO_TIME_LEN+1); format_iso_time(*answer, interval_start_time); |