]> git.ipfire.org Git - thirdparty/tor.git/commitdiff
Reset `buf` after each succesful test to avoid artifacts.
authorAlexander Færøy <ahf@torproject.org>
Mon, 27 Feb 2017 14:57:08 +0000 (15:57 +0100)
committerAlexander Færøy <ahf@torproject.org>
Wed, 1 Mar 2017 20:26:27 +0000 (21:26 +0100)
This patch resets `buf` in test_util_fgets_eagain() after each succesful
ivocation to avoid stray artifacts left in the buffer by erroneous
tor_fgets() calls.

src/test/test_util.c

index 7a6991650d2bdfc8e248ddcc92b3330a193b6bd8..3e4d45d35b4b840208676436c1b0f3d18386a0b0 100644 (file)
@@ -3986,6 +3986,7 @@ test_util_fgets_eagain(void *ptr)
   tt_ptr_op(retptr, OP_EQ, buf);
   tt_str_op(buf, OP_EQ, "B\n");
   errno = 0;
+  memset(buf, '\0', sizeof(buf));
 
   /* Send in a full line */
   retlen = write(test_pipe[1], "CD\n", 3);
@@ -3995,6 +3996,7 @@ test_util_fgets_eagain(void *ptr)
   tt_ptr_op(retptr, OP_EQ, buf);
   tt_str_op(buf, OP_EQ, "CD\n");
   errno = 0;
+  memset(buf, '\0', sizeof(buf));
 
   /* Send in a partial line */
   retlen = write(test_pipe[1], "E", 1);
@@ -4013,6 +4015,7 @@ test_util_fgets_eagain(void *ptr)
   tt_ptr_op(retptr, OP_EQ, buf);
   tt_str_op(buf, OP_EQ, "F\n");
   errno = 0;
+  memset(buf, '\0', sizeof(buf));
 
   /* Send in a full line and close */
   retlen = write(test_pipe[1], "GH", 2);
@@ -4036,6 +4039,7 @@ test_util_fgets_eagain(void *ptr)
 
   /* Check that buf is unchanged according to C99 and C11 */
   tt_str_op(buf, OP_EQ, "GH");
+  memset(buf, '\0', sizeof(buf));
 
  done:
   if (test_stream != NULL)