Skip to content
This repository has been archived by the owner on Sep 28, 2023. It is now read-only.

Buffer overflow #172

Closed
bertrandmarc opened this issue Aug 17, 2019 · 3 comments
Closed

Buffer overflow #172

bertrandmarc opened this issue Aug 17, 2019 · 3 comments

Comments

@bertrandmarc
Copy link

Dear developer,

While packaging the 2.15.2 version for Debian, I got a buffer overflow at startup. I seems to come from src/LocaleRes.cpp:110
strcat(codeset, "//TRANSLIT");
I think the length of codeset (hardcoded 16) is too short to fit "ISO-8859-1//TRANSLIT". Changing the length to 30 fixes the buffer overflow (see here).

Best Regards,
Bertrand

@the3dfxdude
Copy link
Owner

It's already fixed:
5f765d8

Your patch is wrong.

@bertrandmarc
Copy link
Author

My patch might not be good, I never claimed it was.

This is anyway not a good reason to be that rude.

@the3dfxdude
Copy link
Owner

I'm sorry you perceive rudeness. I gave you some information with the hope you'd use it since you are breaking the binary format.

Narrat added a commit to Narrat/PKGBuilds that referenced this issue Jan 10, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants