- Wine-Commits - list.winehq.org

Eric Pouech : ucrtbase: Handle undecoration of function's reference qualifiers (C++11).
by Alexandre Julliard Nov. 21, 2022

Nov. 21, 2022

Module: wine Branch: master Commit: d1eb78a75deb49cea3b273742b05362f227c8d22 URL: https://gitlab.winehq.org/wine/wine/-/commit/d1eb78a75deb49cea3b273742b0536… Author: Eric Pouech <eric.pouech(a)gmail.com> Date: Mon Nov 21 11:30:13 2022 +0100 ucrtbase: Handle undecoration of function's reference qualifiers (C++11). Introducing get_function_qualifier(). Feature is not available in msvcrt. Signed-off-by: Eric Pouech <eric.pouech(a)gmail.com> --- dlls/msvcrt/undname.c | 59 +++++++++++++++++++++++++++++------------------ dlls/ucrtbase/tests/cpp.c | 1 - 2 files changed, 37 insertions(+), 23 deletions(-) diff --git a/dlls/msvcrt/undname.c b/dlls/msvcrt/undname.c index 8db8e8f01b9..5a7283caac7 100644 --- a/dlls/msvcrt/undname.c +++ b/dlls/msvcrt/undname.c @@ -80,6 +80,7 @@ struct parsed_symbol enum datatype_e { DT_NO_LEADING_WS = 0x01, + DT_NO_LRSEP_WS = 0x02, }; /* Type for parsing mangled types */ @@ -407,28 +408,35 @@ static char* get_args(struct parsed_symbol* sym, struct array* pmt_ref, BOOL z_t return args_str; } -static void append_extended_qualifier(struct parsed_symbol *sym, const char **where, const char *str) +static void append_extended_qualifier(struct parsed_symbol *sym, const char **where, + const char *str, BOOL is_ms_keyword) { - if (!(sym->flags & UNDNAME_NO_MS_KEYWORDS)) + if (!is_ms_keyword || !(sym->flags & UNDNAME_NO_MS_KEYWORDS)) { - if (sym->flags & UNDNAME_NO_LEADING_UNDERSCORES) + if (is_ms_keyword && (sym->flags & UNDNAME_NO_LEADING_UNDERSCORES)) str += 2; - *where = *where ? str_printf(sym, "%s %s", *where, str) : str; + *where = *where ? str_printf(sym, "%s%s%s%s", *where, is_ms_keyword ? " " : "", str, is_ms_keyword ? "" : " ") : + str_printf(sym, "%s%s", str, is_ms_keyword ? "" : " "); } } static void get_extended_qualifier(struct parsed_symbol *sym, struct datatype_t *xdt) { + unsigned fl = 0; xdt->left = xdt->right = NULL; xdt->flags = 0; for (;;) { switch (*sym->current) { - case 'E': append_extended_qualifier(sym, &xdt->right, "__ptr64"); break; - case 'F': append_extended_qualifier(sym, &xdt->left, "__unaligned"); break; - case 'I': append_extended_qualifier(sym, &xdt->right, "__restrict"); break; - default: return; + case 'E': append_extended_qualifier(sym, &xdt->right, "__ptr64", TRUE); fl |= 2; break; + case 'F': append_extended_qualifier(sym, &xdt->left, "__unaligned", TRUE); fl |= 2; break; +#ifdef _UCRT + case 'G': append_extended_qualifier(sym, &xdt->right, "&", FALSE); fl |= 1; break; + case 'H': append_extended_qualifier(sym, &xdt->right, "&&", FALSE); fl |= 1; break; +#endif + case 'I': append_extended_qualifier(sym, &xdt->right, "__restrict", TRUE); fl |= 2; break; + default: if (fl == 1 || (fl == 3 && (sym->flags & UNDNAME_NO_MS_KEYWORDS))) xdt->flags = DT_NO_LRSEP_WS; return; } sym->current++; } @@ -457,7 +465,10 @@ static BOOL get_qualifier(struct parsed_symbol *sym, struct datatype_t *xdt, con default: return FALSE; } if (qualif) + { + xdt->flags &= ~DT_NO_LRSEP_WS; xdt->left = xdt->left ? str_printf(sym, "%s %s", qualif, xdt->left) : qualif; + } if (ch >= 'Q' && ch <= 'T') /* pointer to member, fetch class */ { const char* class = get_class_name(sym); @@ -473,6 +484,16 @@ static BOOL get_qualifier(struct parsed_symbol *sym, struct datatype_t *xdt, con return TRUE; } +static BOOL get_function_qualifier(struct parsed_symbol *sym, const char** qualif) +{ + struct datatype_t xdt; + + if (!get_qualifier(sym, &xdt, NULL)) return FALSE; + *qualif = (xdt.left || xdt.right) ? + str_printf(sym, "%s%s%s", xdt.left, (xdt.flags & DT_NO_LRSEP_WS) ? "" : " ", xdt.right) : NULL; + return TRUE; +} + static BOOL get_qualified_type(struct datatype_t *ct, struct parsed_symbol* sym, struct array *pmt_ref, char qualif, enum datatype_flags flags) { @@ -949,24 +970,20 @@ static BOOL demangle_datatype(struct parsed_symbol* sym, struct datatype_t* ct, { struct function_signature fs; const char* class; - struct datatype_t xdt; + const char* function_qualifier; sym->current++; if (!(class = get_class_name(sym))) goto done; - if (!get_qualifier(sym, &xdt, NULL)) + if (!get_function_qualifier(sym, &function_qualifier)) goto done; if (!get_function_signature(sym, pmt_ref, &fs)) goto done; - if (xdt.left) - xdt.left = str_printf(sym, "%s %s", xdt.left, xdt.right); - else if (xdt.right) - xdt.left = str_printf(sym, " %s", xdt.right); ct->left = str_printf(sym, "%s%s (%s %s::*", fs.return_ct.left, fs.return_ct.right, fs.call_conv, class); - ct->right = str_printf(sym, ")%s%s", fs.arguments, xdt.left); + ct->right = str_printf(sym, ")%s%s", fs.arguments, function_qualifier); } else if (*sym->current == '6') { @@ -1233,7 +1250,7 @@ static BOOL handle_method(struct parsed_symbol* sym, BOOL cast_op) const char* member_type = NULL; struct datatype_t ct_ret; const char* call_conv; - struct datatype_t xdt = {NULL}; + const char* function_qualifier = NULL; const char* exported; const char* args_str = NULL; const char* name = NULL; @@ -1362,9 +1379,7 @@ static BOOL handle_method(struct parsed_symbol* sym, BOOL cast_op) (accmem <= 'X' && (accmem - 'A') % 8 != 2 && (accmem - 'A') % 8 != 3))) { /* Implicit 'this' pointer */ - /* If there is an implicit this pointer, const qualifier follows */ - if (!get_qualifier(sym, &xdt, NULL)) goto done; - if (xdt.left || xdt.right) xdt.left = str_printf(sym, "%s %s", xdt.left, xdt.right); + if (!get_function_qualifier(sym, &function_qualifier)) goto done; } if (!get_calling_convention(*sym->current++, &call_conv, &exported, @@ -1395,8 +1410,8 @@ static BOOL handle_method(struct parsed_symbol* sym, BOOL cast_op) mark = sym->stack.num; if (has_args && !(args_str = get_args(sym, &array_pmt, TRUE, '(', ')'))) goto done; - if (sym->flags & UNDNAME_NAME_ONLY) args_str = xdt.left = NULL; - if (sym->flags & UNDNAME_NO_THISTYPE) xdt.left = NULL; + if (sym->flags & UNDNAME_NAME_ONLY) args_str = function_qualifier = NULL; + if (sym->flags & UNDNAME_NO_THISTYPE) function_qualifier = NULL; sym->stack.num = mark; /* Note: '()' after 'Z' means 'throws', but we don't care here @@ -1406,7 +1421,7 @@ static BOOL handle_method(struct parsed_symbol* sym, BOOL cast_op) access, member_type, ct_ret.left, (ct_ret.left && !ct_ret.right) ? " " : NULL, call_conv, call_conv ? " " : NULL, exported, - name, args_str, xdt.left, ct_ret.right); + name, args_str, function_qualifier, ct_ret.right); ret = TRUE; done: return ret; diff --git a/dlls/ucrtbase/tests/cpp.c b/dlls/ucrtbase/tests/cpp.c index b9b36a1e740..9a66d66d814 100644 --- a/dlls/ucrtbase/tests/cpp.c +++ b/dlls/ucrtbase/tests/cpp.c @@ -219,7 +219,6 @@ static void test___unDName(void) for (i = 0; i < ARRAY_SIZE(und_tests); i++) { char *name = p___unDName(0, und_tests[i].in, 0, malloc, free, und_tests[i].flags); - todo_wine_if(i >= 3) ok(!strcmp(name, und_tests[i].out) || broken(und_tests[i].broken && !strcmp(und_tests[i].broken, name)), "unDName returned %s for #%u\n", wine_dbgstr_a(name), i);

