vulpineawoo/solanum
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Replace int fd; in local client object with an fde object.

Browse source
This commit is contained in:
William Pitcock 2007-12-28 15:06:33 -06:00
parent 35cf4c7998
commit c5c2f506c1
14 changed files with 47 additions and 46 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
include/client.h
View file

@ -251,7 +251,7 @@ struct LocalUser
char *fullcaps; char *fullcaps;
int caps; /* capabilities bit-field */ int caps; /* capabilities bit-field */
int fd; /* >= 0, for local clients */ fde_t *F; /* >= 0, for local clients */
/* time challenge response is valid for */ /* time challenge response is valid for */
time_t chal_time; time_t chal_time;

1
libcharybdis/commio.h
View file

@ -203,5 +203,6 @@ extern void mangle_mapped_sockaddr(struct sockaddr *in);
extern int comm_get_maxconnections(void); extern int comm_get_maxconnections(void);
extern fde_t *comm_locate_fd(int fd); extern fde_t *comm_locate_fd(int fd);
extern fde_t *comm_add_fd(int fd);
#endif /* INCLUDED_commio_h */ #endif /* INCLUDED_commio_h */

4
modules/core/m_nick.c
View file

@ -741,7 +741,7 @@ set_initial_nick(struct Client *client_p, struct Client *source_p, char *nick)
add_to_client_hash(nick, source_p); add_to_client_hash(nick, source_p);
/* fd_desc is long enough */ /* fd_desc is long enough */
comm_note(client_p->localClient->fd, "Nick: %s", nick); comm_note(client_p->localClient->F->fd, "Nick: %s", nick);
if(source_p->flags & FLAGS_SENTUSER) if(source_p->flags & FLAGS_SENTUSER)
{ {
@ -850,7 +850,7 @@ change_local_nick(struct Client *client_p, struct Client *source_p,
} }
/* fd_desc is long enough */ /* fd_desc is long enough */
comm_note(client_p->localClient->fd, "Nick: %s", nick); comm_note(client_p->localClient->F->fd, "Nick: %s", nick);
return; return;
} }

2
modules/m_services.c
View file

@ -237,7 +237,7 @@ me_rsfnc(struct Client *client_p, struct Client *source_p,
del_all_accepts(target_p); del_all_accepts(target_p);
comm_note(target_p->localClient->fd, "Nick: %s", target_p->name); comm_note(target_p->localClient->F->fd, "Nick: %s", target_p->name);
return 0; return 0;
} }

2
modules/m_signon.c
View file

@ -258,7 +258,7 @@ me_svslogin(struct Client *client_p, struct Client *source_p,
else else
{ {
send_signon(NULL, target_p, nick, user, host, CurrentTime, login); send_signon(NULL, target_p, nick, user, host, CurrentTime, login);
comm_note(target_p->localClient->fd, "Nick: %s", target_p->name); comm_note(target_p->localClient->F->fd, "Nick: %s", target_p->name);
} }
return 0; return 0;

16
src/client.c
View file

@ -162,7 +162,7 @@ make_client(struct Client *from)
client_p->localClient->lasttime = client_p->localClient->firsttime = CurrentTime; client_p->localClient->lasttime = client_p->localClient->firsttime = CurrentTime;
client_p->localClient->fd = -1; client_p->localClient->F = NULL;
client_p->localClient->ctrlfd = -1; client_p->localClient->ctrlfd = -1;
client_p->preClient = (struct PreClient *) BlockHeapAlloc(pclient_heap); client_p->preClient = (struct PreClient *) BlockHeapAlloc(pclient_heap);
@ -222,8 +222,8 @@ free_local_client(struct Client *client_p)
client_p->localClient->listener = 0; client_p->localClient->listener = 0;
} }
if(client_p->localClient->fd >= 0) if(client_p->localClient->F->fd >= 0)
comm_close(client_p->localClient->fd); comm_close(client_p->localClient->F->fd);
if(client_p->localClient->passwd) if(client_p->localClient->passwd)
{ {
@ -2101,14 +2101,14 @@ close_connection(struct Client *client_p)
else else
ServerStats->is_ni++; ServerStats->is_ni++;
if(-1 < client_p->localClient->fd) if(client_p->localClient->F)
{ {
/* attempt to flush any pending dbufs. Evil, but .. -- adrian */ /* attempt to flush any pending dbufs. Evil, but .. -- adrian */
if(!IsIOError(client_p)) if(!IsIOError(client_p))
send_queued_write(client_p->localClient->fd, client_p); send_queued_write(client_p->localClient->F->fd, client_p);
comm_close(client_p->localClient->fd); comm_close(client_p->localClient->F->fd);
client_p->localClient->fd = -1; client_p->localClient->F = NULL;
} }
if(-1 < client_p->localClient->ctrlfd) if(-1 < client_p->localClient->ctrlfd)
@ -2144,7 +2144,7 @@ error_exit_client(struct Client *client_p, int error)
* for reading even though it ends up being an EOF. -avalon * for reading even though it ends up being an EOF. -avalon
*/ */
char errmsg[255]; char errmsg[255];
int current_error = comm_get_sockerr(client_p->localClient->fd); int current_error = comm_get_sockerr(client_p->localClient->F->fd);
SetIOError(client_p); SetIOError(client_p);

2
src/listener.c
View file

@ -465,7 +465,7 @@ add_connection(listener_t *listener, int fd, struct sockaddr *sai, int exempt)
strlcpy(new_client->host, new_client->sockhost, sizeof(new_client->host)); strlcpy(new_client->host, new_client->sockhost, sizeof(new_client->host));
new_client->localClient->fd = fd; new_client->localClient->F = comm_add_fd(fd);
new_client->localClient->listener = listener; new_client->localClient->listener = listener;
++listener->ref_count; ++listener->ref_count;

8
src/packet.c
View file

@ -356,13 +356,13 @@ read_packet(int fd, void *data)
* I personally think it makes the code too hairy to make sane. * I personally think it makes the code too hairy to make sane.
* -- adrian * -- adrian
*/ */
length = read(client_p->localClient->fd, readBuf, READBUF_SIZE); length = client_p->localClient->F->read_impl(client_p->localClient->F, readBuf, READBUF_SIZE);
if(length <= 0) if(length <= 0)
{ {
if((length == -1) && ignoreErrno(errno)) if((length == -1) && ignoreErrno(errno))
{ {
comm_setselect(client_p->localClient->fd, FDLIST_IDLECLIENT, comm_setselect(client_p->localClient->F->fd, FDLIST_IDLECLIENT,
COMM_SELECT_READ, read_packet, client_p, 0); COMM_SELECT_READ, read_packet, client_p, 0);
return; return;
} }
@ -416,12 +416,12 @@ read_packet(int fd, void *data)
/* If we get here, we need to register for another COMM_SELECT_READ */ /* If we get here, we need to register for another COMM_SELECT_READ */
if(PARSE_AS_SERVER(client_p)) if(PARSE_AS_SERVER(client_p))
{ {
comm_setselect(client_p->localClient->fd, FDLIST_SERVER, COMM_SELECT_READ, comm_setselect(client_p->localClient->F->fd, FDLIST_SERVER, COMM_SELECT_READ,
read_packet, client_p, 0); read_packet, client_p, 0);
} }
else else
{ {
comm_setselect(client_p->localClient->fd, FDLIST_IDLECLIENT, comm_setselect(client_p->localClient->F->fd, FDLIST_IDLECLIENT,
COMM_SELECT_READ, read_packet, client_p, 0); COMM_SELECT_READ, read_packet, client_p, 0);
} }
} }

2
src/parse.c
View file

@ -133,7 +133,7 @@ parse(struct Client *client_p, char *pbuffer, char *bufend)
struct Message *mptr; struct Message *mptr;
s_assert(MyConnect(client_p)); s_assert(MyConnect(client_p));
s_assert(client_p->localClient->fd >= 0); s_assert(client_p->localClient->F->fd >= 0);
if(IsAnyDead(client_p)) if(IsAnyDead(client_p))
return; return;

4
src/reject.c
View file

@ -179,7 +179,7 @@ check_reject(struct Client *client_p)
{ {
ServerStats->is_rej++; ServerStats->is_rej++;
SetReject(client_p); SetReject(client_p);
comm_setselect(client_p->localClient->fd, FDLIST_NONE, COMM_SELECT_WRITE | COMM_SELECT_READ, NULL, NULL, 0); comm_setselect(client_p->localClient->F->fd, FDLIST_NONE, COMM_SELECT_WRITE | COMM_SELECT_READ, NULL, NULL, 0);
SetClosing(client_p); SetClosing(client_p);
dlinkMoveNode(&client_p->localClient->tnode, &unknown_list, &delay_exit); dlinkMoveNode(&client_p->localClient->tnode, &unknown_list, &delay_exit);
return 1; return 1;
@ -277,7 +277,7 @@ add_unknown_ip(struct Client *client_p)
{ {
SetExUnknown(client_p); SetExUnknown(client_p);
SetReject(client_p); SetReject(client_p);
comm_setselect(client_p->localClient->fd, FDLIST_NONE, COMM_SELECT_WRITE | COMM_SELECT_READ, NULL, NULL, 0); comm_setselect(client_p->localClient->F->fd, FDLIST_NONE, COMM_SELECT_WRITE | COMM_SELECT_READ, NULL, NULL, 0);
SetClosing(client_p); SetClosing(client_p);
dlinkMoveNode(&client_p->localClient->tnode, &unknown_list, &delay_exit); dlinkMoveNode(&client_p->localClient->tnode, &unknown_list, &delay_exit);
return 1; return 1;

14
src/s_auth.c
View file

@ -149,8 +149,8 @@ release_auth_client(struct AuthRequest *auth)
client->localClient->auth_request = NULL; client->localClient->auth_request = NULL;
dlinkDelete(&auth->node, &auth_poll_list); dlinkDelete(&auth->node, &auth_poll_list);
free_auth_request(auth); free_auth_request(auth);
if(client->localClient->fd > highest_fd) if(client->localClient->F->fd > highest_fd)
highest_fd = client->localClient->fd; highest_fd = client->localClient->F->fd;
/* /*
* When a client has auth'ed, we want to start reading what it sends * When a client has auth'ed, we want to start reading what it sends
@ -158,9 +158,9 @@ release_auth_client(struct AuthRequest *auth)
* -- adrian * -- adrian
*/ */
client->localClient->allow_read = MAX_FLOOD; client->localClient->allow_read = MAX_FLOOD;
comm_setflush(client->localClient->fd, 1000, flood_recalc, client); comm_setflush(client->localClient->F->fd, 1000, flood_recalc, client);
dlinkAddTail(client, &client->node, &global_client_list); dlinkAddTail(client, &client->node, &global_client_list);
read_packet(client->localClient->fd, client); read_packet(client->localClient->F->fd, client);
} }
/* /*
@ -309,7 +309,7 @@ start_auth_query(struct AuthRequest *auth)
* and machines with multiple IP addresses are common now * and machines with multiple IP addresses are common now
*/ */
memset(&localaddr, 0, locallen); memset(&localaddr, 0, locallen);
getsockname(auth->client->localClient->fd, getsockname(auth->client->localClient->F->fd,
(struct sockaddr *) &localaddr, &locallen); (struct sockaddr *) &localaddr, &locallen);
mangle_mapped_sockaddr((struct sockaddr *)&localaddr); mangle_mapped_sockaddr((struct sockaddr *)&localaddr);
@ -492,9 +492,9 @@ auth_connect_callback(int fd, int error, void *data)
} }
if(getsockname if(getsockname
(auth->client->localClient->fd, (struct sockaddr *) &us, (auth->client->localClient->F->fd, (struct sockaddr *) &us,
(socklen_t *) & ulen) (socklen_t *) & ulen)
|| getpeername(auth->client->localClient->fd, || getpeername(auth->client->localClient->F->fd,
(struct sockaddr *) &them, (socklen_t *) & tlen)) (struct sockaddr *) &them, (socklen_t *) & tlen))
{ {
ilog(L_IOERROR, "auth get{sock,peer}name error for %s:%m", ilog(L_IOERROR, "auth get{sock,peer}name error for %s:%m",

30
src/s_serv.c
View file

@ -1063,7 +1063,7 @@ server_estab(struct Client *client_p)
(me.info[0]) ? (me.info) : "IRCers United"); (me.info[0]) ? (me.info) : "IRCers United");
} }
if(!comm_set_buffers(client_p->localClient->fd, READBUF_SIZE)) if(!comm_set_buffers(client_p->localClient->F->fd, READBUF_SIZE))
report_error(SETBUF_ERROR_MSG, report_error(SETBUF_ERROR_MSG,
get_server_name(client_p, SHOW_IP), get_server_name(client_p, SHOW_IP),
log_client_name(client_p, SHOW_IP), errno); log_client_name(client_p, SHOW_IP), errno);
@ -1143,11 +1143,11 @@ server_estab(struct Client *client_p)
/* we won't overflow FD_DESC_SZ here, as it can hold /* we won't overflow FD_DESC_SZ here, as it can hold
* client_p->name + 64 * client_p->name + 64
*/ */
comm_note(client_p->localClient->fd, "slink data: %s", client_p->name); comm_note(client_p->localClient->F->fd, "slink data: %s", client_p->name);
comm_note(client_p->localClient->ctrlfd, "slink ctrl: %s", client_p->name); comm_note(client_p->localClient->ctrlfd, "slink ctrl: %s", client_p->name);
} }
else else
comm_note(client_p->localClient->fd, "Server: %s", client_p->name); comm_note(client_p->localClient->F->fd, "Server: %s", client_p->name);
/* /*
** Old sendto_serv_but_one() call removed because we now ** Old sendto_serv_but_one() call removed because we now
@ -1371,7 +1371,7 @@ fork_server(struct Client *server)
{ {
if((i == ctrl_fds[1]) || (i == data_fds[1]) || (i == server->localClient->fd)) if((i == ctrl_fds[1]) || (i == data_fds[1]) || (i == server->localClient->F->fd))
{ {
comm_set_nb(i); comm_set_nb(i);
} }
@ -1386,7 +1386,7 @@ fork_server(struct Client *server)
ircsnprintf(fd_str[0], sizeof(fd_str[0]), "%d", ctrl_fds[1]); ircsnprintf(fd_str[0], sizeof(fd_str[0]), "%d", ctrl_fds[1]);
ircsnprintf(fd_str[1], sizeof(fd_str[1]), "%d", data_fds[1]); ircsnprintf(fd_str[1], sizeof(fd_str[1]), "%d", data_fds[1]);
ircsnprintf(fd_str[2], sizeof(fd_str[2]), "%d", server->localClient->fd); ircsnprintf(fd_str[2], sizeof(fd_str[2]), "%d", server->localClient->F->fd);
kid_argv[0] = slink; kid_argv[0] = slink;
kid_argv[1] = fd_str[0]; kid_argv[1] = fd_str[0];
kid_argv[2] = fd_str[1]; kid_argv[2] = fd_str[1];
@ -1401,7 +1401,7 @@ fork_server(struct Client *server)
} }
else else
{ {
comm_close(server->localClient->fd); comm_close(server->localClient->F->fd);
/* close the childs end of the pipes */ /* close the childs end of the pipes */
close(ctrl_fds[1]); close(ctrl_fds[1]);
@ -1409,9 +1409,9 @@ fork_server(struct Client *server)
s_assert(server->localClient); s_assert(server->localClient);
server->localClient->ctrlfd = ctrl_fds[0]; server->localClient->ctrlfd = ctrl_fds[0];
server->localClient->fd = data_fds[0]; server->localClient->F->fd = data_fds[0];
if(!comm_set_nb(server->localClient->fd)) if(!comm_set_nb(server->localClient->F->fd))
{ {
report_error(NONB_ERROR_MSG, report_error(NONB_ERROR_MSG,
get_server_name(server, SHOW_IP), get_server_name(server, SHOW_IP),
@ -1428,10 +1428,10 @@ fork_server(struct Client *server)
} }
comm_open(server->localClient->ctrlfd, FD_SOCKET, NULL); comm_open(server->localClient->ctrlfd, FD_SOCKET, NULL);
comm_open(server->localClient->fd, FD_SOCKET, NULL); comm_open(server->localClient->F->fd, FD_SOCKET, NULL);
read_ctrl_packet(server->localClient->ctrlfd, server); read_ctrl_packet(server->localClient->ctrlfd, server);
read_packet(server->localClient->fd, server); read_packet(server->localClient->F->fd, server);
} }
return 0; return 0;
@ -1517,7 +1517,7 @@ serv_connect(struct server_conf *server_p, struct Client *by)
strlcpy(client_p->name, server_p->name, sizeof(client_p->name)); strlcpy(client_p->name, server_p->name, sizeof(client_p->name));
strlcpy(client_p->host, server_p->host, sizeof(client_p->host)); strlcpy(client_p->host, server_p->host, sizeof(client_p->host));
strlcpy(client_p->sockhost, server_p->host, sizeof(client_p->sockhost)); strlcpy(client_p->sockhost, server_p->host, sizeof(client_p->sockhost));
client_p->localClient->fd = fd; client_p->localClient->F->fd = fd;
/* /*
* Set up the initial server evilness, ripped straight from * Set up the initial server evilness, ripped straight from
@ -1525,7 +1525,7 @@ serv_connect(struct server_conf *server_p, struct Client *by)
* -- adrian * -- adrian
*/ */
if(!comm_set_buffers(client_p->localClient->fd, READBUF_SIZE)) if(!comm_set_buffers(client_p->localClient->F->fd, READBUF_SIZE))
{ {
report_error(SETBUF_ERROR_MSG, report_error(SETBUF_ERROR_MSG,
get_server_name(client_p, SHOW_IP), get_server_name(client_p, SHOW_IP),
@ -1598,7 +1598,7 @@ serv_connect(struct server_conf *server_p, struct Client *by)
#endif #endif
(server_p->aftype == AF_INET ? "IPv4" : "?")); (server_p->aftype == AF_INET ? "IPv4" : "?"));
comm_connect_tcp(client_p->localClient->fd, server_p->host, comm_connect_tcp(client_p->localClient->F->fd, server_p->host,
server_p->port, NULL, 0, serv_connect_callback, server_p->port, NULL, 0, serv_connect_callback,
client_p, server_p->aftype, client_p, server_p->aftype,
ConfigFileEntry.connect_timeout); ConfigFileEntry.connect_timeout);
@ -1614,7 +1614,7 @@ serv_connect(struct server_conf *server_p, struct Client *by)
(server_p->aftype == AF_INET ? "IPv4" : "?"), vhoststr); (server_p->aftype == AF_INET ? "IPv4" : "?"), vhoststr);
comm_connect_tcp(client_p->localClient->fd, server_p->host, comm_connect_tcp(client_p->localClient->F->fd, server_p->host,
server_p->port, (struct sockaddr *) &myipnum, server_p->port, (struct sockaddr *) &myipnum,
GET_SS_LEN(myipnum), serv_connect_callback, client_p, GET_SS_LEN(myipnum), serv_connect_callback, client_p,
myipnum.ss_family, ConfigFileEntry.connect_timeout); myipnum.ss_family, ConfigFileEntry.connect_timeout);
@ -1641,7 +1641,7 @@ serv_connect_callback(int fd, int status, void *data)
/* First, make sure its a real client! */ /* First, make sure its a real client! */
s_assert(client_p != NULL); s_assert(client_p != NULL);
s_assert(client_p->localClient->fd == fd); s_assert(client_p->localClient->F->fd == fd);
if(client_p == NULL) if(client_p == NULL)
return; return;

2
src/s_user.c
View file

@ -296,7 +296,7 @@ register_local_user(struct Client *client_p, struct Client *source_p, const char
del_from_client_hash(source_p->name, source_p); del_from_client_hash(source_p->name, source_p);
strlcpy(source_p->name, source_p->preClient->spoofnick, NICKLEN + 1); strlcpy(source_p->name, source_p->preClient->spoofnick, NICKLEN + 1);
add_to_client_hash(source_p->name, source_p); add_to_client_hash(source_p->name, source_p);
comm_note(source_p->localClient->fd, "Nick: %s", source_p->name); comm_note(source_p->localClient->F->fd, "Nick: %s", source_p->name);
} }
if(!valid_hostname(source_p->host)) if(!valid_hostname(source_p->host))

4
src/send.c
View file

@ -108,7 +108,7 @@ _send_linebuf(struct Client *to, buf_head_t *linebuf)
to->localClient->sendM += 1; to->localClient->sendM += 1;
me.localClient->sendM += 1; me.localClient->sendM += 1;
if(linebuf_len(&to->localClient->buf_sendq) > 0) if(linebuf_len(&to->localClient->buf_sendq) > 0)
send_queued_write(to->localClient->fd, to); send_queued_write(to->localClient->F->fd, to);
return 0; return 0;
} }
@ -168,7 +168,7 @@ send_queued_write(int fd, void *data)
#ifdef USE_IODEBUG_HOOKS #ifdef USE_IODEBUG_HOOKS
hook_data_int hd; hook_data_int hd;
#endif #endif
fde_t *F = comm_locate_fd(to->localClient->fd); fde_t *F = comm_locate_fd(to->localClient->F->fd);
if (!F) if (!F)
return; return;