[tor-commits] [tor/master] scan-build: truncate tinytest hexified outputs to 1024 bytes.

nickm at torproject.org nickm at torproject.org
Fri Apr 25 05:30:23 UTC 2014


commit 9c9e07963dddff6e11330e9dc8ad7a6d37da4aa4
Author: Nick Mathewson <nickm at torproject.org>
Date:   Sat Apr 19 12:44:31 2014 -0400

    scan-build: truncate tinytest hexified outputs to 1024 bytes.
    
    scan-build didn't like the unlimited version since we might need to
    overflow size_t to hexify a string that took up half our address
    space. (!)
---
 src/ext/tinytest.c |    9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/src/ext/tinytest.c b/src/ext/tinytest.c
index 3a8e331..cc054ad 100644
--- a/src/ext/tinytest.c
+++ b/src/ext/tinytest.c
@@ -478,16 +478,23 @@ tinytest_format_hex_(const void *val_, unsigned long len)
 	const unsigned char *val = val_;
 	char *result, *cp;
 	size_t i;
+	int ellipses = 0;
 
 	if (!val)
 		return strdup("null");
-	if (!(result = malloc(len*2+1)))
+	if (len > 1024) {
+		ellipses = 3;
+		len = 1024;
+	}
+	if (!(result = malloc(len*2+4)))
 		return strdup("<allocation failure>");
 	cp = result;
 	for (i=0;i<len;++i) {
 		*cp++ = "0123456789ABCDEF"[val[i] >> 4];
 		*cp++ = "0123456789ABCDEF"[val[i] & 0x0f];
 	}
+	while (ellipses--)
+		*cp++ = '.';
 	*cp = 0;
 	return result;
 }





More information about the tor-commits mailing list