virtualx-engine/drivers
Max Hilbrunner b48cbb5da9 Fix Windows platform file access
This restores Windows platform file handling back to open files non-exlusively by default, as was the case before October 2018. (See b902a2f2a7)
Back then, while fixing warnings for MSVC, the function used for opening files was changed from _wfopen() to _wfopen_s() as suggsted by the warning C4996. ("This function may be unsafe, consider using _wfopen_s instead.")

This new function
1. did parameter validation and thus avoided some possible security issues due to nil pointers or wrongly terminated strings
2. it also changed the default file sharing for opened files from _SH_DENYNO (which was the implicit default for the previous _wfopen()) to _SH_SECURE.

_SH_DENYNO means every opened file could be opened by other calls (like is the default on other operating systems).
_SH_SECURE means if the file is opened with READ access, others can still read the same file, but if it is opened with WRITE access, others can't open it at all, not even to read.

This led to rarely occuring bugs on Windows, i.e. due to random access by Antivirus processes, or Godot/Windows not closing a file handle fast enough while trying to open it again elsewhere (i.e. project.godot, instead showing the Project manager, or saving shaders/debugging the game).

What this PR does it change the file access to a third method, _wfsopen(). This is still secure, doing parameter validation and thus avoids the warning, but it allows us to actually SET the file sharing parameter. And we set it to _SH_DENYNO, as it was implicitely before the change. (And as it currently is on all non-Windows platforms, where file sharing restrictions don't exist by default.)

Warning C4996 should really have been pointing this out. It should've been _wfsopen() all along. Let's hope this banishes those annoying, rare errors for all eternity.

Fixes #28036.
2021-08-09 14:41:38 +02:00
..
alsa Omit some functions in alsa wrapper 2021-02-22 19:22:53 +01:00
alsamidi Don't clobber original library symbols 2021-02-18 00:21:10 +01:00
coreaudio Core: Drop custom copymem/zeromem defines 2021-04-27 16:26:27 +02:00
coremidi Update copyright statements to 2021 2021-01-01 20:19:21 +01:00
png Rename instance()->instantiate() when it's a verb 2021-06-19 20:49:18 -06:00
pulseaudio Fix crash when using ALSA MIDI with PulseAudio 2021-06-03 10:10:10 +02:00
spirv-reflect SCons: Add explicit dependencies on thirdparty code in cloned env 2020-12-18 10:29:34 +01:00
unix Use const references where possible for List range iterators 2021-07-25 12:22:25 +02:00
vulkan Use subpasses to do 3D rendering and resolve in mobile renderer 2021-08-06 23:43:26 +10:00
wasapi Reorganize Project Settings 2021-02-18 11:23:34 -03:00
windows Fix Windows platform file access 2021-08-09 14:41:38 +02:00
winmidi Update copyright statements to 2021 2021-01-01 20:19:21 +01:00
xaudio2 Reorganize Project Settings 2021-02-18 11:23:34 -03:00
register_driver_types.cpp Rename instance()->instantiate() when it's a verb 2021-06-19 20:49:18 -06:00
register_driver_types.h Update copyright statements to 2021 2021-01-01 20:19:21 +01:00
SCsub LinuxBSD now compiles without vulkan/x11. 2021-06-01 16:27:54 +02:00