[git commit] disable format security warnings

Mike Frysinger vapier at gentoo.org
Sat Aug 4 20:14:24 UTC 2012


commit: http://git.busybox.net/busybox/commit/?id=ca81a937225714f75157eb596d90255243ae2e29
branch: http://git.busybox.net/busybox/commit/?id=refs/heads/master

We pass strings to printf directly a lot.  Some distros enable some flags
in their gcc by default like -Wformat-security.  With these two things, we
end up with a lot of build warnings like so:
	loginutils/chpasswd.c:42:3: warning: format not a string literal and
		no format arguments [-Wformat-security]

But we don't care.  Our focus is first and foremost on size, so adding a
lot of dummy calls like:
-	printf(some_constant_string);
+	printf("%s", some_constant_string);
is pointless bloat.

Disable this warning flag if the compiler supports it.

Signed-off-by: Mike Frysinger <vapier at gentoo.org>
---
 Makefile.flags |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/Makefile.flags b/Makefile.flags
index c43c8dc..f5f4465 100644
--- a/Makefile.flags
+++ b/Makefile.flags
@@ -25,6 +25,7 @@ CFLAGS += $(call cc-option,-Wstrict-prototypes,)
 CFLAGS += $(call cc-option,-Wunused -Wunused-parameter,)
 CFLAGS += $(call cc-option,-Wunused-function -Wunused-value,)
 CFLAGS += $(call cc-option,-Wmissing-prototypes -Wmissing-declarations,)
+CFLAGS += $(call cc-option,-Wno-format-security,)
 # warn about C99 declaration after statement
 CFLAGS += $(call cc-option,-Wdeclaration-after-statement,)
 # If you want to add more -Wsomething above, make sure that it is


More information about the busybox-cvs mailing list