zydronium/SMBLibrary
1
0
Fork 0
mirror of https://github.com/TalAloni/SMBLibrary.git synced 2025-07-04 00:33:18 +02:00
Code Issues Projects Releases Packages Wiki Activity

Server: Dispose of ReceiveBuffer after receiving invalid packets

Browse source
This commit is contained in:
Tal Aloni 2023-12-30 20:46:17 +02:00
parent e31df2a797
commit d34f014eab
1 changed files with 5 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

5
SMBLibrary/Server/SMBServer.cs
View file

@ -315,6 +315,7 @@ namespace SMBLibrary.Server
{ {
state.LogToServer(Severity.Verbose, "Rejected SMB1 message"); state.LogToServer(Severity.Verbose, "Rejected SMB1 message");
state.ClientSocket.Close(); state.ClientSocket.Close();
state.ReceiveBuffer.Dispose();
return; return;
} }
@ -327,6 +328,7 @@ namespace SMBLibrary.Server
{ {
state.LogToServer(Severity.Warning, "Invalid SMB1 message: " + ex.Message); state.LogToServer(Severity.Warning, "Invalid SMB1 message: " + ex.Message);
state.ClientSocket.Close(); state.ClientSocket.Close();
state.ReceiveBuffer.Dispose();
return; return;
} }
state.LogToServer(Severity.Verbose, "SMB1 message received: {0} requests, First request: {1}, Packet length: {2}", message.Commands.Count, message.Commands[0].CommandName.ToString(), packet.Length); state.LogToServer(Severity.Verbose, "SMB1 message received: {0} requests, First request: {1}, Packet length: {2}", message.Commands.Count, message.Commands[0].CommandName.ToString(), packet.Length);
@ -365,6 +367,7 @@ namespace SMBLibrary.Server
{ {
state.LogToServer(Severity.Verbose, "Rejected SMB2 message"); state.LogToServer(Severity.Verbose, "Rejected SMB2 message");
state.ClientSocket.Close(); state.ClientSocket.Close();
state.ReceiveBuffer.Dispose();
return; return;
} }
@ -377,6 +380,7 @@ namespace SMBLibrary.Server
{ {
state.LogToServer(Severity.Warning, "Invalid SMB2 request chain: " + ex.Message); state.LogToServer(Severity.Warning, "Invalid SMB2 request chain: " + ex.Message);
state.ClientSocket.Close(); state.ClientSocket.Close();
state.ReceiveBuffer.Dispose();
return; return;
} }
state.LogToServer(Severity.Verbose, "SMB2 request chain received: {0} requests, First request: {1}, Packet length: {2}", requestChain.Count, requestChain[0].CommandName.ToString(), packet.Length); state.LogToServer(Severity.Verbose, "SMB2 request chain received: {0} requests, First request: {1}, Packet length: {2}", requestChain.Count, requestChain[0].CommandName.ToString(), packet.Length);
@ -401,6 +405,7 @@ namespace SMBLibrary.Server
{ {
state.LogToServer(Severity.Warning, "Inappropriate NetBIOS session packet"); state.LogToServer(Severity.Warning, "Inappropriate NetBIOS session packet");
state.ClientSocket.Close(); state.ClientSocket.Close();
state.ReceiveBuffer.Dispose();
return; return;
} }
} }