Fix buffer overflow in ffindex_from_fasta#265
Fix buffer overflow in ffindex_from_fasta#265konstin wants to merge 1 commit intosoedinglab:masterfrom
Conversation
I could verify with valgrind that this previously could lead to memory corruption
|
Mainline does not have this problem: |
|
@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 |
|
I wrote FFindex as the database format for Johannes' hhblits, when we were colleagues at LMU. 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. So I assume all features are present in mainline. If not just file an incident on mainline GitHub. |
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?