svn commit: r259666 - in head/contrib/gcc: . cp

Pedro F. Giffuni pfg at FreeBSD.org
Fri Dec 20 20:57:17 UTC 2013


Author: pfg
Date: Fri Dec 20 20:57:13 2013
New Revision: 259666
URL: http://svnweb.freebsd.org/changeset/base/259666

Log:
  gcc: clean some warnings from -Wformat-security
  
  Obtained from:	Apple GCC 4.2 - 5646 (Radar 5764921)
  MFC after:	1 week

Modified:
  head/contrib/gcc/c-common.c
  head/contrib/gcc/c-convert.c
  head/contrib/gcc/c-incpath.c
  head/contrib/gcc/c-typeck.c
  head/contrib/gcc/cfg.c
  head/contrib/gcc/collect2.c
  head/contrib/gcc/cp/cvt.c
  head/contrib/gcc/cp/pt.c
  head/contrib/gcc/cp/typeck.c
  head/contrib/gcc/fold-const.c
  head/contrib/gcc/gcc.c
  head/contrib/gcc/gcov.c
  head/contrib/gcc/tlink.c

Modified: head/contrib/gcc/c-common.c
==============================================================================
--- head/contrib/gcc/c-common.c	Fri Dec 20 20:22:10 2013	(r259665)
+++ head/contrib/gcc/c-common.c	Fri Dec 20 20:57:13 2013	(r259666)
@@ -6014,11 +6014,11 @@ c_parse_error (const char *gmsgid, enum 
       message = NULL;
     }
   else
-    error (gmsgid);
+    error (gmsgid, "");
 
   if (message)
     {
-      error (message);
+      error (message, "");
       free (message);
     }
 #undef catenate_messages

Modified: head/contrib/gcc/c-convert.c
==============================================================================
--- head/contrib/gcc/c-convert.c	Fri Dec 20 20:22:10 2013	(r259665)
+++ head/contrib/gcc/c-convert.c	Fri Dec 20 20:57:13 2013	(r259666)
@@ -80,7 +80,7 @@ convert (tree type, tree expr)
   if ((invalid_conv_diag
        = targetm.invalid_conversion (TREE_TYPE (expr), type)))
     {
-      error (invalid_conv_diag);
+      error (invalid_conv_diag, "");
       return error_mark_node;
     }
 

