July 2020 - Wine-Commits - list.winehq.org

Alexandre Julliard : ntdll: Silence a noisy error.
by Alexandre Julliard July 10, 2020

July 10, 2020

Module: wine Branch: master Commit: a2e77268f2007f2819c2e3e8bd736a056f309a4c URL: https://source.winehq.org/git/wine.git/?a=commit;h=a2e77268f2007f2819c2e3e8… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Jul 10 07:55:28 2020 +0200 ntdll: Silence a noisy error. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntdll/unix/thread.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/ntdll/unix/thread.c b/dlls/ntdll/unix/thread.c index 7cdfc911db..78da6ae638 100644 --- a/dlls/ntdll/unix/thread.c +++ b/dlls/ntdll/unix/thread.c @@ -806,7 +806,7 @@ BOOL get_thread_times(int unix_pid, int unix_tid, LARGE_INTEGER *kernel_time, LA sprintf( buf, "/proc/%u/task/%u/stat", unix_pid, unix_tid ); if (!(f = fopen( buf, "r" ))) { - ERR("Failed to open %s: %s\n", buf, strerror(errno)); + WARN("Failed to open %s: %s\n", buf, strerror(errno)); return FALSE; }

1 0

Alexandre Julliard : ntdll: Add a helper function to retrieve the NT headers of the main exe.
by Alexandre Julliard July 10, 2020

July 10, 2020

Module: wine Branch: master Commit: eccd21e38735fb08e31a4be1a8897ebe248747b4 URL: https://source.winehq.org/git/wine.git/?a=commit;h=eccd21e38735fb08e31a4be1… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Jul 10 07:56:18 2020 +0200 ntdll: Add a helper function to retrieve the NT headers of the main exe. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntdll/unix/server.c | 2 +- dlls/ntdll/unix/unix_private.h | 6 ++++++ dlls/ntdll/unix/virtual.c | 4 ++-- 3 files changed, 9 insertions(+), 3 deletions(-) diff --git a/dlls/ntdll/unix/server.c b/dlls/ntdll/unix/server.c index 813c926cf4..4e65b0eab2 100644 --- a/dlls/ntdll/unix/server.c +++ b/dlls/ntdll/unix/server.c @@ -1454,7 +1454,7 @@ void server_init_process(void) void CDECL server_init_process_done( void *relay ) { PEB *peb = NtCurrentTeb()->Peb; - IMAGE_NT_HEADERS *nt = RtlImageNtHeader( peb->ImageBaseAddress ); + IMAGE_NT_HEADERS *nt = get_exe_nt_header(); void *entry = (char *)peb->ImageBaseAddress + nt->OptionalHeader.AddressOfEntryPoint; NTSTATUS status; int suspend; diff --git a/dlls/ntdll/unix/unix_private.h b/dlls/ntdll/unix/unix_private.h index 6090190ee0..2a73f17e10 100644 --- a/dlls/ntdll/unix/unix_private.h +++ b/dlls/ntdll/unix/unix_private.h @@ -264,6 +264,12 @@ static inline void ascii_to_unicode( WCHAR *dst, const char *src, size_t len ) while (len--) *dst++ = (unsigned char)*src++; } +static inline IMAGE_NT_HEADERS *get_exe_nt_header(void) +{ + IMAGE_DOS_HEADER *module = (IMAGE_DOS_HEADER *)NtCurrentTeb()->Peb->ImageBaseAddress; + return (IMAGE_NT_HEADERS *)((char *)module + module->e_lfanew); +} + static inline size_t ntdll_wcslen( const WCHAR *str ) { const WCHAR *s = str; diff --git a/dlls/ntdll/unix/virtual.c b/dlls/ntdll/unix/virtual.c index 868dd57055..7592234ccc 100644 --- a/dlls/ntdll/unix/virtual.c +++ b/dlls/ntdll/unix/virtual.c @@ -2728,7 +2728,7 @@ NTSTATUS CDECL virtual_alloc_thread_stack( INITIAL_TEB *stack, SIZE_T reserve_si if (!reserve_size || !commit_size) { - IMAGE_NT_HEADERS *nt = RtlImageNtHeader( NtCurrentTeb()->Peb->ImageBaseAddress ); + IMAGE_NT_HEADERS *nt = get_exe_nt_header(); if (!reserve_size) reserve_size = nt->OptionalHeader.SizeOfStackReserve; if (!commit_size) commit_size = nt->OptionalHeader.SizeOfStackCommit; } @@ -3325,7 +3325,7 @@ void CDECL virtual_release_address_space(void) */ void CDECL virtual_set_large_address_space(void) { - IMAGE_NT_HEADERS *nt = RtlImageNtHeader( NtCurrentTeb()->Peb->ImageBaseAddress ); + IMAGE_NT_HEADERS *nt = get_exe_nt_header(); if (!(nt->FileHeader.Characteristics & IMAGE_FILE_LARGE_ADDRESS_AWARE)) return; /* no large address space on win9x */

1 0

Alexandre Julliard : ntdll: Avoid using LdrQueryProcessModuleInformation() in the Unix library.
by Alexandre Julliard July 10, 2020

July 10, 2020