1 0

Alexandre Julliard : ntdll: Use unsigned int for the return value of server calls.
by Alexandre Julliard Nov. 21, 2022

Nov. 21, 2022

Module: wine Branch: master Commit: af4378d46dbb72e682017485212442bf865c2226 URL: https://gitlab.winehq.org/wine/wine/-/commit/af4378d46dbb72e682017485212442… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Nov 21 16:21:43 2022 +0100 ntdll: Use unsigned int for the return value of server calls. To avoid typecasts in debug traces. --- dlls/ntdll/unix/env.c | 2 +- dlls/ntdll/unix/file.c | 102 +++++++++++++++++++------------------- dlls/ntdll/unix/loader.c | 10 ++-- dlls/ntdll/unix/process.c | 24 ++++----- dlls/ntdll/unix/registry.c | 28 +++++------ dlls/ntdll/unix/security.c | 24 ++++----- dlls/ntdll/unix/serial.c | 4 +- dlls/ntdll/unix/server.c | 10 ++-- dlls/ntdll/unix/signal_i386.c | 2 +- dlls/ntdll/unix/socket.c | 9 ++-- dlls/ntdll/unix/sync.c | 110 ++++++++++++++++++++--------------------- dlls/ntdll/unix/system.c | 8 +-- dlls/ntdll/unix/thread.c | 28 +++++------ dlls/ntdll/unix/unix_private.h | 4 +- dlls/ntdll/unix/virtual.c | 77 ++++++++++++++--------------- 15 files changed, 219 insertions(+), 223 deletions(-)

1 0

Alexandre Julliard : ntdll: Use int instead of long types in a few places.
by Alexandre Julliard Nov. 21, 2022

Nov. 21, 2022

Module: wine Branch: master Commit: 74a5851ef48cafe02e4383df4dd5742ae55ba75f URL: https://gitlab.winehq.org/wine/wine/-/commit/74a5851ef48cafe02e4383df4dd574… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Nov 21 16:21:16 2022 +0100 ntdll: Use int instead of long types in a few places. To avoid typecasts in debug traces. --- dlls/ntdll/unix/debug.c | 2 +- dlls/ntdll/unix/env.c | 6 ++--- dlls/ntdll/unix/file.c | 16 ++++++------ dlls/ntdll/unix/serial.c | 18 +++++++------- dlls/ntdll/unix/signal_arm.c | 49 ++++++++++++++++++------------------ dlls/ntdll/unix/signal_i386.c | 58 +++++++++++++++++++++---------------------- dlls/ntdll/unix/socket.c | 2 +- dlls/ntdll/unix/tape.c | 2 +- include/wine/afd.h | 4 +-- 9 files changed, 78 insertions(+), 79 deletions(-)

1 0

Alexandre Julliard : ntdll: Fix int/long type mismatches.
by Alexandre Julliard Nov. 21, 2022

Nov. 21, 2022

Module: wine Branch: master Commit: d5e2ab1a9858135ba778ee642a5b91325d33e928 URL: https://gitlab.winehq.org/wine/wine/-/commit/d5e2ab1a9858135ba778ee642a5b91… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Nov 21 16:18:44 2022 +0100 ntdll: Fix int/long type mismatches. --- dlls/ntdll/unix/server.c | 5 +++-- dlls/ntdll/unix/socket.c | 4 ++-- dlls/ntdll/unix/sync.c | 12 ++++++------ dlls/ntdll/unix/system.c | 19 ++++++++++--------- dlls/ntdll/unix/thread.c | 2 +- 5 files changed, 22 insertions(+), 20 deletions(-) diff --git a/dlls/ntdll/unix/server.c b/dlls/ntdll/unix/server.c index 07d9c8d3acd..3daaf95d8ee 100644 --- a/dlls/ntdll/unix/server.c +++ b/dlls/ntdll/unix/server.c @@ -474,11 +474,12 @@ static void invoke_system_apc( const apc_call_t *call, apc_result_t *result, BOO size = call->virtual_protect.size; if ((ULONG_PTR)addr == call->virtual_protect.addr && size == call->virtual_protect.size) { + ULONG prot; result->virtual_protect.status = NtProtectVirtualMemory( NtCurrentProcess(), &addr, &size, - call->virtual_protect.prot, - &result->virtual_protect.prot ); + call->virtual_protect.prot, &prot ); result->virtual_protect.addr = wine_server_client_ptr( addr ); result->virtual_protect.size = size; + result->virtual_protect.prot = prot; } else result->virtual_protect.status = STATUS_INVALID_PARAMETER; break; diff --git a/dlls/ntdll/unix/socket.c b/dlls/ntdll/unix/socket.c index 7dbf03365bb..beb8382687a 100644 --- a/dlls/ntdll/unix/socket.c +++ b/dlls/ntdll/unix/socket.c @@ -113,7 +113,7 @@ struct async_recv_ioctl void *control; struct WS_sockaddr *addr; int *addr_len; - DWORD *ret_flags; + unsigned int *ret_flags; int unix_flags; unsigned int count; BOOL icmp_over_dgram; @@ -907,7 +907,7 @@ static NTSTATUS sock_recv( HANDLE handle, HANDLE event, PIO_APC_ROUTINE apc, voi static NTSTATUS sock_ioctl_recv( HANDLE handle, HANDLE event, PIO_APC_ROUTINE apc, void *apc_user, IO_STATUS_BLOCK *io, int fd, const void *buffers_ptr, unsigned int count, WSABUF *control, - struct WS_sockaddr *addr, int *addr_len, DWORD *ret_flags, int unix_flags, int force_async ) + struct WS_sockaddr *addr, int *addr_len, unsigned int *ret_flags, int unix_flags, int force_async ) { struct async_recv_ioctl *async; DWORD async_size; diff --git a/dlls/ntdll/unix/sync.c b/dlls/ntdll/unix/sync.c index 9112572c67d..c085bffe34d 100644 --- a/dlls/ntdll/unix/sync.c +++ b/dlls/ntdll/unix/sync.c @@ -109,7 +109,7 @@ static inline ULONGLONG monotonic_counter(void) static int futex_private = 128; -static inline int futex_wait( const int *addr, int val, struct timespec *timeout ) +static inline int futex_wait( const LONG *addr, int val, struct timespec *timeout ) { #if (defined(__i386__) || defined(__arm__)) && _TIME_BITS==64 if (timeout && sizeof(*timeout) != 8) @@ -125,14 +125,14 @@ static inline int futex_wait( const int *addr, int val, struct timespec *timeout return syscall( __NR_futex, addr, FUTEX_WAIT | futex_private, val, timeout, 0, 0 ); } -static inline int futex_wake( const int *addr, int val ) +static inline int futex_wake( const LONG *addr, int val ) { return syscall( __NR_futex, addr, FUTEX_WAKE | futex_private, val, NULL, 0, 0 ); } static inline int use_futexes(void) { - static int supported = -1; + static LONG supported = -1; if (supported == -1) { @@ -2309,7 +2309,7 @@ union tid_alert_entry #else HANDLE event; #ifdef __linux__ - int futex; + LONG futex; #endif #endif }; @@ -2395,7 +2395,7 @@ NTSTATUS WINAPI NtAlertThreadByThreadId( HANDLE tid ) #ifdef __linux__ if (use_futexes()) { - int *futex = &entry->futex; + LONG *futex = &entry->futex; if (!InterlockedExchange( futex, 1 )) futex_wake( futex, 1 ); return STATUS_SUCCESS; @@ -2496,7 +2496,7 @@ NTSTATUS WINAPI NtWaitForAlertByThreadId( const void *address, const LARGE_INTEG #ifdef __linux__ if (use_futexes()) { - int *futex = &entry->futex; + LONG *futex = &entry->futex; ULONGLONG end; int ret; diff --git a/dlls/ntdll/unix/system.c b/dlls/ntdll/unix/system.c index ba2051da65e..4551c9603a3 100644 --- a/dlls/ntdll/unix/system.c +++ b/dlls/ntdll/unix/system.c @@ -777,7 +777,7 @@ static BOOL logical_proc_info_add_group( DWORD num_cpus, ULONG_PTR mask ) static BOOL sysfs_parse_bitmap(const char *filename, ULONG_PTR *mask) { FILE *f; - DWORD r; + unsigned int r; f = fopen(filename, "r"); if (!f) return FALSE; @@ -802,7 +802,7 @@ static BOOL sysfs_parse_bitmap(const char *filename, ULONG_PTR *mask) * - /sys/devices/system/cpu/cpu0/cache/index0/shared_cpu_list * - /sys/devices/system/cpu/cpu0/topology/thread_siblings_list. */ -static BOOL sysfs_count_list_elements(const char *filename, DWORD *result) +static BOOL sysfs_count_list_elements(const char *filename, unsigned int *result) { FILE *f; @@ -812,7 +812,7 @@ static BOOL sysfs_count_list_elements(const char *filename, DWORD *result) while (!feof(f)) { char op; - DWORD beg, end; + unsigned int beg, end; if (!fscanf(f, "%u%c ", &beg, &op)) break; if(op == '-') @@ -834,7 +834,7 @@ static NTSTATUS create_logical_proc_info(void) static const char numa_info[] = "/sys/devices/system/node/node%u/cpumap"; FILE *fcpu_list, *fnuma_list, *f; - DWORD beg, end, i, j, r, num_cpus = 0, max_cpus = 0; + unsigned int beg, end, i, j, r, num_cpus = 0, max_cpus = 0; char op, name[MAX_PATH]; ULONG_PTR all_cpus_mask = 0; @@ -863,7 +863,7 @@ static NTSTATUS create_logical_proc_info(void) for(i = beg; i <= end; i++) { - DWORD phys_core = 0; + unsigned int phys_core = 0; ULONG_PTR thread_mask = 0; if (i > 8 * sizeof(ULONG_PTR)) @@ -3682,14 +3682,15 @@ NTSTATUS WINAPI NtPowerInformation( POWER_INFORMATION_LEVEL level, void *input, if ((out_size / sizeof(PROCESSOR_POWER_INFORMATION)) < out_cpus) return STATUS_BUFFER_TOO_SMALL; #if defined(linux) { + unsigned int val; char filename[128]; FILE* f; for(i = 0; i < out_cpus; i++) { sprintf(filename, "/sys/devices/system/cpu/cpu%d/cpufreq/cpuinfo_max_freq", i); f = fopen(filename, "r"); - if (f && (fscanf(f, "%d", &cpu_power[i].MaxMhz) == 1)) { - cpu_power[i].MaxMhz /= 1000; + if (f && (fscanf(f, "%u", &val) == 1)) { + cpu_power[i].MaxMhz = val / 1000; fclose(f); cpu_power[i].CurrentMhz = cpu_power[i].MaxMhz; } @@ -3707,8 +3708,8 @@ NTSTATUS WINAPI NtPowerInformation( POWER_INFORMATION_LEVEL level, void *input, sprintf(filename, "/sys/devices/system/cpu/cpu%d/cpufreq/scaling_max_freq", i); f = fopen(filename, "r"); - if(f && (fscanf(f, "%d", &cpu_power[i].MhzLimit) == 1)) { - cpu_power[i].MhzLimit /= 1000; + if(f && (fscanf(f, "%u", &val) == 1)) { + cpu_power[i].MhzLimit = val / 1000; fclose(f); } else diff --git a/dlls/ntdll/unix/thread.c b/dlls/ntdll/unix/thread.c index 711cb4cb13c..5d140c90c42 100644 --- a/dlls/ntdll/unix/thread.c +++ b/dlls/ntdll/unix/thread.c @@ -79,7 +79,7 @@ WINE_DECLARE_DEBUG_CHANNEL(threadname); pthread_key_t teb_key = 0; -static int nb_threads = 1; +static LONG nb_threads = 1; static inline int get_unix_exit_code( NTSTATUS status ) {

1 0

Piotr Caban : winprint: Implement RAW print processor.
by Alexandre Julliard Nov. 21, 2022

Nov. 21, 2022

