[tor-commits] [tor/release-0.2.8] Fix warnings in test_util_formats.

nickm at torproject.org nickm at torproject.org
Sun Jul 17 17:55:21 UTC 2016


commit bec4e41f4b48c288613a13021da9a29d5fb1ecac
Author: Nick Mathewson <nickm at torproject.org>
Date:   Sun Jul 17 13:51:45 2016 -0400

    Fix warnings in test_util_formats.
    
    Storing 255 into a char gives a warning when char is signed.
    
    Fixes bug 19682; bugfix on 0.2.8.1-alpha, where these tests were added.
---
 changes/bug19682            | 3 +++
 src/test/test_util_format.c | 8 ++++----
 2 files changed, 7 insertions(+), 4 deletions(-)

diff --git a/changes/bug19682 b/changes/bug19682
new file mode 100644
index 0000000..c799c41
--- /dev/null
+++ b/changes/bug19682
@@ -0,0 +1,3 @@
+  o Minor bugfixes (compilation):
+    - Fix compilation warning in the unit tests on systems where
+      char is signed. Fixes bug 19682; bugfix on 0.2.8.1-alpha.
diff --git a/src/test/test_util_format.c b/src/test/test_util_format.c
index a25054c..3d02930 100644
--- a/src/test/test_util_format.c
+++ b/src/test/test_util_format.c
@@ -106,10 +106,10 @@ test_util_format_base64_encode(void *ignored)
   for (i = 0;i<50;i++) {
     src[i] = 0;
   }
-  src[50] = 255;
-  src[51] = 255;
-  src[52] = 255;
-  src[53] = 255;
+  src[50] = (char)255;
+  src[51] = (char)255;
+  src[52] = (char)255;
+  src[53] = (char)255;
 
   res = base64_encode(dst, 1000, src, 54, BASE64_ENCODE_MULTILINE);
   tt_int_op(res, OP_EQ, 74);





More information about the tor-commits mailing list