October 2018 - Wine-commits

Nikolay Sivov : dxgi: Add partial implementation for dxgi_adapter_GetDesc2 () and dxgi_adapter_GetDesc3().
by Alexandre Julliard 24 Oct '18

24 Oct '18

Module: wine Branch: master Commit: 1e8d17d0a4ce5dfd892e70ba8cb131dc56989528 URL: https://source.winehq.org/git/wine.git/?a=commit;h=1e8d17d0a4ce5dfd892e70ba… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Wed Oct 24 08:22:03 2018 +0300 dxgi: Add partial implementation for dxgi_adapter_GetDesc2() and dxgi_adapter_GetDesc3(). Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/dxgi/adapter.c | 57 ++++++++++++++++++++++++++++++++++++++++------------- 1 file changed, 43 insertions(+), 14 deletions(-) diff --git a/dlls/dxgi/adapter.c b/dlls/dxgi/adapter.c index a913870..b96904e 100644 --- a/dlls/dxgi/adapter.c +++ b/dlls/dxgi/adapter.c @@ -147,18 +147,12 @@ static HRESULT STDMETHODCALLTYPE dxgi_adapter_EnumOutputs(IWineDXGIAdapter *ifac return S_OK; } -static HRESULT STDMETHODCALLTYPE dxgi_adapter_GetDesc1(IWineDXGIAdapter *iface, DXGI_ADAPTER_DESC1 *desc) +static HRESULT dxgi_adapter_get_desc(struct dxgi_adapter *adapter, DXGI_ADAPTER_DESC3 *desc) { - struct dxgi_adapter *adapter = impl_from_IWineDXGIAdapter(iface); struct wined3d_adapter_identifier adapter_id; char description[128]; HRESULT hr; - TRACE("iface %p, desc %p.\n", iface, desc); - - if (!desc) - return E_INVALIDARG; - adapter_id.driver_size = 0; adapter_id.description = description; adapter_id.description_size = sizeof(description); @@ -187,13 +181,16 @@ static HRESULT STDMETHODCALLTYPE dxgi_adapter_GetDesc1(IWineDXGIAdapter *iface, desc->SharedSystemMemory = 0; /* FIXME */ desc->AdapterLuid = adapter_id.adapter_luid; desc->Flags = 0; + desc->GraphicsPreemptionGranularity = 0; /* FIXME */ + desc->ComputePreemptionGranularity = 0; /* FIXME */ return hr; } static HRESULT STDMETHODCALLTYPE dxgi_adapter_GetDesc(IWineDXGIAdapter *iface, DXGI_ADAPTER_DESC *desc) { - DXGI_ADAPTER_DESC1 desc1; + struct dxgi_adapter *adapter = impl_from_IWineDXGIAdapter(iface); + DXGI_ADAPTER_DESC3 desc3; HRESULT hr; TRACE("iface %p, desc %p.\n", iface, desc); @@ -201,8 +198,8 @@ static HRESULT STDMETHODCALLTYPE dxgi_adapter_GetDesc(IWineDXGIAdapter *iface, D if (!desc) return E_INVALIDARG; - if (SUCCEEDED(hr = dxgi_adapter_GetDesc1(iface, &desc1))) - memcpy(desc, &desc1, sizeof(*desc)); + if (SUCCEEDED(hr = dxgi_adapter_get_desc(adapter, &desc3))) + memcpy(desc, &desc3, sizeof(*desc)); return hr; } @@ -249,11 +246,38 @@ static HRESULT STDMETHODCALLTYPE dxgi_adapter_CheckInterfaceSupport(IWineDXGIAda return S_OK; } +static HRESULT STDMETHODCALLTYPE dxgi_adapter_GetDesc1(IWineDXGIAdapter *iface, DXGI_ADAPTER_DESC1 *desc) +{ + struct dxgi_adapter *adapter = impl_from_IWineDXGIAdapter(iface); + DXGI_ADAPTER_DESC3 desc3; + HRESULT hr; + + TRACE("iface %p, desc %p.\n", iface, desc); + + if (!desc) + return E_INVALIDARG; + + if (SUCCEEDED(hr = dxgi_adapter_get_desc(adapter, &desc3))) + memcpy(desc, &desc3, sizeof(*desc)); + + return hr; +} + static HRESULT STDMETHODCALLTYPE dxgi_adapter_GetDesc2(IWineDXGIAdapter *iface, DXGI_ADAPTER_DESC2 *desc) { - FIXME("iface %p, desc %p stub!\n", iface, desc); + struct dxgi_adapter *adapter = impl_from_IWineDXGIAdapter(iface); + DXGI_ADAPTER_DESC3 desc3; + HRESULT hr; - return E_NOTIMPL; + TRACE("iface %p, desc %p.\n", iface, desc); + + if (!desc) + return E_INVALIDARG; + + if (SUCCEEDED(hr = dxgi_adapter_get_desc(adapter, &desc3))) + memcpy(desc, &desc3, sizeof(*desc)); + + return hr; } static HRESULT STDMETHODCALLTYPE dxgi_adapter_RegisterHardwareContentProtectionTeardownStatusEvent( @@ -304,9 +328,14 @@ static void STDMETHODCALLTYPE dxgi_adapter_UnregisterVideoMemoryBudgetChangeNoti static HRESULT STDMETHODCALLTYPE dxgi_adapter_GetDesc3(IWineDXGIAdapter *iface, DXGI_ADAPTER_DESC3 *desc) { - FIXME("iface %p, desc %p stub!\n", iface, desc); + struct dxgi_adapter *adapter = impl_from_IWineDXGIAdapter(iface); - return E_NOTIMPL; + TRACE("iface %p, desc %p.\n", iface, desc); + + if (!desc) + return E_INVALIDARG; + + return dxgi_adapter_get_desc(adapter, desc); } static const struct IWineDXGIAdapterVtbl dxgi_adapter_vtbl =

1 0

Jacek Caban : server: Use server_get_file_info for all info classes not implemented on client side.
by Alexandre Julliard 24 Oct '18

24 Oct '18

