Sigh. There appears to be no way to get Windows to allow us to rename a
authorGuy Harris <guy@alum.mit.edu>
Fri, 1 Jun 2012 08:05:12 +0000 (08:05 -0000)
committerGuy Harris <guy@alum.mit.edu>
Fri, 1 Jun 2012 08:05:12 +0000 (08:05 -0000)
commit129c881fcf7c8707f379a09d2ed2cdfcc178b807
treec3e2e1a9b39235d5b89594d3947e849d9152c09a
parent86c69b01e7965f37eecbe6195e797a52ab81e23e
Sigh.  There appears to be no way to get Windows to allow us to rename a
file that we ourselves have open.  In the "safe save" code path for
capture files, on Windows temporarily close the file descriptors for the
currently-open capture before doing the rename and then, if the rename
failed, reopen them, leaving the rest of the wtap and capture_file
structures intact.

Rename filed_open() to file_fdopen(), to make its name match what it
does a bit better (it's an fdopen()-style routine, i.e. do the
equivalent of an open with an already-open file descriptor rather than a
pathname, in the file_wrappers.c set of routines).

Remove the file_ routines from the .def file for Wiretap - they should
only be called by code inside Wiretap.

Closing a descriptor open for input has no reason to fail (closing a
descriptor open for *writing* could fail if the file is on a server and
dirty pages are pushed asynchronously to the server and synchronously on
a close), so just have file_close() return void.

svn path=/trunk/; revision=42961
file.c
wiretap/file_access.c
wiretap/file_wrappers.c
wiretap/file_wrappers.h
wiretap/wtap.c
wiretap/wtap.def
wiretap/wtap.h