September 2020 - Wine-Commits

Zebediah Figura : server: Remove the no longer used create_socket request.
by Alexandre Julliard Sept. 22, 2020

Sept. 22, 2020

Module: wine Branch: master Commit: 1c84ac61e6674e2951691d97f27d75f667b00c97 URL: https://source.winehq.org/git/wine.git/?a=commit;h=1c84ac61e6674e2951691d97… Author: Zebediah Figura <z.figura12(a)gmail.com> Date: Sat Sep 19 17:13:38 2020 -0500 server: Remove the no longer used create_socket request. Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- include/wine/server_protocol.h | 25 +------------------------ server/protocol.def | 13 ------------- server/request.h | 11 ----------- server/sock.c | 14 -------------- server/trace.c | 18 ------------------ 5 files changed, 1 insertion(+), 80 deletions(-) diff --git a/include/wine/server_protocol.h b/include/wine/server_protocol.h index 2d4e016478..8e2ea08500 100644 --- a/include/wine/server_protocol.h +++ b/include/wine/server_protocol.h @@ -1692,26 +1692,6 @@ struct unlock_file_reply -struct create_socket_request -{ - struct request_header __header; - unsigned int access; - unsigned int attributes; - int family; - int type; - int protocol; - unsigned int flags; - char __pad_36[4]; -}; -struct create_socket_reply -{ - struct reply_header __header; - obj_handle_t handle; - char __pad_12[4]; -}; - - - struct accept_socket_request { struct request_header __header; @@ -5500,7 +5480,6 @@ enum request REQ_get_volume_info, REQ_lock_file, REQ_unlock_file, - REQ_create_socket, REQ_accept_socket, REQ_accept_into_socket, REQ_set_socket_event, @@ -5788,7 +5767,6 @@ union generic_request struct get_volume_info_request get_volume_info_request; struct lock_file_request lock_file_request; struct unlock_file_request unlock_file_request; - struct create_socket_request create_socket_request; struct accept_socket_request accept_socket_request; struct accept_into_socket_request accept_into_socket_request; struct set_socket_event_request set_socket_event_request; @@ -6074,7 +6052,6 @@ union generic_reply struct get_volume_info_reply get_volume_info_reply; struct lock_file_reply lock_file_reply; struct unlock_file_reply unlock_file_reply; - struct create_socket_reply create_socket_reply; struct accept_socket_reply accept_socket_reply; struct accept_into_socket_reply accept_into_socket_reply; struct set_socket_event_reply set_socket_event_reply; @@ -6305,7 +6282,7 @@ union generic_reply /* ### protocol_version begin ### */ -#define SERVER_PROTOCOL_VERSION 642 +#define SERVER_PROTOCOL_VERSION 643 /* ### protocol_version end ### */ diff --git a/server/protocol.def b/server/protocol.def index fd9b721d69..94ae63fec5 100644 --- a/server/protocol.def +++ b/server/protocol.def @@ -1380,19 +1380,6 @@ enum server_fd_type @END -/* Create a socket */ -(a)REQ(create_socket) - unsigned int access; /* wanted access rights */ - unsigned int attributes; /* object attributes */ - int family; /* family, see socket manpage */ - int type; /* type, see socket manpage */ - int protocol; /* protocol, see socket manpage */ - unsigned int flags; /* socket flags */ -(a)REPLY - obj_handle_t handle; /* handle to the new socket */ -(a)END - - /* Accept a socket */ @REQ(accept_socket) obj_handle_t lhandle; /* handle to the listening socket */ diff --git a/server/request.h b/server/request.h index 39f3ce7b2e..de82151698 100644 --- a/server/request.h +++ b/server/request.h @@ -173,7 +173,6 @@ DECL_HANDLER(get_file_info); DECL_HANDLER(get_volume_info); DECL_HANDLER(lock_file); DECL_HANDLER(unlock_file); -DECL_HANDLER(create_socket); DECL_HANDLER(accept_socket); DECL_HANDLER(accept_into_socket); DECL_HANDLER(set_socket_event); @@ -460,7 +459,6 @@ static const req_handler req_handlers[REQ_NB_REQUESTS] = (req_handler)req_get_volume_info, (req_handler)req_lock_file, (req_handler)req_unlock_file, - (req_handler)req_create_socket, (req_handler)req_accept_socket, (req_handler)req_accept_into_socket, (req_handler)req_set_socket_event, @@ -1052,15 +1050,6 @@ C_ASSERT( FIELD_OFFSET(struct unlock_file_request, handle) == 12 ); C_ASSERT( FIELD_OFFSET(struct unlock_file_request, offset) == 16 ); C_ASSERT( FIELD_OFFSET(struct unlock_file_request, count) == 24 ); C_ASSERT( sizeof(struct unlock_file_request) == 32 ); -C_ASSERT( FIELD_OFFSET(struct create_socket_request, access) == 12 ); -C_ASSERT( FIELD_OFFSET(struct create_socket_request, attributes) == 16 ); -C_ASSERT( FIELD_OFFSET(struct create_socket_request, family) == 20 ); -C_ASSERT( FIELD_OFFSET(struct create_socket_request, type) == 24 ); -C_ASSERT( FIELD_OFFSET(struct create_socket_request, protocol) == 28 ); -C_ASSERT( FIELD_OFFSET(struct create_socket_request, flags) == 32 ); -C_ASSERT( sizeof(struct create_socket_request) == 40 ); -C_ASSERT( FIELD_OFFSET(struct create_socket_reply, handle) == 8 ); -C_ASSERT( sizeof(struct create_socket_reply) == 16 ); C_ASSERT( FIELD_OFFSET(struct accept_socket_request, lhandle) == 12 ); C_ASSERT( FIELD_OFFSET(struct accept_socket_request, access) == 16 ); C_ASSERT( FIELD_OFFSET(struct accept_socket_request, attributes) == 20 ); diff --git a/server/sock.c b/server/sock.c index 7148702165..346d3758f3 100644 --- a/server/sock.c +++ b/server/sock.c @@ -1249,20 +1249,6 @@ struct object *create_socket_device( struct object *root, const struct unicode_s return create_named_object( root, &socket_device_ops, name, 0, NULL ); } -/* create a socket */ -DECL_HANDLER(create_socket) -{ - struct sock *sock; - - reply->handle = 0; - if ((sock = create_socket()) != NULL) - { - if (!init_socket( sock, req->family, req->type, req->protocol, req->flags )) - reply->handle = alloc_handle( current->process, &sock->obj, req->access, req->attributes ); - release_object( sock ); - } -} - /* accept a socket */ DECL_HANDLER(accept_socket) { diff --git a/server/trace.c b/server/trace.c index 707a952357..3aa6ecc1b7 100644 --- a/server/trace.c +++ b/server/trace.c @@ -1945,21 +1945,6 @@ static void dump_unlock_file_request( const struct unlock_file_request *req ) dump_uint64( ", count=", &req->count ); } -static void dump_create_socket_request( const struct create_socket_request *req ) -{ - fprintf( stderr, " access=%08x", req->access ); - fprintf( stderr, ", attributes=%08x", req->attributes ); - fprintf( stderr, ", family=%d", req->family ); - fprintf( stderr, ", type=%d", req->type ); - fprintf( stderr, ", protocol=%d", req->protocol ); - fprintf( stderr, ", flags=%08x", req->flags ); -} - -static void dump_create_socket_reply( const struct create_socket_reply *req ) -{ - fprintf( stderr, " handle=%04x", req->handle ); -} - static void dump_accept_socket_request( const struct accept_socket_request *req ) { fprintf( stderr, " lhandle=%04x", req->lhandle ); @@ -4479,7 +4464,6 @@ static const dump_func req_dumpers[REQ_NB_REQUESTS] = { (dump_func)dump_get_volume_info_request, (dump_func)dump_lock_file_request, (dump_func)dump_unlock_file_request, - (dump_func)dump_create_socket_request, (dump_func)dump_accept_socket_request, (dump_func)dump_accept_into_socket_request, (dump_func)dump_set_socket_event_request, @@ -4763,7 +4747,6 @@ static const dump_func reply_dumpers[REQ_NB_REQUESTS] = { (dump_func)dump_get_volume_info_reply, (dump_func)dump_lock_file_reply, NULL, - (dump_func)dump_create_socket_reply, (dump_func)dump_accept_socket_reply, NULL, NULL, @@ -5047,7 +5030,6 @@ static const char * const req_names[REQ_NB_REQUESTS] = { "get_volume_info", "lock_file", "unlock_file", - "create_socket", "accept_socket", "accept_into_socket", "set_socket_event",