Module: wine Branch: master Commit: 04094a66694703789768cb230f40ceed2671c0ea URL: https://source.winehq.org/git/wine.git/?a=commit;h=04094a66694703789768cb23… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue Oct 23 17:45:06 2018 +0200 server: Use server_get_file_info for all info classes not implemented on client side. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntdll/file.c | 7 ++----- server/change.c | 2 +- server/device.c | 2 +- server/fd.c | 6 ++++++ server/file.c | 2 +- server/file.h | 1 + server/mailslot.c | 6 +++--- server/named_pipe.c | 26 ++++++++++++++++---------- server/serial.c | 2 +- server/sock.c | 2 +- 10 files changed, 33 insertions(+), 23 deletions(-) diff --git a/dlls/ntdll/file.c b/dlls/ntdll/file.c index 77d2d32..c784ee6 100644 --- a/dlls/ntdll/file.c +++ b/dlls/ntdll/file.c @@ -2200,7 +2200,7 @@ static NTSTATUS server_get_file_info( HANDLE handle, IO_STATUS_BLOCK *io, void * io->Information = wine_server_reply_size( reply ); } SERVER_END_REQ; - if (io->u.Status == STATUS_OBJECT_TYPE_MISMATCH) + if (io->u.Status == STATUS_NOT_IMPLEMENTED) FIXME( "Unsupported info class %x\n", info_class ); return io->u.Status; @@ -2305,10 +2305,7 @@ NTSTATUS WINAPI NtQueryInformationFile( HANDLE hFile, PIO_STATUS_BLOCK io, if (class <= 0 || class >= FileMaximumInformation) return io->u.Status = STATUS_INVALID_INFO_CLASS; if (!info_sizes[class]) - { - FIXME("Unsupported class (%d)\n", class); - return io->u.Status = STATUS_NOT_IMPLEMENTED; - } + return server_get_file_info( hFile, io, ptr, len, class ); if (len < info_sizes[class]) return io->u.Status = STATUS_INFO_LENGTH_MISMATCH; diff --git a/server/change.c b/server/change.c index 1f2f7c5..d00a885 100644 --- a/server/change.c +++ b/server/change.c @@ -187,7 +187,7 @@ static const struct fd_ops dir_fd_ops = no_fd_read, /* read */ no_fd_write, /* write */ no_fd_flush, /* flush */ - no_fd_get_file_info, /* get_file_info */ + default_fd_get_file_info, /* get_file_info */ no_fd_get_volume_info, /* get_volume_info */ default_fd_ioctl, /* ioctl */ default_fd_queue_async, /* queue_async */ diff --git a/server/device.c b/server/device.c index dcc2946..081fd2d 100644 --- a/server/device.c +++ b/server/device.c @@ -209,7 +209,7 @@ static const struct fd_ops device_file_fd_ops = device_file_read, /* read */ device_file_write, /* write */ device_file_flush, /* flush */ - no_fd_get_file_info, /* get_file_info */ + default_fd_get_file_info, /* get_file_info */ no_fd_get_volume_info, /* get_volume_info */ device_file_ioctl, /* ioctl */ default_fd_queue_async, /* queue_async */ diff --git a/server/fd.c b/server/fd.c index 4d07797..85d15d9 100644 --- a/server/fd.c +++ b/server/fd.c @@ -2175,6 +2175,12 @@ void no_fd_get_file_info( struct fd *fd, obj_handle_t handle, unsigned int info_ set_error( STATUS_OBJECT_TYPE_MISMATCH ); } +/* default get_file_info() routine */ +void default_fd_get_file_info( struct fd *fd, obj_handle_t handle, unsigned int info_class ) +{ + set_error( STATUS_NOT_IMPLEMENTED ); +} + /* default get_volume_info() routine */ void no_fd_get_volume_info( struct fd *fd, unsigned int info_class ) { diff --git a/server/file.c b/server/file.c index 446621a..37f7afe 100644 --- a/server/file.c +++ b/server/file.c @@ -107,7 +107,7 @@ static const struct fd_ops file_fd_ops = no_fd_read, /* read */ no_fd_write, /* write */ file_flush, /* flush */ - no_fd_get_file_info, /* get_file_info */ + default_fd_get_file_info, /* get_file_info */ no_fd_get_volume_info, /* get_volume_info */ default_fd_ioctl, /* ioctl */ default_fd_queue_async, /* queue_async */ diff --git a/server/file.h b/server/file.h index 8a47333..c30bc96 100644 --- a/server/file.h +++ b/server/file.h @@ -112,6 +112,7 @@ extern int no_fd_read( struct fd *fd, struct async *async, file_pos_t pos ); extern int no_fd_write( struct fd *fd, struct async *async, file_pos_t pos ); extern int no_fd_flush( struct fd *fd, struct async *async ); extern void no_fd_get_file_info( struct fd *fd, obj_handle_t handle, unsigned int info_class ); +extern void default_fd_get_file_info( struct fd *fd, obj_handle_t handle, unsigned int info_class ); extern void no_fd_get_volume_info( struct fd *fd, unsigned int info_class ); extern int no_fd_ioctl( struct fd *fd, ioctl_code_t code, struct async *async ); extern int default_fd_ioctl( struct fd *fd, ioctl_code_t code, struct async *async ); diff --git a/server/mailslot.c b/server/mailslot.c index f4c7007..95308c4 100644 --- a/server/mailslot.c +++ b/server/mailslot.c @@ -104,7 +104,7 @@ static const struct fd_ops mailslot_fd_ops = no_fd_read, /* read */ no_fd_write, /* write */ no_fd_flush, /* flush */ - no_fd_get_file_info, /* get_file_info */ + default_fd_get_file_info, /* get_file_info */ no_fd_get_volume_info, /* get_volume_info */ default_fd_ioctl, /* ioctl */ mailslot_queue_async, /* queue_async */ @@ -159,7 +159,7 @@ static const struct fd_ops mail_writer_fd_ops = no_fd_read, /* read */ no_fd_write, /* write */ no_fd_flush, /* flush */ - no_fd_get_file_info, /* get_file_info */ + default_fd_get_file_info, /* get_file_info */ no_fd_get_volume_info, /* get_volume_info */ default_fd_ioctl, /* ioctl */ default_fd_queue_async, /* queue_async */ @@ -214,7 +214,7 @@ static const struct fd_ops mailslot_device_fd_ops = no_fd_read, /* read */ no_fd_write, /* write */ no_fd_flush, /* flush */ - no_fd_get_file_info, /* get_file_info */ + default_fd_get_file_info, /* get_file_info */ no_fd_get_volume_info, /* get_volume_info */ default_fd_ioctl, /* ioctl */ default_fd_queue_async, /* queue_async */ diff --git a/server/named_pipe.c b/server/named_pipe.c index ebd8c4f..f643140 100644 --- a/server/named_pipe.c +++ b/server/named_pipe.c @@ -271,7 +271,7 @@ static const struct fd_ops named_pipe_device_fd_ops = no_fd_read, /* read */ no_fd_write, /* write */ no_fd_flush, /* flush */ - no_fd_get_file_info, /* get_file_info */ + default_fd_get_file_info, /* get_file_info */ no_fd_get_volume_info, /* get_volume_info */ named_pipe_device_ioctl, /* ioctl */ default_fd_queue_async, /* queue_async */ @@ -520,12 +520,6 @@ static void pipe_end_get_file_info( struct fd *fd, obj_handle_t handle, unsigned struct pipe_end *pipe_end = get_fd_user( fd ); struct named_pipe *pipe = pipe_end->pipe; - if (!pipe) - { - set_error( STATUS_PIPE_DISCONNECTED ); - return; - } - switch (info_class) { case FileNameInformation: @@ -540,13 +534,13 @@ static void pipe_end_get_file_info( struct fd *fd, obj_handle_t handle, unsigned return; } - name = get_object_name( &pipe->obj, &name_len ); /* FIXME: We should be able to return on unlinked pipe */ - if (!name) + if (!pipe || !(name = get_object_name( &pipe->obj, &name_len ))) { set_error( STATUS_PIPE_DISCONNECTED ); return; } + reply_size = offsetof( FILE_NAME_INFORMATION, FileName[name_len/sizeof(WCHAR) + 1] ); if (reply_size > get_reply_max_size()) { @@ -577,6 +571,12 @@ static void pipe_end_get_file_info( struct fd *fd, obj_handle_t handle, unsigned return; } + if (!pipe) + { + set_error( STATUS_PIPE_DISCONNECTED ); + return; + } + if (!(pipe_info = set_reply_data_size( sizeof(*pipe_info) ))) return; pipe_info->ReadMode = (pipe_end->flags & NAMED_PIPE_MESSAGE_STREAM_READ) ? FILE_PIPE_MESSAGE_MODE : FILE_PIPE_BYTE_STREAM_MODE; @@ -600,6 +600,12 @@ static void pipe_end_get_file_info( struct fd *fd, obj_handle_t handle, unsigned return; } + if (!pipe) + { + set_error( STATUS_PIPE_DISCONNECTED ); + return; + } + if (!(pipe_info = set_reply_data_size( sizeof(*pipe_info) ))) return; pipe_info->NamedPipeType = (pipe_end->flags & NAMED_PIPE_MESSAGE_STREAM_WRITE) != 0; switch (pipe->sharing) @@ -626,7 +632,7 @@ static void pipe_end_get_file_info( struct fd *fd, obj_handle_t handle, unsigned break; } default: - no_fd_get_file_info( fd, handle, info_class ); + default_fd_get_file_info( fd, handle, info_class ); } } diff --git a/server/serial.c b/server/serial.c index bb976a1..f0adf92 100644 --- a/server/serial.c +++ b/server/serial.c @@ -114,7 +114,7 @@ static const struct fd_ops serial_fd_ops = no_fd_read, /* read */ no_fd_write, /* write */ no_fd_flush, /* flush */ - no_fd_get_file_info, /* get_file_info */ + default_fd_get_file_info, /* get_file_info */ no_fd_get_volume_info, /* get_volume_info */ serial_ioctl, /* ioctl */ serial_queue_async, /* queue_async */ diff --git a/server/sock.c b/server/sock.c index 84f54f6..a8e6e28 100644 --- a/server/sock.c +++ b/server/sock.c @@ -167,7 +167,7 @@ static const struct fd_ops sock_fd_ops = no_fd_read, /* read */ no_fd_write, /* write */ no_fd_flush, /* flush */ - no_fd_get_file_info, /* get_file_info */ + default_fd_get_file_info, /* get_file_info */ no_fd_get_volume_info, /* get_volume_info */ sock_ioctl, /* ioctl */ sock_queue_async, /* queue_async */

