"filename" memory should be freed on exit/failure

```OpenDataFiles()``` uses a ```filename``` variable to construct filenames, and this variable is initialized with a ```malloc(...)``` call. However, it is not freed when the function exits on failure.