Module: wine Branch: master Commit: afb16545671e7420c7a8bbb4ef97333c33001564 URL: https://source.winehq.org/git/wine.git/?a=commit;h=afb16545671e7420c7a8bbb4… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Jul 10 07:58:35 2020 +0200 ntdll: Avoid using LdrQueryProcessModuleInformation() in the Unix library. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntdll/unix/system.c | 35 +++++++++++++++++++++++++++++++++-- 1 file changed, 33 insertions(+), 2 deletions(-) diff --git a/dlls/ntdll/unix/system.c b/dlls/ntdll/unix/system.c index 07a3ad2154..c3dca7ffe2 100644 --- a/dlls/ntdll/unix/system.c +++ b/dlls/ntdll/unix/system.c @@ -2292,10 +2292,41 @@ NTSTATUS WINAPI NtQuerySystemInformation( SYSTEM_INFORMATION_CLASS class, } case SystemModuleInformation: - /* FIXME: should be system-wide */ + { + /* FIXME: return some fake info for now */ + static const char *fake_modules[] = + { + "\\SystemRoot\\system32\\ntoskrnl.exe", + "\\SystemRoot\\system32\\hal.dll", + "\\SystemRoot\\system32\\drivers\\mountmgr.sys" + }; + if (!info) ret = STATUS_ACCESS_VIOLATION; - else ret = LdrQueryProcessModuleInformation( info, size, &len ); + else + { + ULONG i; + SYSTEM_MODULE_INFORMATION *smi = info; + + len = offsetof( SYSTEM_MODULE_INFORMATION, Modules[ARRAY_SIZE(fake_modules)] ); + if (len <= size) + { + memset( smi, 0, len ); + for (i = 0; i < ARRAY_SIZE(fake_modules); i++) + { + SYSTEM_MODULE *sm = &smi->Modules[i]; + sm->ImageBaseAddress = (char *)0x10000000 + 0x200000 * i; + sm->ImageSize = 0x200000; + sm->LoadOrderIndex = i; + sm->LoadCount = 1; + strcpy( (char *)sm->Name, fake_modules[i] ); + sm->NameOffset = strrchr( fake_modules[i], '\\' ) - fake_modules[i] + 1; + } + smi->ModulesCount = i; + } + else ret = STATUS_INFO_LENGTH_MISMATCH; + } break; + } case SystemHandleInformation: {

1 0

Alexandre Julliard : ntdll: Move the FSCTL_PIPE_IMPERSONATE semi-stub to the server.
by Alexandre Julliard July 10, 2020

July 10, 2020

Module: wine Branch: master Commit: 9ac3f24f744bf76c252fafb2c2e254bbbf38ed1e URL: https://source.winehq.org/git/wine.git/?a=commit;h=9ac3f24f744bf76c252fafb2… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Jul 10 07:57:07 2020 +0200 ntdll: Move the FSCTL_PIPE_IMPERSONATE semi-stub to the server. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntdll/unix/file.c | 4 ++-- server/named_pipe.c | 11 +++++++++++ 2 files changed, 13 insertions(+), 2 deletions(-) diff --git a/dlls/ntdll/unix/file.c b/dlls/ntdll/unix/file.c index b4c9b75b3e..070aac3083 100644 --- a/dlls/ntdll/unix/file.c +++ b/dlls/ntdll/unix/file.c @@ -5665,8 +5665,8 @@ NTSTATUS WINAPI NtFsControlFile( HANDLE handle, HANDLE event, PIO_APC_ROUTINE ap case FSCTL_PIPE_IMPERSONATE: FIXME("FSCTL_PIPE_IMPERSONATE: impersonating self\n"); - status = RtlImpersonateSelf( SecurityImpersonation ); - break; + return server_ioctl_file( handle, event, apc, apc_context, io, code, + in_buffer, in_size, out_buffer, out_size ); case FSCTL_IS_VOLUME_MOUNTED: case FSCTL_LOCK_VOLUME: diff --git a/server/named_pipe.c b/server/named_pipe.c index f97b141641..b259abb8de 100644 --- a/server/named_pipe.c +++ b/server/named_pipe.c @@ -1138,6 +1138,17 @@ static int pipe_server_ioctl( struct fd *fd, ioctl_code_t code, struct async *as pipe_end_disconnect( &server->pipe_end, STATUS_PIPE_DISCONNECTED ); return 1; + case FSCTL_PIPE_IMPERSONATE: + if (current->process->token) /* FIXME: use the client token */ + { + struct token *token; + if (!(token = token_duplicate( current->process->token, 0, SecurityImpersonation, NULL ))) + return 0; + if (current->token) release_object( current->token ); + current->token = token; + } + return 1; + default: return pipe_end_ioctl( &server->pipe_end, code, async ); }

1 0

Alexandre Julliard : ntdll: Avoid using RtlDosPathNameToNtPathName_U() in the Unix library.
by Alexandre Julliard July 10, 2020

July 10, 2020

Module: wine Branch: master Commit: bcc3410732c60e32eaf3abd808be7482218f57d7 URL: https://source.winehq.org/git/wine.git/?a=commit;h=bcc3410732c60e32eaf3abd8… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Jul 10 07:58:14 2020 +0200 ntdll: Avoid using RtlDosPathNameToNtPathName_U() in the Unix library. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntdll/unix/process.c | 24 +++++++++++++++++++++--- 1 file changed, 21 insertions(+), 3 deletions(-) diff --git a/dlls/ntdll/unix/process.c b/dlls/ntdll/unix/process.c index 8a4da2cd55..67450fdb3d 100644 --- a/dlls/ntdll/unix/process.c +++ b/dlls/ntdll/unix/process.c @@ -476,6 +476,10 @@ static ULONG get_env_size( const RTL_USER_PROCESS_PARAMETERS *params, char **win */ static int get_unix_curdir( const RTL_USER_PROCESS_PARAMETERS *params ) { + static const WCHAR ntprefixW[] = {'\\','?','?','\\',0}; + static const WCHAR uncprefixW[] = {'U','N','C','\\',0}; + const UNICODE_STRING *curdir = &params->CurrentDirectory.DosPath; + const WCHAR *dir = curdir->Buffer; UNICODE_STRING nt_name; OBJECT_ATTRIBUTES attr; IO_STATUS_BLOCK io; @@ -483,13 +487,27 @@ static int get_unix_curdir( const RTL_USER_PROCESS_PARAMETERS *params ) HANDLE handle; int fd = -1; - if (!RtlDosPathNameToNtPathName_U( params->CurrentDirectory.DosPath.Buffer, &nt_name, NULL, NULL )) - return -1; + if (!(nt_name.Buffer = malloc( curdir->Length + 8 * sizeof(WCHAR) ))) return -1; + + /* simplified version of RtlDosPathNameToNtPathName_U */ + wcscpy( nt_name.Buffer, ntprefixW ); + if (dir[0] == '\\' && dir[1] == '\\') + { + if ((dir[2] == '.' || dir[2] == '?') && dir[3] == '\\') dir += 4; + else + { + wcscat( nt_name.Buffer, uncprefixW ); + dir += 2; + } + } + wcscat( nt_name.Buffer, dir ); + nt_name.Length = wcslen( nt_name.Buffer ) * sizeof(WCHAR); + InitializeObjectAttributes( &attr, &nt_name, OBJ_CASE_INSENSITIVE, 0, NULL ); status = NtOpenFile( &handle, FILE_TRAVERSE | SYNCHRONIZE, &attr, &io, FILE_SHARE_READ | FILE_SHARE_WRITE, FILE_DIRECTORY_FILE | FILE_SYNCHRONOUS_IO_NONALERT ); - RtlFreeUnicodeString( &nt_name ); + free( nt_name.Buffer ); if (status) return -1; server_handle_to_fd( handle, FILE_TRAVERSE, &fd, NULL ); NtClose( handle );

1 0

Alexandre Julliard : ntdll: Avoid using RtlInitUnicodeString() in the Unix library.
by Alexandre Julliard July 10, 2020

July 10, 2020

Module: wine Branch: master Commit: 0357d2ca75be92e927bb785a786d3766c94f3c83 URL: https://source.winehq.org/git/wine.git/?a=commit;h=0357d2ca75be92e927bb785a… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Jul 10 07:57:59 2020 +0200 ntdll: Avoid using RtlInitUnicodeString() in the Unix library. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntdll/unix/file.c | 3 ++- dlls/ntdll/unix/system.c | 16 +++++++++------- dlls/ntdll/unix/virtual.c | 10 ++++------ 3 files changed, 15 insertions(+), 14 deletions(-) diff --git a/dlls/ntdll/unix/file.c b/dlls/ntdll/unix/file.c index 070aac3083..af7c214984 100644 --- a/dlls/ntdll/unix/file.c +++ b/dlls/ntdll/unix/file.c @@ -1993,7 +1993,8 @@ static NTSTATUS get_mountmgr_fs_info( HANDLE handle, int fd, struct mountmgr_uni else drive->letter = 'a' + letter; - RtlInitUnicodeString( &string, MOUNTMGR_DEVICE_NAME ); + string.Buffer = (WCHAR *)MOUNTMGR_DEVICE_NAME; + string.Length = sizeof(MOUNTMGR_DEVICE_NAME) - sizeof(WCHAR); InitializeObjectAttributes( &attr, &string, 0, NULL, NULL ); status = NtOpenFile( &mountmgr, GENERIC_READ | SYNCHRONIZE, &attr, &io, FILE_SHARE_READ | FILE_SHARE_WRITE, FILE_SYNCHRONOUS_IO_NONALERT ); diff --git a/dlls/ntdll/unix/system.c b/dlls/ntdll/unix/system.c index 33fb37ec33..07a3ad2154 100644 --- a/dlls/ntdll/unix/system.c +++ b/dlls/ntdll/unix/system.c @@ -1729,7 +1729,8 @@ static BOOL reg_query_value( HKEY key, LPCWSTR name, DWORD type, void *data, DWO if (count > sizeof(buf) - sizeof(KEY_VALUE_PARTIAL_INFORMATION)) return FALSE; - RtlInitUnicodeString( &nameW, name ); + nameW.Buffer = (WCHAR *)name; + nameW.Length = wcslen( name ) * sizeof(WCHAR); if (NtQueryValueKey( key, &nameW, KeyValuePartialInformation, buf, sizeof(buf), &count )) return FALSE; @@ -1756,7 +1757,7 @@ static void find_reg_tz_info(RTL_DYNAMIC_TIME_ZONE_INFORMATION *tzi, const char* HANDLE key, subkey, subkey_dyn = 0; ULONG idx, len; OBJECT_ATTRIBUTES attr; - UNICODE_STRING nameW, nameDynamicW; + UNICODE_STRING nameW; WCHAR yearW[16]; char buffer[128]; KEY_BASIC_INFORMATION *info = (KEY_BASIC_INFORMATION *)buffer; @@ -1764,9 +1765,8 @@ static void find_reg_tz_info(RTL_DYNAMIC_TIME_ZONE_INFORMATION *tzi, const char* sprintf( buffer, "%u", year ); ascii_to_unicode( yearW, buffer, strlen(buffer) + 1 ); - RtlInitUnicodeString( &nameW, Time_ZonesW ); - RtlInitUnicodeString( &nameDynamicW, Dynamic_DstW ); - + nameW.Buffer = (WCHAR *)Time_ZonesW; + nameW.Length = sizeof(Time_ZonesW) - sizeof(WCHAR); InitializeObjectAttributes( &attr, &nameW, 0, 0, NULL ); if (NtOpenKey( &key, KEY_READ, &attr )) return; @@ -1785,7 +1785,7 @@ static void find_reg_tz_info(RTL_DYNAMIC_TIME_ZONE_INFORMATION *tzi, const char* nameW.Buffer = info->Name; nameW.Length = info->NameLength; - InitializeObjectAttributes( &attr, &nameW, 0, key, NULL ); + attr.RootDirectory = key; if (NtOpenKey( &subkey, KEY_READ, &attr )) continue; memset( &reg_tzi, 0, sizeof(reg_tzi) ); @@ -1801,7 +1801,9 @@ static void find_reg_tz_info(RTL_DYNAMIC_TIME_ZONE_INFORMATION *tzi, const char* goto next; /* Check for Dynamic DST entry first */ - InitializeObjectAttributes( &attr, &nameDynamicW, 0, subkey, NULL ); + nameW.Buffer = (WCHAR *)Dynamic_DstW; + nameW.Length = sizeof(Dynamic_DstW) - sizeof(WCHAR); + attr.RootDirectory = subkey; if (!NtOpenKey( &subkey_dyn, KEY_READ, &attr )) { is_dynamic = reg_query_value( subkey_dyn, yearW, REG_BINARY, &tz_data, sizeof(tz_data) ); diff --git a/dlls/ntdll/unix/virtual.c b/dlls/ntdll/unix/virtual.c index 7592234ccc..4e5b1acf0b 100644 --- a/dlls/ntdll/unix/virtual.c +++ b/dlls/ntdll/unix/virtual.c @@ -2804,16 +2804,14 @@ void virtual_clear_thread_stack( void *stack_end ) */ void virtual_map_user_shared_data(void) { - static const WCHAR wine_usdW[] = {'\\','K','e','r','n','e','l','O','b','j','e','c','t','s', - '\\','_','_','w','i','n','e','_','u','s','e','r','_','s','h','a','r','e','d','_','d','a','t','a',0}; - OBJECT_ATTRIBUTES attr = {sizeof(attr)}; - UNICODE_STRING wine_usd_str; + static const WCHAR nameW[] = {'\\','K','e','r','n','e','l','O','b','j','e','c','t','s', + '\\','_','_','w','i','n','e','_','u','s','e','r','_','s','h','a','r','e','d','_','d','a','t','a',0}; + UNICODE_STRING name_str = { sizeof(nameW) - sizeof(WCHAR), sizeof(nameW), (WCHAR *)nameW }; + OBJECT_ATTRIBUTES attr = { sizeof(attr), 0, &name_str }; NTSTATUS status; HANDLE section; int res, fd, needs_close; - RtlInitUnicodeString( &wine_usd_str, wine_usdW ); - InitializeObjectAttributes( &attr, &wine_usd_str, OBJ_OPENIF, NULL, NULL ); if ((status = NtOpenSection( &section, SECTION_ALL_ACCESS, &attr ))) { ERR( "failed to open the USD section: %08x\n", status );

1 0

Alexandre Julliard : ntdll: Add a local helper function to check DOS 8.3 names.
by Alexandre Julliard July 10, 2020

July 10, 2020

Module: wine Branch: master Commit: 4a49af0cbeb1bb5570d92f679c98ad05abc693e6 URL: https://source.winehq.org/git/wine.git/?a=commit;h=4a49af0cbeb1bb5570d92f67… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Jul 10 07:56:39 2020 +0200 ntdll: Add a local helper function to check DOS 8.3 names. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntdll/unix/file.c | 82 +++++++++++++++++++++++++++++--------------------- 1 file changed, 48 insertions(+), 34 deletions(-) diff --git a/dlls/ntdll/unix/file.c b/dlls/ntdll/unix/file.c index e985c75a41..b4c9b75b3e 100644 --- a/dlls/ntdll/unix/file.c +++ b/dlls/ntdll/unix/file.c @@ -1263,11 +1263,11 @@ static BOOL is_hidden_file( const UNICODE_STRING *name ) * 'buffer' must be at least 12 characters long. * Returns length of short name in bytes; short name is NOT null-terminated. */ -static ULONG hash_short_file_name( const UNICODE_STRING *name, LPWSTR buffer ) +static ULONG hash_short_file_name( const WCHAR *name, int length, LPWSTR buffer ) { static const char hash_chars[32] = "ABCDEFGHIJKLMNOPQRSTUVWXYZ012345"; - LPCWSTR p, ext, end = name->Buffer + name->Length / sizeof(WCHAR); + LPCWSTR p, ext, end = name + length; LPWSTR dst; unsigned short hash; int i; @@ -1277,22 +1277,22 @@ static ULONG hash_short_file_name( const UNICODE_STRING *name, LPWSTR buffer ) /* insert a better algorithm here... */ if (!is_case_sensitive) { - for (p = name->Buffer, hash = 0xbeef; p < end - 1; p++) + for (p = name, hash = 0xbeef; p < end - 1; p++) hash = (hash<<3) ^ (hash>>5) ^ towlower(*p) ^ (towlower(p[1]) << 8); hash = (hash<<3) ^ (hash>>5) ^ towlower(*p); /* Last character */ } else { - for (p = name->Buffer, hash = 0xbeef; p < end - 1; p++) + for (p = name, hash = 0xbeef; p < end - 1; p++) hash = (hash << 3) ^ (hash >> 5) ^ *p ^ (p[1] << 8); hash = (hash << 3) ^ (hash >> 5) ^ *p; /* Last character */ } /* Find last dot for start of the extension */ - for (p = name->Buffer + 1, ext = NULL; p < end - 1; p++) if (*p == '.') ext = p; + for (p = name + 1, ext = NULL; p < end - 1; p++) if (*p == '.') ext = p; /* Copy first 4 chars, replacing invalid chars with '_' */ - for (i = 4, p = name->Buffer, dst = buffer; i > 0; i--, p++) + for (i = 4, p = name, dst = buffer; i > 0; i--, p++) { if (p == end || p == ext) break; *dst++ = is_invalid_dos_char(*p) ? '_' : *p; @@ -1332,12 +1332,11 @@ static ULONG hash_short_file_name( const UNICODE_STRING *name, LPWSTR buffer ) * *test1.txt* test1.txt * * h?l?o*t.dat hellothisisatest.dat * */ -static BOOLEAN match_filename( const UNICODE_STRING *name_str, const UNICODE_STRING *mask_str ) +static BOOLEAN match_filename( const WCHAR *name, int length, const UNICODE_STRING *mask_str ) { BOOL mismatch; - const WCHAR *name = name_str->Buffer; const WCHAR *mask = mask_str->Buffer; - const WCHAR *name_end = name + name_str->Length / sizeof(WCHAR); + const WCHAR *name_end = name + length; const WCHAR *mask_end = mask + mask_str->Length / sizeof(WCHAR); const WCHAR *lastjoker = NULL; const WCHAR *next_to_retry = NULL; @@ -1399,6 +1398,38 @@ static BOOLEAN match_filename( const UNICODE_STRING *name_str, const UNICODE_STR } +/*********************************************************************** + * is_legal_8dot3_name + * + * Simplified version of RtlIsNameLegalDOS8Dot3. + */ +static BOOLEAN is_legal_8dot3_name( const WCHAR *name, int len ) +{ + static const WCHAR invalid_chars[] = { INVALID_DOS_CHARS,':','/','\\',0 }; + int i, dot = -1; + + if (len > 12) return FALSE; + + /* a starting . is invalid, except for . and .. */ + if (len > 0 && name[0] == '.') return (len == 1 || (len == 2 && name[1] == '.')); + + for (i = 0; i < len; i++) + { + if (name[i] > 0x7f) return FALSE; + if (wcschr( invalid_chars, name[i] )) return FALSE; + if (name[i] == '.') + { + if (dot != -1) return FALSE; + dot = i; + } + } + + if (dot == -1) return (len <= 8); + if (dot > 8) return FALSE; + return (len - dot > 1 && len - dot < 5); +} + + /*********************************************************************** * append_entry * @@ -1410,16 +1441,11 @@ static BOOL append_entry( struct dir_data *data, const char *long_name, int long_len, short_len; WCHAR long_nameW[MAX_DIR_ENTRY_LEN + 1]; WCHAR short_nameW[13]; - UNICODE_STRING str; long_len = ntdll_umbstowcs( long_name, strlen(long_name), long_nameW, ARRAY_SIZE(long_nameW) ); if (long_len == ARRAY_SIZE(long_nameW)) return TRUE; long_nameW[long_len] = 0; - str.Buffer = long_nameW; - str.Length = long_len * sizeof(WCHAR); - str.MaximumLength = sizeof(long_nameW); - if (short_name) { short_len = ntdll_umbstowcs( short_name, strlen(short_name), @@ -1427,11 +1453,9 @@ static BOOL append_entry( struct dir_data *data, const char *long_name, } else /* generate a short name if necessary */ { - BOOLEAN spaces; - short_len = 0; - if (!RtlIsNameLegalDOS8Dot3( &str, NULL, &spaces ) || spaces) - short_len = hash_short_file_name( &str, short_nameW ); + if (!is_legal_8dot3_name( long_nameW, long_len )) + short_len = hash_short_file_name( long_nameW, long_len, short_nameW ); } short_nameW[short_len] = 0; wcsupr( short_nameW ); @@ -1439,13 +1463,10 @@ static BOOL append_entry( struct dir_data *data, const char *long_name, TRACE( "long %s short %s mask %s\n", debugstr_w( long_nameW ), debugstr_w( short_nameW ), debugstr_us( mask )); - if (mask && !match_filename( &str, mask )) + if (mask && !match_filename( long_nameW, long_len, mask )) { if (!short_len) return TRUE; /* no short name to match */ - str.Buffer = short_nameW; - str.Length = short_len * sizeof(WCHAR); - str.MaximumLength = sizeof(short_nameW); - if (!match_filename( &str, mask )) return TRUE; + if (!match_filename( short_nameW, short_len, mask )) return TRUE; } return add_dir_data_names( data, long_nameW, short_nameW, long_name ); @@ -2496,8 +2517,7 @@ static NTSTATUS find_file_in_dir( char *unix_name, int pos, const WCHAR *name, i BOOLEAN check_case, BOOLEAN *is_win_dir ) { WCHAR buffer[MAX_DIR_ENTRY_LEN]; - UNICODE_STRING str; - BOOLEAN spaces, is_name_8_dot_3; + BOOLEAN is_name_8_dot_3; DIR *dir; struct dirent *de; struct stat st; @@ -2523,10 +2543,7 @@ static NTSTATUS find_file_in_dir( char *unix_name, int pos, const WCHAR *name, i /* check if it fits in 8.3 so that we don't look for short names if we won't need them */ - str.Buffer = (WCHAR *)name; - str.Length = length * sizeof(WCHAR); - str.MaximumLength = str.Length; - is_name_8_dot_3 = RtlIsNameLegalDOS8Dot3( &str, NULL, &spaces ) && !spaces; + is_name_8_dot_3 = is_legal_8dot3_name( name, length ); #ifndef VFAT_IOCTL_READDIR_BOTH is_name_8_dot_3 = is_name_8_dot_3 && length >= 8 && name[4] == '~'; #endif @@ -2587,8 +2604,6 @@ static NTSTATUS find_file_in_dir( char *unix_name, int pos, const WCHAR *name, i else return STATUS_ACCESS_DENIED; } unix_name[pos - 1] = '/'; - str.Buffer = buffer; - str.MaximumLength = sizeof(buffer); while ((de = readdir( dir ))) { ret = ntdll_umbstowcs( de->d_name, strlen(de->d_name), buffer, MAX_DIR_ENTRY_LEN ); @@ -2601,11 +2616,10 @@ static NTSTATUS find_file_in_dir( char *unix_name, int pos, const WCHAR *name, i if (!is_name_8_dot_3) continue; - str.Length = ret * sizeof(WCHAR); - if (!RtlIsNameLegalDOS8Dot3( &str, NULL, &spaces ) || spaces) + if (!is_legal_8dot3_name( buffer, ret )) { WCHAR short_nameW[12]; - ret = hash_short_file_name( &str, short_nameW ); + ret = hash_short_file_name( buffer, ret, short_nameW ); if (ret == length && !wcsnicmp( short_nameW, name, length )) { strcpy( unix_name + pos, de->d_name );