1 0

Jacek Caban : server: Use default_fd_get_file_info for FileAccessInformation implementation.
by Alexandre Julliard 24 Oct '18

24 Oct '18

Module: wine Branch: master Commit: 8b8eba7e6cf168541c88b9d8ef1d48af011a5dfc URL: https://source.winehq.org/git/wine.git/?a=commit;h=8b8eba7e6cf168541c88b9d8… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue Oct 23 17:45:21 2018 +0200 server: Use default_fd_get_file_info for FileAccessInformation implementation. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntdll/file.c | 24 ++++-------------------- server/fd.c | 18 +++++++++++++++++- 2 files changed, 21 insertions(+), 21 deletions(-) diff --git a/dlls/ntdll/file.c b/dlls/ntdll/file.c index c784ee6..b571284 100644 --- a/dlls/ntdll/file.c +++ b/dlls/ntdll/file.c @@ -2236,7 +2236,7 @@ NTSTATUS WINAPI NtQueryInformationFile( HANDLE hFile, PIO_STATUS_BLOCK io, sizeof(FILE_STANDARD_INFORMATION), /* FileStandardInformation */ sizeof(FILE_INTERNAL_INFORMATION), /* FileInternalInformation */ sizeof(FILE_EA_INFORMATION), /* FileEaInformation */ - sizeof(FILE_ACCESS_INFORMATION), /* FileAccessInformation */ + 0, /* FileAccessInformation */ sizeof(FILE_NAME_INFORMATION), /* FileNameInformation */ sizeof(FILE_RENAME_INFORMATION)-sizeof(WCHAR), /* FileRenameInformation */ 0, /* FileLinkInformation */ @@ -2309,13 +2309,10 @@ NTSTATUS WINAPI NtQueryInformationFile( HANDLE hFile, PIO_STATUS_BLOCK io, if (len < info_sizes[class]) return io->u.Status = STATUS_INFO_LENGTH_MISMATCH; - if (class != FileAccessInformation) + if ((io->u.Status = server_get_unix_fd( hFile, 0, &fd, &needs_close, NULL, NULL ))) { - if ((io->u.Status = server_get_unix_fd( hFile, 0, &fd, &needs_close, NULL, NULL ))) - { - if (io->u.Status != STATUS_BAD_DEVICE_TYPE) return io->u.Status; - return server_get_file_info( hFile, io, ptr, len, class ); - } + if (io->u.Status != STATUS_BAD_DEVICE_TYPE) return io->u.Status; + return server_get_file_info( hFile, io, ptr, len, class ); } switch (class) @@ -2358,19 +2355,6 @@ NTSTATUS WINAPI NtQueryInformationFile( HANDLE hFile, PIO_STATUS_BLOCK io, info->EaSize = 0; } break; - case FileAccessInformation: - { - FILE_ACCESS_INFORMATION *info = ptr; - SERVER_START_REQ( get_object_info ) - { - req->handle = wine_server_obj_handle( hFile ); - io->u.Status = wine_server_call( req ); - if (io->u.Status == STATUS_SUCCESS) - info->AccessFlags = reply->access; - } - SERVER_END_REQ; - } - break; case FileEndOfFileInformation: if (fd_get_file_info( fd, &st, &attr ) == -1) io->u.Status = FILE_GetNtStatus(); else fill_file_info( &st, attr, ptr, class ); diff --git a/server/fd.c b/server/fd.c index 85d15d9..65dc8a2 100644 --- a/server/fd.c +++ b/server/fd.c @@ -2178,7 +2178,23 @@ void no_fd_get_file_info( struct fd *fd, obj_handle_t handle, unsigned int info_ /* default get_file_info() routine */ void default_fd_get_file_info( struct fd *fd, obj_handle_t handle, unsigned int info_class ) { - set_error( STATUS_NOT_IMPLEMENTED ); + switch (info_class) + { + case FileAccessInformation: + { + FILE_ACCESS_INFORMATION info; + if (get_reply_max_size() < sizeof(info)) + { + set_error( STATUS_INFO_LENGTH_MISMATCH ); + return; + } + info.AccessFlags = get_handle_access( current->process, handle ); + set_reply_data( &info, sizeof(info) ); + break; + } + default: + set_error( STATUS_NOT_IMPLEMENTED ); + } } /* default get_volume_info() routine */

