Skip to content
This repository has been archived by the owner on Jan 13, 2022. It is now read-only.

Fix memory leak in TCPInterface (CVSS score: 4.8) #125

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Luke1410
Copy link

@Luke1410 Luke1410 commented May 24, 2018

This is a backport of a security relevant fix for RakNet which was brought to our attention by a user. The issue has already been fixed in SLikeNet 0.1.1 (see https://www.slikenet.com/).
We provide this backport for people who prefer to stick with the RakNet project and also in order to easier share this fix with other RakNet forks.

CVSS Base score: 5.3
CVSS Temporal score: 4.8
CVSS Overall score: 4.8
CVSS v3 Vector: AV:N/AC:H/PR:L/UI:N/S:U/C:N/I:N/A:H/E:P/RL:O/RC:C

The security implications of the issue are medium to low in our opinion. Ultimately, the vulnerability can be utilized to run a DoS attack on a susceptible server. A successful attack will result in the server running out of memory and will eventually cause the server application to crash.
An application is however only susceptive, if it utilizes TCP connections. Furthermore, the attack is dependent on conditions which are usually outside the realm of the attacker (i.e. depends on when the server closes a TCP connection and whether internal data was already processed).

For further details, please refer to the original issue report: SLikeSoft#18

@Luke1410 Luke1410 changed the title - fix memory leak in TCPInterface (fixes #18) - fix memory leak in TCPInterface May 24, 2018
@Luke1410 Luke1410 changed the title - fix memory leak in TCPInterface Fix memory leak in TCPInterface May 24, 2018
@Luke1410 Luke1410 changed the title Fix memory leak in TCPInterface Fix memory leak in TCPInterface (CVSS score: 5.7) May 24, 2018
@Luke1410 Luke1410 changed the title Fix memory leak in TCPInterface (CVSS score: 5.7) Fix memory leak in TCPInterface (CVSS score: 4.8) May 24, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant