config: Change conf.c remove compiler warnings
authorPaul Schulz <paul@mawsonlakes.org>
Thu, 21 Jun 2018 08:49:39 +0000 (18:19 +0930)
committerJohn Crispin <john@phrozen.org>
Mon, 30 Jul 2018 08:43:36 +0000 (10:43 +0200)
Compiler is producing the warning:
   warning: format not a string literal and no format arguments
   [-Wformat-security]

This patch makes the format a literal string in printf statements.

This with: gcc version 7.3.0 (Ubuntu 7.3.0-16ubuntu3) in Ubuntu 18.04

Signed-off-by: Paul Schulz <paul@mawsonlakes.org>
scripts/config/conf.c

index 6d35957afe60c4520432696721b8a45b752b373a..cf20e0a2164b760183497ad7763d8ce1c1f02b49 100644 (file)
@@ -75,9 +75,9 @@ static void strip(char *str)
 static void check_stdin(void)
 {
        if (!valid_stdin) {
-               printf(_("aborted!\n\n"));
-               printf(_("Console input/output is redirected. "));
-               printf(_("Run 'make oldconfig' to update configuration.\n\n"));
+               printf("%s",_("aborted!\n\n"));
+               printf("%s",_("Console input/output is redirected. "));
+               printf("%s",_("Run 'make oldconfig' to update configuration.\n\n"));
                exit(1);
        }
 }
@@ -87,7 +87,7 @@ static int conf_askvalue(struct symbol *sym, const char *def)
        enum symbol_type type = sym_get_type(sym);
 
        if (!sym_has_value(sym))
-               printf(_("(NEW) "));
+               printf("%s",_("(NEW) "));
 
        line[0] = '\n';
        line[1] = 0;
@@ -288,7 +288,7 @@ static int conf_choice(struct menu *menu)
                        if (child->sym->name)
                                printf(" (%s)", child->sym->name);
                        if (!sym_has_value(child->sym))
-                               printf(_(" (NEW)"));
+                               printf("%s",_(" (NEW)"));
                        printf("\n");
                }
                printf(_("%*schoice"), indent - 1, "");
@@ -436,7 +436,7 @@ static void check_conf(struct menu *menu)
                                }
                        } else if (input_mode != olddefconfig) {
                                if (!conf_cnt++)
-                                       printf(_("*\n* Restart config...\n*\n"));
+                                       printf("%s",_("*\n* Restart config...\n*\n"));
                                rootEntry = menu_get_parent_menu(menu);
                                conf(rootEntry);
                        }