Module: wine Branch: master Commit: f63a9b592ce29bb5cd081d441ab4dbdc1b373478 URL: https://gitlab.winehq.org/wine/wine/-/commit/f63a9b592ce29bb5cd081d441ab4db… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Wed Oct 26 13:33:37 2022 +0200 winprint: Implement RAW print processor. --- dlls/winprint/Makefile.in | 1 + dlls/winprint/printproc.c | 161 +++++++++++++++++++++++++++++++++++++++++----- 2 files changed, 147 insertions(+), 15 deletions(-) diff --git a/dlls/winprint/Makefile.in b/dlls/winprint/Makefile.in index 002004b1d0d..57851fc5b04 100644 --- a/dlls/winprint/Makefile.in +++ b/dlls/winprint/Makefile.in @@ -1,4 +1,5 @@ MODULE = winprint.dll +IMPORTS = winspool C_SRCS = \ printproc.c diff --git a/dlls/winprint/printproc.c b/dlls/winprint/printproc.c index 7cc8759d801..d7d9cce9d93 100644 --- a/dlls/winprint/printproc.c +++ b/dlls/winprint/printproc.c @@ -18,6 +18,20 @@ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA */ +/* Missing datatypes support: + * RAW [FF appended] + * RAW [FF auto] + * NT EMF 1.003 + * NT EMF 1.006 + * NT EMF 1.007 + * NT EMF 1.008 + * TEXT + * XPS2GDI + * + * Implement print processor features like e.g. printing multiple copies. */ + +#include <stdlib.h> + #include "windows.h" #include "winspool.h" #include "ddk/winsplp.h" @@ -26,34 +40,133 @@ WINE_DEFAULT_DEBUG_CHANNEL(winprint); +#define PP_MAGIC 0x952173fd + +struct pp_data +{ + DWORD magic; + HANDLE hport; + WCHAR *doc_name; + WCHAR *out_file; +}; + +static struct pp_data* get_handle_data(HANDLE pp) +{ + struct pp_data *ret = (struct pp_data *)pp; + + if (!ret || ret->magic != PP_MAGIC) + { + SetLastError(ERROR_INVALID_HANDLE); + return NULL; + } + return ret; +} + BOOL WINAPI EnumPrintProcessorDatatypesW(WCHAR *server, WCHAR *name, DWORD level, BYTE *datatypes, DWORD size, DWORD *needed, DWORD *no) { - FIXME("%s, %s, %ld, %p, %ld, %p, %p\n", debugstr_w(server), debugstr_w(name), + static const WCHAR raw[] = L"RAW"; + + DATATYPES_INFO_1W *info = (DATATYPES_INFO_1W *)datatypes; + + TRACE("%s, %s, %ld, %p, %ld, %p, %p\n", debugstr_w(server), debugstr_w(name), level, datatypes, size, needed, no); - SetLastError(ERROR_CALL_NOT_IMPLEMENTED); - return FALSE; + + if (!needed || !no) + { + SetLastError(ERROR_INVALID_PARAMETER); + return FALSE; + } + + *no = 0; + *needed = sizeof(*info) + sizeof(raw); + + if (level != 1 || !datatypes) + { + SetLastError(ERROR_INVALID_PARAMETER); + return FALSE; + } + + if (size < *needed) + { + SetLastError(ERROR_INSUFFICIENT_BUFFER); + return FALSE; + } + + *no = 1; + info->pName = (WCHAR*)(info + 1); + memcpy(info + 1, raw, sizeof(raw)); + return TRUE; } HANDLE WINAPI OpenPrintProcessor(WCHAR *port, PRINTPROCESSOROPENDATA *open_data) { - FIXME("%s, %p\n", debugstr_w(port), open_data); - SetLastError(ERROR_CALL_NOT_IMPLEMENTED); - return NULL; -} + struct pp_data *data; + HANDLE hport; -BOOL WINAPI ClosePrintProcessor(HANDLE pp) -{ - FIXME("%p\n", pp); - SetLastError(ERROR_CALL_NOT_IMPLEMENTED); - return FALSE; + TRACE("%s, %p\n", debugstr_w(port), open_data); + + if (!port || !open_data || !open_data->pDatatype) + { + SetLastError(ERROR_INVALID_PARAMETER); + return NULL; + } + if (!wcscmp(open_data->pDatatype, L"RAW")) + { + SetLastError(ERROR_INVALID_DATATYPE); + return NULL; + } + + if (!OpenPrinterW(port, &hport, NULL)) + return NULL; + + data = LocalAlloc(LMEM_FIXED | LMEM_ZEROINIT, sizeof(*data)); + if (!data) + return NULL; + data->magic = PP_MAGIC; + data->hport = hport; + data->doc_name = wcsdup(open_data->pDocumentName); + data->out_file = wcsdup(open_data->pOutputFile); + return (HANDLE)data; } BOOL WINAPI PrintDocumentOnPrintProcessor(HANDLE pp, WCHAR *doc_name) { - FIXME("%p, %s\n", pp, debugstr_w(doc_name)); - SetLastError(ERROR_CALL_NOT_IMPLEMENTED); - return FALSE; + struct pp_data *data = get_handle_data(pp); + HANDLE spool_data; + DOC_INFO_1W info; + BYTE buf[0x1000]; + DWORD r, w; + + TRACE("%p, %s\n", pp, debugstr_w(doc_name)); + + if (!data) + return FALSE; + + if (!OpenPrinterW(doc_name, &spool_data, NULL)) + return FALSE; + + info.pDocName = data->doc_name; + info.pOutputFile = data->out_file; + info.pDatatype = (WCHAR *)L"RAW"; + if (!StartDocPrinterW(data->hport, 1, (BYTE *)&info)) + { + ClosePrinter(spool_data); + return FALSE; + } + + while (ReadPrinter(spool_data, buf, sizeof(buf), &r) && r) + { + if (!WritePrinter(data->hport, buf, r, &w) || r != w) + { + ClosePrinter(spool_data); + EndDocPrinter(data->hport); + return FALSE; + } + } + ClosePrinter(spool_data); + + return EndDocPrinter(data->hport); } BOOL WINAPI ControlPrintProcessor(HANDLE pp, DWORD cmd) @@ -62,3 +175,21 @@ BOOL WINAPI ControlPrintProcessor(HANDLE pp, DWORD cmd) SetLastError(ERROR_CALL_NOT_IMPLEMENTED); return FALSE; } + +BOOL WINAPI ClosePrintProcessor(HANDLE pp) +{ + struct pp_data *data = get_handle_data(pp); + + TRACE("%p\n", pp); + + if (!data) + return FALSE; + + ClosePrinter(data->hport); + free(data->doc_name); + free(data->out_file); + + memset(data, 0, sizeof(*data)); + LocalFree(data); + return TRUE; +}

1 0

Piotr Caban : winprint: Add stub dll.
by Alexandre Julliard Nov. 21, 2022

Nov. 21, 2022

Module: wine Branch: master Commit: b7d7b23a617bdbb4177830c074cc0c8807ec9bce URL: https://gitlab.winehq.org/wine/wine/-/commit/b7d7b23a617bdbb4177830c074cc0c… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Wed Oct 26 13:12:27 2022 +0200 winprint: Add stub dll. --- configure | 2 ++ configure.ac | 1 + dlls/winprint/Makefile.in | 4 +++ dlls/winprint/printproc.c | 64 +++++++++++++++++++++++++++++++++++++++++++++ dlls/winprint/winprint.spec | 6 +++++ loader/wine.inf.in | 2 ++ 6 files changed, 79 insertions(+) diff --git a/configure b/configure index 99fefd2c172..e17b71555f2 100755 --- a/configure +++ b/configure @@ -1472,6 +1472,7 @@ enable_winhttp enable_wininet enable_winmm enable_winnls32 +enable_winprint enable_winscard enable_winspool_drv enable_winsta @@ -21918,6 +21919,7 @@ wine_fn_config_makefile dlls/winmm enable_winmm wine_fn_config_makefile dlls/winmm/tests enable_tests wine_fn_config_makefile dlls/winnls.dll16 enable_win16 wine_fn_config_makefile dlls/winnls32 enable_winnls32 +wine_fn_config_makefile dlls/winprint enable_winprint wine_fn_config_makefile dlls/winscard enable_winscard wine_fn_config_makefile dlls/winsock.dll16 enable_win16 wine_fn_config_makefile dlls/winspool.drv enable_winspool_drv diff --git a/configure.ac b/configure.ac index abf03f41b09..b7d443c1182 100644 --- a/configure.ac +++ b/configure.ac @@ -3203,6 +3203,7 @@ WINE_CONFIG_MAKEFILE(dlls/winmm) WINE_CONFIG_MAKEFILE(dlls/winmm/tests) WINE_CONFIG_MAKEFILE(dlls/winnls.dll16,enable_win16) WINE_CONFIG_MAKEFILE(dlls/winnls32) +WINE_CONFIG_MAKEFILE(dlls/winprint) WINE_CONFIG_MAKEFILE(dlls/winscard) WINE_CONFIG_MAKEFILE(dlls/winsock.dll16,enable_win16) WINE_CONFIG_MAKEFILE(dlls/winspool.drv) diff --git a/dlls/winprint/Makefile.in b/dlls/winprint/Makefile.in new file mode 100644 index 00000000000..002004b1d0d --- /dev/null +++ b/dlls/winprint/Makefile.in @@ -0,0 +1,4 @@ +MODULE = winprint.dll + +C_SRCS = \ + printproc.c diff --git a/dlls/winprint/printproc.c b/dlls/winprint/printproc.c new file mode 100644 index 00000000000..7cc8759d801 --- /dev/null +++ b/dlls/winprint/printproc.c @@ -0,0 +1,64 @@ +/* + * Print processor implementation. + * + * Copyright 2022 Piotr Caban for CodeWeavers + * + * This library is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public + * License as published by the Free Software Foundation; either + * version 2.1 of the License, or (at your option) any later version. + * + * This library is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public + * License along with this library; if not, write to the Free Software + * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA + */ + +#include "windows.h" +#include "winspool.h" +#include "ddk/winsplp.h" + +#include "wine/debug.h" + +WINE_DEFAULT_DEBUG_CHANNEL(winprint); + +BOOL WINAPI EnumPrintProcessorDatatypesW(WCHAR *server, WCHAR *name, DWORD level, + BYTE *datatypes, DWORD size, DWORD *needed, DWORD *no) +{ + FIXME("%s, %s, %ld, %p, %ld, %p, %p\n", debugstr_w(server), debugstr_w(name), + level, datatypes, size, needed, no); + SetLastError(ERROR_CALL_NOT_IMPLEMENTED); + return FALSE; +} + +HANDLE WINAPI OpenPrintProcessor(WCHAR *port, PRINTPROCESSOROPENDATA *open_data) +{ + FIXME("%s, %p\n", debugstr_w(port), open_data); + SetLastError(ERROR_CALL_NOT_IMPLEMENTED); + return NULL; +} + +BOOL WINAPI ClosePrintProcessor(HANDLE pp) +{ + FIXME("%p\n", pp); + SetLastError(ERROR_CALL_NOT_IMPLEMENTED); + return FALSE; +} + +BOOL WINAPI PrintDocumentOnPrintProcessor(HANDLE pp, WCHAR *doc_name) +{ + FIXME("%p, %s\n", pp, debugstr_w(doc_name)); + SetLastError(ERROR_CALL_NOT_IMPLEMENTED); + return FALSE; +} + +BOOL WINAPI ControlPrintProcessor(HANDLE pp, DWORD cmd) +{ + FIXME("%p, %ld\n", pp, cmd); + SetLastError(ERROR_CALL_NOT_IMPLEMENTED); + return FALSE; +} diff --git a/dlls/winprint/winprint.spec b/dlls/winprint/winprint.spec new file mode 100644 index 00000000000..b7b4642b695 --- /dev/null +++ b/dlls/winprint/winprint.spec @@ -0,0 +1,6 @@ +@ stdcall ClosePrintProcessor(ptr) +@ stdcall ControlPrintProcessor(ptr long) +@ stdcall EnumPrintProcessorDatatypesW(ptr ptr long ptr long ptr ptr) +@ stub GetPrintProcessorCapabilities +@ stdcall OpenPrintProcessor(ptr ptr) +@ stdcall PrintDocumentOnPrintProcessor(ptr ptr) diff --git a/loader/wine.inf.in b/loader/wine.inf.in index e7b435ed0f0..127d1f9ad6a 100644 --- a/loader/wine.inf.in +++ b/loader/wine.inf.in @@ -2149,6 +2149,7 @@ HKLM,%CurrentVersion%\Telephony\Country List\998,"SameAreaRule",,"G" 16427,System\OLE DB,msdasql.dll 16427,System\ADO,msado15.dll 66000,3,wineps.drv +55,,winprint.dll 12,,*.sys,- 11,,*.msstyles,- 11,,* @@ -2211,6 +2212,7 @@ HKLM,%CurrentVersion%\Telephony\Country List\998,"SameAreaRule",,"G" 16427,System\OLE DB,msdasql.dll 16427,System\ADO,msado15.dll 66000,3,wineps.drv +55,,winprint.dll 12,,*.sys 11,,*

