--- sourceforge.net/trunk/rdesktop/rdpdr.c 2004/01/21 18:02:38 570 +++ sourceforge.net/trunk/rdesktop/rdpdr.c 2004/02/06 10:32:13 598 @@ -1,4 +1,6 @@ #include +#include +#include #include "rdesktop.h" #define IRP_MJ_CREATE 0x00 @@ -20,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 */ @@ -39,14 +38,20 @@ 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 */ +}; + +struct async_iorequest *g_iorequest; /* Return device_id for a given handle */ int @@ -78,32 +83,42 @@ 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++) + if (g_iorequest == NULL) { - iorq = &g_iorequest[i]; - - if (iorq->fd == 0) - { - 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; - } - } - error("IO request table full. Increase MAX_ASYNC_IO_REQUESTS in rdpdr.c!\n"); - return False; + g_iorequest = (struct async_iorequest *) xmalloc(sizeof(struct async_iorequest)); + g_iorequest->fd = 0; + g_iorequest->next = NULL; + } + + iorq = g_iorequest; + + while (iorq->fd != 0) + { + // create new element if needed + if (iorq->next == NULL) + { + iorq->next = + (struct async_iorequest *) xmalloc(sizeof(struct async_iorequest)); + iorq->next->fd = 0; + iorq->next->next = NULL; + } + iorq = iorq->next; + } + 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) { @@ -186,29 +201,30 @@ 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); + if (printerinfo->blob) + xfree(printerinfo->blob); /* Blob is sent twice if reconnecting */ + break; + default: + out_uint32(s, 0); } } #if 0 @@ -280,7 +296,6 @@ buffer = (uint8 *) xmalloc(1024); buffer[0] = 0; - switch (g_rdpdr_device[device].device_type) { case DEVICE_TYPE_SERIAL: @@ -303,6 +318,7 @@ case DEVICE_TYPE_DISK: fns = &disk_fns; + rw_blocking = False; break; case DEVICE_TYPE_SCARD: @@ -389,7 +405,6 @@ status = STATUS_CANCELLED; break; - case IRP_MJ_WRITE: buffer_len = 1; @@ -659,7 +674,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); @@ -669,15 +687,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) { @@ -700,25 +716,28 @@ 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; + uint32 req_size = 0; if (timed_out) { @@ -726,55 +745,123 @@ 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); + + req_size = + (iorq->length - iorq->partial_len) > + 8192 ? 8192 : (iorq->length - + iorq->partial_len); + /* never read larger chunks than 8k - chances are that it will block */ + status = fns->read(iorq->fd, + iorq->buffer + iorq->partial_len, + req_size, 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 */ + /* or if result was 0 and status success - EOF */ + if ((iorq->partial_len == iorq->length) || + (result == 0)) + { +#if WITH_DEBUG_RDP5 + DEBUG(("RDPDR: AIO total %u bytes read of %u\n", iorq->partial_len, iorq->length)); +#endif + /* send the data */ + status = STATUS_SUCCESS; + rdpdr_send_completion(iorq->device, + iorq->id, status, + iorq->partial_len, + iorq->buffer, + iorq->partial_len); + xfree(iorq->buffer); + iorq->fd = 0; + if (prev != NULL) + { + prev->next = iorq->next; + xfree(iorq); + } + else + { + // Even if NULL + g_iorequest = 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; + req_size = + (iorq->length - iorq->partial_len) > + 8192 ? 8192 : (iorq->length - + iorq->partial_len); + + /* never write larger chunks than 8k - chances are that it will block */ + status = fns->write(iorq->fd, + iorq->buffer + + iorq->partial_len, req_size, 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 */ + /* or we couldn't write */ + if ((iorq->partial_len == iorq->length) + || (result == 0)) + { +#if WITH_DEBUG_RDP5 + DEBUG(("RDPDR: AIO total %u bytes written of %u\n", iorq->partial_len, iorq->length)); +#endif + /* send a status success */ + status = STATUS_SUCCESS; + rdpdr_send_completion(iorq->device, + iorq->id, status, + iorq->partial_len, "", + 1); + + xfree(iorq->buffer); + iorq->fd = 0; + if (prev != NULL) + { + prev->next = iorq->next; + xfree(iorq); + } + else + { + // Even if NULL + g_iorequest = iorq->next; + xfree(iorq); + } + } } break; } + } + prev = iorq; + iorq = iorq->next; } + } /* Abort a pending io request for a given handle and major */ @@ -782,13 +869,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)) @@ -797,8 +884,23 @@ 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); + } + else + { + // Even if NULL + g_iorequest = iorq->next; + xfree(iorq); + } return True; } + + prev = iorq; + iorq = iorq->next; } + return False; }