diff options
author | Mark Lodato <lodatom@gmail.com> | 2010-09-05 00:09:57 (JST) |
---|---|---|
committer | Mark Lodato <lodatom@gmail.com> | 2010-09-05 00:11:40 (JST) |
commit | a4c58d92bfee0cf2e35ab1c5e828a6d790108a69 (patch) | |
tree | 4b6073081a04216525facaf06db956b4a08dffac | |
parent | bf8293d3196d49d6c4a48b254a04415183ca2762 (diff) | |
download | cgit-a4c58d92bfee0cf2e35ab1c5e828a6d790108a69.zip cgit-a4c58d92bfee0cf2e35ab1c5e828a6d790108a69.tar.gz |
use __attribute__ to catch printf format mistakes
Use "__attribute__((format (printf,N,M)))", as is done in git, do catch
mistakes in printf-style format strings.
Signed-off-by: Mark Lodato <lodatom@gmail.com>
-rw-r--r-- | cache.h | 1 | ||||
-rw-r--r-- | cgit.h | 1 | ||||
-rw-r--r-- | html.h | 3 |
3 files changed, 5 insertions, 0 deletions
@@ -30,6 +30,7 @@ extern int cache_process(int size, const char *path, const char *key, int ttl, | |||
30 | extern int cache_ls(const char *path); | 30 | extern int cache_ls(const char *path); |
31 | 31 | ||
32 | /* Print a message to stdout */ | 32 | /* Print a message to stdout */ |
33 | __attribute__((format (printf,1,2))) | ||
33 | extern void cache_log(const char *format, ...); | 34 | extern void cache_log(const char *format, ...); |
34 | 35 | ||
35 | extern unsigned long hash_str(const char *str); | 36 | extern unsigned long hash_str(const char *str); |
@@ -293,6 +293,7 @@ extern void cgit_diff_tree(const unsigned char *old_sha1, | |||
293 | 293 | ||
294 | extern void cgit_diff_commit(struct commit *commit, filepair_fn fn); | 294 | extern void cgit_diff_commit(struct commit *commit, filepair_fn fn); |
295 | 295 | ||
296 | __attribute__((format (printf,1,2))) | ||
296 | extern char *fmt(const char *format,...); | 297 | extern char *fmt(const char *format,...); |
297 | 298 | ||
298 | extern struct commitinfo *cgit_parse_commit(struct commit *commit); | 299 | extern struct commitinfo *cgit_parse_commit(struct commit *commit); |
@@ -5,7 +5,10 @@ extern int htmlfd; | |||
5 | 5 | ||
6 | extern void html_raw(const char *txt, size_t size); | 6 | extern void html_raw(const char *txt, size_t size); |
7 | extern void html(const char *txt); | 7 | extern void html(const char *txt); |
8 | |||
9 | __attribute__((format (printf,1,2))) | ||
8 | extern void htmlf(const char *format,...); | 10 | extern void htmlf(const char *format,...); |
11 | |||
9 | extern void html_status(int code, const char *msg, int more_headers); | 12 | extern void html_status(int code, const char *msg, int more_headers); |
10 | extern void html_txt(const char *txt); | 13 | extern void html_txt(const char *txt); |
11 | extern void html_ntxt(int len, const char *txt); | 14 | extern void html_ntxt(int len, const char *txt); |