aboutsummaryrefslogtreecommitdiff
path: root/changes
diff options
context:
space:
mode:
authorNick Mathewson <nickm@torproject.org>2012-05-30 12:14:38 -0400
committerNick Mathewson <nickm@torproject.org>2012-05-30 12:14:38 -0400
commite28489467233bff4500a70f8a7b22e42ca3b3e68 (patch)
treefccba02270c6a44fca8978d71d0f4bc8b548a4ee /changes
parent75fc4dbbcabaedc715f0f9e883ccab1c9634e787 (diff)
downloadtor-e28489467233bff4500a70f8a7b22e42ca3b3e68.tar
tor-e28489467233bff4500a70f8a7b22e42ca3b3e68.tar.gz
Add __attribute__(format)s for our varargs printf/scanf wrappers
It turns out that if you set the third argument of __attribute__(format) to 0, GCC and Clang will check the format argument without expecting to find variadic arguments. This is the correct behavior for vsnprintf, vasprintf, and vscanf. I'm hoping this will fix bug 5969 (a clang warning) by telling clang that the format argument to tor_vasprintf is indeed a format string.
Diffstat (limited to 'changes')
-rw-r--r--changes/bug5969_0227
1 files changed, 7 insertions, 0 deletions
diff --git a/changes/bug5969_022 b/changes/bug5969_022
new file mode 100644
index 000000000..57c874426
--- /dev/null
+++ b/changes/bug5969_022
@@ -0,0 +1,7 @@
+ o Minor bugfixes
+ - Fix a build warning with Clang 3.1 related to our use of vasprint.
+ Fix for bug 5969. Bugfix on 0.2.2.11-alpha.
+
+ o Compilation improvements:
+ - Tell GCC and Clang to check for any errors in format strings passed
+ to the tor_v*(print|scan)f functions.