AMajor/SoftEtherVPN
1
0
Fork 0
mirror of https://github.com/SoftEtherVPN/SoftEtherVPN.git synced 2024-11-22 17:39:53 +03:00
Code Releases Activity

Cedar/Session.c: fix "use after free" in CleanupSession()

Browse Source 
Fixes #1226.
This commit is contained in:
Ilya Shipitsin 2020-10-31 19:59:57 +01:00 committed by Davide Beatrici
parent a2d15615f3
commit bb2e2ff997
1 changed files with 4 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/Cedar/Session.c
View File

@ -1223,6 +1223,9 @@ void CleanupSession(SESSION *s)
// Release the client connection options // Release the client connection options
if (s->ClientOption != NULL) if (s->ClientOption != NULL)
{ {
#ifdef OS_UNIX
UnixVLanSetState(s->ClientOption->DeviceName, false);
#endif
Free(s->ClientOption); Free(s->ClientOption);
} }
@ -1278,12 +1281,7 @@ void CleanupSession(SESSION *s)
{ {
FreePacketAdapter(s->PacketAdapter); FreePacketAdapter(s->PacketAdapter);
} }
#ifdef OS_UNIX
if (s->ClientOption != NULL)
{
UnixVLanSetState(s->ClientOption->DeviceName, false);
}
#endif
if (s->OldTraffic != NULL) if (s->OldTraffic != NULL)
{ {
FreeTraffic(s->OldTraffic); FreeTraffic(s->OldTraffic);