]> git.ipfire.org Git - thirdparty/binutils-gdb.git/commitdiff
* gen-aout.c (main): Fix formatting. Close file.
authorNick Clifton <nickc@redhat.com>
Mon, 14 Oct 2013 09:15:09 +0000 (09:15 +0000)
committerNick Clifton <nickc@redhat.com>
Mon, 14 Oct 2013 09:15:09 +0000 (09:15 +0000)
* emultempl/aix.em (_read_file): Close file at end of function.

* gas/all/itbl-test.c (main): Close fas.

* read.c (add_include_dir): Use xrealloc.
* config/tc-score.c (do_macro_bcmp): Initialise inst_main.
* config/tc-tic6x.c (tic6x_parse_operand): Initialise second_reg.

* readelf.c (decode_arm_unwind): Initialise addr structure.
(process_symbol_table): Free lengths.
* srcconv.c (wr_sc): Free info.

* chew.c (perform): Free next.

15 files changed:
bfd/ChangeLog
bfd/doc/ChangeLog
bfd/doc/chew.c
bfd/gen-aout.c
binutils/ChangeLog
binutils/readelf.c
binutils/srconv.c
gas/ChangeLog
gas/config/tc-score.c
gas/config/tc-tic6x.c
gas/read.c
gas/testsuite/ChangeLog
gas/testsuite/gas/all/itbl-test.c
ld/ChangeLog
ld/emultempl/aix.em

index f7cc20f3ec198d4298e4098c9c95600b616df04a..f4dcecbc736d9730f7f7ac1e330b66ce81085a45 100644 (file)
@@ -1,3 +1,7 @@
+2013-10-14  Nick Clifton  <nickc@redhat.com>
+
+       * gen-aout.c (main): Fix formatting.  Close file.
+
 2013-10-13  Richard Sandiford  <rdsandiford@googlemail.com>
 
        * elfxx-mips.c (mips_use_local_got_p): New function.
index c33eb3b3b31f29d4cb5652d8c1dc355acd8549f7..08c278081c5c84103ea3e316009a35c4f2d17d7a 100644 (file)
@@ -1,3 +1,7 @@
+2013-10-14  Nick Clifton  <nickc@redhat.com>
+
+       * chew.c (perform): Free next.
+
 2013-04-15  Alan Modra  <amodra@gmail.com>
 
        * Makefile.am ($(MKDOC)): Append $(EXEEXT_FOR_BUILD) to temp file.
index f949e1f7ffc0e797ac79daf1d64435bc767a96be..b019837b13b48755d9b4e8a4e6d3a0fe9b9d4942 100644 (file)
@@ -1256,7 +1256,7 @@ perform ()
                fprintf (stderr, "warning, %s is not recognised\n", next);
              skip_past_newline ();
            }
-
+         free (next);
        }
       else
        skip_past_newline ();
