file.c (file_read): Copy name when duplicating input file spec.
If only one filespec is provided to the file endpoint, then it copies
the input spec (which it's already parsed) to the output. But this
isn't enough because the spec might contain a dynamically allocated
filename. The function `file_destroy' unconditionally frees both names,
which is a double-free bug.
Fix this in the stupid way, by allocating a separate copy of the
filename if we duplicate the input filespec. (I could fix it in
`file_destroy' instead, but that becomes a little fiddly, and it's
rather brittle.
~mdw / fwd / commit