oysteikt/mpd
0
0
Fork 0
Code Issues Pull Requests Releases Activity

moved code to sockaddr_to_tmp_string()

Browse Source 
Unclutter the client_new() constructor by moving unrelated complex
code into a separate function.
This commit is contained in:
Max Kellermann 2008-08-28 20:20:10 +02:00
parent 34735fab66
commit e0fd63ecf8
1 changed files with 17 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

26
src/client.c
View File

@ -248,16 +248,10 @@ static void client_close(struct client *client)
free(client);
}
void client_new(int fd, const struct sockaddr *addr)
static const char *
sockaddr_to_tmp_string(const struct sockaddr *addr)
{
const char *hostname;
struct client *client;
if (num_clients >= client_max_connections) {
ERROR("Max Connections Reached!\n");
xclose(fd);
return;
}
switch (addr->sa_family) {
#ifdef HAVE_TCP
@ -293,11 +287,25 @@ void client_new(int fd, const struct sockaddr *addr)
hostname = "unknown";
}
return hostname;
}
void client_new(int fd, const struct sockaddr *addr)
{
struct client *client;
if (num_clients >= client_max_connections) {
ERROR("Max Connections Reached!\n");
xclose(fd);
return;
}
client = xcalloc(1, sizeof(*client));
list_add(&client->siblings, &clients);
++num_clients;
client_init(client, fd);
SECURE("client %i: opened from %s\n", client->num, hostname);
SECURE("client %i: opened from %s\n", client->num,
sockaddr_to_tmp_string(addr));
}
static int client_process_line(struct client *client)