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

Prevent crash in fm_remove_free. #722

Closed
wants to merge 1 commit into from
Closed

Conversation

shimaore
Copy link
Contributor

@shimaore shimaore commented Dec 9, 2015

This is a workaround for #721
This doesn't address the base question (why would a fragement have ->prev set to NULL), so this isn't a fix, but it should prevent crashes for this case.

@razvancrainea
Copy link
Member

Unfortunately this issue is a bit more delicate. The problem is that ->prev should never be NULL, otherwise it wouldn't have been in the qm->free_hash[hash] hash.
This is a concurrency problem we are tracing for a while, but unfortunately we couldn't find a fix for it.

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

Successfully merging this pull request may close these issues.

None yet

2 participants