summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNick Mathewson <nickm@torproject.org>2010-04-20 01:09:40 -0400
committerNick Mathewson <nickm@torproject.org>2010-04-20 01:09:40 -0400
commitf2c30e97cc61204d43040fc9d82ea114a87afdeb (patch)
tree20b23d49137f6952d1598c3f0476771cb5ceee11
parentde4a49adc2bc7bc8a2dfe44d75a47a828056a52c (diff)
downloadtor-f2c30e97cc61204d43040fc9d82ea114a87afdeb.tar.gz
tor-f2c30e97cc61204d43040fc9d82ea114a87afdeb.zip
Switch geoip_get_request_history to asprintf; fix bug 1365
-rw-r--r--changes/fix_bug_13654
-rw-r--r--src/or/geoip.c4
2 files changed, 6 insertions, 2 deletions
diff --git a/changes/fix_bug_1365 b/changes/fix_bug_1365
new file mode 100644
index 0000000000..73027310f3
--- /dev/null
+++ b/changes/fix_bug_1365
@@ -0,0 +1,4 @@
+ o Minor bugfixes:
+ - Fix a segfault when DirReqStatistics is enabled. Bug found by keb.
+ Fixes bug 1365; bugfix on 0.2.2.11-alpha.
+
diff --git a/src/or/geoip.c b/src/or/geoip.c
index b5a0374c49..ea11dc323d 100644
--- a/src/or/geoip.c
+++ b/src/or/geoip.c
@@ -945,8 +945,8 @@ geoip_get_request_history(time_t now, geoip_client_action_t action)
strings = smartlist_create();
SMARTLIST_FOREACH(entries, c_hist_t *, ent, {
- char buf[32];
- tor_snprintf(buf, sizeof(buf), "%s=%u", ent->country, ent->total);
+ char *buf = NULL;
+ tor_asprintf(&buf, "%s=%u", ent->country, ent->total);
smartlist_add(strings, buf);
});
result = smartlist_join_strings(strings, ",", 0, NULL);