1 0

Alexandre Julliard : server: Add an object operation to retrieve an object name.
by Alexandre Julliard Sept. 22, 2020

Sept. 22, 2020

1 0

Jacek Caban : conhost: Fix a leak in create_screen_buffer.
by Alexandre Julliard Sept. 22, 2020

Sept. 22, 2020

Module: wine Branch: master Commit: c840de2839f67793483bae633362b2daf00e3949 URL: https://source.winehq.org/git/wine.git/?a=commit;h=c840de2839f67793483bae63… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue Sep 22 16:46:09 2020 +0200 conhost: Fix a leak in create_screen_buffer. Based on patch by Haidong Yu. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- programs/conhost/conhost.c | 44 ++++++++++++++++++-------------------------- 1 file changed, 18 insertions(+), 26 deletions(-) diff --git a/programs/conhost/conhost.c b/programs/conhost/conhost.c index c0a0e08e6d..82cee8bdf0 100644 --- a/programs/conhost/conhost.c +++ b/programs/conhost/conhost.c @@ -166,41 +166,48 @@ static int screen_buffer_compare_id( const void *key, const struct wine_rb_entry static struct wine_rb_tree screen_buffer_map = { screen_buffer_compare_id }; +static void destroy_screen_buffer( struct screen_buffer *screen_buffer ) +{ + if (screen_buffer->console->active == screen_buffer) + screen_buffer->console->active = NULL; + wine_rb_remove( &screen_buffer_map, &screen_buffer->entry ); + free( screen_buffer->data ); + free( screen_buffer ); +} + static struct screen_buffer *create_screen_buffer( struct console *console, int id, int width, int height ) { struct screen_buffer *screen_buffer; unsigned int i; - if (!(screen_buffer = malloc( sizeof(*screen_buffer) ))) return NULL; + if (!(screen_buffer = calloc( 1, sizeof(*screen_buffer) ))) return NULL; screen_buffer->console = console; screen_buffer->id = id; screen_buffer->mode = ENABLE_PROCESSED_OUTPUT | ENABLE_WRAP_AT_EOL_OUTPUT; screen_buffer->cursor_size = 100; screen_buffer->cursor_visible = 1; - screen_buffer->cursor_x = 0; - screen_buffer->cursor_y = 0; screen_buffer->width = width; screen_buffer->height = height; screen_buffer->attr = 0x07; screen_buffer->popup_attr = 0xf5; screen_buffer->max_width = 80; screen_buffer->max_height = 25; - screen_buffer->win.left = 0; screen_buffer->win.right = min( screen_buffer->max_width - 1, width - 1 ); - screen_buffer->win.top = 0; screen_buffer->win.bottom = min( screen_buffer->max_height - 1, height - 1); - screen_buffer->font.width = 0; - screen_buffer->font.height = 0; screen_buffer->font.weight = FW_NORMAL; screen_buffer->font.pitch_family = FIXED_PITCH | FF_DONTCARE; - screen_buffer->font.face_name = NULL; - screen_buffer->font.face_len = 0; - memset( screen_buffer->color_map, 0, sizeof(screen_buffer->color_map) ); + + if (wine_rb_put( &screen_buffer_map, LongToPtr(id), &screen_buffer->entry )) + { + free( screen_buffer ); + ERR( "id %x already exists\n", id ); + return NULL; + } if (!(screen_buffer->data = malloc( screen_buffer->width * screen_buffer->height * sizeof(*screen_buffer->data) ))) { - free( screen_buffer ); + destroy_screen_buffer( screen_buffer ); return NULL; } @@ -211,24 +218,9 @@ static struct screen_buffer *create_screen_buffer( struct console *console, int memcpy( &screen_buffer->data[i * screen_buffer->width], screen_buffer->data, screen_buffer->width * sizeof(char_info_t) ); - if (wine_rb_put( &screen_buffer_map, LongToPtr(id), &screen_buffer->entry )) - { - ERR( "id %x already exists\n", id ); - return NULL; - } - return screen_buffer; } -static void destroy_screen_buffer( struct screen_buffer *screen_buffer ) -{ - if (screen_buffer->console->active == screen_buffer) - screen_buffer->console->active = NULL; - wine_rb_remove( &screen_buffer_map, &screen_buffer->entry ); - free( screen_buffer->data ); - free( screen_buffer ); -} - static BOOL is_active( struct screen_buffer *screen_buffer ) { return screen_buffer == screen_buffer->console->active;

