Adam Sampson

I ran into this as well, building with -D_FORTIFY_SOURCE=3 - it's because the snprintf calls in giftext.c that write into AsciiForm have the length set as 3 rather than 2. Here's a patch.

ftdi timings wrong with FT232RL

Ahh - the original code from zynAddSubFX was only working by accident! If it's trying to get the opaque value (which makes sense given the test above), it should have been looking at node->child->value.opaque; it happened to work because value.element.name will occupy the same place in the union. So my mxmlGetElement is wrong; it should be mxmlGetOpaque. Updated patch attached...

Fix XMLwrapper.cpp for mxml 3

Improvements to some Coverity fixes

Don't dereference an uninitialised pointer

Avoid a NULL pointer dereference

Explicitly initialise two more static pointers