1 0

Alexandre Julliard : ntdll: Add a helper function to open a file from its Unix path.
by Alexandre Julliard July 10, 2020

July 10, 2020

Module: wine Branch: master Commit: 9ec262ebcc7f14d7373841d4ca082b855ed8090f URL: https://source.winehq.org/git/wine.git/?a=commit;h=9ec262ebcc7f14d7373841d4… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Jul 10 07:56:02 2020 +0200 ntdll: Add a helper function to open a file from its Unix path. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntdll/unix/file.c | 63 +++++++++++++++++++++++++----------------- dlls/ntdll/unix/loader.c | 34 ++++++++--------------- dlls/ntdll/unix/unix_private.h | 3 ++ 3 files changed, 53 insertions(+), 47 deletions(-) diff --git a/dlls/ntdll/unix/file.c b/dlls/ntdll/unix/file.c index a0ad361fcb..e985c75a41 100644 --- a/dlls/ntdll/unix/file.c +++ b/dlls/ntdll/unix/file.c @@ -3528,6 +3528,42 @@ void CDECL set_show_dot_files( BOOL enable ) } +/****************************************************************************** + * open_unix_file + * + * Helper for NtCreateFile that takes a Unix path. + */ +NTSTATUS open_unix_file( HANDLE *handle, const char *unix_name, ACCESS_MASK access, + OBJECT_ATTRIBUTES *attr, ULONG attributes, ULONG sharing, ULONG disposition, + ULONG options, void *ea_buffer, ULONG ea_length ) +{ + static UNICODE_STRING empty_string; + struct object_attributes *objattr; + OBJECT_ATTRIBUTES unix_attr = *attr; + NTSTATUS status; + data_size_t len; + + unix_attr.ObjectName = &empty_string; /* we send the unix name instead */ + if ((status = alloc_object_attributes( &unix_attr, &objattr, &len ))) return status; + + SERVER_START_REQ( create_file ) + { + req->access = access; + req->sharing = sharing; + req->create = disposition; + req->options = options; + req->attrs = attributes; + wine_server_add_data( req, objattr, len ); + wine_server_add_data( req, unix_name, strlen(unix_name) ); + status = wine_server_call( req ); + *handle = wine_server_ptr_handle( reply->handle ); + } + SERVER_END_REQ; + free( objattr ); + return status; +} + + /****************************************************************************** * NtCreateFile (NTDLL.@) */ @@ -3580,31 +3616,8 @@ NTSTATUS WINAPI NtCreateFile( HANDLE *handle, ACCESS_MASK access, OBJECT_ATTRIBU if (io->u.Status == STATUS_SUCCESS) { - static UNICODE_STRING empty_string; - OBJECT_ATTRIBUTES unix_attr = *attr; - data_size_t len; - struct object_attributes *objattr; - - unix_attr.ObjectName = &empty_string; /* we send the unix name instead */ - if ((io->u.Status = alloc_object_attributes( &unix_attr, &objattr, &len ))) - { - RtlFreeHeap( GetProcessHeap(), 0, unix_name ); - return io->u.Status; - } - SERVER_START_REQ( create_file ) - { - req->access = access; - req->sharing = sharing; - req->create = disposition; - req->options = options; - req->attrs = attributes; - wine_server_add_data( req, objattr, len ); - wine_server_add_data( req, unix_name, strlen(unix_name) ); - io->u.Status = wine_server_call( req ); - *handle = wine_server_ptr_handle( reply->handle ); - } - SERVER_END_REQ; - free( objattr ); + io->u.Status = open_unix_file( handle, unix_name, access, attr, attributes, + sharing, disposition, options, ea_buffer, ea_length ); RtlFreeHeap( GetProcessHeap(), 0, unix_name ); } else WARN( "%s not found (%x)\n", debugstr_us(attr->ObjectName), io->u.Status ); diff --git a/dlls/ntdll/unix/loader.c b/dlls/ntdll/unix/loader.c index 14fbfae8cd..c90ad5969e 100644 --- a/dlls/ntdll/unix/loader.c +++ b/dlls/ntdll/unix/loader.c @@ -990,31 +990,26 @@ static inline char *prepend( char *buffer, const char *str, size_t len ) * * Open a file for a new dll. Helper for find_dll_file. */ -static NTSTATUS open_dll_file( const WCHAR *name, void **module, pe_image_info_t *image_info ) +static NTSTATUS open_dll_file( const char *name, void **module, pe_image_info_t *image_info ) { struct builtin_module *builtin; - FILE_BASIC_INFORMATION info; - OBJECT_ATTRIBUTES attr; + OBJECT_ATTRIBUTES attr = { sizeof(attr) }; IO_STATUS_BLOCK io; - UNICODE_STRING nt_name; LARGE_INTEGER size; FILE_OBJECTID_BUFFER id; + struct stat st; SIZE_T len = 0; NTSTATUS status; HANDLE handle, mapping; - RtlInitUnicodeString( &nt_name, name ); - InitializeObjectAttributes( &attr, &nt_name, OBJ_CASE_INSENSITIVE, 0, NULL ); - if ((status = NtOpenFile( &handle, GENERIC_READ | SYNCHRONIZE, &attr, &io, - FILE_SHARE_READ | FILE_SHARE_DELETE, - FILE_SYNCHRONOUS_IO_NONALERT | FILE_NON_DIRECTORY_FILE ))) + if ((status = open_unix_file( &handle, name, GENERIC_READ | SYNCHRONIZE, &attr, 0, + FILE_SHARE_READ | FILE_SHARE_DELETE, FILE_OPEN, + FILE_SYNCHRONOUS_IO_NONALERT | FILE_NON_DIRECTORY_FILE, NULL, 0 ))) { - if (status != STATUS_OBJECT_PATH_NOT_FOUND && - status != STATUS_OBJECT_NAME_NOT_FOUND && - !NtQueryAttributesFile( &attr, &info )) + if (status != STATUS_OBJECT_PATH_NOT_FOUND && status != STATUS_OBJECT_NAME_NOT_FOUND) { /* if the file exists but failed to open, report the error */ - return status; + if (!stat( name, &st )) return status; } /* otherwise continue searching */ return STATUS_DLL_NOT_FOUND; @@ -1026,7 +1021,7 @@ static NTSTATUS open_dll_file( const WCHAR *name, void **module, pe_image_info_t { if (!memcmp( &builtin->id, id.ObjectId, sizeof(builtin->id) )) { - TRACE( "%s is the same file as existing module %p\n", debugstr_w(name), + TRACE( "%s is the same file as existing module %p\n", debugstr_a(name), builtin->module ); NtClose( handle ); NtUnmapViewOfSection( NtCurrentProcess(), *module ); @@ -1057,12 +1052,12 @@ static NTSTATUS open_dll_file( const WCHAR *name, void **module, pe_image_info_t /* ignore non-builtins */ if (!(image_info->image_flags & IMAGE_FLAGS_WineBuiltin)) { - WARN( "%s found in WINEDLLPATH but not a builtin, ignoring\n", debugstr_w(name) ); + WARN( "%s found in WINEDLLPATH but not a builtin, ignoring\n", debugstr_a(name) ); status = STATUS_DLL_NOT_FOUND; } else if (image_info->cpu != client_cpu) { - TRACE( "%s is for CPU %u, continuing search\n", debugstr_w(name), image_info->cpu ); + TRACE( "%s is for CPU %u, continuing search\n", debugstr_a(name), image_info->cpu ); status = STATUS_IMAGE_MACHINE_TYPE_MISMATCH; } @@ -1082,15 +1077,10 @@ static NTSTATUS open_dll_file( const WCHAR *name, void **module, pe_image_info_t */ static NTSTATUS open_builtin_file( char *name, void **module, pe_image_info_t *image_info ) { - WCHAR *nt_name = NULL; NTSTATUS status; int fd; - if ((status = unix_to_nt_file_name( name, &nt_name ))) return status; - - status = open_dll_file( nt_name, module, image_info ); - RtlFreeHeap( GetProcessHeap(), 0, nt_name ); - + status = open_dll_file( name, module, image_info ); if (status != STATUS_DLL_NOT_FOUND) return status; /* try .so file */ diff --git a/dlls/ntdll/unix/unix_private.h b/dlls/ntdll/unix/unix_private.h index 711a0bed2b..6090190ee0 100644 --- a/dlls/ntdll/unix/unix_private.h +++ b/dlls/ntdll/unix/unix_private.h @@ -237,6 +237,9 @@ extern NTSTATUS tape_DeviceIoControl( HANDLE device, HANDLE event, PIO_APC_ROUTI extern NTSTATUS errno_to_status( int err ) DECLSPEC_HIDDEN; extern NTSTATUS nt_to_unix_file_name( const UNICODE_STRING *nameW, char **name_ret, UINT disposition ) DECLSPEC_HIDDEN; extern NTSTATUS unix_to_nt_file_name( const char *name, WCHAR **nt ) DECLSPEC_HIDDEN; +extern NTSTATUS open_unix_file( HANDLE *handle, const char *unix_name, ACCESS_MASK access, + OBJECT_ATTRIBUTES *attr, ULONG attributes, ULONG sharing, ULONG disposition, + ULONG options, void *ea_buffer, ULONG ea_length ) DECLSPEC_HIDDEN; extern void init_files(void) DECLSPEC_HIDDEN; extern void init_cpu_info(void) DECLSPEC_HIDDEN;