1 0

Alexandre Julliard : server: Implement object name information for registry keys.
by Alexandre Julliard Sept. 22, 2020

Sept. 22, 2020

Module: wine Branch: master Commit: 96dcee64307b63ed3c07d13832fee15c3dc21ff1 URL: https://source.winehq.org/git/wine.git/?a=commit;h=96dcee64307b63ed3c07d138… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Sep 22 16:57:04 2020 +0200 server: Implement object name information for registry keys. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntdll/tests/om.c | 2 +- server/registry.c | 51 ++++++++++++++++++++++++++++++--------------------- 2 files changed, 31 insertions(+), 22 deletions(-) diff --git a/dlls/ntdll/tests/om.c b/dlls/ntdll/tests/om.c index 15f905330e..d3b932bec1 100644 --- a/dlls/ntdll/tests/om.c +++ b/dlls/ntdll/tests/om.c @@ -1470,7 +1470,7 @@ static void test_query_object(void) status = pNtCreateKey( &handle, KEY_READ, &attr, 0, 0, 0, 0 ); ok( status == STATUS_SUCCESS, "NtCreateKey failed status %x\n", status ); - test_object_name( handle, L"\\REGISTRY\\MACHINE", TRUE ); + test_object_name( handle, L"\\REGISTRY\\MACHINE", FALSE ); test_object_type( handle, L"Key" ); pNtClose( handle ); diff --git a/server/registry.c b/server/registry.c index f4386d9bfb..438ec6c6fc 100644 --- a/server/registry.c +++ b/server/registry.c @@ -148,6 +148,7 @@ static void key_dump( struct object *obj, int verbose ); static struct object_type *key_get_type( struct object *obj ); static unsigned int key_map_access( struct object *obj, unsigned int access ); static struct security_descriptor *key_get_sd( struct object *obj ); +static WCHAR *key_get_full_name( struct object *obj, data_size_t *len ); static int key_close_handle( struct object *obj, struct process *process, obj_handle_t handle ); static void key_destroy( struct object *obj ); @@ -165,7 +166,7 @@ static const struct object_ops key_ops = key_map_access, /* map_access */ key_get_sd, /* get_sd */ default_set_sd, /* set_sd */ - no_get_full_name, /* get_full_name */ + key_get_full_name, /* get_full_name */ no_lookup_name, /* lookup_name */ no_link_name, /* link_name */ NULL, /* unlink_name */ @@ -394,6 +395,29 @@ static struct security_descriptor *key_get_sd( struct object *obj ) return key_default_sd; } +static WCHAR *key_get_full_name( struct object *obj, data_size_t *ret_len ) +{ + static const WCHAR backslash = '\\'; + struct key *key = (struct key *) obj; + data_size_t len = sizeof(root_name) - sizeof(WCHAR); + char *ret; + + for (key = (struct key *)obj; key != root_key; key = key->parent) len += key->namelen + sizeof(WCHAR); + if (!(ret = malloc( len ))) return NULL; + + *ret_len = len; + key = (struct key *)obj; + for (key = (struct key *)obj; key != root_key; key = key->parent) + { + memcpy( ret + len - key->namelen, key->name, key->namelen ); + dump_strW( (WCHAR *)(ret + len - key->namelen), key->namelen, stderr, "" ); + len -= key->namelen + sizeof(WCHAR); + memcpy( ret + len, &backslash, sizeof(WCHAR) ); + } + memcpy( ret, root_name, sizeof(root_name) - sizeof(WCHAR) ); + return (WCHAR *)ret; +} + /* close the notification associated with a handle */ static int key_close_handle( struct object *obj, struct process *process, obj_handle_t handle ) { @@ -876,15 +900,13 @@ static struct key *create_key_recursive( struct key *key, const struct unicode_s } /* query information about a key or a subkey */ -static void enum_key( const struct key *key, int index, int info_class, - struct enum_key_reply *reply ) +static void enum_key( struct key *key, int index, int info_class, struct enum_key_reply *reply ) { - static const WCHAR backslash[] = { '\\' }; int i; data_size_t len, namelen, classlen; data_size_t max_subkey = 0, max_class = 0; data_size_t max_value = 0, max_data = 0; - const struct key *k; + WCHAR *fullname = NULL; char *data; if (index != -1) /* -1 means use the specified key directly */ @@ -903,11 +925,7 @@ static void enum_key( const struct key *key, int index, int info_class, switch(info_class) { case KeyNameInformation: - namelen = 0; - for (k = key; k != root_key; k = k->parent) - namelen += k->namelen + sizeof(backslash); - if (!namelen) return; - namelen += sizeof(root_name) - sizeof(backslash); + if (!(fullname = key->obj.ops->get_full_name( &key->obj, &namelen ))) return; /* fall through */ case KeyBasicInformation: classlen = 0; /* only return the name */ @@ -959,18 +977,8 @@ static void enum_key( const struct key *key, int index, int info_class, } else if (info_class == KeyNameInformation) { - data_size_t pos = namelen; reply->namelen = namelen; - for (k = key; k != root_key; k = k->parent) - { - pos -= k->namelen; - if (pos < len) memcpy( data + pos, k->name, - min( k->namelen, len - pos ) ); - pos -= sizeof(backslash); - if (pos < len) memcpy( data + pos, backslash, - min( sizeof(backslash), len - pos ) ); - } - memcpy( data, root_name, min( sizeof(root_name) - sizeof(backslash), len ) ); + memcpy( data, fullname, namelen ); } else { @@ -978,6 +986,7 @@ static void enum_key( const struct key *key, int index, int info_class, memcpy( data, key->name, len ); } } + free( fullname ); if (debug_level > 1) dump_operation( key, NULL, "Enum" ); }

