[8246] parsetexi duplicate a string

[gavinsmith0123]

Revision: 8246
          http://svn.sv.gnu.org/viewvc/?view=rev&root=texinfo&revision=8246
Author:   gavin
Date:     2018-09-29 10:41:20 -0400 (Sat, 29 Sep 2018)
Log Message:
-----------
parsetexi duplicate a string

Modified Paths:
--------------
    trunk/tp/Texinfo/XS/parsetexi/input.c
    trunk/tp/Texinfo/XS/parsetexi/macro.c
    trunk/tp/Texinfo/XS/parsetexi/tree.c

Modified: trunk/tp/Texinfo/XS/parsetexi/input.c
===================================================================
--- trunk/tp/Texinfo/XS/parsetexi/input.c       2018-09-29 10:09:44 UTC (rev 
8245)
+++ trunk/tp/Texinfo/XS/parsetexi/input.c       2018-09-29 14:41:20 UTC (rev 
8246)
@@ -431,7 +431,7 @@
   input_stack[input_number].line_nr.line_nr = line_number;
   input_stack[input_number].line_nr.file_name
                                        = filename ? strdup (filename) : 0;
-  input_stack[input_number].line_nr.macro = macro;
+  input_stack[input_number].line_nr.macro = macro ? strdup (macro) : 0;
   input_number++;
 }
 

Modified: trunk/tp/Texinfo/XS/parsetexi/macro.c
===================================================================
--- trunk/tp/Texinfo/XS/parsetexi/macro.c       2018-09-29 10:09:44 UTC (rev 
8245)
+++ trunk/tp/Texinfo/XS/parsetexi/macro.c       2018-09-29 14:41:20 UTC (rev 
8246)
@@ -486,7 +486,7 @@
 }
 
 // 3898
-/* Handle macro expandsion.  CMD is the macro command. */
+/* Handle macro expansion.  CMD is the macro command. */
 ELEMENT *
 handle_macro (ELEMENT *current, char **line_inout, enum command_id cmd)
 {

Modified: trunk/tp/Texinfo/XS/parsetexi/tree.c
===================================================================
--- trunk/tp/Texinfo/XS/parsetexi/tree.c        2018-09-29 10:09:44 UTC (rev 
8245)
+++ trunk/tp/Texinfo/XS/parsetexi/tree.c        2018-09-29 14:41:20 UTC (rev 
8246)
@@ -103,8 +103,12 @@
             {
               NODE_SPEC_EXTRA *nse = (NODE_SPEC_EXTRA *) e->extra[i].value;
 
-              free_node_contents (nse->manual_content);
-              free_node_contents (nse->node_content);
+              //free_node_contents (nse->manual_content);
+              //free_node_contents (nse->node_content);
+              /* Problem - some of the elements in 'node_content' may have
+                 been in the main tree and have been free'd already.  If
+                 that is the case, we can't rely on checking whether the
+                 elements are 'route_not_in_tree'. */
               free (nse);
               break;
             }