1 0

Henri Verbeet : vkd3d-shader: Rename the vkd3d_sysval_semantic enumeration to vkd3d_shader_sysval_semantic.
by Alexandre Julliard July 10, 2020

July 10, 2020

Module: vkd3d Branch: master Commit: 3a9eb1839eabaff8a6167d5112b2562fc761870d URL: https://source.winehq.org/git/vkd3d.git/?a=commit;h=3a9eb1839eabaff8a6167d5… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Fri Jul 10 17:02:01 2020 +0430 vkd3d-shader: Rename the vkd3d_sysval_semantic enumeration to vkd3d_shader_sysval_semantic. Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- include/vkd3d_shader.h | 44 ++++++++++++++++++++++---------------------- libs/vkd3d-shader/spirv.c | 32 ++++++++++++++++---------------- 2 files changed, 38 insertions(+), 38 deletions(-) diff --git a/include/vkd3d_shader.h b/include/vkd3d_shader.h index 61785be..054b512 100644 --- a/include/vkd3d_shader.h +++ b/include/vkd3d_shader.h @@ -650,27 +650,27 @@ enum vkd3d_shader_component_type VKD3D_FORCE_32_BIT_ENUM(VKD3D_SHADER_COMPONENT_TYPE), }; -enum vkd3d_sysval_semantic -{ - VKD3D_SV_NONE = 0, - VKD3D_SV_POSITION = 1, - VKD3D_SV_CLIP_DISTANCE = 2, - VKD3D_SV_CULL_DISTANCE = 3, - VKD3D_SV_RENDER_TARGET_ARRAY_INDEX = 4, - VKD3D_SV_VIEWPORT_ARRAY_INDEX = 5, - VKD3D_SV_VERTEX_ID = 6, - VKD3D_SV_PRIMITIVE_ID = 7, - VKD3D_SV_INSTANCE_ID = 8, - VKD3D_SV_IS_FRONT_FACE = 9, - VKD3D_SV_SAMPLE_INDEX = 10, - VKD3D_SV_TESS_FACTOR_QUADEDGE = 11, - VKD3D_SV_TESS_FACTOR_QUADINT = 12, - VKD3D_SV_TESS_FACTOR_TRIEDGE = 13, - VKD3D_SV_TESS_FACTOR_TRIINT = 14, - VKD3D_SV_TESS_FACTOR_LINEDET = 15, - VKD3D_SV_TESS_FACTOR_LINEDEN = 16, - - VKD3D_FORCE_32_BIT_ENUM(VKD3D_SYSVAL_SEMANTIC), +enum vkd3d_shader_sysval_semantic +{ + VKD3D_SHADER_SV_NONE = 0x00, + VKD3D_SHADER_SV_POSITION = 0x01, + VKD3D_SHADER_SV_CLIP_DISTANCE = 0x02, + VKD3D_SHADER_SV_CULL_DISTANCE = 0x03, + VKD3D_SHADER_SV_RENDER_TARGET_ARRAY_INDEX = 0x04, + VKD3D_SHADER_SV_VIEWPORT_ARRAY_INDEX = 0x05, + VKD3D_SHADER_SV_VERTEX_ID = 0x06, + VKD3D_SHADER_SV_PRIMITIVE_ID = 0x07, + VKD3D_SHADER_SV_INSTANCE_ID = 0x08, + VKD3D_SHADER_SV_IS_FRONT_FACE = 0x09, + VKD3D_SHADER_SV_SAMPLE_INDEX = 0x0a, + VKD3D_SHADER_SV_TESS_FACTOR_QUADEDGE = 0x0b, + VKD3D_SHADER_SV_TESS_FACTOR_QUADINT = 0x0c, + VKD3D_SHADER_SV_TESS_FACTOR_TRIEDGE = 0x0d, + VKD3D_SHADER_SV_TESS_FACTOR_TRIINT = 0x0e, + VKD3D_SHADER_SV_TESS_FACTOR_LINEDET = 0x0f, + VKD3D_SHADER_SV_TESS_FACTOR_LINEDEN = 0x10, + + VKD3D_FORCE_32_BIT_ENUM(VKD3D_SHADER_SYSVAL_SEMANTIC), }; enum vkd3d_shader_minimum_precision @@ -687,7 +687,7 @@ struct vkd3d_shader_signature_element const char *semantic_name; unsigned int semantic_index; unsigned int stream_index; - enum vkd3d_sysval_semantic sysval_semantic; + enum vkd3d_shader_sysval_semantic sysval_semantic; enum vkd3d_shader_component_type component_type; unsigned int register_index; unsigned int mask; diff --git a/libs/vkd3d-shader/spirv.c b/libs/vkd3d-shader/spirv.c index 257b443..667d851 100644 --- a/libs/vkd3d-shader/spirv.c +++ b/libs/vkd3d-shader/spirv.c @@ -132,30 +132,30 @@ static const void *vkd3d_find_struct_(const struct vkd3d_struct *chain, return NULL; } -static enum vkd3d_shader_input_sysval_semantic vkd3d_siv_from_sysval_indexed(enum vkd3d_sysval_semantic sysval, +static enum vkd3d_shader_input_sysval_semantic vkd3d_siv_from_sysval_indexed(enum vkd3d_shader_sysval_semantic sysval, unsigned int index) { switch (sysval) { - case VKD3D_SV_NONE: + case VKD3D_SHADER_SV_NONE: return VKD3D_SIV_NONE; - case VKD3D_SV_POSITION: + case VKD3D_SHADER_SV_POSITION: return VKD3D_SIV_POSITION; - case VKD3D_SV_CLIP_DISTANCE: + case VKD3D_SHADER_SV_CLIP_DISTANCE: return VKD3D_SIV_CLIP_DISTANCE; - case VKD3D_SV_CULL_DISTANCE: + case VKD3D_SHADER_SV_CULL_DISTANCE: return VKD3D_SIV_CULL_DISTANCE; - case VKD3D_SV_TESS_FACTOR_QUADEDGE: + case VKD3D_SHADER_SV_TESS_FACTOR_QUADEDGE: return VKD3D_SIV_QUAD_U0_TESS_FACTOR + index; - case VKD3D_SV_TESS_FACTOR_QUADINT: + case VKD3D_SHADER_SV_TESS_FACTOR_QUADINT: return VKD3D_SIV_QUAD_U_INNER_TESS_FACTOR + index; - case VKD3D_SV_TESS_FACTOR_TRIEDGE: + case VKD3D_SHADER_SV_TESS_FACTOR_TRIEDGE: return VKD3D_SIV_TRIANGLE_U_TESS_FACTOR + index; - case VKD3D_SV_TESS_FACTOR_TRIINT: + case VKD3D_SHADER_SV_TESS_FACTOR_TRIINT: return VKD3D_SIV_TRIANGLE_INNER_TESS_FACTOR; - case VKD3D_SV_TESS_FACTOR_LINEDET: + case VKD3D_SHADER_SV_TESS_FACTOR_LINEDET: return VKD3D_SIV_LINE_DETAIL_TESS_FACTOR; - case VKD3D_SV_TESS_FACTOR_LINEDEN: + case VKD3D_SHADER_SV_TESS_FACTOR_LINEDEN: return VKD3D_SIV_LINE_DENSITY_TESS_FACTOR; default: FIXME("Unhandled sysval %#x, index %u.\n", sysval, index); @@ -163,7 +163,7 @@ static enum vkd3d_shader_input_sysval_semantic vkd3d_siv_from_sysval_indexed(enu } } -static enum vkd3d_shader_input_sysval_semantic vkd3d_siv_from_sysval(enum vkd3d_sysval_semantic sysval) +static enum vkd3d_shader_input_sysval_semantic vkd3d_siv_from_sysval(enum vkd3d_shader_sysval_semantic sysval) { return vkd3d_siv_from_sysval_indexed(sysval, 0); } @@ -4325,11 +4325,11 @@ static void vkd3d_dxbc_compiler_emit_shader_signature_outputs(struct vkd3d_dxbc_ switch (e->sysval_semantic) { - case VKD3D_SV_CLIP_DISTANCE: + case VKD3D_SHADER_SV_CLIP_DISTANCE: calculate_clip_or_cull_distance_mask(e, &clip_distance_mask); break; - case VKD3D_SV_CULL_DISTANCE: + case VKD3D_SHADER_SV_CULL_DISTANCE: calculate_clip_or_cull_distance_mask(e, &cull_distance_mask); break; @@ -4360,13 +4360,13 @@ static void vkd3d_dxbc_compiler_emit_shader_signature_outputs(struct vkd3d_dxbc_ switch (e->sysval_semantic) { - case VKD3D_SV_CLIP_DISTANCE: + case VKD3D_SHADER_SV_CLIP_DISTANCE: compiler->output_info[i].id = clip_distance_id; compiler->output_info[i].component_type = VKD3D_SHADER_COMPONENT_FLOAT; compiler->output_info[i].array_element_mask = clip_distance_mask; break; - case VKD3D_SV_CULL_DISTANCE: + case VKD3D_SHADER_SV_CULL_DISTANCE: compiler->output_info[i].id = cull_distance_id; compiler->output_info[i].component_type = VKD3D_SHADER_COMPONENT_FLOAT; compiler->output_info[i].array_element_mask = cull_distance_mask;

1 0

Henri Verbeet : vkd3d-shader: Rename the vkd3d_component_type enumeration to vkd3d_shader_component_type.
by Alexandre Julliard July 10, 2020

July 10, 2020

Module: vkd3d Branch: master Commit: 2da59a105623df88b767f7f74621c989de188823 URL: https://source.winehq.org/git/vkd3d.git/?a=commit;h=2da59a105623df88b767f7f… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Fri Jul 10 17:02:00 2020 +0430 vkd3d-shader: Rename the vkd3d_component_type enumeration to vkd3d_shader_component_type. Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- include/vkd3d_shader.h | 17 +- libs/vkd3d-shader/spirv.c | 376 ++++++++++++++++--------------- libs/vkd3d-shader/vkd3d_shader_private.h | 18 +- 3 files changed, 206 insertions(+), 205 deletions(-) Diff: https://source.winehq.org/git/vkd3d.git/?a=commitdiff;h=2da59a105623df88b76…

1 0