[Wireshark-dev] [Patch] Process all XML dtds directories

[Mike Duigou]

The enclosed patch extends the way in which the xml dissector loads DTD definitions. Rather than loading only a single dtd directory this patch will cause the contents of *both* the ~/.wireshark/dtds (user) and the /usr/local/share/wireshark/dtds (built-ins) to be loaded.

Mike

The current dtd parsing logic doesn't have any special handling for the same DTD being defined more than once (and this patch doesn't change that). Specifically this means you can't reliably override a built-in DTD by installing an updated DTD in your ~/.wireshark/dtds directory. Additionally, I wanted to create the ~/.wireshark/dtds directory if it doesn't already exist but wasn't sure if there was a clean way to do it. The required utilities seem to be available in 'file_utils.c' but that I wasn't sure if a dissector should be using that functionality. Any reason that file_utils can't be made more global?

alldtds.18765.patch.gz
Description: GNU Zip compressed data

_______________________________________________
Wireshark-dev mailing list
Wireshark-dev@wireshark.org
http://www.wireshark.org/mailman/listinfo/wireshark-dev