diff options
| author | Mike Frysinger | 2012-08-04 16:14:24 -0400 |
|---|---|---|
| committer | Mike Frysinger | 2012-08-04 16:14:24 -0400 |
| commit | ca81a937225714f75157eb596d90255243ae2e29 (patch) | |
| tree | a5866fefd4e41f9f7a9c0ebb6ec565300040e4a2 | |
| parent | 293c94564ccee60c8e954bb6347fd36034cb7590 (diff) | |
| download | busybox-ca81a937225714f75157eb596d90255243ae2e29.zip busybox-ca81a937225714f75157eb596d90255243ae2e29.tar.gz | |
disable format security warnings
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@gentoo.org>
| -rw-r--r-- | 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 |