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

Out of memory condition in OS causes AV [CORE1983] #2421

Closed
firebird-automations opened this issue Jul 8, 2008 · 7 comments
Closed

Out of memory condition in OS causes AV [CORE1983] #2421

firebird-automations opened this issue Jul 8, 2008 · 7 comments

Comments

@firebird-automations
Copy link
Collaborator

Submitted by: @AlexPeshkoff

mmap() call returns MAP_FAILED (-1) in case of any error, including out of memory condition.
In alloc.cpp:external_alloc() value, returned by MMAP, is returned directly (except Solaris), no matter of the fact that caller expects NULL in case of allocation error, not -1. Therefore (void*)(-1) is used as a normally allocated memory block, definitely causing segfault at the moment of first use.

Commits: 3f3a206 d6d7df3 4d2e830

@firebird-automations
Copy link
Collaborator Author

Modified by: @AlexPeshkoff

assignee: Alexander Peshkov [ alexpeshkoff ]

@firebird-automations
Copy link
Collaborator Author

Modified by: @AlexPeshkoff

Version: 1.5.5 [ 10220 ] =>

Version: 1.5.4 [ 10100 ] =>

@firebird-automations
Copy link
Collaborator Author

Modified by: @AlexPeshkoff

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

Target: 2.5 Beta 1, 2.1.2, 2.0.5 [ 10251, 10270, 10222 ]

@firebird-automations
Copy link
Collaborator Author

Modified by: @AlexPeshkoff

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

resolution: Fixed [ 1 ]

Fix Version: 2.5 Beta 1 [ 10251 ]

Fix Version: 2.1.2 [ 10270 ]

Fix Version: 2.0.5 [ 10222 ]

@firebird-automations
Copy link
Collaborator Author

Modified by: @pcisar

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

@firebird-automations
Copy link
Collaborator Author

Modified by: @pavel-zotov

QA Status: No test

@firebird-automations
Copy link
Collaborator Author

Modified by: @pavel-zotov

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

QA Status: No test => Cannot be tested

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