1 0

Jacek Caban : ntdll/tests: Add more NtQueryInformationFile tests.
by Alexandre Julliard 24 Oct '18

24 Oct '18

Module: wine Branch: master Commit: 6c2ac81084f4e8d66ab3f4361710680f0fdcc74b URL: https://source.winehq.org/git/wine.git/?a=commit;h=6c2ac81084f4e8d66ab3f436… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue Oct 23 17:45:31 2018 +0200 ntdll/tests: Add more NtQueryInformationFile tests. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntdll/tests/om.c | 43 +++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 43 insertions(+) diff --git a/dlls/ntdll/tests/om.c b/dlls/ntdll/tests/om.c index 7eef55c..c340839 100644 --- a/dlls/ntdll/tests/om.c +++ b/dlls/ntdll/tests/om.c @@ -69,6 +69,7 @@ static NTSTATUS (WINAPI *pNtWaitForKeyedEvent)( HANDLE, const void *, BOOLEAN, c static NTSTATUS (WINAPI *pNtReleaseKeyedEvent)( HANDLE, const void *, BOOLEAN, const LARGE_INTEGER * ); static NTSTATUS (WINAPI *pNtCreateIoCompletion)(PHANDLE, ACCESS_MASK, POBJECT_ATTRIBUTES, ULONG); static NTSTATUS (WINAPI *pNtOpenIoCompletion)( PHANDLE, ACCESS_MASK, POBJECT_ATTRIBUTES ); +static NTSTATUS (WINAPI *pNtQueryInformationFile)(HANDLE, PIO_STATUS_BLOCK, void *, ULONG, FILE_INFORMATION_CLASS); #define KEYEDEVENT_WAIT 0x0001 #define KEYEDEVENT_WAKE 0x0002 @@ -1270,6 +1271,37 @@ static void test_symboliclink(void) pNtClose(dir); } +#define test_file_info(a) _test_file_info(__LINE__,a) +static void _test_file_info(unsigned line, HANDLE handle) +{ + IO_STATUS_BLOCK io; + char buf[256]; + NTSTATUS status; + + status = pNtQueryInformationFile(handle, &io, buf, sizeof(buf), 0xdeadbeef); + ok_(__FILE__,line)(status == STATUS_INVALID_INFO_CLASS || status == STATUS_NOT_IMPLEMENTED, + "expected STATUS_NOT_IMPLEMENTED, got %x\n", status); + + status = pNtQueryInformationFile(handle, &io, buf, sizeof(buf), FileAccessInformation); + ok_(__FILE__,line)(status == STATUS_SUCCESS, "FileAccessInformation returned %x\n", status); +} + +#define test_no_file_info(a) _test_no_file_info(__LINE__,a) +static void _test_no_file_info(unsigned line, HANDLE handle) +{ + IO_STATUS_BLOCK io; + char buf[256]; + NTSTATUS status; + + status = pNtQueryInformationFile(handle, &io, buf, sizeof(buf), 0xdeadbeef); + ok_(__FILE__,line)(status == STATUS_INVALID_INFO_CLASS || status == STATUS_NOT_IMPLEMENTED, + "expected STATUS_NOT_IMPLEMENTED, got %x\n", status); + + status = pNtQueryInformationFile(handle, &io, buf, sizeof(buf), FileAccessInformation); + ok_(__FILE__,line)(status == STATUS_OBJECT_TYPE_MISMATCH, + "FileAccessInformation returned %x\n", status); +} + static void test_query_object(void) { static const WCHAR name[] = {'\\','B','a','s','e','N','a','m','e','d','O','b','j','e','c','t','s', @@ -1351,6 +1383,7 @@ static void test_query_object(void) ok( status == STATUS_INFO_LENGTH_MISMATCH, "NtQueryObject failed %x\n", status ); ok( len >= sizeof(OBJECT_TYPE_INFORMATION) + sizeof(type_event) + sizeof(WCHAR), "unexpected len %u\n", len ); + test_no_file_info( handle ); pNtClose( handle ); handle = CreateEventA( NULL, FALSE, FALSE, NULL ); @@ -1361,6 +1394,7 @@ static void test_query_object(void) str = (UNICODE_STRING *)buffer; ok( str->Length == 0, "unexpected len %u\n", len ); ok( str->Buffer == NULL, "unexpected ptr %p\n", str->Buffer ); + test_no_file_info( handle ); pNtClose( handle ); GetWindowsDirectoryA( dir, MAX_PATH ); @@ -1401,6 +1435,7 @@ static void test_query_object(void) ok( len >= expected_len, "unexpected len %u\n", len ); ok( str->Buffer && !memcmp( str->Buffer, type_file, sizeof(type_file) ), "wrong/bad type name %s (%p)\n", wine_dbgstr_w(str->Buffer), str->Buffer ); + test_file_info( handle ); pNtClose( handle ); @@ -1418,6 +1453,7 @@ static void test_query_object(void) ok( str->Buffer && !memcmp( str->Buffer, type_file, sizeof(type_file) ), "wrong/bad type name %s (%p)\n", wine_dbgstr_w(str->Buffer), str->Buffer ); DeleteFileA( file1 ); + test_file_info( handle ); pNtClose( handle ); status = pNtCreateIoCompletion( &handle, IO_COMPLETION_ALL_ACCESS, NULL, 0 ); @@ -1432,6 +1468,7 @@ static void test_query_object(void) ok( len >= expected_len, "unexpected len %u\n", len ); ok( str->Buffer && !memcmp( str->Buffer, type_iocompletion, sizeof(type_iocompletion) ), "wrong/bad type name %s (%p)\n", wine_dbgstr_w(str->Buffer), str->Buffer ); + test_no_file_info( handle ); pNtClose( handle ); status = pNtCreateDirectoryObject( &handle, DIRECTORY_QUERY, NULL ); @@ -1446,6 +1483,7 @@ static void test_query_object(void) ok( len >= expected_len, "unexpected len %u\n", len ); ok( str->Buffer && !memcmp( str->Buffer, type_directory, sizeof(type_directory) ), "wrong/bad type name %s (%p)\n", wine_dbgstr_w(str->Buffer), str->Buffer ); + test_no_file_info( handle ); pNtClose( handle ); size.u.LowPart = 256; @@ -1462,6 +1500,7 @@ static void test_query_object(void) ok( len >= expected_len, "unexpected len %u\n", len ); ok( str->Buffer && !memcmp( str->Buffer, type_section, sizeof(type_section) ), "wrong/bad type name %s (%p)\n", wine_dbgstr_w(str->Buffer), str->Buffer ); + test_no_file_info( handle ); pNtClose( handle ); handle = CreateMailslotA( "\\\\.\\mailslot\\test_mailslot", 100, 1000, NULL ); @@ -1478,6 +1517,7 @@ static void test_query_object(void) ok( len > sizeof(UNICODE_STRING) + sizeof("\\test_mailslot") * sizeof(WCHAR), "name too short %s\n", wine_dbgstr_w(str->Buffer) ); trace( "got %s len %u\n", wine_dbgstr_w(str->Buffer), len ); + test_file_info( handle ); pNtClose( handle ); handle = CreateNamedPipeA( "\\\\.\\pipe\\test_pipe", PIPE_ACCESS_DUPLEX, PIPE_READMODE_BYTE, @@ -1508,6 +1548,7 @@ static void test_query_object(void) ok( len >= sizeof(OBJECT_TYPE_INFORMATION) + str->Length + sizeof(WCHAR), "unexpected len %u\n", len ); ok( str->Buffer && !memcmp( str->Buffer, type_file, sizeof(type_file) ), "wrong/bad type name %s (%p)\n", wine_dbgstr_w(str->Buffer), str->Buffer ); + test_file_info( handle ); client = CreateFileA( "\\\\.\\pipe\\test_pipe", GENERIC_READ | GENERIC_WRITE, 0, NULL, OPEN_EXISTING, 0, 0 ); @@ -1522,6 +1563,7 @@ static void test_query_object(void) ok( len >= sizeof(OBJECT_TYPE_INFORMATION) + str->Length + sizeof(WCHAR), "unexpected len %u\n", len ); ok( str->Buffer && !memcmp( str->Buffer, type_file, sizeof(type_file) ), "wrong/bad type name %s (%p)\n", wine_dbgstr_w(str->Buffer), str->Buffer ); + test_file_info( client ); pNtClose( client ); pNtClose( handle ); @@ -2064,6 +2106,7 @@ START_TEST(om) pNtReleaseKeyedEvent = (void *)GetProcAddress(hntdll, "NtReleaseKeyedEvent"); pNtCreateIoCompletion = (void *)GetProcAddress(hntdll, "NtCreateIoCompletion"); pNtOpenIoCompletion = (void *)GetProcAddress(hntdll, "NtOpenIoCompletion"); + pNtQueryInformationFile = (void *)GetProcAddress(hntdll, "NtQueryInformationFile"); test_case_sensitive(); test_namespace_pipe();