1 0

Alexandre Julliard : server: Return the device name for named pipe files.
by Alexandre Julliard Sept. 22, 2020

Sept. 22, 2020

Module: wine Branch: master Commit: bdb2fe0cad3b34bb6c5b70926cc848f0b9e9e60f URL: https://source.winehq.org/git/wine.git/?a=commit;h=bdb2fe0cad3b34bb6c5b7092… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Sep 22 16:56:58 2020 +0200 server: Return the device name for named pipe files. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntdll/tests/om.c | 4 ++-- server/named_pipe.c | 20 +++++++++++++++++--- 2 files changed, 19 insertions(+), 5 deletions(-) diff --git a/dlls/ntdll/tests/om.c b/dlls/ntdll/tests/om.c index 788fa0bca1..15f905330e 100644 --- a/dlls/ntdll/tests/om.c +++ b/dlls/ntdll/tests/om.c @@ -1443,7 +1443,7 @@ static void test_query_object(void) 1, 1000, 1000, 1000, NULL ); ok( handle != INVALID_HANDLE_VALUE, "CreateNamedPipe failed err %u\n", GetLastError() ); - test_object_name( handle, L"\\Device\\NamedPipe\\test_pipe", TRUE ); + test_object_name( handle, L"\\Device\\NamedPipe\\test_pipe", FALSE ); test_object_type( handle, L"File" ); test_file_info( handle ); @@ -1460,7 +1460,7 @@ static void test_query_object(void) handle = CreateFileA( "\\\\.\\pipe", 0, 0, NULL, OPEN_EXISTING, 0, 0 ); ok( handle != INVALID_HANDLE_VALUE, "CreateFile failed (%d)\n", GetLastError() ); - test_object_name( handle, L"\\Device\\NamedPipe", TRUE ); + test_object_name( handle, L"\\Device\\NamedPipe", FALSE ); test_object_type( handle, L"File" ); test_file_info( handle ); diff --git a/server/named_pipe.c b/server/named_pipe.c index 4f9890258b..1ac35cf118 100644 --- a/server/named_pipe.c +++ b/server/named_pipe.c @@ -141,6 +141,7 @@ static struct fd *pipe_end_get_fd( struct object *obj ); static struct security_descriptor *pipe_end_get_sd( struct object *obj ); static int pipe_end_set_sd( struct object *obj, const struct security_descriptor *sd, unsigned int set_info ); +static WCHAR *pipe_end_get_full_name( struct object *obj, data_size_t *len ); static int pipe_end_read( struct fd *fd, struct async *async, file_pos_t pos ); static int pipe_end_write( struct fd *fd, struct async *async_data, file_pos_t pos ); static int pipe_end_flush( struct fd *fd, struct async *async ); @@ -167,7 +168,7 @@ static const struct object_ops pipe_server_ops = default_fd_map_access, /* map_access */ pipe_end_get_sd, /* get_sd */ pipe_end_set_sd, /* set_sd */ - no_get_full_name, /* get_full_name */ + pipe_end_get_full_name, /* get_full_name */ no_lookup_name, /* lookup_name */ no_link_name, /* link_name */ NULL, /* unlink_name */ @@ -210,7 +211,7 @@ static const struct object_ops pipe_client_ops = default_fd_map_access, /* map_access */ pipe_end_get_sd, /* get_sd */ pipe_end_set_sd, /* set_sd */ - no_get_full_name, /* get_full_name */ + pipe_end_get_full_name, /* get_full_name */ no_lookup_name, /* lookup_name */ no_link_name, /* link_name */ NULL, /* unlink_name */ @@ -269,6 +270,7 @@ static const struct object_ops named_pipe_device_ops = static void named_pipe_device_file_dump( struct object *obj, int verbose ); static struct fd *named_pipe_device_file_get_fd( struct object *obj ); +static WCHAR *named_pipe_device_file_get_full_name( struct object *obj, data_size_t *len ); static int named_pipe_device_ioctl( struct fd *fd, ioctl_code_t code, struct async *async ); static enum server_fd_type named_pipe_device_file_get_fd_type( struct fd *fd ); static void named_pipe_device_file_destroy( struct object *obj ); @@ -287,7 +289,7 @@ static const struct object_ops named_pipe_device_file_ops = default_fd_map_access, /* map_access */ default_get_sd, /* get_sd */ default_set_sd, /* set_sd */ - no_get_full_name, /* get_full_name */ + named_pipe_device_file_get_full_name, /* get_full_name */ no_lookup_name, /* lookup_name */ no_link_name, /* link_name */ NULL, /* unlink_name */ @@ -535,6 +537,12 @@ static struct fd *named_pipe_device_file_get_fd( struct object *obj ) return (struct fd *)grab_object( file->fd ); } +static WCHAR *named_pipe_device_file_get_full_name( struct object *obj, data_size_t *len ) +{ + struct named_pipe_device_file *file = (struct named_pipe_device_file *)obj; + return file->device->obj.ops->get_full_name( &file->device->obj, len ); +} + static enum server_fd_type named_pipe_device_file_get_fd_type( struct fd *fd ) { return FD_TYPE_DEVICE; @@ -704,6 +712,12 @@ static int pipe_end_set_sd( struct object *obj, const struct security_descriptor return 0; } +static WCHAR *pipe_end_get_full_name( struct object *obj, data_size_t *len ) +{ + struct pipe_end *pipe_end = (struct pipe_end *) obj; + return pipe_end->pipe->obj.ops->get_full_name( &pipe_end->pipe->obj, len ); +} + static void pipe_end_get_volume_info( struct fd *fd, unsigned int info_class ) { switch (info_class)

1 0

Alexandre Julliard : server: Return the device name for device files.
by Alexandre Julliard Sept. 22, 2020

Sept. 22, 2020

Module: wine Branch: master Commit: 5e9b1a3c6752e953baeaa10c13f1ef887ddfb471 URL: https://source.winehq.org/git/wine.git/?a=commit;h=5e9b1a3c6752e953baeaa10c… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Sep 22 16:56:42 2020 +0200 server: Return the device name for device files. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntoskrnl.exe/tests/ntoskrnl.c | 2 +- server/device.c | 9 ++++++++- 2 files changed, 9 insertions(+), 2 deletions(-) diff --git a/dlls/ntoskrnl.exe/tests/ntoskrnl.c b/dlls/ntoskrnl.exe/tests/ntoskrnl.c index 61b1e401c5..c2285aef6f 100644 --- a/dlls/ntoskrnl.exe/tests/ntoskrnl.c +++ b/dlls/ntoskrnl.exe/tests/ntoskrnl.c @@ -511,7 +511,7 @@ static void test_object_info(void) status = NtQueryObject(device, ObjectNameInformation, buffer, sizeof(buffer), NULL); ok(!status, "got %#x\n", status); - todo_wine ok(compare_unicode_string(name_info->Name.Buffer, name_info->Name.Length, L"\\Device\\WineTestDriver"), + ok(compare_unicode_string(name_info->Name.Buffer, name_info->Name.Length, L"\\Device\\WineTestDriver"), "wrong name %s\n", debugstr_w(name_info->Name.Buffer)); status = NtQueryObject(device, ObjectTypeInformation, buffer, sizeof(buffer), NULL); diff --git a/server/device.c b/server/device.c index 4241f37e89..371943acee 100644 --- a/server/device.c +++ b/server/device.c @@ -184,6 +184,7 @@ struct device_file static void device_file_dump( struct object *obj, int verbose ); static struct fd *device_file_get_fd( struct object *obj ); +static WCHAR *device_file_get_full_name( struct object *obj, data_size_t *len ); static struct list *device_file_get_kernel_obj_list( struct object *obj ); static int device_file_close_handle( struct object *obj, struct process *process, obj_handle_t handle ); static void device_file_destroy( struct object *obj ); @@ -208,7 +209,7 @@ static const struct object_ops device_file_ops = default_fd_map_access, /* map_access */ default_get_sd, /* get_sd */ default_set_sd, /* set_sd */ - no_get_full_name, /* get_full_name */ + device_file_get_full_name, /* get_full_name */ no_lookup_name, /* lookup_name */ no_link_name, /* link_name */ NULL, /* unlink_name */ @@ -506,6 +507,12 @@ static struct fd *device_file_get_fd( struct object *obj ) return (struct fd *)grab_object( file->fd ); } +static WCHAR *device_file_get_full_name( struct object *obj, data_size_t *len ) +{ + struct device_file *file = (struct device_file *)obj; + return file->device->obj.ops->get_full_name( &file->device->obj, len ); +} + static struct list *device_file_get_kernel_obj_list( struct object *obj ) { struct device_file *file = (struct device_file *)obj;

