From: Nick Mathewson <nickm@torproject.org>
Date: Wed, 19 Dec 2012 21:51:53 +0000 (-0500)
Subject: Fix an instance of snprintf; don't use _snprintf directly
X-Git-Tag: tor-0.2.4.10-alpha~31^2
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=b7cf7bd9ae31aeddd03ad2094dd480d2b1d34d4f;p=thirdparty%2Ftor.git

Fix an instance of snprintf; don't use _snprintf directly
---

diff --git a/changes/7304 b/changes/7304
new file mode 100644
index 0000000000..bfb797b107
--- /dev/null
+++ b/changes/7304
@@ -0,0 +1,7 @@
+  o Minor bugfixes:
+    - Replace all calls to snprintf outside of src/ext with tor_snprintf.
+      Fix for bug 7304.
+  o Code simplification and refactoring:
+    - Remove the #define to replace snprintf with _snprintf on windows;
+      they have different semantics, and all of our callers should be
+      using tor_snprintf anyway. Partial fix for bug 7304.
diff --git a/src/common/compat.h b/src/common/compat.h
index 86ea0c41d4..630682e638 100644
--- a/src/common/compat.h
+++ b/src/common/compat.h
@@ -59,7 +59,6 @@
 #include <io.h>
 #include <math.h>
 #include <projects.h>
-#define snprintf _snprintf
 /* this is not exported as W .... */
 #define SHGetPathFromIDListW SHGetPathFromIDList
 /* wcecompat has vasprintf */
diff --git a/src/or/or.h b/src/or/or.h
index 459e18e466..6ec7c3a79c 100644
--- a/src/or/or.h
+++ b/src/or/or.h
@@ -81,7 +81,6 @@
 #include <process.h>
 #include <direct.h>
 #include <windows.h>
-#define snprintf _snprintf
 #endif
 
 #ifdef USE_BUFFEREVENTS
diff --git a/src/or/routerparse.c b/src/or/routerparse.c
index 1aee4e5332..48744d6994 100644
--- a/src/or/routerparse.c
+++ b/src/or/routerparse.c
@@ -3828,7 +3828,7 @@ get_next_token(memarea_t *area,
   if ((size_t)(eol-next) != 9+obname_len+5 ||
       strcmp_len(next+9, tok->object_type, obname_len) ||
       strcmp_len(eol-5, "-----", 5)) {
-    snprintf(ebuf, sizeof(ebuf), "Malformed object: mismatched end tag %s",
+    tor_snprintf(ebuf, sizeof(ebuf), "Malformed object: mismatched end tag %s",
              tok->object_type);
     ebuf[sizeof(ebuf)-1] = '\0';
     RET_ERR(ebuf);