1 0

Jacek Caban : server: Add NtQueryInformationFile( FileIoCompletionNotificationInformation) implementation.
by Alexandre Julliard 24 Oct '18

24 Oct '18

Module: wine Branch: master Commit: 66e20ce576fd815cffd016f3d36f84c619ea7bc4 URL: https://source.winehq.org/git/wine.git/?a=commit;h=66e20ce576fd815cffd016f3… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue Oct 23 17:45:42 2018 +0200 server: Add NtQueryInformationFile(FileIoCompletionNotificationInformation) implementation. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntdll/tests/om.c | 10 ++++++++++ server/fd.c | 12 ++++++++++++ 2 files changed, 22 insertions(+) diff --git a/dlls/ntdll/tests/om.c b/dlls/ntdll/tests/om.c index c340839..5240854 100644 --- a/dlls/ntdll/tests/om.c +++ b/dlls/ntdll/tests/om.c @@ -1284,6 +1284,11 @@ static void _test_file_info(unsigned line, HANDLE handle) status = pNtQueryInformationFile(handle, &io, buf, sizeof(buf), FileAccessInformation); ok_(__FILE__,line)(status == STATUS_SUCCESS, "FileAccessInformation returned %x\n", status); + + status = pNtQueryInformationFile(handle, &io, buf, sizeof(buf), + FileIoCompletionNotificationInformation); + ok_(__FILE__,line)(status == STATUS_SUCCESS || broken(status == STATUS_INVALID_INFO_CLASS), + "FileIoCompletionNotificationInformation returned %x\n", status); } #define test_no_file_info(a) _test_no_file_info(__LINE__,a) @@ -1300,6 +1305,11 @@ static void _test_no_file_info(unsigned line, HANDLE handle) status = pNtQueryInformationFile(handle, &io, buf, sizeof(buf), FileAccessInformation); ok_(__FILE__,line)(status == STATUS_OBJECT_TYPE_MISMATCH, "FileAccessInformation returned %x\n", status); + + status = pNtQueryInformationFile(handle, &io, buf, sizeof(buf), + FileIoCompletionNotificationInformation); + ok_(__FILE__,line)(status == STATUS_OBJECT_TYPE_MISMATCH || broken(status == STATUS_INVALID_INFO_CLASS), + "FileIoCompletionNotificationInformation returned %x\n", status); } static void test_query_object(void) diff --git a/server/fd.c b/server/fd.c index 65dc8a2..3e73fae 100644 --- a/server/fd.c +++ b/server/fd.c @@ -2192,6 +2192,18 @@ void default_fd_get_file_info( struct fd *fd, obj_handle_t handle, unsigned int set_reply_data( &info, sizeof(info) ); break; } + case FileIoCompletionNotificationInformation: + { + FILE_IO_COMPLETION_NOTIFICATION_INFORMATION info; + if (get_reply_max_size() < sizeof(info)) + { + set_error( STATUS_INFO_LENGTH_MISMATCH ); + return; + } + info.Flags = 0; /* FIXME */ + set_reply_data( &info, sizeof(info) ); + break; + } default: set_error( STATUS_NOT_IMPLEMENTED ); }

