DEADSOFTWARE

Net: Add sanity checks. Forward ports only after host is created.
authorStas'M <x86corez@gmail.com>
Sat, 14 Mar 2020 19:50:40 +0000 (22:50 +0300)
committerStas'M <x86corez@gmail.com>
Sat, 14 Mar 2020 19:50:40 +0000 (22:50 +0300)
Addendum to 10f31fd.

src/game/g_net.pas

index 07d9713477ed7da1251d54bfffac53b3cfd5d9b7..7dad24c613d1a30243f263f945b77e714f783353 100644 (file)
@@ -1492,8 +1492,6 @@ begin
   NetAddr.host := IPAddr;
   NetAddr.port := Port;
 
-  if NetForwardPorts then NetPortThread := BeginThread(ForwardThread);
-
   NetHost := enet_host_create(@NetAddr, NET_MAXCLIENTS, NET_CHANS, 0, 0);
 
   if (NetHost = nil) then
@@ -1504,6 +1502,8 @@ begin
     Exit;
   end;
 
+  if NetForwardPorts then NetPortThread := BeginThread(ForwardThread);
+
   NetPongSock := enet_socket_create(ENET_SOCKET_TYPE_DATAGRAM);
   if NetPongSock <> ENET_SOCKET_NULL then
   begin
@@ -1606,7 +1606,7 @@ var
   Ping: array [0..9] of Byte;
   NPl: Byte;
 begin
-  if NetPongSock = ENET_SOCKET_NULL then Exit;
+  if (NetPongSock = ENET_SOCKET_NULL) or (NetHost = nil) then Exit;
 
   Buf.data := Addr(Ping[0]);
   Buf.dataLength := 2+8;
@@ -2238,6 +2238,9 @@ var
 begin
   Result := False;
 
+  if NetHost = nil then
+    exit;
+
   if NetPortForwarded = NetHost.address.port then
   begin
     Result := True;