index 45331e0c813abf9c3d95c4ba501f296d375b9974..5e791ec15aa4cc839568f3f7358e148ce0f52371 100644 (file)
@@ -1,6 +1,5 @@
 /* Generate parameters for an a.out system.
-   Copyright 1990, 1991, 1992, 1993, 1994, 1995, 2001, 2002, 2005, 2007
-   Free Software Foundation, Inc.
+   Copyright 1990-2013 Free Software Foundation, Inc.
 
    This file is part of BFD, the Binary File Descriptor library.
 
 #endif
 
 int
-main (argc, argv)
-     int argc; char** argv;
+main (int argc, char** argv)
 {
   struct exec my_exec;
   int page_size;
-  char *target = "unknown", *arch = "unknown";
-  FILE *file = fopen("gen-aout", "r");
+  char * target;
+  char * arch = "unknown";
+  FILE * file;
 
-  if (file == NULL) {
-      fprintf(stderr, "Cannot open gen-aout!\n");
+  target = argv[1];
+  if (target == NULL)
+    {
+      fprintf (stderr, "Usage: gen-aout target_name\n");
+      exit (1);
+    }
+
+  file = fopen ("gen-aout", "r");
+  if (file == NULL)
+    {
+      fprintf (stderr, "Cannot open gen-aout!\n");
       return -1;
-  }
-  if (fread(&my_exec, sizeof(struct exec), 1, file) != 1) {
+    }
+
+  if (fread (&my_exec, sizeof (struct exec), 1, file) != 1)
+    {
       fprintf(stderr, "Cannot read gen-aout!\n");
       return -1;
-  }
+    }
 
-  target = argv[1];
-  if (target == NULL) {
-      fprintf(stderr, "Usage: gen-aout target_name\n");
-      exit (1);
-  }
+  fclose (file);
 
 #ifdef N_TXTOFF
   page_size = N_TXTOFF(my_exec);
   if (page_size == 0)
-    printf("#define N_HEADER_IN_TEXT(x) 1\n");
+    printf ("#define N_HEADER_IN_TEXT(x) 1\n");
   else
-    printf("#define N_HEADER_IN_TEXT(x) 0\n");
+    printf ("#define N_HEADER_IN_TEXT(x) 0\n");
 #endif
 
   printf("#define BYTES_IN_WORD %d\n", sizeof (int));
-  if (my_exec.a_entry == 0) {
-      printf("#define ENTRY_CAN_BE_ZERO\n");
-      printf("#define N_SHARED_LIB(x) 0 /* Avoids warning */\n");
-  }
-  else {
-      printf("/*#define ENTRY_CAN_BE_ZERO*/\n");
-      printf("/*#define N_SHARED_LIB(x) 0*/\n");
-  }
+  if (my_exec.a_entry == 0)
+    {
+      printf ("#define ENTRY_CAN_BE_ZERO\n");
+      printf ("#define N_SHARED_LIB(x) 0 /* Avoids warning */\n");
+    }
+  else
+    {
+      printf ("/*#define ENTRY_CAN_BE_ZERO*/\n");
+      printf ("/*#define N_SHARED_LIB(x) 0*/\n");
+    }
 
-  printf("#define TEXT_START_ADDR %d\n", my_exec.a_entry);
+  printf ("#define TEXT_START_ADDR %d\n", my_exec.a_entry);
 
 #ifdef PAGSIZ
   if (page_size == 0)
     page_size = PAGSIZ;
 #endif
+
   if (page_size != 0)
-    printf("#define TARGET_PAGE_SIZE %d\n", page_size);
+    printf ("#define TARGET_PAGE_SIZE %d\n", page_size);
   else
-    printf("/* #define TARGET_PAGE_SIZE ??? */\n");
-  printf("#define SEGMENT_SIZE TARGET_PAGE_SIZE\n");
+    printf ("/* #define TARGET_PAGE_SIZE ??? */\n");
+
+  printf ("#define SEGMENT_SIZE TARGET_PAGE_SIZE\n");
 
 #ifdef vax
   arch = "vax";
@@ -92,19 +102,19 @@ main (argc, argv)
       fprintf (stderr, _("         fix DEFAULT_ARCH in the output file yourself\n"));
       arch = "unknown";
     }
-  printf("#define DEFAULT_ARCH bfd_arch_%s\n\n", arch);
-
-  printf("/* Do not \"beautify\" the CONCAT* macro args.  Traditional C will not");
-  printf("   remove whitespace added here, and thus will fail to concatenate");
-  printf("   the tokens.  */");
-  printf("\n#define MY(OP) CONCAT2 (%s_,OP)\n\n", target);
-  printf("#define TARGETNAME \"a.out-%s\"\n\n", target);
-
-  printf("#include \"sysdep.h\"\n");
-  printf("#include \"bfd.h\"\n");
-  printf("#include \"libbfd.h\"\n");
-  printf("#include \"libaout.h\"\n");
-  printf("\n#include \"aout-target.h\"\n");
+  printf ("#define DEFAULT_ARCH bfd_arch_%s\n\n", arch);
+
+  printf ("/* Do not \"beautify\" the CONCAT* macro args.  Traditional C will not");
+  printf ("   remove whitespace added here, and thus will fail to concatenate");
+  printf ("   the tokens.  */");
+  printf ("\n#define MY(OP) CONCAT2 (%s_,OP)\n\n", target);
+  printf ("#define TARGETNAME \"a.out-%s\"\n\n", target);
+
+  printf ("#include \"sysdep.h\"\n");
+  printf ("#include \"bfd.h\"\n");
+  printf ("#include \"libbfd.h\"\n");
+  printf ("#include \"libaout.h\"\n");
+  printf ("\n#include \"aout-target.h\"\n");
 
   return 0;
 }