1 0

Alexandre Julliard : kernel32: Get std handles from the process parameters.
by Alexandre Julliard 24 Oct '18

24 Oct '18

Module: wine Branch: master Commit: 33c3cee81771c6437cd73564ed84ab530e7f0185 URL: https://source.winehq.org/git/wine.git/?a=commit;h=33c3cee81771c6437cd73564… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Oct 24 11:32:16 2018 +0200 kernel32: Get std handles from the process parameters. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/kernel32/process.c | 36 ++++++++++++++++-------------------- 1 file changed, 16 insertions(+), 20 deletions(-) diff --git a/dlls/kernel32/process.c b/dlls/kernel32/process.c index 799519e..59b6961 100644 --- a/dlls/kernel32/process.c +++ b/dlls/kernel32/process.c @@ -1663,7 +1663,7 @@ static char **build_envp( const WCHAR *envW ) * * Fork and exec a new Unix binary, checking for errors. */ -static int fork_and_exec( const RTL_USER_PROCESS_PARAMETERS *params, const char *newdir, DWORD flags ) +static int fork_and_exec( const RTL_USER_PROCESS_PARAMETERS *params, const char *newdir ) { int fd[2], stdin_fd = -1, stdout_fd = -1, stderr_fd = -1; int pid, err; @@ -1698,7 +1698,8 @@ static int fork_and_exec( const RTL_USER_PROCESS_PARAMETERS *params, const char { close( fd[0] ); - if (flags & (CREATE_NEW_PROCESS_GROUP | CREATE_NEW_CONSOLE | DETACHED_PROCESS)) + if (params->ConsoleFlags || params->ConsoleHandle == KERNEL32_CONSOLE_ALLOC || + (params->hStdInput == INVALID_HANDLE_VALUE && params->hStdOutput == INVALID_HANDLE_VALUE)) { int nullfd = open( "/dev/null", O_RDWR ); setsid(); @@ -2035,11 +2036,11 @@ static BOOL terminate_main_thread(void) /*********************************************************************** * spawn_loader */ -static pid_t spawn_loader( const RTL_USER_PROCESS_PARAMETERS *params, unsigned int flags, int socketfd, - int stdin_fd, int stdout_fd, const char *unixdir, char *winedebug, - const pe_image_info_t *pe_info ) +static pid_t spawn_loader( const RTL_USER_PROCESS_PARAMETERS *params, int socketfd, + const char *unixdir, char *winedebug, const pe_image_info_t *pe_info ) { pid_t pid; + int stdin_fd = -1, stdout_fd = -1; char *wineloader = NULL; const char *loader = NULL; char **argv; @@ -2049,6 +2050,9 @@ static pid_t spawn_loader( const RTL_USER_PROCESS_PARAMETERS *params, unsigned i if (!is_win64 ^ !is_64bit_arch( pe_info->cpu )) loader = get_alternate_loader( &wineloader ); + wine_server_handle_to_fd( params->hStdInput, FILE_READ_DATA, &stdin_fd, NULL ); + wine_server_handle_to_fd( params->hStdOutput, FILE_WRITE_DATA, &stdout_fd, NULL ); + if (!(pid = fork())) /* child */ { if (!(pid = fork())) /* grandchild */ @@ -2057,7 +2061,8 @@ static pid_t spawn_loader( const RTL_USER_PROCESS_PARAMETERS *params, unsigned i ULONGLONG res_start = pe_info->base; ULONGLONG res_end = pe_info->base + pe_info->map_size; - if (flags & (CREATE_NEW_PROCESS_GROUP | CREATE_NEW_CONSOLE | DETACHED_PROCESS)) + if (params->ConsoleFlags || params->ConsoleHandle == KERNEL32_CONSOLE_ALLOC || + (params->hStdInput == INVALID_HANDLE_VALUE && params->hStdOutput == INVALID_HANDLE_VALUE)) { int fd = open( "/dev/null", O_RDWR ); setsid(); @@ -2107,6 +2112,8 @@ static pid_t spawn_loader( const RTL_USER_PROCESS_PARAMETERS *params, unsigned i } while (wret < 0 && errno == EINTR); } + if (stdin_fd != -1) close( stdin_fd ); + if (stdout_fd != -1) close( stdout_fd ); HeapFree( GetProcessHeap(), 0, wineloader ); HeapFree( GetProcessHeap(), 0, argv ); return pid; @@ -2292,7 +2299,7 @@ static BOOL create_process( HANDLE hFile, LPSECURITY_ATTRIBUTES psa, LPSECURITY_ char *winedebug = NULL; startup_info_t *startup_info; DWORD startup_info_size; - int socketfd[2], stdin_fd = -1, stdout_fd = -1; + int socketfd[2]; pid_t pid; int err; @@ -2401,23 +2408,12 @@ static BOOL create_process( HANDLE hFile, LPSECURITY_ATTRIBUTES psa, LPSECURITY_ return FALSE; } - if (!(flags & (CREATE_NEW_CONSOLE | DETACHED_PROCESS))) - { - if (startup_info->hstdin) - wine_server_handle_to_fd( wine_server_ptr_handle(startup_info->hstdin), - FILE_READ_DATA, &stdin_fd, NULL ); - if (startup_info->hstdout) - wine_server_handle_to_fd( wine_server_ptr_handle(startup_info->hstdout), - FILE_WRITE_DATA, &stdout_fd, NULL ); - } HeapFree( GetProcessHeap(), 0, startup_info ); /* create the child process */ - pid = spawn_loader( params, flags, socketfd[0], stdin_fd, stdout_fd, unixdir, winedebug, pe_info ); + pid = spawn_loader( params, socketfd[0], unixdir, winedebug, pe_info ); - if (stdin_fd != -1) close( stdin_fd ); - if (stdout_fd != -1) close( stdout_fd ); close( socketfd[0] ); HeapFree( GetProcessHeap(), 0, winedebug ); if (pid == -1) @@ -2782,7 +2778,7 @@ static BOOL create_process_impl( LPCWSTR app_name, LPWSTR cmd_line, LPSECURITY_A case BINARY_UNIX_EXE: /* unknown file, try as unix executable */ TRACE( "starting %s as Unix binary\n", debugstr_w(name) ); - retv = (fork_and_exec( params, unixdir, flags ) != -1); + retv = (fork_and_exec( params, unixdir ) != -1); break; } if (hFile) CloseHandle( hFile );

