Rename variable named "tmp"

Please don't name variables "tmp" unless it's a temp file, but even then at least call it "tempfile"
Avoid warning from call to free
Make MMDB_get_tree void

It only ever had one return value, MMDB_SUCCESS, so why return anything at all?