Code

http-backend: Let gcc check the format of more printf-type functions.
authorTarmigan Casebolt <tarmigan+git@gmail.com>
Sat, 14 Nov 2009 21:10:58 +0000 (13:10 -0800)
committerJunio C Hamano <gitster@pobox.com>
Mon, 16 Nov 2009 06:15:01 +0000 (22:15 -0800)
We already have these checks in many printf-type functions that have
prototypes which are in header files.  Add these same checks to
static functions in http-backend.c

Signed-off-by: Tarmigan Casebolt <tarmigan+git@gmail.com>
Acked-by: Shawn O. Pearce <spearce@spearce.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
http-backend.c

index 8e08f057dd6536b06d1ee017e4733ad1585e782e..f729488fc5f787c0f4997aacdd1bb14732d717f7 100644 (file)
@@ -108,6 +108,7 @@ static const char *get_parameter(const char *name)
        return i ? i->util : NULL;
 }
 
+__attribute__((format (printf, 2, 3)))
 static void format_write(int fd, const char *fmt, ...)
 {
        static char buffer[1024];
@@ -165,6 +166,7 @@ static void end_headers(void)
        safe_write(1, "\r\n", 2);
 }
 
+__attribute__((format (printf, 1, 2)))
 static NORETURN void not_found(const char *err, ...)
 {
        va_list params;
@@ -180,6 +182,7 @@ static NORETURN void not_found(const char *err, ...)
        exit(0);
 }
 
+__attribute__((format (printf, 1, 2)))
 static NORETURN void forbidden(const char *err, ...)
 {
        va_list params;