index d7421c10c6e119868bc9ab09cc81c25bbfdb88c9..078b74d8b6b751259ddec4869bea8c10b8fb05c0 100644 (file)
@@ -1,3 +1,9 @@
+2013-10-14  Nick Clifton  <nickc@redhat.com>
+
+       * readelf.c (decode_arm_unwind): Initialise addr structure.
+       (process_symbol_table): Free lengths.
+       * srcconv.c (wr_sc): Free info.
+
 2013-10-11  Roland McGrath  <mcgrathr@google.com>
 
        * winduni.c (languages): Use \345 (octal syntax) rather than
index 19b46c9c05c17229645916fb1d3635b037c315b5..32e0cf00144c73cb115a7d5289d93f70058e53c4 100644 (file)
@@ -7170,7 +7170,7 @@ decode_arm_unwind (struct arm_unw_aux_info *  aux,
 {
   int per_index;
   unsigned int more_words = 0;
-  struct absaddr addr;
+  struct absaddr addr = { 0 };
   bfd_vma sym_name = (bfd_vma) -1;
 
   if (remaining == 0)
@@ -9858,6 +9858,7 @@ process_symbol_table (FILE * file)
       counts = (unsigned long *) calloc (maxlength + 1, sizeof (*counts));
       if (counts == NULL)
        {
+         free (lengths);
          error (_("Out of memory\n"));
          return 0;
        }
@@ -9926,6 +9927,7 @@ process_symbol_table (FILE * file)
       counts = (unsigned long *) calloc (maxlength + 1, sizeof (*counts));
       if (counts == NULL)
        {
+         free (lengths);
          error (_("Out of memory\n"));
          return 0;
        }
index 7b3d4a7550cd8e29ce068aaae7e754465b1b750c..dcf33b9a7012f7676e41cdad1fe6a6f93a41e9e5 100644 (file)
@@ -1578,6 +1578,7 @@ wr_sc (struct coff_ofile *ptr, struct coff_sfile *sfile)
       sysroff_swap_sc_out (file, &sc);
       scount++;
     }
+  free (info);
   return scount;
 }
 
index ccc6f04900bf71b28c6a8f916a7078e4d73c3e4c..5d9cc80dfe00af7830c6a04fd0ccfa40998b54ba 100644 (file)
@@ -1,3 +1,9 @@
+2013-10-14  Nick Clifton  <nickc@redhat.com>
+
+       * read.c (add_include_dir): Use xrealloc.
+       * config/tc-score.c (do_macro_bcmp): Initialise inst_main.
+       * config/tc-tic6x.c (tic6x_parse_operand): Initialise second_reg.
+
 2013-10-13  Sandra Loosemore  <sandra@codesourcery.com>
 
        * config/tc-nios2.c (nios2_consume_arg): Make the "ba" warning
index 822b9cfd5baecafcbd875631650dc625e06fb15b..2bb3fbe16961a40d27a1aeb44adeb6283b1cc3d7 100644 (file)
@@ -4489,7 +4489,7 @@ s3_do_macro_bcmp (char *str)
   char* ptemp;
   int i = 0;
   struct s3_score_it inst_expand[2];
-  struct s3_score_it inst_main;
+  struct s3_score_it inst_main = { 0 };
 
   memset (inst_expand, 0, sizeof inst_expand);
   s3_skip_whitespace (str);
