]> git.saurik.com Git - bison.git/commitdiff
fix a memory leak
authorAkim Demaille <akim@lrde.epita.fr>
Wed, 29 May 2013 14:26:53 +0000 (16:26 +0200)
committerAkim Demaille <akim@lrde.epita.fr>
Thu, 30 May 2013 09:07:57 +0000 (11:07 +0200)
* src/print-xml.c (num_escape_bufs): New.
(print_xml): Be sure to release all the escape_bufs.

src/print-xml.c

index a3ef15117d773647f9305305aa6a10aef27042a1..e5c25c544f80144563fe82e843200ee5185cb203 100644 (file)
@@ -45,7 +45,8 @@ struct escape_buf
   char *ptr;
   size_t size;
 };
-static struct escape_buf escape_bufs[3];
+enum { num_escape_bufs = 3 };
+static struct escape_buf escape_bufs[num_escape_bufs];
 
 
 /*--------------------------------.
@@ -496,7 +497,6 @@ xml_escape (char const *str)
 void
 print_xml (void)
 {
-  state_number i;
   int level = 0;
 
   FILE *out = xfopen (spec_xml_file, "w");
@@ -522,8 +522,11 @@ print_xml (void)
   /* print automaton */
   fputc ('\n', out);
   xml_puts (out, level + 1, "<automaton>");
-  for (i = 0; i < nstates; i++)
-    print_state (out, level + 2, states[i]);
+  {
+    state_number i;
+    for (i = 0; i < nstates; i++)
+      print_state (out, level + 2, states[i]);
+  }
   xml_puts (out, level + 1, "</automaton>");
 
   bitset_free (no_reduce_set);
@@ -531,8 +534,11 @@ print_xml (void)
 
   xml_puts (out, 0, "</bison-xml-report>");
 
-  free (escape_bufs[0].ptr);
-  free (escape_bufs[1].ptr);
+  {
+    int i;
+    for (i = 0; i < num_escape_bufs; ++i)
+      free (escape_bufs[i].ptr);
+  }
 
   xfclose (out);
 }