1 0

Nikolay Sivov : include: Add an entry for ProcessIoPriority.
by Alexandre Julliard Sept. 22, 2020

Sept. 22, 2020

Module: wine Branch: master Commit: 7e564c43043d6a68eb7a631b80219d59951ba2d2 URL: https://source.winehq.org/git/wine.git/?a=commit;h=7e564c43043d6a68eb7a631b… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Tue Sep 22 13:24:18 2020 +0300 include: Add an entry for ProcessIoPriority. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/kernel32/tests/process.c | 2 +- include/winternl.h | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/dlls/kernel32/tests/process.c b/dlls/kernel32/tests/process.c index b0371c615e..8996dd9144 100644 --- a/dlls/kernel32/tests/process.c +++ b/dlls/kernel32/tests/process.c @@ -3685,7 +3685,7 @@ static void test_process_info(HANDLE hproc) case ProcessPriorityClass: case ProcessPriorityBoost: case ProcessLUIDDeviceMapsEnabled: - case 33 /* ProcessIoPriority */: + case ProcessIoPriority: case ProcessIoCounters: case ProcessVmCounters: case ProcessWow64Information: diff --git a/include/winternl.h b/include/winternl.h index e23a02b9b2..20b2a44ba2 100644 --- a/include/winternl.h +++ b/include/winternl.h @@ -1153,6 +1153,7 @@ typedef enum _PROCESSINFOCLASS { ProcessDebugObjectHandle = 30, ProcessDebugFlags = 31, ProcessHandleTracing = 32, + ProcessIoPriority = 33, ProcessExecuteFlags = 34, ProcessTlsInformation = 35, ProcessCookie = 36,

