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

fcaseopen.c buffer overflow #29

Open
minerscale opened this issue Dec 29, 2023 · 1 comment
Open

fcaseopen.c buffer overflow #29

minerscale opened this issue Dec 29, 2023 · 1 comment

Comments

@minerscale
Copy link

Hi!

Whilst trying to open a file case-insensitively I came across a codebase that seemed to do the job. But clang's address sanitizer was having none of it! It turns out that there is an off by one error in the buffer size allocation of fcaseopen causing a null terminator to be strcpy'd into unmanaged memory. I reported the problem to the original project but I decided I'd also let everyone I could find who still had the vulnerability floating around copies of it know as well.

You can find more information about it here: OneSadCookie/fcaseopen#2

Thanks,
Aaron.

@xesf
Copy link
Member

xesf commented Dec 29, 2023

Thanks for the details. If you want a more up to date version of this codebase have a look at the ScummVM implementation here: https://github.com/scummvm/scummvm/tree/master/engines/twine

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

No branches or pull requests

2 participants