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

Hotfix CVE-2021-43518 #3018

Merged
merged 1 commit into from Feb 3, 2022
Merged

Hotfix CVE-2021-43518 #3018

merged 1 commit into from Feb 3, 2022

Conversation

Robyt3
Copy link
Contributor

@Robyt3 Robyt3 commented Jan 15, 2022

Hotfix for CVE-2021-43518. Closes #2981.

Adding all the datafile validation (#2931) ended up being a rather large patch and it still needs some work. This hotfix just addresses the exploitable stack buffer overflow write.

Fixing the other issues individually would be pointless, as there are at least 50-100 more distinct maps that can crash the client or cause ASAN/UBSAN errors.

@oy oy merged commit 7e81ae1 into teeworlds:master Feb 3, 2022
@Robyt3 Robyt3 deleted the hotfix-CVE-2021-43518 branch February 4, 2022 18:21
@ncfavier
Copy link

I'm confused, why wasn't this released?

@ChillerDragon
Copy link
Contributor

I'm confused, why wasn't this released?

I guess because the proper fix it still a work in progress in #2931

@ncfavier
Copy link

I don't understand the point of a hotfix if it isn't meant to be released immediately while the proper fix is in progress.

@ChillerDragon
Copy link
Contributor

I don't understand the point of a hotfix if it isn't meant to be released immediately while the proper fix is in progress.

Safety for developers and others who build from source :D
But yea I guess you have a point here.

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.

Stack buffer overflow (write) while loading map in CMapLayers::LoadEnvPoints, maplayers.cpp:184
4 participants