--- sourceforge.net/trunk/rdesktop/rdpdr.c 2004/01/22 20:31:59 576 +++ sourceforge.net/trunk/rdesktop/rdpdr.c 2004/01/30 14:10:32 592 @@ -22,15 +22,12 @@ #define IRP_MN_QUERY_DIRECTORY 0x01 #define IRP_MN_NOTIFY_CHANGE_DIRECTORY 0x02 -#define MAX_ASYNC_IO_REQUESTS 10 - extern char hostname[16]; extern DEVICE_FNS serial_fns; extern DEVICE_FNS printer_fns; extern DEVICE_FNS parallel_fns; extern DEVICE_FNS disk_fns; - static VCHANNEL *rdpdr_channel; /* If select() times out, the request for the device with handle g_min_timeout_fd is aborted */ @@ -41,14 +38,18 @@ RDPDR_DEVICE g_rdpdr_device[RDPDR_MAX_DEVICES]; /* Used to store incoming io request, until they are ready to be completed */ +/* using a linked list ensures that they are processed in the right order, */ +/* if multiple ios are being done on the same fd */ struct async_iorequest { - uint32 fd, major, minor, offset, device, id, length; + uint32 fd, major, minor, offset, device, id, length, partial_len; long timeout, /* Total timeout */ itv_timeout; /* Interval timeout (between serial characters) */ uint8 *buffer; DEVICE_FNS *fns; -} g_iorequest[MAX_ASYNC_IO_REQUESTS]; + + struct async_iorequest *next; /* next element in list */ +} g_iorequest; /* Return device_id for a given handle */ int @@ -80,31 +81,56 @@ add_async_iorequest(uint32 device, uint32 file, uint32 id, uint32 major, uint32 length, DEVICE_FNS * fns, long total_timeout, long interval_timeout, uint8 * buffer) { - int i; struct async_iorequest *iorq; - for (i = 0; i < MAX_ASYNC_IO_REQUESTS; i++) + iorq = &g_iorequest; + while (iorq->fd != 0) { - iorq = &g_iorequest[i]; + // create new element if needed + if (iorq->next == NULL) + iorq->next = + (struct async_iorequest *) xmalloc(sizeof(struct async_iorequest)); + + iorq = iorq->next; + } - if (iorq->fd == 0) + /* first element is special since it doesn't get deleted */ + /* don't want to get io out of order */ + if (g_iorequest.fd == 0) + { + iorq = &g_iorequest; + /* look for first occurrence of fd */ + while (iorq->next != NULL) { - iorq->device = device; - iorq->fd = file; - iorq->id = id; - iorq->major = major; - iorq->length = length; - iorq->fns = fns; - iorq->timeout = total_timeout; - iorq->itv_timeout = interval_timeout; - iorq->buffer = buffer; - return True; + if (iorq->fd == file) + break; + iorq = iorq->next; } + /* if same create new link at end of chain instead */ + if (iorq->fd == file) + { + while (iorq->next != NULL) + iorq = iorq->next; + iorq->next = + (struct async_iorequest *) xmalloc(sizeof(struct async_iorequest)); + iorq = iorq->next; + } + else + iorq = &g_iorequest; /* didn't find fs use first entry */ } - error("IO request table full. Increase MAX_ASYNC_IO_REQUESTS in rdpdr.c!\n"); - return False; -} + iorq->device = device; + iorq->fd = file; + iorq->id = id; + iorq->major = major; + iorq->length = length; + iorq->partial_len = 0; + iorq->fns = fns; + iorq->timeout = total_timeout; + iorq->itv_timeout = interval_timeout; + iorq->buffer = buffer; + return True; +} void rdpdr_send_connect(void) @@ -188,29 +214,29 @@ out_uint32_le(s, i); /* RDP Device ID */ out_uint8p(s, g_rdpdr_device[i].name, 8); - if (g_rdpdr_device[i].device_type == DEVICE_TYPE_PRINTER) + switch (g_rdpdr_device[i].device_type) { - printerinfo = (PRINTER *) g_rdpdr_device[i].pdevice_data; + case DEVICE_TYPE_PRINTER: + printerinfo = (PRINTER *) g_rdpdr_device[i].pdevice_data; - driverlen = 2 * strlen(printerinfo->driver) + 2; - printerlen = 2 * strlen(printerinfo->printer) + 2; - bloblen = printerinfo->bloblen; - - out_uint32_le(s, 24 + driverlen + printerlen + bloblen); /* length of extra info */ - out_uint32_le(s, printerinfo->default_printer ? 2 : 0); - out_uint8s(s, 8); /* unknown */ - out_uint32_le(s, driverlen); - out_uint32_le(s, printerlen); - out_uint32_le(s, bloblen); - rdp_out_unistr(s, printerinfo->driver, driverlen - 2); - rdp_out_unistr(s, printerinfo->printer, printerlen - 2); - out_uint8a(s, printerinfo->blob, bloblen); + driverlen = 2 * strlen(printerinfo->driver) + 2; + printerlen = 2 * strlen(printerinfo->printer) + 2; + bloblen = printerinfo->bloblen; + + out_uint32_le(s, 24 + driverlen + printerlen + bloblen); /* length of extra info */ + out_uint32_le(s, printerinfo->default_printer ? 2 : 0); + out_uint8s(s, 8); /* unknown */ + out_uint32_le(s, driverlen); + out_uint32_le(s, printerlen); + out_uint32_le(s, bloblen); + rdp_out_unistr(s, printerinfo->driver, driverlen - 2); + rdp_out_unistr(s, printerinfo->printer, printerlen - 2); + out_uint8a(s, printerinfo->blob, bloblen); - xfree(printerinfo->blob); /* Blob is sent twice if reconnecting */ - } - else - { - out_uint32(s, 0); + xfree(printerinfo->blob); /* Blob is sent twice if reconnecting */ + break; + default: + out_uint32(s, 0); } } #if 0 @@ -282,7 +308,6 @@ buffer = (uint8 *) xmalloc(1024); buffer[0] = 0; - switch (g_rdpdr_device[device].device_type) { case DEVICE_TYPE_SERIAL: @@ -304,6 +329,7 @@ case DEVICE_TYPE_DISK: + /*rw_blocking = False; */ fns = &disk_fns; break; @@ -391,7 +417,6 @@ status = STATUS_CANCELLED; break; - case IRP_MJ_WRITE: buffer_len = 1; @@ -661,7 +686,10 @@ { if (g_num_devices > 0) { - rdpdr_channel = channel_register("rdpdr", CHANNEL_OPTION_INITIALIZED | CHANNEL_OPTION_COMPRESS_RDP, rdpdr_process); + rdpdr_channel = + channel_register("rdpdr", + CHANNEL_OPTION_INITIALIZED | CHANNEL_OPTION_COMPRESS_RDP, + rdpdr_process); } return (rdpdr_channel != NULL); @@ -671,15 +699,13 @@ void rdpdr_add_fds(int *n, fd_set * rfds, fd_set * wfds, struct timeval *tv, BOOL * timeout) { - int i; long select_timeout = 0; // Timeout value to be used for select() (in millisecons). struct async_iorequest *iorq; - for (i = 0; i < MAX_ASYNC_IO_REQUESTS; i++) + iorq = &g_iorequest; + while (iorq != NULL) { - iorq = &g_iorequest[i]; - - if (iorq->fd != 0) // Found a pending io request + if (iorq->fd != 0) { switch (iorq->major) { @@ -702,25 +728,27 @@ break; case IRP_MJ_WRITE: - FD_SET(iorq->fd, wfds); break; } *n = MAX(*n, iorq->fd); } + + iorq = iorq->next; } } + /* Check if select() returned with one of the rdpdr file descriptors, and complete io if it did */ void rdpdr_check_fds(fd_set * rfds, fd_set * wfds, BOOL timed_out) { - int i; NTSTATUS status; - uint32 result = 0, buffer_len = 0; + uint32 result = 0; DEVICE_FNS *fns; struct async_iorequest *iorq; + struct async_iorequest *prev; if (timed_out) { @@ -728,55 +756,89 @@ return; } - // Walk through array of pending io_rq's - for (i = 0; i < MAX_ASYNC_IO_REQUESTS; i++) + iorq = &g_iorequest; + prev = NULL; + while (iorq != NULL) { - iorq = &g_iorequest[i]; if (iorq->fd != 0) { switch (iorq->major) { - case IRP_MJ_READ: - if (FD_ISSET(iorq->fd, rfds)) { - // Read, and send data. + /* Read the data */ fns = iorq->fns; - status = fns->read(iorq->fd, iorq->buffer, - iorq->length, 0, &result); - buffer_len = result; - - rdpdr_send_completion(iorq->device, iorq->id, - status, result, iorq->buffer, - buffer_len); + status = fns->read(iorq->fd, + iorq->buffer + iorq->partial_len, + iorq->length - iorq->partial_len, + 0, &result); + iorq->partial_len += result; #if WITH_DEBUG_RDP5 DEBUG(("RDPDR: %d bytes of data read\n", result)); #endif - xfree(iorq->buffer); - iorq->fd = 0; + /* only delete link if all data has been transfered */ + if (iorq->partial_len == iorq->length) + { + /* send the data */ + status = STATUS_SUCCESS; + rdpdr_send_completion(iorq->device, + iorq->id, status, + iorq->length, + iorq->buffer, result); + + xfree(iorq->buffer); + iorq->fd = 0; + if (prev != NULL) + { + prev->next = iorq->next; + xfree(iorq); + } + } } break; - case IRP_MJ_WRITE: - if (FD_ISSET(iorq->fd, wfds)) { - // Write data and send completion. + /* Write data. */ fns = iorq->fns; - status = fns->write(iorq->fd, iorq->buffer, - iorq->length, 0, &result); - rdpdr_send_completion(iorq->device, iorq->id, - status, result, "", 1); - - xfree(iorq->buffer); - iorq->fd = 0; + status = fns->write(iorq->fd, + iorq->buffer + + iorq->partial_len, + iorq->length - + iorq->partial_len, 0, &result); + iorq->partial_len += result; +#if WITH_DEBUG_RDP5 + DEBUG(("RDPDR: %d bytes of data written\n", + result)); +#endif + /* only delete link if all data has been transfered */ + if (iorq->partial_len == iorq->length) + { + /* send a status success */ + status = STATUS_SUCCESS; + rdpdr_send_completion(iorq->device, + iorq->id, status, + iorq->length, "", 1); + + xfree(iorq->buffer); + iorq->fd = 0; + if (prev != NULL) + { + prev->next = iorq->next; + xfree(iorq); + } + } } break; } + } + prev = iorq; + iorq = iorq->next; } + } /* Abort a pending io request for a given handle and major */ @@ -784,13 +846,13 @@ rdpdr_abort_io(uint32 fd, uint32 major, NTSTATUS status) { uint32 result; - int i; struct async_iorequest *iorq; + struct async_iorequest *prev; - for (i = 0; i < MAX_ASYNC_IO_REQUESTS; i++) + iorq = &g_iorequest; + prev = NULL; + while (iorq != NULL) { - iorq = &g_iorequest[i]; - // Only remove from table when major is not set, or when correct major is supplied. // Abort read should not abort a write io request. if ((iorq->fd == fd) && (major == 0 || iorq->major == major)) @@ -799,8 +861,17 @@ rdpdr_send_completion(iorq->device, iorq->id, status, result, "", 1); xfree(iorq->buffer); iorq->fd = 0; + if (prev != NULL) + { + prev->next = iorq->next; + xfree(iorq); + } return True; } + + prev = iorq; + iorq = iorq->next; } + return False; }