anewer icon indicating copy to clipboard operation
anewer copied to clipboard

Implicitly check for file existence

Open stoeckmann opened this issue 3 years ago • 0 comments

Checking if a file exists by using its pathname and later on opening the file by using its pathname again leads to a TOCTOU race condition.

Just try to open the file. If it fails because the file does not exist then we move on without error output. If it exists but cannot be open, e.g. because read permissions are missing, then error out. Otherwise there is no further need to check for availability because it has been already opened.

The file could also be opened with read/write permission so it does not have to be opened writable again. But this should be a different commit, if it is even desired behavior.

stoeckmann avatar Mar 15 '22 19:03 stoeckmann