1 0

Alexandre Julliard : ntdll: Use unsigned int instead of NTSTATUS in async callbacks.
by Alexandre Julliard Nov. 21, 2022

Nov. 21, 2022

Module: wine Branch: master Commit: 1a7cacc48949b9a4182539d7f7f2c0db32be7cee URL: https://gitlab.winehq.org/wine/wine/-/commit/1a7cacc48949b9a4182539d7f7f2c0… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Nov 21 16:19:08 2022 +0100 ntdll: Use unsigned int instead of NTSTATUS in async callbacks. To avoid typecasts in debug traces. --- dlls/ntdll/unix/file.c | 8 ++++---- dlls/ntdll/unix/server.c | 2 +- dlls/ntdll/unix/socket.c | 6 +++--- dlls/ntdll/unix/unix_private.h | 2 +- 4 files changed, 9 insertions(+), 9 deletions(-) diff --git a/dlls/ntdll/unix/file.c b/dlls/ntdll/unix/file.c index 95ca5fa9b51..c40447933f0 100644 --- a/dlls/ntdll/unix/file.c +++ b/dlls/ntdll/unix/file.c @@ -4903,7 +4903,7 @@ struct async_fileio *alloc_fileio( DWORD size, async_callback_t callback, HANDLE } /* callback for irp async I/O completion */ -static BOOL irp_completion( void *user, ULONG_PTR *info, NTSTATUS *status ) +static BOOL irp_completion( void *user, ULONG_PTR *info, unsigned int *status ) { struct async_irp *async = user; @@ -4921,7 +4921,7 @@ static BOOL irp_completion( void *user, ULONG_PTR *info, NTSTATUS *status ) return TRUE; } -static BOOL async_read_proc( void *user, ULONG_PTR *info, NTSTATUS *status ) +static BOOL async_read_proc( void *user, ULONG_PTR *info, unsigned int *status ) { struct async_fileio_read *fileio = user; int fd, needs_close, result; @@ -4971,7 +4971,7 @@ static BOOL async_read_proc( void *user, ULONG_PTR *info, NTSTATUS *status ) return TRUE; } -static BOOL async_write_proc( void *user, ULONG_PTR *info, NTSTATUS *status ) +static BOOL async_write_proc( void *user, ULONG_PTR *info, unsigned int *status ) { struct async_fileio_write *fileio = user; int result, fd, needs_close; @@ -6327,7 +6327,7 @@ NTSTATUS WINAPI NtUnlockFile( HANDLE handle, IO_STATUS_BLOCK *io_status, LARGE_I } -static BOOL read_changes_apc( void *user, ULONG_PTR *info, NTSTATUS *status ) +static BOOL read_changes_apc( void *user, ULONG_PTR *info, unsigned int *status ) { struct async_fileio_read_changes *fileio = user; int size = 0; diff --git a/dlls/ntdll/unix/server.c b/dlls/ntdll/unix/server.c index 3daaf95d8ee..a960e881395 100644 --- a/dlls/ntdll/unix/server.c +++ b/dlls/ntdll/unix/server.c @@ -371,7 +371,7 @@ static void invoke_system_apc( const apc_call_t *call, apc_result_t *result, BOO { struct async_fileio *user = wine_server_get_ptr( call->async_io.user ); ULONG_PTR info = call->async_io.result; - NTSTATUS status; + unsigned int status; result->type = call->type; status = call->async_io.status; diff --git a/dlls/ntdll/unix/socket.c b/dlls/ntdll/unix/socket.c index 779cc46fe32..3cddc7da965 100644 --- a/dlls/ntdll/unix/socket.c +++ b/dlls/ntdll/unix/socket.c @@ -809,7 +809,7 @@ static NTSTATUS try_recv( int fd, struct async_recv_ioctl *async, ULONG_PTR *siz return status; } -static BOOL async_recv_proc( void *user, ULONG_PTR *info, NTSTATUS *status ) +static BOOL async_recv_proc( void *user, ULONG_PTR *info, unsigned int *status ) { struct async_recv_ioctl *async = user; int fd, needs_close; @@ -1046,7 +1046,7 @@ static NTSTATUS try_send( int fd, struct async_send_ioctl *async ) return STATUS_SUCCESS; } -static BOOL async_send_proc( void *user, ULONG_PTR *info, NTSTATUS *status ) +static BOOL async_send_proc( void *user, ULONG_PTR *info, unsigned int *status ) { struct async_send_ioctl *async = user; int fd, needs_close; @@ -1293,7 +1293,7 @@ static NTSTATUS try_transmit( int sock_fd, int file_fd, struct async_transmit_io return STATUS_SUCCESS; } -static BOOL async_transmit_proc( void *user, ULONG_PTR *info, NTSTATUS *status ) +static BOOL async_transmit_proc( void *user, ULONG_PTR *info, unsigned int *status ) { int sock_fd, file_fd = -1, sock_needs_close = FALSE, file_needs_close = FALSE; struct async_transmit_ioctl *async = user; diff --git a/dlls/ntdll/unix/unix_private.h b/dlls/ntdll/unix/unix_private.h index afdb8397415..b8570e642ac 100644 --- a/dlls/ntdll/unix/unix_private.h +++ b/dlls/ntdll/unix/unix_private.h @@ -76,7 +76,7 @@ static inline struct ntdll_thread_data *ntdll_get_thread_data(void) } /* returns TRUE if the async is complete; FALSE if it should be restarted */ -typedef BOOL async_callback_t( void *user, ULONG_PTR *info, NTSTATUS *status ); +typedef BOOL async_callback_t( void *user, ULONG_PTR *info, unsigned int *status ); struct async_fileio {