1 0

Alexandre Julliard : kernel32: Use the Unicode string length to build the argv array.
by Alexandre Julliard 24 Oct '18

24 Oct '18

Module: wine Branch: master Commit: 1a561b640b52359a2dba8aa345264f2f2a88fafe URL: https://source.winehq.org/git/wine.git/?a=commit;h=1a561b640b52359a2dba8aa3… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Oct 24 12:20:38 2018 +0200 kernel32: Use the Unicode string length to build the argv array. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/kernel32/process.c | 17 ++++++++++------- 1 file changed, 10 insertions(+), 7 deletions(-) diff --git a/dlls/kernel32/process.c b/dlls/kernel32/process.c index 59b6961..ea483b9 100644 --- a/dlls/kernel32/process.c +++ b/dlls/kernel32/process.c @@ -1483,16 +1483,19 @@ void CDECL __wine_kernel_init(void) * Build an argv array from a command-line. * 'reserved' is the number of args to reserve before the first one. */ -static char **build_argv( const WCHAR *cmdlineW, int reserved ) +static char **build_argv( const UNICODE_STRING *cmdlineW, int reserved ) { int argc; char** argv; char *arg,*s,*d,*cmdline; int in_quotes,bcount,len; - len = WideCharToMultiByte( CP_UNIXCP, 0, cmdlineW, -1, NULL, 0, NULL, NULL ); - if (!(cmdline = HeapAlloc( GetProcessHeap(), 0, len ))) return NULL; - WideCharToMultiByte( CP_UNIXCP, 0, cmdlineW, -1, cmdline, len, NULL, NULL ); + len = WideCharToMultiByte( CP_UNIXCP, 0, cmdlineW->Buffer, cmdlineW->Length / sizeof(WCHAR), + NULL, 0, NULL, NULL ); + if (!(cmdline = HeapAlloc( GetProcessHeap(), 0, len + 1 ))) return NULL; + WideCharToMultiByte( CP_UNIXCP, 0, cmdlineW->Buffer, cmdlineW->Length / sizeof(WCHAR), + cmdline, len, NULL, NULL ); + cmdline[len++] = 0; argc=reserved+1; bcount=0; @@ -1689,7 +1692,7 @@ static int fork_and_exec( const RTL_USER_PROCESS_PARAMETERS *params, const char wine_server_handle_to_fd( params->hStdOutput, FILE_WRITE_DATA, &stdout_fd, NULL ); wine_server_handle_to_fd( params->hStdError, FILE_WRITE_DATA, &stderr_fd, NULL ); - argv = build_argv( params->CommandLine.Buffer, 0 ); + argv = build_argv( &params->CommandLine, 0 ); envp = build_envp( params->Environment ); if (!(pid = fork())) /* child */ @@ -2045,7 +2048,7 @@ static pid_t spawn_loader( const RTL_USER_PROCESS_PARAMETERS *params, int socket const char *loader = NULL; char **argv; - argv = build_argv( params->CommandLine.Buffer, 1 ); + argv = build_argv( &params->CommandLine, 1 ); if (!is_win64 ^ !is_64bit_arch( pe_info->cpu )) loader = get_alternate_loader( &wineloader ); @@ -2132,7 +2135,7 @@ static NTSTATUS exec_loader( const RTL_USER_PROCESS_PARAMETERS *params, int sock ULONGLONG res_start = pe_info->base; ULONGLONG res_end = pe_info->base + pe_info->map_size; - if (!(argv = build_argv( params->CommandLine.Buffer, 1 ))) return STATUS_NO_MEMORY; + if (!(argv = build_argv( &params->CommandLine, 1 ))) return STATUS_NO_MEMORY; if (!is_win64 ^ !is_64bit_arch( pe_info->cpu )) loader = get_alternate_loader( &wineloader );

1 0

Alexandre Julliard : kernel32: Avoid setting an empty current directory in the process params.
by Alexandre Julliard 24 Oct '18

24 Oct '18

Module: wine Branch: master Commit: 7a8ff7fab34e765956fbaf29b63226e561cc0045 URL: https://source.winehq.org/git/wine.git/?a=commit;h=7a8ff7fab34e765956fbaf29… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Oct 24 12:21:06 2018 +0200 kernel32: Avoid setting an empty current directory in the process params. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/kernel32/process.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/dlls/kernel32/process.c b/dlls/kernel32/process.c index ea483b9..1490f6d 100644 --- a/dlls/kernel32/process.c +++ b/dlls/kernel32/process.c @@ -1887,8 +1887,9 @@ static RTL_USER_PROCESS_PARAMETERS *create_process_params( LPCWSTR filename, LPC RtlInitUnicodeString( &desktopW, startup->lpDesktop ); runtimeW.Buffer = (WCHAR *)startup->lpReserved2; runtimeW.Length = runtimeW.MaximumLength = startup->cbReserved2; - if (RtlCreateProcessParametersEx( &params, &imageW, NULL, &curdirW, &cmdlineW, envW, &titleW, - &desktopW, NULL, &runtimeW, PROCESS_PARAMS_FLAG_NORMALIZED )) + if (RtlCreateProcessParametersEx( &params, &imageW, NULL, cur_dir ? &curdirW : NULL, + &cmdlineW, envW, &titleW, &desktopW, + NULL, &runtimeW, PROCESS_PARAMS_FLAG_NORMALIZED )) { if (envW != env) HeapFree( GetProcessHeap(), 0, envW ); return NULL;

1 0

Alexandre Julliard : ntdll: Implement RtlCreateUserProcess().
by Alexandre Julliard 24 Oct '18

24 Oct '18

