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
[Bug]: Use after free in network invalid packet error path #11016
Labels
bug
Something isn't working
Comments
JGRennison
added a commit
to JGRennison/Upstream-OpenTTD
that referenced
this issue
Jun 16, 2023
…ndlers This fixes various use after free scenarios in error handling paths
JGRennison
added a commit
to JGRennison/Upstream-OpenTTD
that referenced
this issue
Jun 16, 2023
…ndlers This fixes various use after free scenarios in error handling paths
JGRennison
added a commit
to JGRennison/Upstream-OpenTTD
that referenced
this issue
Jun 17, 2023
…ndlers This fixes various use after free scenarios in error handling paths
LordAro
pushed a commit
to LordAro/OpenTTD
that referenced
this issue
Jun 28, 2023
…ndlers This fixes various use after free scenarios in error handling paths
LordAro
pushed a commit
that referenced
this issue
Jun 28, 2023
This fixes various use after free scenarios in error handling paths
mrmbernardi
pushed a commit
to mrmbernardi/OpenTTD
that referenced
this issue
Jul 2, 2023
…ndlers This fixes various use after free scenarios in error handling paths
shoter
pushed a commit
to shoter/OpenTTD
that referenced
this issue
Jul 16, 2023
…ndlers This fixes various use after free scenarios in error handling paths
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Version of OpenTTD
master
Expected result
No bugs, etc
Actual result
CloseConnection in NetworkGameSocketHandler::HandlePacket calls
delete this
.this
is still used afterwards in NetworkGameSocketHandler::HandlePacket.In the case of clients,
this
then gets used to try to send an error to the server inClientNetworkGameSocketHandler::Receive
which fairly reliably causes a crash.Steps to reproduce
See above and/or inject a packet fault.
The text was updated successfully, but these errors were encountered: