Alexandre Julliard julliard@winehq.org wrote:
Requiring extra access rights can break things, especially in such low-level functions. It's not just a harmless cleanup, so you need to either show an app that requires it, or provide a test case that demonstrates that the extra accesses are required on Windows too.
At least one such a test case exists for the file mappings, and actually making it pass has motivated me for this kind of patches. It shows that a mapping created with access 0 is not accessible. Since all the objects on the server side share a single access check mechanism it's impossible to make one test pass without either changing that single mechanism, or creating separate access checks for every type of an object.