Bug in ezxml

[kmurray]

Originally reported on Google Code with ID 21

We are not currently using the latest version of ezxml.  There is a bug in the version
we are using when adding tags out of order.  The code segfaults.

I tested with the latest version of ezxml and it corrected the problem.  

I encountered it using the following architecture snippet:

                <interconnect>
                    <direct name="direct1" input="ble.in" output="soft_logic.in"/>
                    <mux name="mux1" input="soft_logic.out[1:1] soft_logic.out[0:0]" output="ff[0:0].D"/>
                    <mux name="mux2" input="soft_logic.out[0:0] ff[0:0].Q" output="ble.out[0:0]"/>
                    <mux name="mux3" input="soft_logic.out[1:1] ff[0:0].Q" output="ble.out[1:1]"/>
                    <direct name="direct4" input="ble.clk" output="ff[0:0].clk"/>                                       
                </interconnect>

Reported by jeffrey.goeders on 2012-05-08 00:46:51

[kmurray]

To clarify, the bug is due to the fact that the 'direct' and 'mux' types are mixed,
instead of grouped.  Mixing will not always cause the bug - only certain orderings
will do so.

This is a known issue in past versions of ezxml.

Reported by jeffrey.goeders on 2012-05-08 00:48:02

[kmurray]

Actually, I mistakenly thought I tested with the new ezxml.  In fact I did not.

I didn't realize we had modified the ezxml.c file.  Perhaps our modifications are the
source of the bug.  Diffing the latest version of ezxml with our version, I only notice
the locations that we have changed.

For now we should just be careful in how we write architecture files.

Reported by jeffrey.goeders on 2012-05-08 18:44:50

[kmurray]

When I get the newest ezxml, add in the line numbers update, and run, it seg faults.
 Will probably need to do what Ted Campbell did to fix the numerous memory access bugs
in ezxml if we want to use the newest version.

Reported by JasonKaiLuu on 2012-05-09 20:23:17

[kmurray]

Fixed in 33e01c8, as ezxml has been replaced with the more robust pugixml.