]> de.git.xonotic.org Git - xonotic/darkplaces.git/blobdiff - netconn.c
Lots of str[n]cat, str[n]cpy, and [v]sprintf have been replaced by strlcat, strlcpy...
[xonotic/darkplaces.git] / netconn.c
index a297a054a879000bbc276f81117759adc1258737..c37a42b481e6cf521cfa8efebda262e80d7e10e5 100755 (executable)
--- a/netconn.c
+++ b/netconn.c
@@ -35,7 +35,6 @@ static cvar_t sv_masters [] =
        {CVAR_SAVE, "sv_master3", ""},
        {CVAR_SAVE, "sv_master4", ""},
        {0, "sv_masterextra1", "198.88.152.4"},
-       {0, "sv_masterextra2", "68.102.242.12"},
        {0, NULL, NULL}
 };
 
@@ -69,6 +68,12 @@ static int unreliableMessagesReceived = 0;
 static int reliableMessagesSent = 0;
 static int reliableMessagesReceived = 0;
 
+double masterquerytime = -1000;
+int masterquerycount = 0;
+int masterreplycount = 0;
+int serverquerycount = 0;
+int serverreplycount = 0;
+
 int hostCacheCount = 0;
 hostcache_t hostcache[HOSTCACHESIZE];
 
@@ -317,20 +322,40 @@ void NetConn_CloseClientPorts(void)
                        LHNET_CloseSocket(cl_sockets[cl_numsockets - 1]);
 }
 
+void NetConn_OpenClientPort(const char *addressstring, int defaultport)
+{
+       lhnetaddress_t address;
+       lhnetsocket_t *s;
+       char addressstring2[1024];
+       if (LHNETADDRESS_FromString(&address, addressstring, defaultport))
+       {
+               if ((s = LHNET_OpenSocket_Connectionless(&address)))
+               {
+                       cl_sockets[cl_numsockets++] = s;
+                       LHNETADDRESS_ToString(LHNET_AddressFromSocket(s), addressstring2, sizeof(addressstring2), true);
+                       Con_Printf("Client opened a socket on address %s\n", addressstring2);
+               }
+               else
+               {
+                       LHNETADDRESS_ToString(&address, addressstring2, sizeof(addressstring2), true);
+                       Con_Printf("Client failed to open a socket on address %s\n", addressstring2);
+               }
+       }
+       else
+               Con_Printf("Client unable to parse address %s\n", addressstring);
+}
+
 void NetConn_OpenClientPorts(void)
 {
        int port;
-       lhnetaddress_t address;
        NetConn_CloseClientPorts();
        port = bound(0, cl_netport.integer, 65535);
        if (cl_netport.integer != port)
                Cvar_SetValueQuick(&cl_netport, port);
-       LHNETADDRESS_FromString(&address, "local", port);
-       cl_sockets[cl_numsockets++] = LHNET_OpenSocket_Connectionless(&address);
-       LHNETADDRESS_FromString(&address, cl_netaddress.string, port);
-       cl_sockets[cl_numsockets++] = LHNET_OpenSocket_Connectionless(&address);
-       LHNETADDRESS_FromString(&address, cl_netaddress_ipv6.string, port);
-       cl_sockets[cl_numsockets++] = LHNET_OpenSocket_Connectionless(&address);
+       Con_Printf("Client using port %i\n", port);
+       NetConn_OpenClientPort("local", port);
+       NetConn_OpenClientPort(cl_netaddress.string, port);
+       NetConn_OpenClientPort(cl_netaddress_ipv6.string, port);
 }
 
 void NetConn_CloseServerPorts(void)
@@ -340,25 +365,48 @@ void NetConn_CloseServerPorts(void)
                        LHNET_CloseSocket(sv_sockets[sv_numsockets - 1]);
 }
 
+void NetConn_OpenServerPort(const char *addressstring, int defaultport)
+{
+       lhnetaddress_t address;
+       lhnetsocket_t *s;
+       char addressstring2[1024];
+       if (LHNETADDRESS_FromString(&address, addressstring, defaultport))
+       {
+               if ((s = LHNET_OpenSocket_Connectionless(&address)))
+               {
+                       sv_sockets[sv_numsockets++] = s;
+                       LHNETADDRESS_ToString(LHNET_AddressFromSocket(s), addressstring2, sizeof(addressstring2), true);
+                       Con_Printf("Server listening on address %s\n", addressstring2);
+               }
+               else
+               {
+                       LHNETADDRESS_ToString(&address, addressstring2, sizeof(addressstring2), true);
+                       Con_Printf("Server failed to open socket on address %s\n", addressstring2);
+               }
+       }
+       else
+               Con_Printf("Server unable to parse address %s\n", addressstring);
+}
+
 void NetConn_OpenServerPorts(int opennetports)
 {
        int port;
-       lhnetaddress_t address;
        NetConn_CloseServerPorts();
        port = bound(0, sv_netport.integer, 65535);
        if (port == 0)
                port = 26000;
+       Con_Printf("Server using port %i\n", port);
        if (sv_netport.integer != port)
                Cvar_SetValueQuick(&sv_netport, port);
-       LHNETADDRESS_FromString(&address, "local", port);
-       sv_sockets[sv_numsockets++] = LHNET_OpenSocket_Connectionless(&address);
+       if (cls.state != ca_dedicated)
+               NetConn_OpenServerPort("local", port);
        if (opennetports)
        {
-               LHNETADDRESS_FromString(&address, sv_netaddress.string, port);
-               sv_sockets[sv_numsockets++] = LHNET_OpenSocket_Connectionless(&address);
-               LHNETADDRESS_FromString(&address, sv_netaddress_ipv6.string, port);
-               sv_sockets[sv_numsockets++] = LHNET_OpenSocket_Connectionless(&address);
+               NetConn_OpenServerPort(sv_netaddress.string, port);
+               NetConn_OpenServerPort(sv_netaddress_ipv6.string, port);
        }
+       if (sv_numsockets == 0)
+               Host_Error("NetConn_OpenServerPorts: unable to open any ports!\n");
 }
 
 lhnetsocket_t *NetConn_ChooseClientSocketForAddress(lhnetaddress_t *address)
