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
Database validation bug can cause AV [CORE4510] #4829
Comments
Modified by: @hvladassignee: Vlad Khorsun [ hvlad ] |
Commented by: Sean Leyne (seanleyne) 1- The description does not really make sense, a clearer explanation is required. 2- Should this be associated with "gfix" component? |
Commented by: @hvlad Sean 1. make better, if you can |
Commented by: Sean Leyne (seanleyne) 1- If I followed what the condition was, I would try to edit. Is the problem that: a database can have pages allocated/defined but not have PIP page(s) not created/initialized for the pages (due to db growth factor)? |
Commented by: @hvlad No, it is not a problem. It is normal state. The problem is that validation code have a bug which is triggered by this normal state only and this bug leads to AV. |
Modified by: @hvladstatus: Open [ 1 ] => Resolved [ 5 ] resolution: Fixed [ 1 ] Fix Version: 3.0 Beta 1 [ 10332 ] |
Commented by: Sean Leyne (seanleyne) OK So, why is this not a GFIX problem, since it is the validation logic which has the bug which causes the AV? |
Commented by: @dyemanov Probably because the database validation code resides inside the engine, GFIX is just an interface to that. |
Commented by: Sean Leyne (seanleyne) Aware and understood, but I look at the problem as follows: |
Commented by: @dyemanov Validation can also be performed via plain API (exacty what gfix does internally) and via Services API. So maybe both "engine" and "gfix" components should be assigned. |
Commented by: @hvlad Issue, assigned to component X means issue within that component, isn't is ? |
Modified by: Sean Leyne (seanleyne)summary: AV in validation is possible => Database validation bug can cause AV |
Modified by: Sean Leyne (seanleyne)Component: GFIX [ 10004 ] |
Commented by: @asfernandes I think both components should be listed. |
Commented by: Sean Leyne (seanleyne) Have edited the subject and link the case to the "engine" and "gfix" components. |
Modified by: @pcisarstatus: Resolved [ 5 ] => Closed [ 6 ] |
Modified by: @pavel-zotovstatus: Closed [ 6 ] => Closed [ 6 ] QA Status: No test => Cannot be tested |
Submitted by: @hvlad
When database physically contains more than pagesPerPIP pages but really used is less than pagesPerPIP pages (i.e. second PIP page is not initialized yet) - it is possible to get AV on validation.
Commits: 1445269 FirebirdSQL/fbt-repository@5b08e97
The text was updated successfully, but these errors were encountered: