New_Connection(): don't set the client hostname twice

Setting the hostname twice doesn't do much harm a lot, but isn't elegant.
And for IPv6 addresses, it isn't correct the first time (missing []) ...
This commit is contained in:
Alexander Barton 2012-01-25 17:11:44 +01:00
parent 4888984429
commit c16133c5ee
2 changed files with 1 additions and 2 deletions

View File

@ -186,7 +186,6 @@ Init_New_Client(CONN_ID Idx, CLIENT *Introducer, CLIENT *TopServer,
assert(Idx >= NONE);
assert(Introducer != NULL);
assert(Hostname != NULL);
client = New_Client_Struct();
if (!client)

View File

@ -1438,7 +1438,7 @@ New_Connection(int Sock)
return -1;
}
c = Client_NewLocal(new_sock, ip_str, CLIENT_UNKNOWN, false);
c = Client_NewLocal(new_sock, NULL, CLIENT_UNKNOWN, false);
if (!c) {
Log(LOG_ALERT,
"Can't accept connection: can't create client structure!");