X-Git-Url: https://jfr.im/git/irc/rqf/shadowircd.git/blobdiff_plain/cc200171d7c40be3b204460f9c14d5d37e771866..1d39b466d4ddd974674c9397589d45935c746ed0:/src/packet.c

diff --git a/src/packet.c b/src/packet.c
index b364772..1434f2c 100644
--- a/src/packet.c
+++ b/src/packet.c
@@ -58,7 +58,7 @@ parse_client_queued(struct Client *client_p)
 			if(client_p->localClient->sent_parsed >= client_p->localClient->allow_read)
 				break;
 
-			dolen = linebuf_get(&client_p->localClient->
+			dolen = rb_linebuf_get(&client_p->localClient->
 					    buf_recvq, readBuf, READBUF_SIZE,
 					    LINEBUF_COMPLETE, LINEBUF_PARSED);
 
@@ -88,7 +88,7 @@ parse_client_queued(struct Client *client_p)
 
 	if(IsAnyServer(client_p) || IsExemptFlood(client_p))
 	{
-		while (!IsAnyDead(client_p) && (dolen = linebuf_get(&client_p->localClient->buf_recvq,
+		while (!IsAnyDead(client_p) && (dolen = rb_linebuf_get(&client_p->localClient->buf_recvq,
 					   readBuf, READBUF_SIZE, LINEBUF_COMPLETE,
 					   LINEBUF_PARSED)) > 0)
 		{
@@ -132,7 +132,7 @@ parse_client_queued(struct Client *client_p)
 			else if(client_p->localClient->sent_parsed >= (4 * client_p->localClient->allow_read))
 				break;
 
-			dolen = linebuf_get(&client_p->localClient->
+			dolen = rb_linebuf_get(&client_p->localClient->
 					    buf_recvq, readBuf, READBUF_SIZE,
 					    LINEBUF_COMPLETE, LINEBUF_PARSED);
 
@@ -172,36 +172,56 @@ flood_endgrace(struct Client *client_p)
  * once a second on any given client. We then attempt to flush some data.
  */
 void
-flood_recalc(int fd, void *data)
+flood_recalc(void *unused)
 {
-	struct Client *client_p = data;
-	struct LocalUser *lclient_p = client_p->localClient;
+	rb_dlink_node *ptr, *next;
+	struct Client *client_p;
 
-	/* This can happen in the event that the client detached. */
-	if(!lclient_p)
-		return;
+	RB_DLINK_FOREACH_SAFE(ptr, next, lclient_list.head)
+	{
+		client_p = ptr->data;
 
-	/* allow a bursting client their allocation per second, allow
-	 * a client whos flooding an extra 2 per second
-	 */
-	if(IsFloodDone(client_p))
-		lclient_p->sent_parsed -= 2;
-	else
-		lclient_p->sent_parsed = 0;
+		if(unlikely(IsMe(client_p)))
+			continue;
+			
+		if(unlikely(client_p->localClient == NULL))
+			continue;
+		
+		if(IsFloodDone(client_p))
+			client_p->localClient->sent_parsed -= 2;
+		else
+			client_p->localClient->sent_parsed = 0;
+			
+		if(client_p->localClient->sent_parsed < 0)
+			client_p->localClient->sent_parsed = 0;
+
+		if(--client_p->localClient->actually_read < 0)
+			client_p->localClient->actually_read = 0;
+
+		parse_client_queued(client_p);
+		
+		if(unlikely(IsAnyDead(client_p)))
+			continue;
 
-	if(lclient_p->sent_parsed < 0)
-		lclient_p->sent_parsed = 0;
+	}
 
-	if(--lclient_p->actually_read < 0)
-		lclient_p->actually_read = 0;
+	RB_DLINK_FOREACH_SAFE(ptr, next, unknown_list.head)
+	{
+		client_p = ptr->data;
 
-	parse_client_queued(client_p);
+		if(client_p->localClient == NULL)
+			continue;
 
-	if(IsAnyDead(client_p))
-		return;
+		client_p->localClient->sent_parsed--;
+
+		if(client_p->localClient->sent_parsed < 0)
+			client_p->localClient->sent_parsed = 0;
+
+		if(--client_p->localClient->actually_read < 0)
+			client_p->localClient->actually_read = 0;
 
-	/* and finally, reset the flood check */
-	rb_setflush(fd, 1000, flood_recalc, client_p);
+		parse_client_queued(client_p);
+	}
 }
 
 /*
@@ -209,7 +229,7 @@ flood_recalc(int fd, void *data)
  *                    link and process it.
  */
 void
-read_ctrl_packet(int fd, void *data)
+read_ctrl_packet(rb_fde_t *F, void *data)
 {
 	struct Client *server = data;
 	struct LocalUser *lserver = server->localClient;
@@ -235,11 +255,11 @@ read_ctrl_packet(int fd, void *data)
 		reply->readdata = 0;
 		reply->data = NULL;
 
-		length = read(fd, tmp, 1);
+		length = rb_read(F, tmp, 1);
 
 		if(length <= 0)
 		{
-			if((length == -1) && ignoreErrno(errno))
+			if((length == -1) && rb_ignore_errno(errno))
 				goto nodata;
 			error_exit_client(server, length);
 			return;
@@ -261,10 +281,10 @@ read_ctrl_packet(int fd, void *data)
 	if((replydef->flags & SLINKRPL_FLAG_DATA) && (reply->gotdatalen < 2))
 	{
 		/* we need a datalen u16 which we don't have yet... */
-		length = read(fd, len, (2 - reply->gotdatalen));
+		length = rb_read(F, len, (2 - reply->gotdatalen));
 		if(length <= 0)
 		{
-			if((length == -1) && ignoreErrno(errno))
+			if((length == -1) && rb_ignore_errno(errno))
 				goto nodata;
 			error_exit_client(server, length);
 			return;
@@ -282,7 +302,7 @@ read_ctrl_packet(int fd, void *data)
 			reply->datalen |= *len;
 			reply->gotdatalen++;
 			if(reply->datalen > 0)
-				reply->data = MyMalloc(reply->datalen);
+				reply->data = rb_malloc(reply->datalen);
 		}
 
 		if(reply->gotdatalen < 2)
@@ -291,11 +311,11 @@ read_ctrl_packet(int fd, void *data)
 
 	if(reply->readdata < reply->datalen)	/* try to get any remaining data */
 	{
-		length = read(fd, (reply->data + reply->readdata),
+		length = rb_read(F, (reply->data + reply->readdata),
 			      (reply->datalen - reply->readdata));
 		if(length <= 0)
 		{
-			if((length == -1) && ignoreErrno(errno))
+			if((length == -1) && rb_ignore_errno(errno))
 				goto nodata;
 			error_exit_client(server, length);
 			return;
@@ -319,7 +339,7 @@ read_ctrl_packet(int fd, void *data)
 
 	/* reset SlinkRpl */
 	if(reply->datalen > 0)
-		MyFree(reply->data);
+		rb_free(reply->data);
 	reply->command = 0;
 
 	if(IsAnyDead(server))
@@ -327,14 +347,14 @@ read_ctrl_packet(int fd, void *data)
 
       nodata:
 	/* If we get here, we need to register for another COMM_SELECT_READ */
-	rb_setselect(fd, FDLIST_SERVER, COMM_SELECT_READ, read_ctrl_packet, server, 0);
+	rb_setselect(F, RB_SELECT_READ, read_ctrl_packet, server);
 }
 
 /*
  * read_packet - Read a 'packet' of data from a connection and process it.
  */
 void
-read_packet(int fd, void *data)
+read_packet(rb_fde_t * F, void *data)
 {
 	struct Client *client_p = data;
 	struct LocalUser *lclient_p = client_p->localClient;
@@ -345,81 +365,83 @@ read_packet(int fd, void *data)
 #ifdef USE_IODEBUG_HOOKS
 	hook_data_int hdata;
 #endif
-	if(IsAnyDead(client_p))
-		return;
 
-	/*
-	 * Read some data. We *used to* do anti-flood protection here, but
-	 * I personally think it makes the code too hairy to make sane.
-	 *     -- adrian
-	 */
-	length = client_p->localClient->F->read_impl(client_p->localClient->F, readBuf, READBUF_SIZE);
-
-	if(length <= 0)
+	while(1)
 	{
-		if((length == -1) && ignoreErrno(errno))
+		if(IsAnyDead(client_p))
+			return;
+
+		/*
+		 * Read some data. We *used to* do anti-flood protection here, but
+		 * I personally think it makes the code too hairy to make sane.
+		 *     -- adrian
+		 */
+		length = rb_read(client_p->localClient->F, readBuf, READBUF_SIZE);
+
+		if(length < 0)
 		{
-			rb_setselect(client_p->localClient->F->fd, FDLIST_IDLECLIENT,
-				       COMM_SELECT_READ, read_packet, client_p, 0);
+			if(rb_ignore_errno(errno))
+				rb_setselect(client_p->localClient->F, 
+						RB_SELECT_READ, read_packet, client_p);
+			else
+				error_exit_client(client_p, length);
+			return;
+		}
+		else if(length == 0)
+		{
+			error_exit_client(client_p, length);
 			return;
 		}
-		error_exit_client(client_p, length);
-		return;
-	}
 
 #ifdef USE_IODEBUG_HOOKS
-	hdata.client = client_p;
-	hdata.arg1 = readBuf;
-	hdata.arg2 = length;
-	call_hook(h_iorecv_id, &hdata);
+		hdata.client = client_p;
+		hdata.arg1 = readBuf;
+		hdata.arg2 = length;
+		call_hook(h_iorecv_id, &hdata);
 #endif
 
-	if(client_p->localClient->lasttime < CurrentTime)
-		client_p->localClient->lasttime = CurrentTime;
-	client_p->flags &= ~FLAGS_PINGSENT;
+		if(client_p->localClient->lasttime < rb_current_time())
+			client_p->localClient->lasttime = rb_current_time();
+		client_p->flags &= ~FLAGS_PINGSENT;
 
-	/*
-	 * Before we even think of parsing what we just read, stick
-	 * it on the end of the receive queue and do it when its
-	 * turn comes around.
-	 */
-	if(IsHandshake(client_p) || IsUnknown(client_p))
-		binary = 1;
+		/*
+		 * Before we even think of parsing what we just read, stick
+		 * it on the end of the receive queue and do it when its
+		 * turn comes around.
+		 */
+		if(IsHandshake(client_p) || IsUnknown(client_p))
+			binary = 1;
 
-	lbuf_len = linebuf_parse(&client_p->localClient->buf_recvq, readBuf, length, binary);
+		lbuf_len = rb_linebuf_parse(&client_p->localClient->buf_recvq, readBuf, length, binary);
 
-	lclient_p->actually_read += lbuf_len;
+		lclient_p->actually_read += lbuf_len;
 
-	if(IsAnyDead(client_p))
-		return;
-		
-	/* Attempt to parse what we have */
-	parse_client_queued(client_p);
+		if(IsAnyDead(client_p))
+			return;
+			
+		/* Attempt to parse what we have */
+		parse_client_queued(client_p);
 
-	if(IsAnyDead(client_p))
-		return;
-		
-	/* Check to make sure we're not flooding */
-	if(!IsAnyServer(client_p) &&
-	   (linebuf_alloclen(&client_p->localClient->buf_recvq) > ConfigFileEntry.client_flood))
-	{
-		if(!(ConfigFileEntry.no_oper_flood && IsOper(client_p)))
-		{
-			exit_client(client_p, client_p, client_p, "Excess Flood");
+		if(IsAnyDead(client_p))
 			return;
+			
+		/* Check to make sure we're not flooding */
+		if(!IsAnyServer(client_p) &&
+		   (rb_linebuf_alloclen(&client_p->localClient->buf_recvq) > ConfigFileEntry.client_flood))
+		{
+			if(!(ConfigFileEntry.no_oper_flood && IsOper(client_p)))
+			{
+				exit_client(client_p, client_p, client_p, "Excess Flood");
+				return;
+			}
 		}
-	}
 
-	/* If we get here, we need to register for another COMM_SELECT_READ */
-	if(PARSE_AS_SERVER(client_p))
-	{
-		rb_setselect(client_p->localClient->F->fd, FDLIST_SERVER, COMM_SELECT_READ,
-			      read_packet, client_p, 0);
-	}
-	else
-	{
-		rb_setselect(client_p->localClient->F->fd, FDLIST_IDLECLIENT,
-			       COMM_SELECT_READ, read_packet, client_p, 0);
+		/* bail if short read */
+		if(length < READBUF_SIZE)
+		{
+			rb_setselect(client_p->localClient->F, RB_SELECT_READ, read_packet, client_p);
+			return;
+		}
 	}
 }