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

Fix buffer overflow in ffindex_from_fasta #265

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

konstin
Copy link

@konstin konstin commented May 4, 2021

I could verify with valgrind that previously that this could lead to memory corruption.

Is there any suggested way to check if that has an impact on performance?

I could verify with valgrind that this previously could lead to memory corruption
@ahcm
Copy link

ahcm commented Jul 30, 2021

Mainline does not have this problem:
https://github.com/ahcm/ffindex

@konstin
Copy link
Author

konstin commented Jul 30, 2021

@ahcm What's the relation between this repository and https://github.com/ahcm/ffindex? I was under the impression that ffindex is a part of and maintained by the hh-suite

@ahcm
Copy link

ahcm commented Jul 30, 2021

I wrote FFindex as the database format for Johannes' hhblits, when we were colleagues at LMU.
So it's quite OK to package it. Unfortunately the code changes they made range from unnecessary to dangerous, introducing strange limitations and security flaws.

The switch to cmake I can understand, but think it's not worth the trouble.

To be honest, I never understood why they forked it when they left Munich.
Debian/Ubuntu uses mainline and also their hhsuite package depends on it.

So I assume all features are present in mainline. If not just file an incident on mainline GitHub.

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