From b7cf7bd9ae31aeddd03ad2094dd480d2b1d34d4f Mon Sep 17 00:00:00 2001 From: Nick Mathewson Date: Wed, 19 Dec 2012 16:51:53 -0500 Subject: [PATCH] Fix an instance of snprintf; don't use _snprintf directly --- changes/7304 | 7 +++++++ src/common/compat.h | 1 - src/or/or.h | 1 - src/or/routerparse.c | 2 +- 4 files changed, 8 insertions(+), 3 deletions(-) create mode 100644 changes/7304 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 #include #include -#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 #include #include -#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); -- 2.47.3