1 0

Nikolay Sivov : kernel32/tests: Link to more functions directly.
by Alexandre Julliard Sept. 22, 2020

Sept. 22, 2020

Module: wine Branch: master Commit: bce9cdcc9812a1a22bd744c64309ba83b4eb4e89 URL: https://source.winehq.org/git/wine.git/?a=commit;h=bce9cdcc9812a1a22bd744c6… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Tue Sep 22 13:24:19 2020 +0300 kernel32/tests: Link to more functions directly. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/kernel32/tests/actctx.c | 4 +- dlls/kernel32/tests/environ.c | 7 +-- dlls/kernel32/tests/file.c | 38 +++++---------- dlls/kernel32/tests/pipe.c | 12 ++--- dlls/kernel32/tests/process.c | 28 ++--------- dlls/kernel32/tests/timer.c | 25 ++-------- dlls/kernel32/tests/virtual.c | 28 ++++------- dlls/kernel32/tests/volume.c | 107 +++++++++++++----------------------------- 8 files changed, 71 insertions(+), 178 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=bce9cdcc9812a1a22bd7…

1 0

Nikolay Sivov : server: Implement inherited handles list.
by Alexandre Julliard Sept. 22, 2020

Sept. 22, 2020

Module: wine Branch: master Commit: c58a10c16395c30e7793cde1f748febe0432a6ad URL: https://source.winehq.org/git/wine.git/?a=commit;h=c58a10c16395c30e7793cde1… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Tue Sep 22 13:24:21 2020 +0300 server: Implement inherited handles list. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/kernel32/tests/process.c | 1 - dlls/ntdll/unix/process.c | 39 +++++++++++++++++++++++++++++++++++++++ include/wine/server_protocol.h | 5 +++-- server/handle.c | 38 ++++++++++++++++++++++++++++++-------- server/handle.h | 3 ++- server/process.c | 24 ++++++++++++++++++++---- server/process.h | 3 ++- server/protocol.def | 2 ++ server/request.c | 2 +- server/request.h | 1 + server/trace.c | 2 ++ 11 files changed, 102 insertions(+), 18 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=c58a10c16395c30e7793…