Module: wine Branch: master Commit: 5cc8bcf0b05f6cfd4f60fe371e00ec8a575bec83 URL: https://source.winehq.org/git/wine.git/?a=commit;h=5cc8bcf0b05f6cfd4f60fe37… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Oct 24 12:27:23 2018 +0200 ntdll: Implement RtlCreateUserProcess(). Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntdll/process.c | 583 ++++++++++++++++++++++++++++++++++++++++++++++++++- dlls/ntdll/rtl.c | 13 -- 2 files changed, 578 insertions(+), 18 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=5cc8bcf0b05f6cfd4f60…

1 0

Alexandre Julliard : ntdll: Also return the SECTION_IMAGE_INFORMATION data from RtlCreateUserProcess( ).
by Alexandre Julliard 24 Oct '18

24 Oct '18

Module: wine Branch: master Commit: c998667bf0983ef99cc48847d3d6fc6ca6ff4a2d URL: https://source.winehq.org/git/wine.git/?a=commit;h=c998667bf0983ef99cc48847… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Oct 24 12:32:30 2018 +0200 ntdll: Also return the SECTION_IMAGE_INFORMATION data from RtlCreateUserProcess(). Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntdll/ntdll_misc.h | 2 ++ dlls/ntdll/process.c | 1 + dlls/ntdll/virtual.c | 59 ++++++++++++++++++++++++++++--------------------- 3 files changed, 37 insertions(+), 25 deletions(-) diff --git a/dlls/ntdll/ntdll_misc.h b/dlls/ntdll/ntdll_misc.h index f4de3cf..fdb443b 100644 --- a/dlls/ntdll/ntdll_misc.h +++ b/dlls/ntdll/ntdll_misc.h @@ -185,6 +185,8 @@ extern NTSTATUS virtual_uninterrupted_write_memory( void *addr, const void *buff extern void VIRTUAL_SetForceExec( BOOL enable ) DECLSPEC_HIDDEN; extern void virtual_release_address_space(void) DECLSPEC_HIDDEN; extern void virtual_set_large_address_space(void) DECLSPEC_HIDDEN; +extern void virtual_fill_image_information( const pe_image_info_t *pe_info, + SECTION_IMAGE_INFORMATION *info ) DECLSPEC_HIDDEN; extern struct _KUSER_SHARED_DATA *user_shared_data DECLSPEC_HIDDEN; /* completion */ diff --git a/dlls/ntdll/process.c b/dlls/ntdll/process.c index 372e26e..623bf65 100644 --- a/dlls/ntdll/process.c +++ b/dlls/ntdll/process.c @@ -1314,6 +1314,7 @@ NTSTATUS WINAPI RtlCreateUserProcess( UNICODE_STRING *path, ULONG attributes, info->Thread = thread_handle; info->ClientId.UniqueProcess = ULongToHandle( process_id ); info->ClientId.UniqueThread = ULongToHandle( thread_id ); + virtual_fill_image_information( &pe_info, &info->ImageInformation ); process_handle = thread_handle = 0; status = STATUS_SUCCESS; } diff --git a/dlls/ntdll/virtual.c b/dlls/ntdll/virtual.c index af1509e..5dfbe74 100644 --- a/dlls/ntdll/virtual.c +++ b/dlls/ntdll/virtual.c @@ -3176,6 +3176,39 @@ NTSTATUS WINAPI NtUnmapViewOfSection( HANDLE process, PVOID addr ) /****************************************************************************** + * virtual_fill_image_information + * + * Helper for NtQuerySection. + */ +void virtual_fill_image_information( const pe_image_info_t *pe_info, SECTION_IMAGE_INFORMATION *info ) +{ + info->TransferAddress = wine_server_get_ptr( pe_info->entry_point ); + info->ZeroBits = pe_info->zerobits; + info->MaximumStackSize = pe_info->stack_size; + info->CommittedStackSize = pe_info->stack_commit; + info->SubSystemType = pe_info->subsystem; + info->SubsystemVersionLow = pe_info->subsystem_low; + info->SubsystemVersionHigh = pe_info->subsystem_high; + info->GpValue = pe_info->gp; + info->ImageCharacteristics = pe_info->image_charact; + info->DllCharacteristics = pe_info->dll_charact; + info->Machine = pe_info->machine; + info->ImageContainsCode = pe_info->contains_code; + info->u.ImageFlags = pe_info->image_flags; + info->LoaderFlags = pe_info->loader_flags; + info->ImageFileSize = pe_info->file_size; + info->CheckSum = pe_info->checksum; +#ifndef _WIN64 /* don't return 64-bit values to 32-bit processes */ + if (pe_info->machine == IMAGE_FILE_MACHINE_AMD64 || pe_info->machine == IMAGE_FILE_MACHINE_ARM64) + { + info->TransferAddress = (void *)0x81231234; /* sic */ + info->MaximumStackSize = 0x100000; + info->CommittedStackSize = 0x10000; + } +#endif +} + +/****************************************************************************** * NtQuerySection (NTDLL.@) * ZwQuerySection (NTDLL.@) */ @@ -3217,32 +3250,8 @@ NTSTATUS WINAPI NtQuerySection( HANDLE handle, SECTION_INFORMATION_CLASS class, else if (reply->flags & SEC_IMAGE) { SECTION_IMAGE_INFORMATION *info = ptr; - info->TransferAddress = wine_server_get_ptr( image_info.entry_point ); - info->ZeroBits = image_info.zerobits; - info->MaximumStackSize = image_info.stack_size; - info->CommittedStackSize = image_info.stack_commit; - info->SubSystemType = image_info.subsystem; - info->SubsystemVersionLow = image_info.subsystem_low; - info->SubsystemVersionHigh = image_info.subsystem_high; - info->GpValue = image_info.gp; - info->ImageCharacteristics = image_info.image_charact; - info->DllCharacteristics = image_info.dll_charact; - info->Machine = image_info.machine; - info->ImageContainsCode = image_info.contains_code; - info->u.ImageFlags = image_info.image_flags; - info->LoaderFlags = image_info.loader_flags; - info->ImageFileSize = image_info.file_size; - info->CheckSum = image_info.checksum; + virtual_fill_image_information( &image_info, info ); if (ret_size) *ret_size = sizeof(*info); -#ifndef _WIN64 /* don't return 64-bit values to 32-bit processes */ - if (image_info.machine == IMAGE_FILE_MACHINE_AMD64 || - image_info.machine == IMAGE_FILE_MACHINE_ARM64) - { - info->TransferAddress = (void *)0x81231234; /* sic */ - info->MaximumStackSize = 0x100000; - info->CommittedStackSize = 0x10000; - } -#endif } else status = STATUS_SECTION_NOT_IMAGE; }

1 0