1 0

Alexandre Julliard : ntdll: Use UINT instead of ULONG in ioctl helpers.
by Alexandre Julliard Nov. 21, 2022

Nov. 21, 2022

Module: wine Branch: master Commit: 7afd11232eeb9fd019f7ea3cc5a65d7811a028fb URL: https://gitlab.winehq.org/wine/wine/-/commit/7afd11232eeb9fd019f7ea3cc5a65d… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Nov 21 16:19:31 2022 +0100 ntdll: Use UINT instead of ULONG in ioctl helpers. To avoid typecasts in debug traces. --- dlls/ntdll/unix/cdrom.c | 4 ++-- dlls/ntdll/unix/file.c | 6 +++--- dlls/ntdll/unix/serial.c | 10 +++++----- dlls/ntdll/unix/socket.c | 2 +- dlls/ntdll/unix/tape.c | 6 +++--- dlls/ntdll/unix/unix_private.h | 14 +++++++------- 6 files changed, 21 insertions(+), 21 deletions(-) diff --git a/dlls/ntdll/unix/cdrom.c b/dlls/ntdll/unix/cdrom.c index a4a9400a518..a9acdf7a645 100644 --- a/dlls/ntdll/unix/cdrom.c +++ b/dlls/ntdll/unix/cdrom.c @@ -2814,8 +2814,8 @@ static NTSTATUS GetInquiryData(int fd, PSCSI_ADAPTER_BUS_INFO BufferOut, DWORD O * cdrom_DeviceIoControl */ NTSTATUS cdrom_DeviceIoControl( HANDLE device, HANDLE event, PIO_APC_ROUTINE apc, void *apc_user, - IO_STATUS_BLOCK *io, ULONG code, void *in_buffer, - ULONG in_size, void *out_buffer, ULONG out_size ) + IO_STATUS_BLOCK *io, UINT code, void *in_buffer, + UINT in_size, void *out_buffer, UINT out_size ) { DWORD sz = 0; NTSTATUS status = STATUS_SUCCESS; diff --git a/dlls/ntdll/unix/file.c b/dlls/ntdll/unix/file.c index e80eb368e8a..95ca5fa9b51 100644 --- a/dlls/ntdll/unix/file.c +++ b/dlls/ntdll/unix/file.c @@ -5095,9 +5095,9 @@ static NTSTATUS server_write_file( HANDLE handle, HANDLE event, PIO_APC_ROUTINE /* do an ioctl call through the server */ static NTSTATUS server_ioctl_file( HANDLE handle, HANDLE event, PIO_APC_ROUTINE apc, PVOID apc_context, - IO_STATUS_BLOCK *io, ULONG code, - const void *in_buffer, ULONG in_size, - PVOID out_buffer, ULONG out_size ) + IO_STATUS_BLOCK *io, UINT code, + const void *in_buffer, UINT in_size, + PVOID out_buffer, UINT out_size ) { struct async_irp *async; NTSTATUS status; diff --git a/dlls/ntdll/unix/serial.c b/dlls/ntdll/unix/serial.c index 16c5964b77b..fce09500c91 100644 --- a/dlls/ntdll/unix/serial.c +++ b/dlls/ntdll/unix/serial.c @@ -71,7 +71,7 @@ WINE_DEFAULT_DEBUG_CHANNEL(comm); -static const char* iocode2str(DWORD ioc) +static const char* iocode2str(UINT ioc) { switch (ioc) { @@ -1096,8 +1096,8 @@ static NTSTATUS xmit_immediate(HANDLE hDevice, int fd, const char* ptr) } static NTSTATUS io_control( HANDLE device, HANDLE event, PIO_APC_ROUTINE apc, void *apc_user, - IO_STATUS_BLOCK *io, ULONG code, void *in_buffer, - ULONG in_size, void *out_buffer, ULONG out_size ) + IO_STATUS_BLOCK *io, UINT code, void *in_buffer, + UINT in_size, void *out_buffer, UINT out_size ) { DWORD sz = 0, access = FILE_READ_DATA; NTSTATUS status = STATUS_SUCCESS; @@ -1320,8 +1320,8 @@ static NTSTATUS io_control( HANDLE device, HANDLE event, PIO_APC_ROUTINE apc, vo * serial_DeviceIoControl */ NTSTATUS serial_DeviceIoControl( HANDLE device, HANDLE event, PIO_APC_ROUTINE apc, void *apc_user, - IO_STATUS_BLOCK *io, ULONG code, void *in_buffer, - ULONG in_size, void *out_buffer, ULONG out_size ) + IO_STATUS_BLOCK *io, UINT code, void *in_buffer, + UINT in_size, void *out_buffer, UINT out_size ) { NTSTATUS status; diff --git a/dlls/ntdll/unix/socket.c b/dlls/ntdll/unix/socket.c index beb8382687a..779cc46fe32 100644 --- a/dlls/ntdll/unix/socket.c +++ b/dlls/ntdll/unix/socket.c @@ -1487,7 +1487,7 @@ static int get_sock_type( HANDLE handle ) NTSTATUS sock_ioctl( HANDLE handle, HANDLE event, PIO_APC_ROUTINE apc, void *apc_user, IO_STATUS_BLOCK *io, - ULONG code, void *in_buffer, ULONG in_size, void *out_buffer, ULONG out_size ) + UINT code, void *in_buffer, UINT in_size, void *out_buffer, UINT out_size ) { int fd, needs_close = FALSE; NTSTATUS status; diff --git a/dlls/ntdll/unix/tape.c b/dlls/ntdll/unix/tape.c index 40ac307a632..a7ca04badb0 100644 --- a/dlls/ntdll/unix/tape.c +++ b/dlls/ntdll/unix/tape.c @@ -66,7 +66,7 @@ WINE_DEFAULT_DEBUG_CHANNEL(tape); -static const char *io2str( DWORD io ) +static const char *io2str( unsigned int io ) { switch (io) { @@ -519,8 +519,8 @@ static NTSTATUS TAPE_WriteMarks( int fd, const TAPE_WRITE_MARKS *data ) * tape_DeviceIoControl */ NTSTATUS tape_DeviceIoControl( HANDLE device, HANDLE event, PIO_APC_ROUTINE apc, void *apc_user, - IO_STATUS_BLOCK *io, ULONG code, - void *in_buffer, ULONG in_size, void *out_buffer, ULONG out_size ) + IO_STATUS_BLOCK *io, UINT code, + void *in_buffer, UINT in_size, void *out_buffer, UINT out_size ) { DWORD sz = 0; NTSTATUS status = STATUS_INVALID_PARAMETER; diff --git a/dlls/ntdll/unix/unix_private.h b/dlls/ntdll/unix/unix_private.h index 9cf8989fb0d..afdb8397415 100644 --- a/dlls/ntdll/unix/unix_private.h +++ b/dlls/ntdll/unix/unix_private.h @@ -252,21 +252,21 @@ extern void fill_vm_counters( VM_COUNTERS_EX *pvmi, int unix_pid ) DECLSPEC_HIDD extern NTSTATUS open_hkcu_key( const char *path, HANDLE *key ) DECLSPEC_HIDDEN; extern NTSTATUS cdrom_DeviceIoControl( HANDLE device, HANDLE event, PIO_APC_ROUTINE apc, void *apc_user, - IO_STATUS_BLOCK *io, ULONG code, void *in_buffer, - ULONG in_size, void *out_buffer, ULONG out_size ) DECLSPEC_HIDDEN; + IO_STATUS_BLOCK *io, UINT code, void *in_buffer, + UINT in_size, void *out_buffer, UINT out_size ) DECLSPEC_HIDDEN; extern NTSTATUS serial_DeviceIoControl( HANDLE device, HANDLE event, PIO_APC_ROUTINE apc, void *apc_user, - IO_STATUS_BLOCK *io, ULONG code, void *in_buffer, - ULONG in_size, void *out_buffer, ULONG out_size ) DECLSPEC_HIDDEN; + IO_STATUS_BLOCK *io, UINT code, void *in_buffer, + UINT in_size, void *out_buffer, UINT out_size ) DECLSPEC_HIDDEN; extern NTSTATUS serial_FlushBuffersFile( int fd ) DECLSPEC_HIDDEN; extern NTSTATUS sock_ioctl( HANDLE handle, HANDLE event, PIO_APC_ROUTINE apc, void *apc_user, IO_STATUS_BLOCK *io, - ULONG code, void *in_buffer, ULONG in_size, void *out_buffer, ULONG out_size ) DECLSPEC_HIDDEN; + UINT code, void *in_buffer, UINT in_size, void *out_buffer, UINT out_size ) DECLSPEC_HIDDEN; extern NTSTATUS sock_read( HANDLE handle, int fd, HANDLE event, PIO_APC_ROUTINE apc, void *apc_user, IO_STATUS_BLOCK *io, void *buffer, ULONG length ) DECLSPEC_HIDDEN; extern NTSTATUS sock_write( HANDLE handle, int fd, HANDLE event, PIO_APC_ROUTINE apc, void *apc_user, IO_STATUS_BLOCK *io, const void *buffer, ULONG length ) DECLSPEC_HIDDEN; extern NTSTATUS tape_DeviceIoControl( HANDLE device, HANDLE event, PIO_APC_ROUTINE apc, void *apc_user, - IO_STATUS_BLOCK *io, ULONG code, void *in_buffer, - ULONG in_size, void *out_buffer, ULONG out_size ) DECLSPEC_HIDDEN; + IO_STATUS_BLOCK *io, UINT code, void *in_buffer, + UINT in_size, void *out_buffer, UINT out_size ) DECLSPEC_HIDDEN; extern struct async_fileio *alloc_fileio( DWORD size, async_callback_t callback, HANDLE handle ) DECLSPEC_HIDDEN; extern void release_fileio( struct async_fileio *io ) DECLSPEC_HIDDEN;

1 0

Alex Henrie : dinput/tests: Avoid using GCC's typeof extension.
by Alexandre Julliard Nov. 21, 2022

Nov. 21, 2022

Module: wine Branch: master Commit: f0605758e0b6762d5c2ad173a2026fe8a256a702 URL: https://gitlab.winehq.org/wine/wine/-/commit/f0605758e0b6762d5c2ad173a2026f… Author: Alex Henrie <alexhenrie24(a)gmail.com> Date: Mon Nov 21 08:07:22 2022 -0700 dinput/tests: Avoid using GCC's typeof extension. typeof is not available in MSVC. --- dlls/dinput/tests/force_feedback.c | 12 +++++------- dlls/dinput/tests/hotplug.c | 10 ++++------ dlls/dinput/tests/joystick8.c | 12 +++++------- 3 files changed, 14 insertions(+), 20 deletions(-) diff --git a/dlls/dinput/tests/force_feedback.c b/dlls/dinput/tests/force_feedback.c index 8fb15e38f13..ba37320e9f0 100644 --- a/dlls/dinput/tests/force_feedback.c +++ b/dlls/dinput/tests/force_feedback.c @@ -48,13 +48,11 @@ #include "windows.gaming.input.forcefeedback.h" #undef Size -#define MAKE_FUNC(f) static typeof(f) *p ## f -MAKE_FUNC( RoGetActivationFactory ); -MAKE_FUNC( RoInitialize ); -MAKE_FUNC( WindowsCreateString ); -MAKE_FUNC( WindowsDeleteString ); -MAKE_FUNC( WindowsGetStringRawBuffer ); -#undef MAKE_FUNC +static HRESULT WINAPI (*pRoGetActivationFactory)( HSTRING, REFIID, void** ); +static HRESULT WINAPI (*pRoInitialize)( RO_INIT_TYPE ); +static HRESULT WINAPI (*pWindowsCreateString)( const WCHAR*, UINT32, HSTRING* ); +static HRESULT WINAPI (*pWindowsDeleteString)( HSTRING str ); +static const WCHAR* WINAPI (*pWindowsGetStringRawBuffer)( HSTRING, UINT32* ); static BOOL load_combase_functions(void) { diff --git a/dlls/dinput/tests/hotplug.c b/dlls/dinput/tests/hotplug.c index 9896e7b480e..e915d04745a 100644 --- a/dlls/dinput/tests/hotplug.c +++ b/dlls/dinput/tests/hotplug.c @@ -50,12 +50,10 @@ #include "windows.gaming.input.custom.h" #undef Size -#define MAKE_FUNC(f) static typeof(f) *p ## f -MAKE_FUNC( RoGetActivationFactory ); -MAKE_FUNC( RoInitialize ); -MAKE_FUNC( WindowsCreateString ); -MAKE_FUNC( WindowsDeleteString ); -#undef MAKE_FUNC +static HRESULT WINAPI (*pRoGetActivationFactory)( HSTRING, REFIID, void** ); +static HRESULT WINAPI (*pRoInitialize)( RO_INIT_TYPE ); +static HRESULT WINAPI (*pWindowsCreateString)( const WCHAR*, UINT32, HSTRING* ); +static HRESULT WINAPI (*pWindowsDeleteString)( HSTRING str ); static BOOL load_combase_functions(void) { diff --git a/dlls/dinput/tests/joystick8.c b/dlls/dinput/tests/joystick8.c index 0981d917b2a..f568c9298c6 100644 --- a/dlls/dinput/tests/joystick8.c +++ b/dlls/dinput/tests/joystick8.c @@ -47,13 +47,11 @@ #include "windows.gaming.input.h" #undef Size -#define MAKE_FUNC(f) static typeof(f) *p ## f -MAKE_FUNC( RoGetActivationFactory ); -MAKE_FUNC( RoInitialize ); -MAKE_FUNC( WindowsCreateString ); -MAKE_FUNC( WindowsDeleteString ); -MAKE_FUNC( WindowsGetStringRawBuffer ); -#undef MAKE_FUNC +static HRESULT WINAPI (*pRoGetActivationFactory)( HSTRING, REFIID, void** ); +static HRESULT WINAPI (*pRoInitialize)( RO_INIT_TYPE ); +static HRESULT WINAPI (*pWindowsCreateString)( const WCHAR*, UINT32, HSTRING* ); +static HRESULT WINAPI (*pWindowsDeleteString)( HSTRING str ); +static const WCHAR* WINAPI (*pWindowsGetStringRawBuffer)( HSTRING, UINT32* ); static BOOL load_combase_functions(void) {

1 0

Alex Henrie : msxml3: Use standard C functions for memory allocation in saxreader.c.
by Alexandre Julliard Nov. 21, 2022

Nov. 21, 2022

Module: wine Branch: master Commit: c366fa43407dcb2fdf1cad9a6c22aee90d62568d URL: https://gitlab.winehq.org/wine/wine/-/commit/c366fa43407dcb2fdf1cad9a6c22ae… Author: Alex Henrie <alexhenrie24(a)gmail.com> Date: Sun Nov 20 18:44:24 2022 -0700 msxml3: Use standard C functions for memory allocation in saxreader.c. --- dlls/msxml3/msxml_private.h | 7 ------- dlls/msxml3/saxreader.c | 48 +++++++++++++++++++++++---------------------- 2 files changed, 25 insertions(+), 30 deletions(-) diff --git a/dlls/msxml3/msxml_private.h b/dlls/msxml3/msxml_private.h index e41b7c9979d..cd8ae547c73 100644 --- a/dlls/msxml3/msxml_private.h +++ b/dlls/msxml3/msxml_private.h @@ -78,13 +78,6 @@ typedef enum _XDR_DT { extern HINSTANCE MSXML_hInstance DECLSPEC_HIDDEN; -/* memory allocation functions */ - -static inline void* __WINE_ALLOC_SIZE(2) heap_realloc_zero(void *mem, size_t size) -{ - return HeapReAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, mem, size); -} - /* XSLProcessor parameter list */ struct xslprocessor_par { diff --git a/dlls/msxml3/saxreader.c b/dlls/msxml3/saxreader.c index 8d58ed1d94b..0e886a2d051 100644 --- a/dlls/msxml3/saxreader.c +++ b/dlls/msxml3/saxreader.c @@ -466,13 +466,13 @@ static element_entry* alloc_element_entry(const xmlChar *local, const xmlChar *p element_entry *ret; int i; - ret = heap_alloc(sizeof(*ret)); + ret = malloc(sizeof(*ret)); if (!ret) return ret; ret->local = bstr_from_xmlChar(local); ret->prefix = bstr_from_xmlChar(prefix); ret->qname = build_qname(ret->prefix, ret->local); - ret->ns = nb_ns ? heap_alloc(nb_ns*sizeof(ns)) : NULL; + ret->ns = nb_ns ? malloc(nb_ns * sizeof(ns)) : NULL; ret->ns_count = nb_ns; for (i=0; i < nb_ns; i++) @@ -498,8 +498,8 @@ static void free_element_entry(element_entry *element) SysFreeString(element->local); SysFreeString(element->qname); - heap_free(element->ns); - heap_free(element); + free(element->ns); + free(element); } static void push_element_ns(saxlocator *locator, element_entry *element) @@ -567,7 +567,7 @@ static BOOL bstr_pool_insert(struct bstrpool *pool, BSTR pool_entry) { if (!pool->pool) { - pool->pool = heap_alloc(16 * sizeof(*pool->pool)); + pool->pool = malloc(16 * sizeof(*pool->pool)); if (!pool->pool) return FALSE; @@ -576,12 +576,12 @@ static BOOL bstr_pool_insert(struct bstrpool *pool, BSTR pool_entry) } else if (pool->index == pool->len) { - BSTR *realloc = heap_realloc(pool->pool, pool->len * 2 * sizeof(*realloc)); + BSTR *new_pool = realloc(pool->pool, pool->len * 2 * sizeof(*new_pool)); - if (!realloc) + if (!new_pool) return FALSE; - pool->pool = realloc; + pool->pool = new_pool; pool->len *= 2; } @@ -596,7 +596,7 @@ static void free_bstr_pool(struct bstrpool *pool) for (i = 0; i < pool->index; i++) SysFreeString(pool->pool[i]); - heap_free(pool->pool); + free(pool->pool); pool->pool = NULL; pool->index = pool->len = 0; @@ -1399,7 +1399,7 @@ static BSTR saxreader_get_unescaped_value(const xmlChar *buf, int len) str_len = MultiByteToWideChar(CP_UTF8, 0, (LPCSTR)buf, len, NULL, 0); if (len != -1) str_len++; - str = heap_alloc(str_len*sizeof(WCHAR)); + str = malloc(str_len * sizeof(WCHAR)); if (!str) return NULL; MultiByteToWideChar(CP_UTF8, 0, (LPCSTR)buf, len, str, str_len); @@ -1421,7 +1421,7 @@ static BSTR saxreader_get_unescaped_value(const xmlChar *buf, int len) } bstr = SysAllocString(str); - heap_free(str); + free(str); return bstr; } @@ -1461,13 +1461,15 @@ static HRESULT SAXAttributes_populate(saxlocator *locator, if(locator->attr_count > locator->attr_alloc_count) { int new_size = locator->attr_count * 2; - attrs = heap_realloc_zero(locator->attributes, new_size * sizeof(struct _attributes)); + attrs = realloc(locator->attributes, new_size * sizeof(*locator->attributes)); if(!attrs) { free_attribute_values(locator); locator->attr_count = 0; return E_OUTOFMEMORY; } + memset(attrs + locator->attr_alloc_count, 0, + (new_size - locator->attr_alloc_count) * sizeof(*locator->attributes)); locator->attributes = attrs; locator->attr_alloc_count = new_size; } @@ -1930,7 +1932,7 @@ static void WINAPIV libxmlFatalError(void *ctx, const char *msg, ...) va_end(args); len = MultiByteToWideChar(CP_UNIXCP, 0, message, -1, NULL, 0); - error = heap_alloc(sizeof(WCHAR)*len); + error = malloc(sizeof(WCHAR) * len); if(error) { MultiByteToWideChar(CP_UNIXCP, 0, message, -1, error, len); @@ -1941,7 +1943,7 @@ static void WINAPIV libxmlFatalError(void *ctx, const char *msg, ...) { xmlStopParser(This->pParserCtxt); This->ret = E_FAIL; - heap_free(error); + free(error); return; } @@ -1957,7 +1959,7 @@ static void WINAPIV libxmlFatalError(void *ctx, const char *msg, ...) else ISAXErrorHandler_fatalError(handler->handler, &This->ISAXLocator_iface, error, E_FAIL); - heap_free(error); + free(error); xmlStopParser(This->pParserCtxt); This->ret = E_FAIL; @@ -2335,7 +2337,7 @@ static ULONG WINAPI isaxlocator_Release( SysFreeString(This->attributes[index].szValue); SysFreeString(This->attributes[index].szQName); } - heap_free(This->attributes); + free(This->attributes); /* element stack */ LIST_FOR_EACH_ENTRY_SAFE(element, element2, &This->elements, element_entry, entry) @@ -2345,7 +2347,7 @@ static ULONG WINAPI isaxlocator_Release( } ISAXXMLReader_Release(&This->saxreader->ISAXXMLReader_iface); - heap_free( This ); + free(This); } return ref; @@ -2434,7 +2436,7 @@ static HRESULT SAXLocator_create(saxreader *reader, saxlocator **ppsaxlocator, B saxlocator *locator; - locator = heap_alloc( sizeof (*locator) ); + locator = malloc(sizeof(*locator)); if( !locator ) return E_OUTOFMEMORY; @@ -2461,18 +2463,18 @@ static HRESULT SAXLocator_create(saxreader *reader, saxlocator **ppsaxlocator, B if(!locator->namespaceUri) { ISAXXMLReader_Release(&reader->ISAXXMLReader_iface); - heap_free(locator); + free(locator); return E_OUTOFMEMORY; } locator->attr_alloc_count = 8; locator->attr_count = 0; - locator->attributes = heap_alloc_zero(sizeof(struct _attributes)*locator->attr_alloc_count); + locator->attributes = calloc(locator->attr_alloc_count, sizeof(*locator->attributes)); if(!locator->attributes) { ISAXXMLReader_Release(&reader->ISAXXMLReader_iface); SysFreeString(locator->namespaceUri); - heap_free(locator); + free(locator); return E_OUTOFMEMORY; } @@ -2948,7 +2950,7 @@ static ULONG WINAPI saxxmlreader_Release( SysFreeString(This->xmldecl_version); free_bstr_pool(&This->pool); - heap_free( This ); + free(This); } return ref; @@ -3437,7 +3439,7 @@ HRESULT SAXXMLReader_create(MSXML_VERSION version, LPVOID *ppObj) TRACE("(%p)\n", ppObj); - reader = heap_alloc( sizeof (*reader) ); + reader = malloc(sizeof(*reader)); if( !reader ) return E_OUTOFMEMORY;

1 0