@@ -426,17 +474,17 @@ static int clientport2 = -1;
 static int hostport = -1;
 static void NetConn_UpdateServerStuff(void)
 {
-       if (clientport2 != cl_netport.integer)
-       {
-               clientport2 = cl_netport.integer;
-               if (cls.state == ca_connected)
-                       Con_Printf("Changing \"cl_port\" will not take effect until you reconnect.\n");
-       }
        if (cls.state != ca_dedicated)
        {
-               if (cls.state == ca_disconnected && clientport != cl_netport.integer)
+               if (clientport2 != cl_netport.integer)
                {
-                       clientport = cl_netport.integer;
+                       clientport2 = cl_netport.integer;
+                       if (cls.state == ca_connected)
+                               Con_Printf("Changing \"cl_port\" will not take effect until you reconnect.\n");
+               }
+               if (cls.state == ca_disconnected && clientport != clientport2)
+               {
+                       clientport = clientport2;
                        NetConn_CloseClientPorts();
                }
                if (cl_numsockets == 0)
@@ -578,15 +626,8 @@ void NetConn_ConnectionEstablished(lhnetsocket_t *mysocket, lhnetaddress_t *peer
 
 int NetConn_IsLocalGame(void)
 {
-       int i;
-       if (cls.state == ca_connected && sv.active/* && LHNETADDRESS_GetAddressType(cl.netcon->peeraddress) == LHNETADDRESSTYPE_LOOP*/)
-       {
-               // make sure there are no other connected clients
-               for (i = 1;i < MAX_SCOREBOARD;i++)
-                       if (svs.connectedclients[i])
-                               return false;
+       if (cls.state == ca_connected && sv.active && cl.maxclients == 1)
                return true;
-       }
        return false;
 }
 
@@ -654,81 +695,74 @@ int NetConn_ClientParsePacket(lhnetsocket_t *mysocket, qbyte *data, int length,
                        string += 13;
                        // hostcache only uses text addresses
                        LHNETADDRESS_ToString(peeraddress, cname, sizeof(cname), true);
-                       // search the cache for this server
-                       for (n = 0; n < hostCacheCount; n++)
-                               if (!strcmp(cname, hostcache[n].cname))
-                                       break;
-                       // add it or update it
-                       if (n == hostCacheCount)
-                       {
-                               // if cache is full replace highest ping server (the list is
-                               // kept sorted so this is always the last, and if this server
-                               // is good it will be sorted into an early part of the list)
-                               if (hostCacheCount >= HOSTCACHESIZE)
-                                       n = hostCacheCount - 1;
-                               else
-                                       hostCacheCount++;
-                       }
-                       if ((s = SearchInfostring(string, "gamename"     )) != NULL) strncpy(game, s, sizeof(game) - 1);else game[0] = 0;
-                       if ((s = SearchInfostring(string, "modname"      )) != NULL) strncpy(mod , s, sizeof(mod ) - 1);else mod[0] = 0;
-                       if ((s = SearchInfostring(string, "mapname"      )) != NULL) strncpy(map , s, sizeof(map ) - 1);else map[0] = 0;
-                       if ((s = SearchInfostring(string, "hostname"     )) != NULL) strncpy(name, s, sizeof(name) - 1);else name[0] = 0;
+                       if ((s = SearchInfostring(string, "gamename"     )) != NULL) strlcpy(game, s, sizeof (game));else game[0] = 0;
+                       if ((s = SearchInfostring(string, "modname"      )) != NULL) strlcpy(mod , s, sizeof (mod ));else mod[0]  = 0;
+                       if ((s = SearchInfostring(string, "mapname"      )) != NULL) strlcpy(map , s, sizeof (map ));else map[0]  = 0;
+                       if ((s = SearchInfostring(string, "hostname"     )) != NULL) strlcpy(name, s, sizeof (name));else name[0] = 0;
                        if ((s = SearchInfostring(string, "protocol"     )) != NULL) c = atoi(s);else c = -1;
                        if ((s = SearchInfostring(string, "clients"      )) != NULL) users = atoi(s);else users = 0;
                        if ((s = SearchInfostring(string, "sv_maxclients")) != NULL) maxusers = atoi(s);else maxusers = 0;
-                       pingtime = 9999;
-                       for (i = 0;i < HOSTCACHESIZE;i++)
-                               if (!LHNETADDRESS_Compare(peeraddress, &pingcache[i].peeraddress))
-                                       pingtime = (int)(realtime - pingcache[i].senttime);
-                       pingtime = bound(0, pingtime, 9999);
-                       memset(&hostcache[n], 0, sizeof(hostcache[n]));
-                       // store the data the engine cares about (address and ping)
-                       strcpy(hostcache[n].cname, cname);
-                       hostcache[n].ping = pingtime;
-                       // build description strings for the things users care about
-                       snprintf(hostcache[n].line1, sizeof(hostcache[n].line1), "%5d%c%3u/%3u %-65.65s", (int)pingtime, c != NET_PROTOCOL_VERSION ? '*' : ' ', users, maxusers, name);
-                       snprintf(hostcache[n].line2, sizeof(hostcache[n].line2), "%-21.21s %-19.19s %-17.17s %-20.20s", cname, game, mod, map);
-                       // if ping is especially high, display it as such
-                       if (pingtime >= 300)
-                       {
-                               // orange numbers (lower block)
-                               for (i = 0;i < 5;i++)
-                                       if (hostcache[n].line1[i] != ' ')
-                                               hostcache[n].line1[i] += 128;
-                       }
-                       else if (pingtime >= 200)
-                       {
-                               // yellow numbers (in upper block)
-                               for (i = 0;i < 5;i++)
-                                       if (hostcache[n].line1[i] != ' ')
-                                               hostcache[n].line1[i] -= 30;
-                       }
-                       // if not in the slist menu we should print the server to console
-                       if (m_state != m_slist)
-                               Con_Printf("%s\n%s\n", hostcache[n].line1, hostcache[n].line2);
-                       // and finally, re-sort the list
-                       for (i = 0;i < hostCacheCount;i++)
+                       // search the cache for this server and update it
+                       for (n = 0; n < hostCacheCount; n++)
                        {
-                               for (j = i + 1;j < hostCacheCount;j++)
+                               if (!strcmp(cname, hostcache[n].cname))
                                {
-                                       //if (strcmp(hostcache[j].name, hostcache[i].name) < 0)
-                                       if (hostcache[i].ping > hostcache[j].ping)
+                                       if (hostcache[n].ping == 100000)
+                                               serverreplycount++;
+                                       pingtime = (int)((realtime - hostcache[n].querytime) * 1000.0);
+                                       pingtime = bound(0, pingtime, 9999);
+                                       // update the ping
+                                       hostcache[n].ping = pingtime;
+                                       // build description strings for the things users care about
+                                       snprintf(hostcache[n].line1, sizeof(hostcache[n].line1), "%5d%c%3u/%3u %-65.65s", (int)pingtime, c != NET_PROTOCOL_VERSION ? '*' : ' ', users, maxusers, name);
+                                       snprintf(hostcache[n].line2, sizeof(hostcache[n].line2), "%-21.21s %-19.19s %-17.17s %-20.20s", cname, game, mod, map);
+                                       // if ping is especially high, display it as such
+                                       if (pingtime >= 300)
+                                       {
+                                               // orange numbers (lower block)
+                                               for (i = 0;i < 5;i++)
+                                                       if (hostcache[n].line1[i] != ' ')
+                                                               hostcache[n].line1[i] += 128;
+                                       }
+                                       else if (pingtime >= 200)
                                        {
-                                               memcpy(&temp, &hostcache[j], sizeof(hostcache_t));
-                                               memcpy(&hostcache[j], &hostcache[i], sizeof(hostcache_t));
-                                               memcpy(&hostcache[i], &temp, sizeof(hostcache_t));
+                                               // yellow numbers (in upper block)
+                                               for (i = 0;i < 5;i++)
+                                                       if (hostcache[n].line1[i] != ' ')
+                                                               hostcache[n].line1[i] -= 30;
                                        }
+                                       // if not in the slist menu we should print the server to console
+                                       if (m_state != m_slist)
+                                               Con_Printf("%s\n%s\n", hostcache[n].line1, hostcache[n].line2);
+                                       // and finally, re-sort the list
+                                       for (i = 0;i < hostCacheCount;i++)
+                                       {
+                                               for (j = i + 1;j < hostCacheCount;j++)
+                                               {
+                                                       //if (strcmp(hostcache[j].name, hostcache[i].name) < 0)
+                                                       if (hostcache[i].ping > hostcache[j].ping)
+                                                       {
+                                                               memcpy(&temp, &hostcache[j], sizeof(hostcache_t));
+                                                               memcpy(&hostcache[j], &hostcache[i], sizeof(hostcache_t));
+                                                               memcpy(&hostcache[i], &temp, sizeof(hostcache_t));
+                                                       }
+                                               }
+                                       }
+                                       break;
                                }
                        }
                        return true;
                }
                if (!strncmp(string, "getserversResponse\\", 19) && hostCacheCount < HOSTCACHESIZE)
                {
-                       int i, best;
-                       double besttime;
+                       int i, n, j;
+                       hostcache_t temp;
                        // Extract the IP addresses
                        data += 18;
                        length -= 18;
+                       masterreplycount++;
+                       if (m_state != m_slist)
+                               Con_Printf("received server list...\n");
                        while (length >= 7 && data[0] == '\\' && (data[1] != 0xFF || data[2] != 0xFF || data[3] != 0xFF || data[4] != 0xFF) && data[5] * 256 + data[6] != 0)
                        {
                                sprintf(ipstring, "%u.%u.%u.%u:%u", data[1], data[2], data[3], data[4], (data[5] << 8) | data[6]);
@@ -736,28 +770,54 @@ int NetConn_ClientParsePacket(lhnetsocket_t *mysocket, qbyte *data, int length,
                                        Con_Printf("Requesting info from server %s\n", ipstring);
                                LHNETADDRESS_FromString(&svaddress, ipstring, 0);
                                NetConn_WriteString(mysocket, "\377\377\377\377getinfo", &svaddress);
-                               // replace oldest or matching entry in ping cache
-                               // we scan this later when getting a reply to see how long it took
-                               besttime = realtime;
-                               best = 0;
-                               for (i = 0;i < HOSTCACHESIZE;i++)
+
+
+                               // add to slist (hostCache)
+                               // search the cache for this server
+                               for (n = 0; n < hostCacheCount; n++)
+                                       if (!strcmp(ipstring, hostcache[n].cname))
+                                               break;
+                               // add it or update it
+                               if (n == hostCacheCount)
                                {
-                                       if (!LHNETADDRESS_Compare(&svaddress, &pingcache[i].peeraddress))
+                                       // if cache is full replace highest ping server (the list is
+                                       // kept sorted so this is always the last, and if this server
+                                       // is good it will be sorted into an early part of the list)
+                                       if (hostCacheCount >= HOSTCACHESIZE)
+                                               n = hostCacheCount - 1;
+                                       else
                                        {
-                                               best = i;
-                                               break;
+                                               serverquerycount++;
+                                               hostCacheCount++;
                                        }
-                                       if (besttime > pingcache[i].senttime)
+                               }
+                               memset(&hostcache[n], 0, sizeof(hostcache[n]));
+                               // store the data the engine cares about (address and ping)
+                               strcpy(hostcache[n].cname, ipstring);
+                               hostcache[n].ping = 100000;
+                               hostcache[n].querytime = realtime;
+                               // build description strings for the things users care about
+                               snprintf(hostcache[n].line1, sizeof(hostcache[n].line1), "?");
+                               snprintf(hostcache[n].line2, sizeof(hostcache[n].line2), "%s", ipstring);
+                               // if not in the slist menu we should print the server to console
+                               if (m_state != m_slist)
+                                       Con_Printf("querying %s\n", ipstring);
+                               // and finally, re-sort the list
+                               for (i = 0;i < hostCacheCount;i++)
+                               {
+                                       for (j = i + 1;j < hostCacheCount;j++)
                                        {
-                                               besttime = pingcache[i].senttime;
-                                               best = i;
-                                               // if ping cache isn't full yet we can skip out early
-                                               if (!besttime)
-                                                       break;
+                                               //if (strcmp(hostcache[j].name, hostcache[i].name) < 0)
+                                               if (hostcache[i].ping > hostcache[j].ping)
+                                               {
+                                                       memcpy(&temp, &hostcache[j], sizeof(hostcache_t));
+                                                       memcpy(&hostcache[j], &hostcache[i], sizeof(hostcache_t));
+                                                       memcpy(&hostcache[i], &temp, sizeof(hostcache_t));
+                                               }
                                        }
                                }
-                               pingcache[best].peeraddress = svaddress;
-                               pingcache[best].senttime = realtime;
+
+
                                // move on to next address in packet
                                data += 7;
                                length -= 7;
@@ -809,7 +869,7 @@ int NetConn_ClientParsePacket(lhnetsocket_t *mysocket, qbyte *data, int length,
                        if (developer.integer)
                                Con_Printf("Datagram_ParseConnectionless: received CCREP_REJECT from %s.\n", addressstring2);
                        Con_Printf("%s\n", data);
-                       strncpy(m_return_reason, data, sizeof(m_return_reason) - 1);
+                       strlcpy (m_return_reason, data, sizeof (m_return_reason));
                        break;
 #if 0
                case CCREP_SERVER_INFO:
@@ -838,11 +898,9 @@ int NetConn_ClientParsePacket(lhnetsocket_t *mysocket, qbyte *data, int length,
                                        c = MSG_ReadByte();
                                        if (c != NET_PROTOCOL_VERSION)
                                        {
-                                               strncpy(hostcache[n].cname, hostcache[n].name, sizeof(hostcache[n].cname) - 1);
-                                               hostcache[n].cname[sizeof(hostcache[n].cname) - 1] = 0;
+                                               strlcpy (hostcache[n].cname, hostcache[n].name, sizeof (hostcache[n].cname));
                                                strcpy(hostcache[n].name, "*");
-                                               strncat(hostcache[n].name, hostcache[n].cname, sizeof(hostcache[n].name) - 1);
-                                               hostcache[n].name[sizeof(hostcache[n].name) - 1] = 0;
+                                               strlcat (hostcache[n].name, hostcache[n].cname, sizeof(hostcache[n].name));
                                        }
                                        strcpy(hostcache[n].cname, cname);
                                }
@@ -883,13 +941,26 @@ void NetConn_ClientFrame(void)
                if (cls.connect_remainingtries == 0)
                {
                        cls.connect_trying = false;
-                       Host_Error("Connect failed\n");
+                       if (m_state == m_slist)
+                               strcpy(m_return_reason, "Connect: Failed");
+                       else
+                               Con_Printf("Connect failed\n");
                        return;
                }
                if (cls.connect_nextsendtime)
-                       Con_Printf("Still trying...\n");
+               {
+                       if (m_state == m_slist)
+                               strcpy(m_return_reason, "Connect: Still trying");
+                       else
+                               Con_Printf("Still trying...\n");
+               }
                else
-                       Con_Printf("Trying...\n");
+               {
+                       if (m_state == m_slist)
+                               strcpy(m_return_reason, "Connect: Trying");
+                       else
+                               Con_Printf("Trying...\n");
+               }
                cls.connect_nextsendtime = realtime + 1;
                cls.connect_remainingtries--;
                // try challenge first (newer server)
@@ -944,7 +1015,7 @@ static void NetConn_BuildChallengeString(char *buffer, int bufferlength)
 extern void SV_ConnectClient(int clientnum, netconn_t *netconnection);
 int NetConn_ServerParsePacket(lhnetsocket_t *mysocket, qbyte *data, int length, lhnetaddress_t *peeraddress)
 {
-       int i, n, ret, clientnum, responselength, best, clientcount;
+       int i, n, ret, clientnum, responselength, best;
        double besttime;
        client_t *client;
        netconn_t *conn;
@@ -1014,10 +1085,10 @@ int NetConn_ServerParsePacket(lhnetsocket_t *mysocket, qbyte *data, int length,
                                                else
                                                {
                                                        // see if this is a duplicate connection request
-                                                       for (clientnum = 0;clientnum < MAX_SCOREBOARD;clientnum++)
-                                                               if ((client = svs.connectedclients[clientnum]) && LHNETADDRESS_Compare(peeraddress, &client->netconnection->peeraddress) == 0)
+                                                       for (clientnum = 0, client = svs.clients;clientnum < svs.maxclients;clientnum++, client++)
+                                                               if (client->netconnection && LHNETADDRESS_Compare(peeraddress, &client->netconnection->peeraddress) == 0)
                                                                        break;
-                                                       if (clientnum < MAX_SCOREBOARD)
+                                                       if (clientnum < svs.maxclients)
                                                        {
                                                                // duplicate connection request
                                                                if (realtime - client->netconnection->connecttime < 2.0)
@@ -1038,39 +1109,27 @@ int NetConn_ServerParsePacket(lhnetsocket_t *mysocket, qbyte *data, int length,
                                                        else
                                                        {
                                                                // this is a new client, find a slot
-                                                               for (clientcount = 0, clientnum = 0;clientnum < MAX_SCOREBOARD;clientnum++)
-                                                                       if (svs.connectedclients[clientnum])
-                                                                               clientcount++;
-                                                               for (clientnum = 0;clientnum < MAX_SCOREBOARD;clientnum++)
-                                                                       if (!svs.connectedclients[clientnum])
+                                                               for (clientnum = 0, client = svs.clients;clientnum < svs.maxclients;clientnum++, client++)
+                                                                       if (!client->active)
                                                                                break;
-                                                               if (clientcount < max(1, sv_maxplayers.integer) && clientnum < MAX_SCOREBOARD)
+                                                               if (clientnum < svs.maxclients)
                                                                {
-                                                                       // allocate and prepare the client struct
-                                                                       if ((client = Mem_Alloc(sv_clients_mempool, sizeof(client_t))))
+                                                                       // prepare the client struct
+                                                                       if ((client->entitydatabase4 = EntityFrame4_AllocDatabase(sv_clients_mempool)))
                                                                        {
-                                                                               if ((client->entitydatabase4 = EntityFrame4_AllocDatabase(sv_clients_mempool)))
+                                                                               if ((conn = NetConn_Open(mysocket, peeraddress)))
                                                                                {
-                                                                                       if ((conn = NetConn_Open(mysocket, peeraddress)))
-                                                                                       {
-                                                                                               // allocated connection
-                                                                                               LHNETADDRESS_ToString(peeraddress, conn->address, sizeof(conn->address), true);
-                                                                                               if (developer.integer)
-                                                                                                       Con_Printf("Datagram_ParseConnectionless: sending \"accept\" to %s.\n", conn->address);
-                                                                                               NetConn_WriteString(mysocket, "\377\377\377\377accept", peeraddress);
-                                                                                               // now set up the client struct
-                                                                                               svs.connectedclients[clientnum] = client;
-                                                                                               SV_ConnectClient(clientnum, conn);
-                                                                                               NetConn_Heartbeat(1);
-                                                                                       }
-                                                                                       else
-                                                                                       {
-                                                                                               EntityFrame4_FreeDatabase(client->entitydatabase4);
-                                                                                               Mem_Free(client);
-                                                                                       }
+                                                                                       // allocated connection
+                                                                                       LHNETADDRESS_ToString(peeraddress, conn->address, sizeof(conn->address), true);
+                                                                                       if (developer.integer)
+                                                                                               Con_Printf("Datagram_ParseConnectionless: sending \"accept\" to %s.\n", conn->address);
+                                                                                       NetConn_WriteString(mysocket, "\377\377\377\377accept", peeraddress);
+                                                                                       // now set up the client
+                                                                                       SV_ConnectClient(clientnum, conn);
+                                                                                       NetConn_Heartbeat(1);
                                                                                }
                                                                                else
-                                                                                       Mem_Free(client);
+                                                                                       EntityFrame4_FreeDatabase(client->entitydatabase4);
                                                                        }
                                                                }
                                                                else
@@ -1092,13 +1151,13 @@ int NetConn_ServerParsePacket(lhnetsocket_t *mysocket, qbyte *data, int length,
                                // If there was a challenge in the getinfo message
                                if (length > 8 && string[7] == ' ')
                                        challenge = string + 8;
-                               for (i = 0, n = 0;i < MAX_SCOREBOARD;i++)
-                                       if (svs.connectedclients[i])
+                               for (i = 0, n = 0;i < svs.maxclients;i++)
+                                       if (svs.clients[i].active)
                                                n++;
                                responselength = snprintf(response, sizeof(response), "\377\377\377\377infoResponse\x0A"
                                                        "\\gamename\\%s\\modname\\%s\\sv_maxclients\\%d"
                                                        "\\clients\\%d\\mapname\\%s\\hostname\\%s\\protocol\\%d%s%s",
-                                                       gamename, com_modname, min(sv_maxplayers.integer, MAX_SCOREBOARD), n,
+                                                       gamename, com_modname, svs.maxclients, n,
                                                        sv.name, hostname.string, NET_PROTOCOL_VERSION, challenge ? "\\challenge\\" : "", challenge ? challenge : "");
                                // does it fit in the buffer?
                                if (responselength < (int)sizeof(response))
@@ -1158,10 +1217,10 @@ int NetConn_ServerParsePacket(lhnetsocket_t *mysocket, qbyte *data, int length,
                                                else
                                                {
                                                        // see if this is a duplicate connection request
-                                                       for (clientnum = 0;clientnum < MAX_SCOREBOARD;clientnum++)
-                                                               if ((client = svs.connectedclients[clientnum]) && LHNETADDRESS_Compare(peeraddress, &client->netconnection->peeraddress) == 0)
+                                                       for (clientnum = 0, client = svs.clients;clientnum < svs.maxclients;clientnum++, client++)
+                                                               if (client->netconnection && LHNETADDRESS_Compare(peeraddress, &client->netconnection->peeraddress) == 0)
                                                                        break;
-                                                       if (clientnum < MAX_SCOREBOARD)
+                                                       if (clientnum < svs.maxclients)
                                                        {
                                                                // duplicate connection request
                                                                if (realtime - client->netconnection->connecttime < 2.0)
@@ -1190,11 +1249,10 @@ int NetConn_ServerParsePacket(lhnetsocket_t *mysocket, qbyte *data, int length,
                                                        else
                                                        {
                                                                // this is a new client, find a slot
-                                                               for (clientnum = 0;clientnum < MAX_SCOREBOARD;clientnum++)
-                                                                       if (!(client = svs.connectedclients[clientnum]))
+                                                               for (clientnum = 0, client = svs.clients;clientnum < svs.maxclients;clientnum++, client++)
+                                                                       if (!client->active)
                                                                                break;
-                                                               // WARNING: this is broken code
-                                                               if (clientnum < MAX_SCOREBOARD && (client->netconnection = conn = NetConn_Open(mysocket, peeraddress)) != NULL)
+                                                               if (clientnum < svs.maxclients && (client->netconnection = conn = NetConn_Open(mysocket, peeraddress)) != NULL)
                                                                {
                                                                        // connect to the client
                                                                        // everything is allocated, just fill in the details
@@ -1249,7 +1307,7 @@ int NetConn_ServerParsePacket(lhnetsocket_t *mysocket, qbyte *data, int length,
                                                MSG_WriteString(&net_message, hostname.string);
                                                MSG_WriteString(&net_message, sv.name);
                                                MSG_WriteByte(&net_message, net_activeconnections);
-                                               MSG_WriteByte(&net_message, min(sv_maxplayers.integer, MAX_SCOREBOARD));
+                                               MSG_WriteByte(&net_message, svs.maxclients);
                                                MSG_WriteByte(&net_message, NET_PROTOCOL_VERSION);
                                                *((int *)net_message.data) = BigLong(NETFLAG_CTL | (net_message.cursize & NETFLAG_LENGTH_MASK));
                                                NetConn_Write(mysocket, net_message.data, net_message.cursize, peeraddress);
@@ -1324,25 +1382,14 @@ int NetConn_ServerParsePacket(lhnetsocket_t *mysocket, qbyte *data, int length,
                        }
                }
 #endif
-               for (i = 0;i < MAX_SCOREBOARD;i++)
+               for (i = 0, host_client = svs.clients;i < svs.maxclients;i++, host_client++)
                {
-                       if ((host_client = svs.connectedclients[i]))
+                       if (host_client->netconnection && host_client->netconnection->mysocket == mysocket && !LHNETADDRESS_Compare(&host_client->netconnection->peeraddress, peeraddress))
                        {
-                               if (host_client->netconnection)
-                               {
-                                       if (host_client->netconnection->mysocket == mysocket && !LHNETADDRESS_Compare(&host_client->netconnection->peeraddress, peeraddress))
-                                       {
-                                               sv_player = host_client->edict;
-                                               if ((ret = NetConn_ReceivedMessage(host_client->netconnection, data, length)) == 2)
-                                                       SV_ReadClientMessage();
-                                               return ret;
-                                       }
-                               }
-                               else
-                               {
-                                       Con_Printf("Removing client with no netconnection!\n");
-                                       SV_DropClient(true);
-                               }
+                               sv_player = host_client->edict;
+                               if ((ret = NetConn_ReceivedMessage(host_client->netconnection, data, length)) == 2)
+                                       SV_ReadClientMessage();
+                               return ret;
                        }
                }
        }
@@ -1358,9 +1405,10 @@ void NetConn_ServerFrame(void)
        for (i = 0;i < sv_numsockets;i++)
                while (sv_sockets[i] && (length = NetConn_Read(sv_sockets[i], readbuffer, sizeof(readbuffer), &peeraddress)) > 0)
                        NetConn_ServerParsePacket(sv_sockets[i], readbuffer, length, &peeraddress);
-       for (i = 0;i < MAX_SCOREBOARD;i++)
+       for (i = 0, host_client = svs.clients;i < svs.maxclients;i++, host_client++)
        {
-               if ((host_client = svs.connectedclients[i]) && realtime > host_client->netconnection->timeout)
+               // never timeout loopback connections
+               if (host_client->netconnection && realtime > host_client->netconnection->timeout && LHNETADDRESS_GetAddressType(LHNET_AddressFromSocket(host_client->netconnection->mysocket)) != LHNETADDRESSTYPE_LOOP)
                {
                        Con_Printf("Client \"%s\" connection timed out\n", host_client->name);
                        sv_player = host_client->edict;
@@ -1407,10 +1455,20 @@ void NetConn_QueryMasters(void)
 
                        // search internet
                        for (masternum = 0;sv_masters[masternum].name;masternum++)
+                       {
                                if (sv_masters[masternum].string && LHNETADDRESS_FromString(&masteraddress, sv_masters[masternum].string, MASTER_PORT) && LHNETADDRESS_GetAddressType(&masteraddress) == LHNETADDRESS_GetAddressType(LHNET_AddressFromSocket(cl_sockets[i])))
+                               {
+                                       masterquerycount++;
                                        NetConn_WriteString(cl_sockets[i], request, &masteraddress);
+                               }
+                       }
                }
        }
+       if (!masterquerycount)
+       {
+               Con_Printf("Unable to query master servers, no suitable network sockets active.\n");
+               strcpy(m_return_reason, "No network");
+       }
 }
 
 void NetConn_Heartbeat(int priority)
@@ -1435,7 +1493,7 @@ void NetConn_Heartbeat(int priority)
 
        // make advertising optional and don't advertise singleplayer games, and
        // only send a heartbeat as often as the admin wants
-       if (sv.active && sv_public.integer && (!cl.islocalgame || sv_maxplayers.integer >= 2) && (priority > 1 || realtime > nextheartbeattime))
+       if (sv.active && sv_public.integer && svs.maxclients >= 2 && (priority > 1 || realtime > nextheartbeattime))
        {
                nextheartbeattime = realtime + sv_heartbeatperiod.value;
                for (masternum = 0;sv_masters[masternum].name;masternum++)
@@ -1462,9 +1520,9 @@ int NetConn_SendToAll(sizebuf_t *data, double blocktime)
                count = 0;
                NetConn_ClientFrame();
                NetConn_ServerFrame();
-               for (i = 0;i < MAX_SCOREBOARD;i++)
+               for (i = 0, host_client = svs.clients;i < svs.maxclients;i++, host_client++)
                {
-                       if ((host_client = svs.connectedclients[i]))
+                       if (host_client->netconnection)
                        {
                                if (NetConn_CanSendMessage(host_client->netconnection))
                                {
@@ -1483,7 +1541,10 @@ int NetConn_SendToAll(sizebuf_t *data, double blocktime)
 
 static void Net_Heartbeat_f(void)
 {
-       NetConn_Heartbeat(2);
+       if (sv.active)
+               NetConn_Heartbeat(2);
+       else
+               Con_Printf("No server running, can not heartbeat to master server.\n");
 }
 
 void PrintStats(netconn_t *conn)
@@ -1510,6 +1571,11 @@ void Net_Stats_f(void)
 
 void Net_Slist_f(void)
 {
+       masterquerytime = realtime;
+       masterquerycount = 0;
+       masterreplycount = 0;
+       serverquerycount = 0;
+       serverreplycount = 0;
        hostCacheCount = 0;
        memset(&pingcache, 0, sizeof(pingcache));
        if (m_state != m_slist)
@@ -1521,7 +1587,8 @@ void Net_Slist_f(void)
 
 void NetConn_Init(void)
 {
-       int masternum;
+       int i;
+       lhnetaddress_t tempaddress;
        netconn_mempool = Mem_AllocPool("Networking");
        Cmd_AddCommand("net_stats", Net_Stats_f);
        Cmd_AddCommand("net_slist", Net_Slist_f);
@@ -1545,8 +1612,30 @@ void NetConn_Init(void)
        Cvar_RegisterVariable(&sv_netaddress_ipv6);
        Cvar_RegisterVariable(&sv_public);
        Cvar_RegisterVariable(&sv_heartbeatperiod);
-       for (masternum = 0;sv_masters[masternum].name;masternum++)
-               Cvar_RegisterVariable(&sv_masters[masternum]);
+       for (i = 0;sv_masters[i].name;i++)
+               Cvar_RegisterVariable(&sv_masters[i]);
+       if ((i = COM_CheckParm("-ip")) && i + 1 < com_argc)
+       {
+               if (LHNETADDRESS_FromString(&tempaddress, com_argv[i + 1], 0) == 1)
+               {
+                       Con_Printf("-ip option used, setting cl_netaddress and sv_netaddress to address \"%s\"\n");
+                       Cvar_SetQuick(&cl_netaddress, com_argv[i + 1]);
+                       Cvar_SetQuick(&sv_netaddress, com_argv[i + 1]);
+               }
+               else
+                       Con_Printf("-ip option used, but unable to parse the address \"%s\"\n", com_argv[i + 1]);
+       }
+       if (((i = COM_CheckParm("-port")) || (i = COM_CheckParm("-ipport")) || (i = COM_CheckParm("-udpport"))) && i + 1 < com_argc)
+       {
+               i = atoi(com_argv[i + 1]);
+               if (i >= 0 && i < 65536)
+               {
+                       Con_Printf("-port option used, setting port cvar to %i\n", i);
+                       Cvar_SetValueQuick(&sv_netport, i);
+               }
+               else
+                       Con_Printf("-port option used, but %i is not a valid port number\n", i);
+       }
        cl_numsockets = 0;
        sv_numsockets = 0;
        memset(&pingcache, 0, sizeof(pingcache));