index 81f33f40f134946f809bcec9206275057779f199..fda9cd354eb28a7089c92c80277c8e7c2a039112 100644 (file)
@@ -1596,7 +1596,7 @@ tic6x_parse_operand (char **p, tic6x_operand *op, unsigned int op_forms,
   /* See if this looks like a register or register pair.  */
   if (!operand_parsed && (op_forms & (TIC6X_OP_REG | TIC6X_OP_REGPAIR)))
     {
-      tic6x_register first_reg, second_reg;
+      tic6x_register first_reg, second_reg = { 0 };
       bfd_boolean reg_ok;
       char *rq = q;
 
index 082670c44509d8db231c345af4320d815db97227..02f5a255b64587a46de6fe8118c7b55c51e31fab 100644 (file)
@@ -1,7 +1,5 @@
 /* read.c - read a source file -
-   Copyright 1986, 1987, 1990, 1991, 1992, 1993, 1994, 1995, 1996, 1997,
-   1998, 1999, 2000, 2001, 2002, 2003, 2004, 2005, 2006, 2007, 2008, 2009,
-   2010, 2011, 2012  Free Software Foundation, Inc.
+   Copyright 1986-2013 Free Software Foundation, Inc.
 
    This file is part of GAS, the GNU Assembler.
 
@@ -5794,8 +5792,8 @@ add_include_dir (char *path)
     {
       include_dir_count++;
       include_dirs =
-       (char **) realloc (include_dirs,
-                          include_dir_count * sizeof (*include_dirs));
+       (char **) xrealloc (include_dirs,
+                           include_dir_count * sizeof (*include_dirs));
     }
 
   include_dirs[include_dir_count - 1] = path;  /* New one.  */
index 551386775508cd7ccd08a3585385e498e32fcc4f..9100e4b3399bdfdfce6a51f7edcb064ed7c2d3ba 100644 (file)
@@ -1,3 +1,7 @@
+2013-10-14  Nick Clifton  <nickc@redhat.com>
+
+       * gas/all/itbl-test.c (main): Close fas.
+
 2013-10-13  Sandra Loosemore  <sandra@codesourcery.com>
 
        * gas/nios2/warn_nobreak.l: Update text of warning messages.
index 38fcfc58f1def01d9cbb0b5af6c9b3154d30ef0d..c00ff1ae8730b4deac1d32f8a1cd8ea85b961c76 100644 (file)
@@ -1,6 +1,6 @@
 /* itbl-test.c
 
-   Copyright (C) 1997, 2005, 2007  Free Software Foundation, Inc.
+   Copyright (C) 1997-2013 Free Software Foundation.
 
    This file is part of GAS, the GNU Assembler.
 
@@ -97,6 +97,7 @@ main (int argc, char **argv)
   test_reg (3, e_creg, "c2", 22);
   test_reg (3, e_dreg, "d3", 3);
 
+  fclose (fas);
   return 0;
 }
 
index 14287ddd3a886421db5eb6d165799d47bc9dab4c..b8ebe32c12a2e02af4409b10bf0913e778c3f567 100644 (file)
@@ -1,3 +1,7 @@
+2013-10-14  Nick Clifton  <nickc@redhat.com>
+
+       * emultempl/aix.em (_read_file): Close file at end of function.
+
 2013-10-10  Roland McGrath  <mcgrathr@google.com>
 
        * ldmisc.c (vfinfo): Use Boolean ? "" : ":" in place of ":" + Boolean.
index b48228a134e85368dbb202c637a9b8d3861f9495..aa72ce64c4f9fdff4ea5f88fe60ca06d6be2767b 100644 (file)
@@ -9,9 +9,7 @@ fragment <<EOF
 /* This file is is generated by a shell script.  DO NOT EDIT! */
 
 /* AIX emulation code for ${EMULATION_NAME}
-   Copyright 1991, 1993, 1995, 1996, 1997, 1998, 1999, 2000, 2001, 2002,
-   2003, 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2012
-   Free Software Foundation, Inc.
+   Copyright 1991-2013 Free Software Foundation, Inc.
    Written by Steve Chamberlain <sac@cygnus.com>
    AIX support by Ian Lance Taylor <ian@cygnus.com>
    AIX 64 bit support by Tom Rix <trix@redhat.com>
@@ -1110,6 +1108,7 @@ gld${EMULATION_NAME}_read_file (const char *filename, bfd_boolean import)
     {
       bfd_set_error (bfd_error_system_call);
       einfo ("%F%s: %E\n", filename);
+      return;
     }
 
   keep = FALSE;
@@ -1314,6 +1313,8 @@ gld${EMULATION_NAME}_read_file (const char *filename, bfd_boolean import)
       obstack_free (o, NULL);
       free (o);
     }
+
+  fclose (f);
 }
 
 /* This routine saves us from worrying about declaring free.  */