Modified: head/contrib/gcc/c-incpath.c
==============================================================================
--- head/contrib/gcc/c-incpath.c	Fri Dec 20 20:22:10 2013	(r259665)
+++ head/contrib/gcc/c-incpath.c	Fri Dec 20 20:57:13 2013	(r259666)
@@ -72,7 +72,7 @@ free_path (struct cpp_dir *path, int rea
     case REASON_DUP_SYS:
       fprintf (stderr, _("ignoring duplicate directory \"%s\"\n"), path->name);
       if (reason == REASON_DUP_SYS)
-	fprintf (stderr,
+	fprintf (stderr, "%s",
  _("  as it is a non-system directory that duplicates a system directory\n"));
       break;
 
@@ -292,16 +292,16 @@ merge_include_chains (cpp_reader *pfile,
     {
       struct cpp_dir *p;
 
-      fprintf (stderr, _("#include \"...\" search starts here:\n"));
+      fprintf (stderr, "%s", _("#include \"...\" search starts here:\n"));
       for (p = heads[QUOTE];; p = p->next)
 	{
 	  if (p == heads[BRACKET])
-	    fprintf (stderr, _("#include <...> search starts here:\n"));
+	    fprintf (stderr, "%s", _("#include <...> search starts here:\n"));
 	  if (!p)
 	    break;
 	  fprintf (stderr, " %s\n", p->name);
 	}
-      fprintf (stderr, _("End of search list.\n"));
+      fprintf (stderr, "%s", _("End of search list.\n"));
     }
 }
 

Modified: head/contrib/gcc/c-typeck.c
==============================================================================
--- head/contrib/gcc/c-typeck.c	Fri Dec 20 20:22:10 2013	(r259665)
+++ head/contrib/gcc/c-typeck.c	Fri Dec 20 20:57:13 2013	(r259666)
@@ -2584,7 +2584,7 @@ convert_arguments (tree typelist, tree v
       else if ((invalid_func_diag =
 		targetm.calls.invalid_arg_for_unprototyped_fn (typelist, fundecl, val)))
 	{
-	  error (invalid_func_diag);
+	  error (invalid_func_diag, "");
 	  return error_mark_node;
 	}
       else
@@ -2781,7 +2781,7 @@ build_unary_op (enum tree_code code, tre
   if ((invalid_op_diag
        = targetm.invalid_unary_op (code, TREE_TYPE (xarg))))
     {
-      error (invalid_op_diag);
+      error (invalid_op_diag, "");
       return error_mark_node;
     }
 
@@ -7819,7 +7819,7 @@ build_binary_op (enum tree_code code, tr
   if ((invalid_op_diag
        = targetm.invalid_binary_op (code, type0, type1)))
     {
-      error (invalid_op_diag);
+      error (invalid_op_diag, "");
       return error_mark_node;
     }
 

Modified: head/contrib/gcc/cfg.c
==============================================================================
--- head/contrib/gcc/cfg.c	Fri Dec 20 20:22:10 2013	(r259665)
+++ head/contrib/gcc/cfg.c	Fri Dec 20 20:57:13 2013	(r259666)
@@ -830,7 +830,7 @@ dump_cfg_bb_info (FILE *file, basic_bloc
 	else
 	  fprintf (file, ", ");
 	first = false;
-	fputs (bb_bitnames[i], file);
+	fprintf (file, "%s", bb_bitnames[i]);
       }
   if (!first)
     fprintf (file, ")");

Modified: head/contrib/gcc/collect2.c
==============================================================================
--- head/contrib/gcc/collect2.c	Fri Dec 20 20:22:10 2013	(r259665)
+++ head/contrib/gcc/collect2.c	Fri Dec 20 20:57:13 2013	(r259666)
@@ -1562,10 +1562,10 @@ collect_execute (const char *prog, char 
       if (err != 0)
 	{
 	  errno = err;
-	  fatal_perror (errmsg);
+	  fatal_perror ("%s", errmsg);
 	}
       else
-	fatal (errmsg);
+	fatal ("%s", errmsg);
     }
 
   return pex;
@@ -2050,10 +2050,10 @@ scan_prog_file (const char *prog_name, e
       if (err != 0)
 	{
 	  errno = err;
-	  fatal_perror (errmsg);
+	  fatal_perror ("%s", errmsg);
 	}
       else
-	fatal (errmsg);
+	fatal ("%s", errmsg);
     }
 
   int_handler  = (void (*) (int)) signal (SIGINT,  SIG_IGN);

Modified: head/contrib/gcc/cp/cvt.c
==============================================================================
--- head/contrib/gcc/cp/cvt.c	Fri Dec 20 20:22:10 2013	(r259665)
+++ head/contrib/gcc/cp/cvt.c	Fri Dec 20 20:57:13 2013	(r259666)
@@ -615,7 +615,7 @@ ocp_convert (tree type, tree expr, int c
   if ((invalid_conv_diag
        = targetm.invalid_conversion (TREE_TYPE (expr), type)))
     {
-      error (invalid_conv_diag);
+      error (invalid_conv_diag, "");
       return error_mark_node;
     }
 

Modified: head/contrib/gcc/cp/pt.c
==============================================================================
--- head/contrib/gcc/cp/pt.c	Fri Dec 20 20:22:10 2013	(r259665)
+++ head/contrib/gcc/cp/pt.c	Fri Dec 20 20:57:13 2013	(r259666)
@@ -8925,7 +8925,7 @@ tsubst_copy_and_build (tree t,
 				     /*template_arg_p=*/false,
 				     &error_msg);
 	if (error_msg)
-	  error (error_msg);
+	  error ("%s", error_msg);
 	if (!function_p && TREE_CODE (decl) == IDENTIFIER_NODE)
 	  decl = unqualified_name_lookup_error (decl);
 	return decl;

Modified: head/contrib/gcc/cp/typeck.c
==============================================================================
--- head/contrib/gcc/cp/typeck.c	Fri Dec 20 20:22:10 2013	(r259665)
+++ head/contrib/gcc/cp/typeck.c	Fri Dec 20 20:57:13 2013	(r259666)
@@ -3107,7 +3107,7 @@ build_binary_op (enum tree_code code, tr
   if ((invalid_op_diag
        = targetm.invalid_binary_op (code, type0, type1)))
     {
-      error (invalid_op_diag);
+      error (invalid_op_diag, "");
       return error_mark_node;
     }
 
@@ -4034,7 +4034,7 @@ build_unary_op (enum tree_code code, tre
 				    : code),
 				   TREE_TYPE (xarg))))
     {
-      error (invalid_op_diag);
+      error (invalid_op_diag, "");
       return error_mark_node;
     }
 

Modified: head/contrib/gcc/fold-const.c
==============================================================================
--- head/contrib/gcc/fold-const.c	Fri Dec 20 20:22:10 2013	(r259665)
+++ head/contrib/gcc/fold-const.c	Fri Dec 20 20:57:13 2013	(r259666)
@@ -992,7 +992,7 @@ fold_overflow_warning (const char* gmsgi
 	}
     }
   else if (issue_strict_overflow_warning (wc))
-    warning (OPT_Wstrict_overflow, gmsgid);
+    warning (OPT_Wstrict_overflow, "%s", gmsgid);
 }
 

 /* Return true if the built-in mathematical function specified by CODE

Modified: head/contrib/gcc/gcc.c
==============================================================================
--- head/contrib/gcc/gcc.c	Fri Dec 20 20:22:10 2013	(r259665)
+++ head/contrib/gcc/gcc.c	Fri Dec 20 20:57:13 2013	(r259666)
@@ -2974,7 +2974,7 @@ execute (void)
       if (errmsg != NULL)
 	{
 	  if (err == 0)
-	    fatal (errmsg);
+	    fatal ("%s", errmsg);
 	  else
 	    {
 	      errno = err;
@@ -6525,7 +6525,7 @@ main (int argc, char **argv)
 
       if (! verbose_flag)
 	{
-	  printf (_("\nFor bug reporting instructions, please see:\n"));
+	  printf ("%s", _("\nFor bug reporting instructions, please see:\n"));
 	  printf ("%s.\n", bug_report_url);
 
 	  return (0);

Modified: head/contrib/gcc/gcov.c
==============================================================================
--- head/contrib/gcc/gcov.c	Fri Dec 20 20:22:10 2013	(r259665)
+++ head/contrib/gcc/gcov.c	Fri Dec 20 20:57:13 2013	(r259666)
@@ -414,7 +414,7 @@ print_usage (int error_p)
 static void
 print_version (void)
 {
-  fnotice (stdout, "gcov (GCC) %s\n", version_string);
+  fnotice (stdout, "%s", "gcov (GCC) %s\n", version_string);
   fprintf (stdout, "Copyright %s 2006 Free Software Foundation, Inc.\n",
 	   _("(C)"));
   fnotice (stdout,

Modified: head/contrib/gcc/tlink.c
==============================================================================
--- head/contrib/gcc/tlink.c	Fri Dec 20 20:22:10 2013	(r259665)
+++ head/contrib/gcc/tlink.c	Fri Dec 20 20:57:13 2013	(r259666)
@@ -381,7 +381,7 @@ read_repo_file (file *f)
   FILE *stream = fopen (f->key, "r");
 
   if (tlink_verbose >= 2)
-    fprintf (stderr, _("collect: reading %s\n"), f->key);
+    fprintf (stderr, "%s", _("collect: reading %s\n"), f->key);
 
   while (fscanf (stream, "%c ", &c) == 1)
     {


More information about the svn-src-head mailing list