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

Segfault when shutting down database which got encrypted by another process [CORE6501] #6731

Closed
firebird-automations opened this issue Mar 3, 2021 · 4 comments

Comments

@firebird-automations
Copy link
Collaborator

Submitted by: @AlexPeshkoff

When database is shutting down header page should be fetched and next written to database in order to update transaction numbers. If AST, causing change of encryption state, arrived during DB shutdown right before TRA_update_counters() database encryption manager is already missing environment required to load encryption plugin (like attachment) and segfaults happens when plugin is tried to be loaded.

Update of DB header does not require presence of database crypt plugin - it's anyway not encrypted page. Therefore fix is ti check environment before attempt to load plugin.

Commits: 64fe26f 18019a6

====== Test Details ======

Crash occured several times on Linux only, during execution of test 'functional.util.gbak_zip'.
Separate test not required.

@firebird-automations
Copy link
Collaborator Author

Modified by: @AlexPeshkoff

assignee: Alexander Peshkov [ alexpeshkoff ]

@firebird-automations
Copy link
Collaborator Author

Modified by: @AlexPeshkoff

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

resolution: Fixed [ 1 ]

Fix Version: 4.0.0 [ 10931 ]

Fix Version: 3.0.8 [ 10960 ]

@firebird-automations
Copy link
Collaborator Author

Modified by: @pavel-zotov

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

QA Status: No test => Covered by another test(s)

Test Details: Crash occured several times on Linux only, during execution of test 'functional.util.gbak_zip'.
Separate test not required.

@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