msLoadMap does not properly close map files on parse error

[scott.e@…]

We use C# mapscript on Windows and have been seeing behavior where a map file is
locked by the process that calls msLoadMap.  Thus the file cannot be delete
until the process is finished.  This is only occurs on parse error, and is
because fclose() is not properly called in these circumstances.

Attached is a patch which rectifies this by calling fclose() in all circumstances.

[scott.e@…]

patch

[scott.e@…]

corrected patch