Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Windows "SetEndOfFile" method should not be used with memory mapped files [CORE4987] #5278

Closed
firebird-automations opened this issue Nov 2, 2015 · 7 comments

Comments

@firebird-automations
Copy link
Collaborator

Submitted by: Hristo Venev (h.venev)

According to the documentation released by Microsoft, SetEndOfFile should not work on a file that has memory mappings. This check was implemented in wine-staging 1.7.50. I found this issue while trying to find out why a piece of Windows-only software was not working on wine. I think the issue is in src/jrd/isc_sync.cpp.

@firebird-automations
Copy link
Collaborator Author

Commented by: Sean Leyne (seanleyne)

Edited Summary for readiblity

@firebird-automations
Copy link
Collaborator Author

Modified by: Sean Leyne (seanleyne)

summary: SetEndOfFile called with mapped file => Windows "SetEndOfFile" method should not be used with memory mapped files

@firebird-automations
Copy link
Collaborator Author

Modified by: @hvlad

assignee: Vlad Khorsun [ hvlad ]

@firebird-automations
Copy link
Collaborator Author

Commented by: @hvlad

It was fixed in commit

6a85b46

@firebird-automations
Copy link
Collaborator Author

Modified by: @hvlad

status: Open [ 1 ] => Resolved [ 5 ]

resolution: Fixed [ 1 ]

Fix Version: 4.0 Alpha 1 [ 10731 ]

Fix Version: 3.0.0 [ 10740 ]

@firebird-automations
Copy link
Collaborator Author

Modified by: @pavel-zotov

status: Resolved [ 5 ] => Resolved [ 5 ]

QA Status: No test => Cannot be tested

@firebird-automations
Copy link
Collaborator Author

Modified by: @pavel-zotov

status: Resolved [ 5 ] => Closed [ 6 ]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants