Mercurial > pidgin
annotate libfaim/aim_rxqueue.c @ 840:595ac7759563
[gaim-migrate @ 850]
lots of (mostly useless for us) libfaim changes. should help portability, but it was always portable between unices.... stupid win32ers.
committer: Tailor Script <tailor@pidgin.im>
| author | Eric Warmenhoven <eric@warmenhoven.org> |
|---|---|
| date | Mon, 04 Sep 2000 23:37:32 +0000 |
| parents | 88f8f98de02d |
| children | e1da6a6ec42b |
| rev | line source |
|---|---|
| 2 | 1 /* |
| 237 | 2 * aim_rxqueue.c |
| 3 * | |
| 4 * This file contains the management routines for the receive | |
| 5 * (incoming packet) queue. The actual packet handlers are in | |
| 6 * aim_rxhandlers.c. | |
| 2 | 7 */ |
| 8 | |
|
283
0f14e6d8a51b
[gaim-migrate @ 293]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
279
diff
changeset
|
9 #include <faim/aim.h> |
|
840
595ac7759563
[gaim-migrate @ 850]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
835
diff
changeset
|
10 #include <sys/socket.h> |
|
595ac7759563
[gaim-migrate @ 850]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
835
diff
changeset
|
11 |
|
595ac7759563
[gaim-migrate @ 850]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
835
diff
changeset
|
12 /* |
|
595ac7759563
[gaim-migrate @ 850]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
835
diff
changeset
|
13 * Since not all implementations support MSG_WAITALL, define |
|
595ac7759563
[gaim-migrate @ 850]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
835
diff
changeset
|
14 * an alternate guarenteed read function... |
|
595ac7759563
[gaim-migrate @ 850]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
835
diff
changeset
|
15 * |
|
595ac7759563
[gaim-migrate @ 850]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
835
diff
changeset
|
16 * We keep recv() for systems that can do it because it means |
|
595ac7759563
[gaim-migrate @ 850]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
835
diff
changeset
|
17 * a single system call for the entire packet, where read may |
|
595ac7759563
[gaim-migrate @ 850]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
835
diff
changeset
|
18 * take more for a badly fragmented packet. |
|
595ac7759563
[gaim-migrate @ 850]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
835
diff
changeset
|
19 * |
|
595ac7759563
[gaim-migrate @ 850]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
835
diff
changeset
|
20 */ |
|
595ac7759563
[gaim-migrate @ 850]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
835
diff
changeset
|
21 static int aim_recv(int fd, void *buf, size_t count) |
|
595ac7759563
[gaim-migrate @ 850]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
835
diff
changeset
|
22 { |
|
595ac7759563
[gaim-migrate @ 850]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
835
diff
changeset
|
23 #ifdef MSG_WAITALL |
|
595ac7759563
[gaim-migrate @ 850]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
835
diff
changeset
|
24 return recv(fd, buf, count, MSG_WAITALL); |
|
595ac7759563
[gaim-migrate @ 850]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
835
diff
changeset
|
25 #else |
|
595ac7759563
[gaim-migrate @ 850]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
835
diff
changeset
|
26 int left, ret, cur = 0; |
|
595ac7759563
[gaim-migrate @ 850]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
835
diff
changeset
|
27 |
|
595ac7759563
[gaim-migrate @ 850]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
835
diff
changeset
|
28 left = count; |
|
595ac7759563
[gaim-migrate @ 850]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
835
diff
changeset
|
29 |
|
595ac7759563
[gaim-migrate @ 850]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
835
diff
changeset
|
30 while (left) { |
|
595ac7759563
[gaim-migrate @ 850]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
835
diff
changeset
|
31 ret = read(fd, ((unsigned char *)buf)+cur, left); |
|
595ac7759563
[gaim-migrate @ 850]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
835
diff
changeset
|
32 if (ret == -1) |
|
595ac7759563
[gaim-migrate @ 850]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
835
diff
changeset
|
33 return -1; |
|
595ac7759563
[gaim-migrate @ 850]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
835
diff
changeset
|
34 if (ret == 0) |
|
595ac7759563
[gaim-migrate @ 850]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
835
diff
changeset
|
35 return cur; |
|
595ac7759563
[gaim-migrate @ 850]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
835
diff
changeset
|
36 |
|
595ac7759563
[gaim-migrate @ 850]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
835
diff
changeset
|
37 cur += ret; |
|
595ac7759563
[gaim-migrate @ 850]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
835
diff
changeset
|
38 left -= ret; |
|
595ac7759563
[gaim-migrate @ 850]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
835
diff
changeset
|
39 } |
|
595ac7759563
[gaim-migrate @ 850]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
835
diff
changeset
|
40 |
|
595ac7759563
[gaim-migrate @ 850]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
835
diff
changeset
|
41 return cur; |
|
595ac7759563
[gaim-migrate @ 850]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
835
diff
changeset
|
42 #endif |
|
595ac7759563
[gaim-migrate @ 850]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
835
diff
changeset
|
43 } |
| 2 | 44 |
| 45 /* | |
| 237 | 46 * Grab a single command sequence off the socket, and enqueue |
| 47 * it in the incoming event queue in a seperate struct. | |
| 48 */ | |
|
840
595ac7759563
[gaim-migrate @ 850]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
835
diff
changeset
|
49 faim_export int aim_get_command(struct aim_session_t *sess, struct aim_conn_t *conn) |
| 2 | 50 { |
|
445
e4c34ca88d9b
[gaim-migrate @ 455]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
338
diff
changeset
|
51 unsigned char generic[6]; |
| 237 | 52 struct command_rx_struct *newrx = NULL; |
| 2 | 53 |
| 237 | 54 if (!sess || !conn) |
| 55 return 0; | |
| 2 | 56 |
| 237 | 57 if (conn->fd < 3) /* can happen when people abuse the interface */ |
| 58 return 0; | |
| 2 | 59 |
| 237 | 60 /* |
|
338
9d258a0aa560
[gaim-migrate @ 348]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
283
diff
changeset
|
61 * Rendezvous (client-client) connections do not speak |
|
9d258a0aa560
[gaim-migrate @ 348]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
283
diff
changeset
|
62 * FLAP, so this function will break on them. |
|
9d258a0aa560
[gaim-migrate @ 348]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
283
diff
changeset
|
63 */ |
|
445
e4c34ca88d9b
[gaim-migrate @ 455]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
338
diff
changeset
|
64 if (conn->type == AIM_CONN_TYPE_RENDEZVOUS) |
|
e4c34ca88d9b
[gaim-migrate @ 455]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
338
diff
changeset
|
65 return aim_get_command_rendezvous(sess, conn); |
|
503
6e318907bcce
[gaim-migrate @ 513]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
445
diff
changeset
|
66 if (conn->type == AIM_CONN_TYPE_RENDEZVOUS_OUT) |
|
6e318907bcce
[gaim-migrate @ 513]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
445
diff
changeset
|
67 return 0; |
|
338
9d258a0aa560
[gaim-migrate @ 348]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
283
diff
changeset
|
68 |
|
9d258a0aa560
[gaim-migrate @ 348]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
283
diff
changeset
|
69 /* |
| 237 | 70 * Read FLAP header. Six bytes: |
| 71 * | |
| 72 * 0 char -- Always 0x2a | |
| 73 * 1 char -- Channel ID. Usually 2 -- 1 and 4 are used during login. | |
| 74 * 2 short -- Sequence number | |
| 75 * 4 short -- Number of data bytes that follow. | |
| 76 */ | |
|
279
501e09c51cbc
[gaim-migrate @ 289]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
237
diff
changeset
|
77 faim_mutex_lock(&conn->active); |
|
840
595ac7759563
[gaim-migrate @ 850]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
835
diff
changeset
|
78 if (aim_recv(conn->fd, generic, 6) < 6){ |
|
503
6e318907bcce
[gaim-migrate @ 513]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
445
diff
changeset
|
79 aim_conn_close(conn); |
|
279
501e09c51cbc
[gaim-migrate @ 289]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
237
diff
changeset
|
80 faim_mutex_unlock(&conn->active); |
| 237 | 81 return -1; |
| 82 } | |
| 2 | 83 |
| 237 | 84 /* |
| 85 * This shouldn't happen unless the socket breaks, the server breaks, | |
| 86 * or we break. We must handle it just in case. | |
| 87 */ | |
| 88 if (generic[0] != 0x2a) { | |
| 89 faimdprintf(1, "Bad incoming data!"); | |
|
503
6e318907bcce
[gaim-migrate @ 513]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
445
diff
changeset
|
90 aim_conn_close(conn); |
|
445
e4c34ca88d9b
[gaim-migrate @ 455]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
338
diff
changeset
|
91 faim_mutex_unlock(&conn->active); |
| 237 | 92 return -1; |
| 93 } | |
| 2 | 94 |
| 95 /* allocate a new struct */ | |
|
445
e4c34ca88d9b
[gaim-migrate @ 455]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
338
diff
changeset
|
96 if (!(newrx = (struct command_rx_struct *)malloc(sizeof(struct command_rx_struct)))) { |
|
e4c34ca88d9b
[gaim-migrate @ 455]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
338
diff
changeset
|
97 faim_mutex_unlock(&conn->active); |
| 237 | 98 return -1; |
|
445
e4c34ca88d9b
[gaim-migrate @ 455]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
338
diff
changeset
|
99 } |
| 237 | 100 memset(newrx, 0x00, sizeof(struct command_rx_struct)); |
| 2 | 101 |
| 237 | 102 newrx->lock = 1; /* lock the struct */ |
| 103 | |
|
445
e4c34ca88d9b
[gaim-migrate @ 455]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
338
diff
changeset
|
104 /* we're doing OSCAR if we're here */ |
|
e4c34ca88d9b
[gaim-migrate @ 455]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
338
diff
changeset
|
105 newrx->hdrtype = AIM_FRAMETYPE_OSCAR; |
|
e4c34ca88d9b
[gaim-migrate @ 455]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
338
diff
changeset
|
106 |
| 237 | 107 /* store channel -- byte 2 */ |
|
445
e4c34ca88d9b
[gaim-migrate @ 455]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
338
diff
changeset
|
108 newrx->hdr.oscar.type = (char) generic[1]; |
| 2 | 109 |
| 110 /* store seqnum -- bytes 3 and 4 */ | |
|
445
e4c34ca88d9b
[gaim-migrate @ 455]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
338
diff
changeset
|
111 newrx->hdr.oscar.seqnum = aimutil_get16(generic+2); |
| 2 | 112 |
| 113 /* store commandlen -- bytes 5 and 6 */ | |
| 237 | 114 newrx->commandlen = aimutil_get16(generic+4); |
| 115 | |
| 116 newrx->nofree = 0; /* free by default */ | |
| 2 | 117 |
| 118 /* malloc for data portion */ | |
|
445
e4c34ca88d9b
[gaim-migrate @ 455]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
338
diff
changeset
|
119 if (!(newrx->data = (u_char *) malloc(newrx->commandlen))) { |
| 237 | 120 free(newrx); |
|
445
e4c34ca88d9b
[gaim-migrate @ 455]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
338
diff
changeset
|
121 faim_mutex_unlock(&conn->active); |
| 237 | 122 return -1; |
| 123 } | |
| 2 | 124 |
| 125 /* read the data portion of the packet */ | |
|
840
595ac7759563
[gaim-migrate @ 850]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
835
diff
changeset
|
126 if (aim_recv(conn->fd, newrx->data, newrx->commandlen) < newrx->commandlen){ |
| 237 | 127 free(newrx->data); |
| 128 free(newrx); | |
|
503
6e318907bcce
[gaim-migrate @ 513]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
445
diff
changeset
|
129 aim_conn_close(conn); |
|
279
501e09c51cbc
[gaim-migrate @ 289]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
237
diff
changeset
|
130 faim_mutex_unlock(&conn->active); |
| 237 | 131 return -1; |
| 132 } | |
|
279
501e09c51cbc
[gaim-migrate @ 289]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
237
diff
changeset
|
133 faim_mutex_unlock(&conn->active); |
| 2 | 134 |
| 237 | 135 newrx->conn = conn; |
| 2 | 136 |
| 237 | 137 newrx->next = NULL; /* this will always be at the bottom */ |
| 138 newrx->lock = 0; /* unlock */ | |
| 2 | 139 |
| 140 /* enqueue this packet */ | |
| 237 | 141 if (sess->queue_incoming == NULL) { |
| 142 sess->queue_incoming = newrx; | |
| 143 } else { | |
| 144 struct command_rx_struct *cur; | |
| 145 | |
| 146 /* | |
| 147 * This append operation takes a while. It might be faster | |
| 148 * if we maintain a pointer to the last entry in the queue | |
| 149 * and just update that. Need to determine if the overhead | |
| 150 * to maintain that is lower than the overhead for this loop. | |
| 151 */ | |
| 152 for (cur = sess->queue_incoming; cur->next; cur = cur->next) | |
| 153 ; | |
| 154 cur->next = newrx; | |
| 155 } | |
| 156 | |
| 157 newrx->conn->lastactivity = time(NULL); | |
| 2 | 158 |
| 159 return 0; | |
| 160 } | |
| 161 | |
| 162 /* | |
| 237 | 163 * Purge recieve queue of all handled commands (->handled==1). Also |
| 164 * allows for selective freeing using ->nofree so that the client can | |
| 165 * keep the data for various purposes. | |
| 166 * | |
| 167 * If ->nofree is nonzero, the frame will be delinked from the global list, | |
| 168 * but will not be free'ed. The client _must_ keep a pointer to the | |
| 169 * data -- libfaim will not! If the client marks ->nofree but | |
| 170 * does not keep a pointer, it's lost forever. | |
| 171 * | |
| 2 | 172 */ |
|
840
595ac7759563
[gaim-migrate @ 850]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
835
diff
changeset
|
173 faim_export void aim_purge_rxqueue(struct aim_session_t *sess) |
| 2 | 174 { |
| 237 | 175 struct command_rx_struct *cur = NULL; |
| 176 struct command_rx_struct *tmp; | |
| 2 | 177 |
| 237 | 178 if (sess->queue_incoming == NULL) |
| 179 return; | |
| 180 | |
| 181 if (sess->queue_incoming->next == NULL) { | |
| 182 if (sess->queue_incoming->handled) { | |
| 183 tmp = sess->queue_incoming; | |
| 184 sess->queue_incoming = NULL; | |
| 185 | |
| 186 if (!tmp->nofree) { | |
|
445
e4c34ca88d9b
[gaim-migrate @ 455]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
338
diff
changeset
|
187 if (tmp->hdrtype == AIM_FRAMETYPE_OFT) |
|
e4c34ca88d9b
[gaim-migrate @ 455]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
338
diff
changeset
|
188 free(tmp->hdr.oft.hdr2); |
| 237 | 189 free(tmp->data); |
| 190 free(tmp); | |
| 191 } else | |
| 192 tmp->next = NULL; | |
| 2 | 193 } |
| 237 | 194 return; |
| 195 } | |
| 2 | 196 |
| 237 | 197 for(cur = sess->queue_incoming; cur->next != NULL; ) { |
| 198 if (cur->next->handled) { | |
| 199 tmp = cur->next; | |
| 200 cur->next = tmp->next; | |
| 201 if (!tmp->nofree) { | |
|
445
e4c34ca88d9b
[gaim-migrate @ 455]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
338
diff
changeset
|
202 if (tmp->hdrtype == AIM_FRAMETYPE_OFT) |
|
e4c34ca88d9b
[gaim-migrate @ 455]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
338
diff
changeset
|
203 free(tmp->hdr.oft.hdr2); |
| 237 | 204 free(tmp->data); |
| 205 free(tmp); | |
| 206 } else | |
| 207 tmp->next = NULL; | |
| 208 } | |
| 209 cur = cur->next; | |
| 2 | 210 |
| 237 | 211 /* |
| 212 * Be careful here. Because of the way we just | |
| 213 * manipulated the pointer, cur may be NULL and | |
| 214 * the for() will segfault doing the check unless | |
| 215 * we find this case first. | |
| 216 */ | |
| 217 if (cur == NULL) | |
| 218 break; | |
| 219 } | |
| 220 | |
| 221 return; | |
| 2 | 222 } |
|
445
e4c34ca88d9b
[gaim-migrate @ 455]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
338
diff
changeset
|
223 |
|
e4c34ca88d9b
[gaim-migrate @ 455]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
338
diff
changeset
|
224 /* |
|
e4c34ca88d9b
[gaim-migrate @ 455]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
338
diff
changeset
|
225 * Since aim_get_command will aim_conn_kill dead connections, we need |
|
e4c34ca88d9b
[gaim-migrate @ 455]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
338
diff
changeset
|
226 * to clean up the rxqueue of unprocessed connections on that socket. |
|
e4c34ca88d9b
[gaim-migrate @ 455]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
338
diff
changeset
|
227 * |
|
e4c34ca88d9b
[gaim-migrate @ 455]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
338
diff
changeset
|
228 * XXX: this is something that was handled better in the old connection |
|
e4c34ca88d9b
[gaim-migrate @ 455]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
338
diff
changeset
|
229 * handling method, but eh. |
|
e4c34ca88d9b
[gaim-migrate @ 455]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
338
diff
changeset
|
230 */ |
|
840
595ac7759563
[gaim-migrate @ 850]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
835
diff
changeset
|
231 faim_internal void aim_rxqueue_cleanbyconn(struct aim_session_t *sess, struct aim_conn_t *conn) |
|
445
e4c34ca88d9b
[gaim-migrate @ 455]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
338
diff
changeset
|
232 { |
|
e4c34ca88d9b
[gaim-migrate @ 455]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
338
diff
changeset
|
233 struct command_rx_struct *currx; |
|
e4c34ca88d9b
[gaim-migrate @ 455]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
338
diff
changeset
|
234 |
|
e4c34ca88d9b
[gaim-migrate @ 455]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
338
diff
changeset
|
235 for (currx = sess->queue_incoming; currx; currx = currx->next) { |
|
e4c34ca88d9b
[gaim-migrate @ 455]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
338
diff
changeset
|
236 if ((!currx->handled) && (currx->conn == conn)) |
|
e4c34ca88d9b
[gaim-migrate @ 455]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
338
diff
changeset
|
237 currx->handled = 1; |
|
e4c34ca88d9b
[gaim-migrate @ 455]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
338
diff
changeset
|
238 } |
|
e4c34ca88d9b
[gaim-migrate @ 455]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
338
diff
changeset
|
239 return; |
|
e4c34ca88d9b
[gaim-migrate @ 455]
Eric Warmenhoven <eric@warmenhoven.org>
parents:
338
diff
changeset
|
240 } |
