commit ee12c11dd4e81d44020ded9f275ec030de0a2d1f Author: Nick Mathewson nickm@torproject.org Date: Tue Jul 17 16:47:32 2018 -0400

Increase line coverage in libtor-string to 100%

(On linux.) --- src/lib/string/printf.c | 2 +- src/test/test_util.c | 15 +++++++++++++++ 2 files changed, 16 insertions(+), 1 deletion(-)

diff --git a/src/lib/string/printf.c b/src/lib/string/printf.c index 82d38242d..e23da69d0 100644 --- a/src/lib/string/printf.c +++ b/src/lib/string/printf.c @@ -101,7 +101,7 @@ tor_vasprintf(char **strp, const char *fmt, va_list args) /* If the platform gives us one, use it. */ int r = vasprintf(&strp_tmp, fmt, args); if (r < 0) - *strp = NULL; + *strp = NULL; // LCOV_EXCL_LINE -- no cross-platform way to force this else *strp = strp_tmp; return r; diff --git a/src/test/test_util.c b/src/test/test_util.c index 99fee4c5a..888038bea 100644 --- a/src/test/test_util.c +++ b/src/test/test_util.c @@ -3163,6 +3163,21 @@ test_util_sscanf(void *arg) test_feq(d3, -900123123.2000787); test_feq(d4, 3.2);

+ /* missing float */ + r = tor_sscanf("3 ", "%d %lf", &int1, &d1); + tt_int_op(r, OP_EQ, 1); + tt_int_op(int1, OP_EQ, 3); + + /* not a float */ + r = tor_sscanf("999 notafloat", "%d %lf", &int1, &d1); + tt_int_op(r, OP_EQ, 1); + tt_int_op(int1, OP_EQ, 999); + + /* %s but no buffer. */ + char *nullbuf = NULL; + r = tor_sscanf("hello", "%3s", nullbuf); + tt_int_op(r, OP_EQ, 0); + done: tor_free(huge); }