1 0

Zebediah Figura : server: Introduce IOCTL_AFD_CREATE.
by Alexandre Julliard Sept. 22, 2020

Sept. 22, 2020

Module: wine Branch: master Commit: 29334489795f0f3fbca52b65b7c5a002983a997b URL: https://source.winehq.org/git/wine.git/?a=commit;h=29334489795f0f3fbca52b65… Author: Zebediah Figura <z.figura12(a)gmail.com> Date: Sat Sep 19 17:13:36 2020 -0500 server: Introduce IOCTL_AFD_CREATE. Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- include/wine/afd.h | 34 ++++++++++++++++++++++++++++++++++ server/sock.c | 19 ++++++++++++++++++- 2 files changed, 52 insertions(+), 1 deletion(-) diff --git a/include/wine/afd.h b/include/wine/afd.h new file mode 100644 index 0000000000..41caaa7b4f --- /dev/null +++ b/include/wine/afd.h @@ -0,0 +1,34 @@ +/* + * Socket driver ioctls + * + * Copyright 2020 Zebediah Figura 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 + */ + +#ifndef __WINE_WINE_AFD_H +#define __WINE_WINE_AFD_H + +#include <winioctl.h> + +#define IOCTL_AFD_CREATE CTL_CODE(FILE_DEVICE_NETWORK, 200, METHOD_BUFFERED, FILE_WRITE_ACCESS) + +struct afd_create_params +{ + int family, type, protocol; + unsigned int flags; +}; + +#endif diff --git a/server/sock.c b/server/sock.c index fd26c6c845..7148702165 100644 --- a/server/sock.c +++ b/server/sock.c @@ -58,6 +58,7 @@ #include "winerror.h" #define USE_WS_PREFIX #include "winsock2.h" +#include "wine/afd.h" #include "process.h" #include "file.h" @@ -133,6 +134,7 @@ static int sock_ioctl( struct fd *fd, ioctl_code_t code, struct async *async ); static void sock_queue_async( struct fd *fd, struct async *async, int type, int count ); static void sock_reselect_async( struct fd *fd, struct async_queue *queue ); +static int init_socket( struct sock *sock, int family, int type, int protocol, unsigned int flags ); static int sock_get_ntstatus( int err ); static unsigned int sock_get_error( int err ); @@ -542,10 +544,23 @@ static int sock_ioctl( struct fd *fd, ioctl_code_t code, struct async *async ) assert( sock->obj.ops == &sock_ops ); - if (get_unix_fd( fd ) == -1) return 0; + if (get_unix_fd( fd ) == -1 && code != IOCTL_AFD_CREATE) return 0; switch(code) { + case IOCTL_AFD_CREATE: + { + const struct afd_create_params *params = get_req_data(); + + if (get_req_data_size() != sizeof(*params)) + { + set_error( STATUS_INVALID_PARAMETER ); + return 0; + } + init_socket( sock, params->family, params->type, params->protocol, params->flags ); + return 0; + } + case WS_SIO_ADDRESS_LIST_CHANGE: if ((sock->state & FD_WINE_NONBLOCKING) && async_is_blocking( async )) { @@ -680,6 +695,8 @@ static int init_socket( struct sock *sock, int family, int type, int protocol, u sock->type = type; sock->family = family; + if (sock->fd) release_object( sock->fd ); + if (!(sock->fd = create_anonymous_fd( &sock_fd_ops, sockfd, &sock->obj, (flags & WSA_FLAG_OVERLAPPED) ? 0 : FILE_SYNCHRONOUS_IO_NONALERT ))) {

1 0