wine-commits February 2022

wine-commits@winehq.org

2 participants
1495 discussions

Huw Davies : winealsa: Store the session master volume in the stream.
by Alexandre Julliard 21 Feb '22

21 Feb '22

Module: wine Branch: master Commit: 4f8a2e456b805159ffd459fbdc23bb4c76e018da URL: https://source.winehq.org/git/wine.git/?a=commit;h=4f8a2e456b805159ffd459fb… Author: Huw Davies <huw(a)codeweavers.com> Date: Fri Feb 18 08:29:30 2022 +0000 winealsa: Store the session master volume in the stream. Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Andrew Eikum <aeikum(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/winealsa.drv/mmdevdrv.c | 18 ++++++++++-------- 1 file changed, 10 insertions(+), 8 deletions(-) diff --git a/dlls/winealsa.drv/mmdevdrv.c b/dlls/winealsa.drv/mmdevdrv.c index 222decdb150..959157ade87 100644 --- a/dlls/winealsa.drv/mmdevdrv.c +++ b/dlls/winealsa.drv/mmdevdrv.c @@ -327,7 +327,7 @@ static void set_stream_volumes(ACImpl *This) unsigned int i; for(i = 0; i < stream->fmt->nChannels; i++) - stream->vols[i] = This->vols[i] * This->session->channel_vols[i]; + stream->vols[i] = This->vols[i] * This->session->channel_vols[i] * This->session->master_vol; } HRESULT WINAPI AUDDRV_GetEndpointIDs(EDataFlow flow, WCHAR ***ids_out, GUID **guids_out, @@ -1472,7 +1472,6 @@ static BYTE *remap_channels(struct alsa_stream *stream, BYTE *buf, snd_pcm_ufram static void adjust_buffer_volume(const ACImpl *This, BYTE *buf, snd_pcm_uframes_t frames, BOOL mute) { struct alsa_stream *stream = This->stream; - float vol[ARRAY_SIZE(stream->alsa_channel_map)]; BOOL adjust = FALSE; UINT32 i, channels; BYTE *end; @@ -1492,8 +1491,7 @@ static void adjust_buffer_volume(const ACImpl *This, BYTE *buf, snd_pcm_uframes_ /* Adjust the buffer based on the volume for each channel */ for (i = 0; i < channels; i++) { - vol[i] = stream->vols[i] * This->session->master_vol; - adjust |= vol[i] != 1.0f; + adjust |= stream->vols[i] != 1.0f; } if (!adjust) return; @@ -1510,7 +1508,7 @@ static void adjust_buffer_volume(const ACImpl *This, BYTE *buf, snd_pcm_uframes_ do \ { \ for (i = 0; i < channels; i++) \ - p[i] = p[i] * vol[i]; \ + p[i] = p[i] * stream->vols[i]; \ p += i; \ } while ((BYTE*)p != end); \ } while (0) @@ -1548,7 +1546,7 @@ static void adjust_buffer_volume(const ACImpl *This, BYTE *buf, snd_pcm_uframes_ v[3] = q[2] & ~0xff; for (k = 0; k < 4; k++) { - v[k] = (INT32)((INT32)v[k] * vol[i]); + v[k] = (INT32)((INT32)v[k] * stream->vols[i]); v[k] &= mask; if (++i == channels) i = 0; } @@ -1559,7 +1557,7 @@ static void adjust_buffer_volume(const ACImpl *This, BYTE *buf, snd_pcm_uframes_ p = (BYTE*)q; while (p != end) { - UINT32 v = (INT32)((INT32)(p[0] << 8 | p[1] << 16 | p[2] << 24) * vol[i]); + UINT32 v = (INT32)((INT32)(p[0] << 8 | p[1] << 16 | p[2] << 24) * stream->vols[i]); v &= mask; *p++ = v >> 8 & 0xff; *p++ = v >> 16 & 0xff; @@ -1575,7 +1573,7 @@ static void adjust_buffer_volume(const ACImpl *This, BYTE *buf, snd_pcm_uframes_ do { for (i = 0; i < channels; i++) - p[i] = (int)((p[i] - 128) * vol[i]) + 128; + p[i] = (int)((p[i] - 128) * stream->vols[i]) + 128; p += i; } while ((BYTE*)p != end); break; @@ -3086,6 +3084,7 @@ static HRESULT WINAPI SimpleAudioVolume_SetMasterVolume( { AudioSessionWrapper *This = impl_from_ISimpleAudioVolume(iface); AudioSession *session = This->session; + ACImpl *client; TRACE("(%p)->(%f, %s)\n", session, level, wine_dbgstr_guid(context)); @@ -3101,6 +3100,9 @@ static HRESULT WINAPI SimpleAudioVolume_SetMasterVolume( session->master_vol = level; + LIST_FOR_EACH_ENTRY(client, &session->clients, ACImpl, entry) + set_stream_volumes(client); + LeaveCriticalSection(&g_sessions_lock); return S_OK;

1 0

Huw Davies : winealsa: Store the session channel vols in the stream.
by Alexandre Julliard 21 Feb '22

21 Feb '22

Module: wine Branch: master Commit: a4de96a1f0d2d17329e8f94084e328069d73c6e7 URL: https://source.winehq.org/git/wine.git/?a=commit;h=a4de96a1f0d2d17329e8f940… Author: Huw Davies <huw(a)codeweavers.com> Date: Fri Feb 18 08:29:29 2022 +0000 winealsa: Store the session channel vols in the stream. Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Andrew Eikum <aeikum(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/winealsa.drv/mmdevdrv.c | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/dlls/winealsa.drv/mmdevdrv.c b/dlls/winealsa.drv/mmdevdrv.c index 8f198531cc0..222decdb150 100644 --- a/dlls/winealsa.drv/mmdevdrv.c +++ b/dlls/winealsa.drv/mmdevdrv.c @@ -327,7 +327,7 @@ static void set_stream_volumes(ACImpl *This) unsigned int i; for(i = 0; i < stream->fmt->nChannels; i++) - stream->vols[i] = This->vols[i]; + stream->vols[i] = This->vols[i] * This->session->channel_vols[i]; } HRESULT WINAPI AUDDRV_GetEndpointIDs(EDataFlow flow, WCHAR ***ids_out, GUID **guids_out, @@ -1493,7 +1493,6 @@ static void adjust_buffer_volume(const ACImpl *This, BYTE *buf, snd_pcm_uframes_ for (i = 0; i < channels; i++) { vol[i] = stream->vols[i] * This->session->master_vol; - vol[i] *= This->session->channel_vols[i]; adjust |= vol[i] != 1.0f; } if (!adjust) return; @@ -3375,6 +3374,7 @@ static HRESULT WINAPI ChannelAudioVolume_SetChannelVolume( { AudioSessionWrapper *This = impl_from_IChannelAudioVolume(iface); AudioSession *session = This->session; + ACImpl *client; TRACE("(%p)->(%d, %f, %s)\n", session, index, level, wine_dbgstr_guid(context)); @@ -3394,6 +3394,9 @@ static HRESULT WINAPI ChannelAudioVolume_SetChannelVolume( session->channel_vols[index] = level; + LIST_FOR_EACH_ENTRY(client, &session->clients, ACImpl, entry) + set_stream_volumes(client); + LeaveCriticalSection(&g_sessions_lock); return S_OK; @@ -3425,6 +3428,7 @@ static HRESULT WINAPI ChannelAudioVolume_SetAllVolumes( AudioSessionWrapper *This = impl_from_IChannelAudioVolume(iface); AudioSession *session = This->session; unsigned int i; + ACImpl *client; TRACE("(%p)->(%d, %p, %s)\n", session, count, levels, wine_dbgstr_guid(context)); @@ -3445,6 +3449,9 @@ static HRESULT WINAPI ChannelAudioVolume_SetAllVolumes( for(i = 0; i < count; ++i) session->channel_vols[i] = levels[i]; + LIST_FOR_EACH_ENTRY(client, &session->clients, ACImpl, entry) + set_stream_volumes(client); + LeaveCriticalSection(&g_sessions_lock); return S_OK;

1 0

Huw Davies : winealsa: Remove an unnecessary check on the session channel count.
by Alexandre Julliard 21 Feb '22

21 Feb '22

Module: wine Branch: master Commit: 8130371614b9845f23a935912493d501bbfb6574 URL: https://source.winehq.org/git/wine.git/?a=commit;h=8130371614b9845f23a93591… Author: Huw Davies <huw(a)codeweavers.com> Date: Fri Feb 18 08:29:28 2022 +0000 winealsa: Remove an unnecessary check on the session channel count. This must always be at least equal to any given stream's channels. Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Andrew Eikum <aeikum(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/winealsa.drv/mmdevdrv.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/dlls/winealsa.drv/mmdevdrv.c b/dlls/winealsa.drv/mmdevdrv.c index d1f9a1d35a4..8f198531cc0 100644 --- a/dlls/winealsa.drv/mmdevdrv.c +++ b/dlls/winealsa.drv/mmdevdrv.c @@ -1491,13 +1491,11 @@ static void adjust_buffer_volume(const ACImpl *This, BYTE *buf, snd_pcm_uframes_ /* Adjust the buffer based on the volume for each channel */ for (i = 0; i < channels; i++) - vol[i] = stream->vols[i] * This->session->master_vol; - for (i = 0; i < min(channels, This->session->channel_count); i++) { + vol[i] = stream->vols[i] * This->session->master_vol; vol[i] *= This->session->channel_vols[i]; adjust |= vol[i] != 1.0f; } - while (i < channels) adjust |= vol[i++] != 1.0f; if (!adjust) return; /* Skip the frames we've already adjusted before */

1 0

Huw Davies : winealsa: Store the channel volumes in the stream.
by Alexandre Julliard 21 Feb '22

21 Feb '22

Module: wine Branch: master Commit: f608d78910894de739d4f7bba08f03f179b270e5 URL: https://source.winehq.org/git/wine.git/?a=commit;h=f608d78910894de739d4f7bb… Author: Huw Davies <huw(a)codeweavers.com> Date: Fri Feb 18 08:29:27 2022 +0000 winealsa: Store the channel volumes in the stream. Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Andrew Eikum <aeikum(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/winealsa.drv/mmdevdrv.c | 24 +++++++++++++++++++++++- dlls/winealsa.drv/unixlib.h | 1 + 2 files changed, 24 insertions(+), 1 deletion(-) diff --git a/dlls/winealsa.drv/mmdevdrv.c b/dlls/winealsa.drv/mmdevdrv.c index 48694710719..d1f9a1d35a4 100644 --- a/dlls/winealsa.drv/mmdevdrv.c +++ b/dlls/winealsa.drv/mmdevdrv.c @@ -321,6 +321,15 @@ static snd_pcm_stream_t alsa_get_direction(EDataFlow flow) return (flow == eRender) ? SND_PCM_STREAM_PLAYBACK : SND_PCM_STREAM_CAPTURE; } +static void set_stream_volumes(ACImpl *This) +{ + struct alsa_stream *stream = This->stream; + unsigned int i; + + for(i = 0; i < stream->fmt->nChannels; i++) + stream->vols[i] = This->vols[i]; +} + HRESULT WINAPI AUDDRV_GetEndpointIDs(EDataFlow flow, WCHAR ***ids_out, GUID **guids_out, UINT *num, UINT *def_index) { @@ -591,6 +600,7 @@ static ULONG WINAPI AudioClient_Release(IAudioClient3 *iface) HeapFree(GetProcessHeap(), 0, stream->tmp_buffer); HeapFree(GetProcessHeap(), 0, stream->hw_params); CoTaskMemFree(stream->fmt); + HeapFree(GetProcessHeap(), 0, stream->vols); HeapFree(GetProcessHeap(), 0, stream); } HeapFree(GetProcessHeap(), 0, This); @@ -1171,6 +1181,14 @@ static HRESULT WINAPI AudioClient_Initialize(IAudioClient3 *iface, for(i = 0; i < This->channel_count; ++i) This->vols[i] = 1.f; + stream->vols = HeapAlloc(GetProcessHeap(), 0, fmt->nChannels * sizeof(float)); + if(!stream->vols){ + hr = E_OUTOFMEMORY; + goto exit; + } + for(i = 0; i < fmt->nChannels; ++i) + stream->vols[i] = 1.f; + stream->share = mode; stream->flags = flags; @@ -1194,11 +1212,13 @@ exit: stream->local_buffer = NULL; CoTaskMemFree(stream->fmt); stream->fmt = NULL; + HeapFree(GetProcessHeap(), 0, stream->vols); HeapFree(GetProcessHeap(), 0, stream); HeapFree(GetProcessHeap(), 0, This->vols); This->vols = NULL; }else{ This->stream = stream; + set_stream_volumes(This); } LeaveCriticalSection(&This->lock); @@ -1471,7 +1491,7 @@ static void adjust_buffer_volume(const ACImpl *This, BYTE *buf, snd_pcm_uframes_ /* Adjust the buffer based on the volume for each channel */ for (i = 0; i < channels; i++) - vol[i] = This->vols[i] * This->session->master_vol; + vol[i] = stream->vols[i] * This->session->master_vol; for (i = 0; i < min(channels, This->session->channel_count); i++) { vol[i] *= This->session->channel_vols[i]; @@ -3214,6 +3234,7 @@ static HRESULT WINAPI AudioStreamVolume_SetChannelVolume( EnterCriticalSection(&This->lock); This->vols[index] = level; + set_stream_volumes(This); LeaveCriticalSection(&This->lock); @@ -3258,6 +3279,7 @@ static HRESULT WINAPI AudioStreamVolume_SetAllVolumes( for(i = 0; i < count; ++i) This->vols[i] = levels[i]; + set_stream_volumes(This); LeaveCriticalSection(&This->lock); diff --git a/dlls/winealsa.drv/unixlib.h b/dlls/winealsa.drv/unixlib.h index cd8d303b5d3..f121763952c 100644 --- a/dlls/winealsa.drv/unixlib.h +++ b/dlls/winealsa.drv/unixlib.h @@ -49,6 +49,7 @@ struct alsa_stream BYTE *local_buffer, *tmp_buffer, *remapping_buf, *silence_buf; LONG32 getbuf_last; /* <0 when using tmp_buffer */ + float *vols; }; struct endpoint

1 0

Zebediah Figura : wined3d: Introduce a wined3d_texture_update_sub_resource() helper.
by Alexandre Julliard 21 Feb '22

21 Feb '22

Module: wine Branch: master Commit: 81068b89d8e37a5fb82fe18432104bb92bb0b384 URL: https://source.winehq.org/git/wine.git/?a=commit;h=81068b89d8e37a5fb82fe184… Author: Zebediah Figura <zfigura(a)codeweavers.com> Date: Sun Feb 20 23:22:23 2022 -0600 wined3d: Introduce a wined3d_texture_update_sub_resource() helper. Signed-off-by: Zebediah Figura <zfigura(a)codeweavers.com> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/cs.c | 30 ++---------------------------- dlls/wined3d/texture.c | 26 ++++++++++++++++++++++++++ dlls/wined3d/wined3d_private.h | 3 +++ 3 files changed, 31 insertions(+), 28 deletions(-) diff --git a/dlls/wined3d/cs.c b/dlls/wined3d/cs.c index 7280739b8b2..f294be29091 100644 --- a/dlls/wined3d/cs.c +++ b/dlls/wined3d/cs.c @@ -2739,43 +2739,17 @@ static void wined3d_cs_exec_update_sub_resource(struct wined3d_cs *cs, const voi const struct wined3d_cs_update_sub_resource *op = data; struct wined3d_resource *resource = op->resource; const struct wined3d_box *box = &op->box; - unsigned int width, height, depth, level; struct wined3d_context *context; - struct wined3d_texture *texture; - struct wined3d_box src_box; context = context_acquire(cs->c.device, NULL, 0); if (resource->type == WINED3D_RTYPE_BUFFER) - { wined3d_buffer_update_sub_resource(buffer_from_resource(resource), context, &op->bo, box->left, box->right - box->left); - goto done; - } - - texture = wined3d_texture_from_resource(resource); - - level = op->sub_resource_idx % texture->level_count; - width = wined3d_texture_get_level_width(texture, level); - height = wined3d_texture_get_level_height(texture, level); - depth = wined3d_texture_get_level_depth(texture, level); - - /* Only load the sub-resource for partial updates. */ - if (!box->left && !box->top && !box->front - && box->right == width && box->bottom == height && box->back == depth) - wined3d_texture_prepare_location(texture, op->sub_resource_idx, context, WINED3D_LOCATION_TEXTURE_RGB); else - wined3d_texture_load_location(texture, op->sub_resource_idx, context, WINED3D_LOCATION_TEXTURE_RGB); - - wined3d_box_set(&src_box, 0, 0, box->right - box->left, box->bottom - box->top, 0, box->back - box->front); - texture->texture_ops->texture_upload_data(context, &op->bo.addr, texture->resource.format, &src_box, - op->row_pitch, op->slice_pitch, texture, op->sub_resource_idx, - WINED3D_LOCATION_TEXTURE_RGB, box->left, box->top, box->front); - - wined3d_texture_validate_location(texture, op->sub_resource_idx, WINED3D_LOCATION_TEXTURE_RGB); - wined3d_texture_invalidate_location(texture, op->sub_resource_idx, ~WINED3D_LOCATION_TEXTURE_RGB); + wined3d_texture_update_sub_resource(texture_from_resource(resource), + op->sub_resource_idx, context, &op->bo, box, op->row_pitch, op->slice_pitch); -done: context_release(context); wined3d_resource_release(resource); diff --git a/dlls/wined3d/texture.c b/dlls/wined3d/texture.c index 1323d533eff..3bbdb492998 100644 --- a/dlls/wined3d/texture.c +++ b/dlls/wined3d/texture.c @@ -4533,6 +4533,32 @@ void wined3d_texture_download_from_texture(struct wined3d_texture *dst_texture, wined3d_texture_invalidate_location(dst_texture, dst_sub_resource_idx, ~dst_location); } +void wined3d_texture_update_sub_resource(struct wined3d_texture *texture, unsigned int sub_resource_idx, + struct wined3d_context *context, const struct upload_bo *upload_bo, const struct wined3d_box *box, + unsigned int row_pitch, unsigned int slice_pitch) +{ + unsigned int level = sub_resource_idx % texture->level_count; + unsigned int width = wined3d_texture_get_level_width(texture, level); + unsigned int height = wined3d_texture_get_level_height(texture, level); + unsigned int depth = wined3d_texture_get_level_depth(texture, level); + struct wined3d_box src_box; + + /* Only load the sub-resource for partial updates. */ + if (!box->left && !box->top && !box->front + && box->right == width && box->bottom == height && box->back == depth) + wined3d_texture_prepare_location(texture, sub_resource_idx, context, WINED3D_LOCATION_TEXTURE_RGB); + else + wined3d_texture_load_location(texture, sub_resource_idx, context, WINED3D_LOCATION_TEXTURE_RGB); + + wined3d_box_set(&src_box, 0, 0, box->right - box->left, box->bottom - box->top, 0, box->back - box->front); + texture->texture_ops->texture_upload_data(context, &upload_bo->addr, texture->resource.format, + &src_box, row_pitch, slice_pitch, texture, sub_resource_idx, + WINED3D_LOCATION_TEXTURE_RGB, box->left, box->top, box->front); + + wined3d_texture_validate_location(texture, sub_resource_idx, WINED3D_LOCATION_TEXTURE_RGB); + wined3d_texture_invalidate_location(texture, sub_resource_idx, ~WINED3D_LOCATION_TEXTURE_RGB); +} + static void wined3d_texture_no3d_upload_data(struct wined3d_context *context, const struct wined3d_const_bo_address *src_bo_addr, const struct wined3d_format *src_format, const struct wined3d_box *src_box, unsigned int src_row_pitch, unsigned int src_slice_pitch, diff --git a/dlls/wined3d/wined3d_private.h b/dlls/wined3d/wined3d_private.h index 989fd33d2bf..277d5a94fb0 100644 --- a/dlls/wined3d/wined3d_private.h +++ b/dlls/wined3d/wined3d_private.h @@ -4643,6 +4643,9 @@ BOOL wined3d_texture_can_use_pbo(const struct wined3d_texture *texture, const st void wined3d_texture_sub_resources_destroyed(struct wined3d_texture *texture) DECLSPEC_HIDDEN; void wined3d_texture_translate_drawable_coords(const struct wined3d_texture *texture, HWND window, RECT *rect) DECLSPEC_HIDDEN; +void wined3d_texture_update_sub_resource(struct wined3d_texture *texture, unsigned int sub_resource_idx, + struct wined3d_context *context, const struct upload_bo *upload_bo, const struct wined3d_box *box, + unsigned int row_pitch, unsigned int slice_pitch) DECLSPEC_HIDDEN; void wined3d_texture_upload_from_texture(struct wined3d_texture *dst_texture, unsigned int dst_sub_resource_idx, unsigned int dst_x, unsigned int dst_y, unsigned int dst_z, struct wined3d_texture *src_texture, unsigned int src_sub_resource_idx, const struct wined3d_box *src_box) DECLSPEC_HIDDEN;

1 0

Alexandre Julliard : ntdll: Load the apiset schema at startup.
by Alexandre Julliard 21 Feb '22

21 Feb '22

Module: wine Branch: master Commit: e3c9d73cd1d092542d846a368196e26cd5c44972 URL: https://source.winehq.org/git/wine.git/?a=commit;h=e3c9d73cd1d092542d846a36… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Feb 21 22:23:03 2022 +0100 ntdll: Load the apiset schema at startup. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntdll/unix/loader.c | 77 ++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 77 insertions(+) diff --git a/dlls/ntdll/unix/loader.c b/dlls/ntdll/unix/loader.c index 96301b1654e..98a5c933215 100644 --- a/dlls/ntdll/unix/loader.c +++ b/dlls/ntdll/unix/loader.c @@ -1992,6 +1992,82 @@ static void load_ntdll(void) } +/*********************************************************************** + * load_apiset_dll + */ +static void load_apiset_dll(void) +{ + static WCHAR path[] = {'\\','?','?','\\','C',':','\\','w','i','n','d','o','w','s','\\', + 's','y','s','t','e','m','3','2','\\', + 'a','p','i','s','e','t','s','c','h','e','m','a','.','d','l','l',0}; + const char *pe_dir = get_pe_dir( current_machine ); + const IMAGE_NT_HEADERS *nt; + const IMAGE_SECTION_HEADER *sec; + API_SET_NAMESPACE *map; + OBJECT_ATTRIBUTES attr; + UNICODE_STRING str; + NTSTATUS status; + HANDLE handle, mapping; + SIZE_T size; + char *name; + void *ptr; + UINT i; + + init_unicode_string( &str, path ); + InitializeObjectAttributes( &attr, &str, 0, 0, NULL ); + + name = malloc( strlen( ntdll_dir ) + strlen( pe_dir ) + sizeof("/apisetschema.dll") ); + if (build_dir) sprintf( name, "%s/dlls/apisetschema/apisetschema.dll", build_dir ); + else sprintf( name, "%s%s/apisetschema.dll", dll_dir, pe_dir ); + 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 ); + free( name ); + + if (!status) + { + status = NtCreateSection( &mapping, STANDARD_RIGHTS_REQUIRED | SECTION_QUERY | SECTION_MAP_READ, + NULL, NULL, PAGE_READONLY, SEC_COMMIT, handle ); + NtClose( handle ); + } + if (!status) + { + ptr = NULL; + size = 0; + status = NtMapViewOfSection( mapping, NtCurrentProcess(), &ptr, + is_win64 && wow_peb ? 0x7fffffff : 0, 0, NULL, + &size, ViewShare, 0, PAGE_READONLY ); + NtClose( mapping ); + } + if (!status) + { + nt = get_rva( ptr, ((IMAGE_DOS_HEADER *)ptr)->e_lfanew ); + sec = (IMAGE_SECTION_HEADER *)((char *)&nt->OptionalHeader + nt->FileHeader.SizeOfOptionalHeader); + + for (i = 0; i < nt->FileHeader.NumberOfSections; i++, sec++) + { + if (memcmp( (char *)sec->Name, ".apiset", 8 )) continue; + map = (API_SET_NAMESPACE *)((char *)ptr + sec->PointerToRawData); + if (sec->PointerToRawData < size && + size - sec->PointerToRawData >= sec->Misc.VirtualSize && + map->Version == 6 && + map->Size <= sec->Misc.VirtualSize) + { + NtCurrentTeb()->Peb->ApiSetMap = map; + if (wow_peb) wow_peb->ApiSetMap = PtrToUlong(map); + NtUnmapViewOfSection( NtCurrentProcess(), ptr ); + TRACE( "loaded %s apiset at %p\n", debugstr_w(path), map ); + return; + } + break; + } + NtUnmapViewOfSection( NtCurrentProcess(), ptr ); + status = STATUS_APISET_NOT_PRESENT; + } + ERR( "failed to load apiset: %x\n", status ); +} + + /*********************************************************************** * load_wow64_ntdll */ @@ -2098,6 +2174,7 @@ static void start_main_thread(void) NtCreateKeyedEvent( &keyed_event, GENERIC_READ | GENERIC_WRITE, NULL, 0 ); load_ntdll(); if (main_image_info.Machine != current_machine) load_wow64_ntdll( main_image_info.Machine ); + load_apiset_dll(); ntdll_init_syscalls( 0, &syscall_table, p__wine_syscall_dispatcher ); status = p__wine_set_unix_funcs( NTDLL_UNIXLIB_VERSION, &unix_funcs ); if (status == STATUS_REVISION_MISMATCH)

1 0

Alexandre Julliard : ntdll: Add a global variable for the WoW PEB.
by Alexandre Julliard 21 Feb '22

21 Feb '22

Module: wine Branch: master Commit: a31f3374e65c02f97829eb9ed712a35f484c4461 URL: https://source.winehq.org/git/wine.git/?a=commit;h=a31f3374e65c02f97829eb9e… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Feb 21 11:41:44 2022 +0100 ntdll: Add a global variable for the WoW PEB. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntdll/unix/env.c | 44 ++++++++++++++++++++---------------------- dlls/ntdll/unix/server.c | 1 + dlls/ntdll/unix/unix_private.h | 4 ++++ 3 files changed, 26 insertions(+), 23 deletions(-) diff --git a/dlls/ntdll/unix/env.c b/dlls/ntdll/unix/env.c index c7e0674e083..3f9f7f1b3b7 100644 --- a/dlls/ntdll/unix/env.c +++ b/dlls/ntdll/unix/env.c @@ -60,6 +60,7 @@ WINE_DEFAULT_DEBUG_CHANNEL(environ); PEB *peb = NULL; +WOW_PEB *wow_peb = NULL; USHORT *uctable = NULL, *lctable = NULL; SIZE_T startup_info_size = 0; BOOL is_prefix_bootstrap = FALSE; @@ -2129,37 +2130,34 @@ static void init_peb( RTL_USER_PROCESS_PARAMETERS *params, void *module ) { NtCurrentTeb()->WowTebOffset = teb_offset; NtCurrentTeb()->Tib.ExceptionList = (void *)((char *)NtCurrentTeb() + teb_offset); + wow_peb = (PEB32 *)((char *)peb + page_size); set_thread_id( NtCurrentTeb(), GetCurrentProcessId(), GetCurrentThreadId() ); } #endif load_global_options( &params->ImagePathName ); - if (NtCurrentTeb()->WowTebOffset) + if (wow_peb) { void *wow64_params = build_wow64_parameters( params ); -#ifdef _WIN64 - PEB32 *wow64_peb = (PEB32 *)((char *)peb + page_size); -#else - PEB64 *wow64_peb = (PEB64 *)((char *)peb - page_size); -#endif - wow64_peb->ImageBaseAddress = PtrToUlong( peb->ImageBaseAddress ); - wow64_peb->ProcessParameters = PtrToUlong( wow64_params ); - wow64_peb->NumberOfProcessors = peb->NumberOfProcessors; - wow64_peb->NtGlobalFlag = peb->NtGlobalFlag; - wow64_peb->CriticalSectionTimeout.QuadPart = peb->CriticalSectionTimeout.QuadPart; - wow64_peb->HeapSegmentReserve = peb->HeapSegmentReserve; - wow64_peb->HeapSegmentCommit = peb->HeapSegmentCommit; - wow64_peb->HeapDeCommitTotalFreeThreshold = peb->HeapDeCommitTotalFreeThreshold; - wow64_peb->HeapDeCommitFreeBlockThreshold = peb->HeapDeCommitFreeBlockThreshold; - wow64_peb->OSMajorVersion = peb->OSMajorVersion; - wow64_peb->OSMinorVersion = peb->OSMinorVersion; - wow64_peb->OSBuildNumber = peb->OSBuildNumber; - wow64_peb->OSPlatformId = peb->OSPlatformId; - wow64_peb->ImageSubSystem = peb->ImageSubSystem; - wow64_peb->ImageSubSystemMajorVersion = peb->ImageSubSystemMajorVersion; - wow64_peb->ImageSubSystemMinorVersion = peb->ImageSubSystemMinorVersion; - wow64_peb->SessionId = peb->SessionId; + + wow_peb->ImageBaseAddress = PtrToUlong( peb->ImageBaseAddress ); + wow_peb->ProcessParameters = PtrToUlong( wow64_params ); + wow_peb->NumberOfProcessors = peb->NumberOfProcessors; + wow_peb->NtGlobalFlag = peb->NtGlobalFlag; + wow_peb->CriticalSectionTimeout.QuadPart = peb->CriticalSectionTimeout.QuadPart; + wow_peb->HeapSegmentReserve = peb->HeapSegmentReserve; + wow_peb->HeapSegmentCommit = peb->HeapSegmentCommit; + wow_peb->HeapDeCommitTotalFreeThreshold = peb->HeapDeCommitTotalFreeThreshold; + wow_peb->HeapDeCommitFreeBlockThreshold = peb->HeapDeCommitFreeBlockThreshold; + wow_peb->OSMajorVersion = peb->OSMajorVersion; + wow_peb->OSMinorVersion = peb->OSMinorVersion; + wow_peb->OSBuildNumber = peb->OSBuildNumber; + wow_peb->OSPlatformId = peb->OSPlatformId; + wow_peb->ImageSubSystem = peb->ImageSubSystem; + wow_peb->ImageSubSystemMajorVersion = peb->ImageSubSystemMajorVersion; + wow_peb->ImageSubSystemMinorVersion = peb->ImageSubSystemMinorVersion; + wow_peb->SessionId = peb->SessionId; } } diff --git a/dlls/ntdll/unix/server.c b/dlls/ntdll/unix/server.c index 9d0594d3374..ab9d99c4f47 100644 --- a/dlls/ntdll/unix/server.c +++ b/dlls/ntdll/unix/server.c @@ -1518,6 +1518,7 @@ size_t server_init_process(void) is_wow64 = TRUE; NtCurrentTeb()->GdiBatchCount = PtrToUlong( (char *)NtCurrentTeb() - teb_offset ); NtCurrentTeb()->WowTebOffset = -teb_offset; + wow_peb = (PEB64 *)((char *)peb - page_size); #endif } else diff --git a/dlls/ntdll/unix/unix_private.h b/dlls/ntdll/unix/unix_private.h index 75f03706401..b1595c792c0 100644 --- a/dlls/ntdll/unix/unix_private.h +++ b/dlls/ntdll/unix/unix_private.h @@ -393,12 +393,16 @@ static inline client_ptr_t iosb_client_ptr( IO_STATUS_BLOCK *io ) #ifdef _WIN64 typedef TEB32 WOW_TEB; +typedef PEB32 WOW_PEB; static inline TEB64 *NtCurrentTeb64(void) { return NULL; } #else typedef TEB64 WOW_TEB; +typedef PEB64 WOW_PEB; static inline TEB64 *NtCurrentTeb64(void) { return (TEB64 *)NtCurrentTeb()->GdiBatchCount; } #endif +extern WOW_PEB *wow_peb DECLSPEC_HIDDEN; + static inline WOW_TEB *get_wow_teb( TEB *teb ) { return teb->WowTebOffset ? (WOW_TEB *)((char *)teb + teb->WowTebOffset) : NULL;

1 0

Alexandre Julliard : include: Add API set definitions.
by Alexandre Julliard 21 Feb '22

21 Feb '22

Module: wine Branch: master Commit: 0d09268558e4ad26a74d0554af3e7a9671ee3681 URL: https://source.winehq.org/git/wine.git/?a=commit;h=0d09268558e4ad26a74d0554… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Feb 21 11:40:43 2022 +0100 include: Add API set definitions. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- include/winternl.h | 44 +++++++++++++++++++++++++++++++++++++++++--- 1 file changed, 41 insertions(+), 3 deletions(-) diff --git a/include/winternl.h b/include/winternl.h index 81fd5f45665..ae331562026 100644 --- a/include/winternl.h +++ b/include/winternl.h @@ -321,7 +321,7 @@ typedef struct _PEB PVOID KernelCallbackTable; /* 02c/058 */ ULONG Reserved; /* 030/060 */ ULONG AtlThunkSListPtr32; /* 034/064 */ - PVOID /*PPEB_FREE_BLOCK*/ FreeList; /* 038/068 */ + PVOID ApiSetMap; /* 038/068 */ ULONG TlsExpansionCounter; /* 03c/070 */ PRTL_BITMAP TlsBitmap; /* 040/078 */ ULONG TlsBitmapBits[2]; /* 044/080 */ @@ -731,7 +731,7 @@ typedef struct _PEB32 ULONG KernelCallbackTable; /* 002c */ ULONG Reserved; /* 0030 */ ULONG AtlThunkSListPtr32; /* 0034 */ - ULONG FreeList; /* 0038 */ + ULONG ApiSetMap; /* 0038 */ ULONG TlsExpansionCounter; /* 003c */ ULONG TlsBitmap; /* 0040 */ ULONG TlsBitmapBits[2]; /* 0044 */ @@ -822,7 +822,7 @@ typedef struct _PEB64 ULONG64 KernelCallbackTable; /* 0058 */ ULONG Reserved; /* 0060 */ ULONG AtlThunkSListPtr32; /* 0064 */ - ULONG64 FreeList; /* 0068 */ + ULONG64 ApiSetMap; /* 0068 */ ULONG TlsExpansionCounter; /* 0070 */ ULONG64 TlsBitmap; /* 0078 */ ULONG TlsBitmapBits[2]; /* 0080 */ @@ -3824,6 +3824,44 @@ typedef struct BYTE *ArgumentTable; } SYSTEM_SERVICE_TABLE; +/* ApiSet structures (format for version 6) */ + +typedef struct _API_SET_NAMESPACE +{ + ULONG Version; + ULONG Size; + ULONG Flags; + ULONG Count; + ULONG EntryOffset; + ULONG HashOffset; + ULONG HashFactor; +} API_SET_NAMESPACE; + +typedef struct _API_SET_HASH_ENTRY +{ + ULONG Hash; + ULONG Index; +} API_SET_HASH_ENTRY; + +typedef struct _API_SET_NAMESPACE_ENTRY +{ + ULONG Flags; + ULONG NameOffset; + ULONG NameLength; + ULONG HashedLength; + ULONG ValueOffset; + ULONG ValueCount; +} API_SET_NAMESPACE_ENTRY; + +typedef struct _API_SET_VALUE_ENTRY +{ + ULONG Flags; + ULONG NameOffset; + ULONG NameLength; + ULONG ValueOffset; + ULONG ValueLength; +} API_SET_VALUE_ENTRY; + /*********************************************************************** * Function declarations */

1 0

Eric Pouech : user32/tests: Don't outdent todo_wine*.
by Alexandre Julliard 21 Feb '22

21 Feb '22

Module: wine Branch: master Commit: ccf6204cf70071aa9215402f2e6b3f65aa359a86 URL: https://source.winehq.org/git/wine.git/?a=commit;h=ccf6204cf70071aa9215402f… Author: Eric Pouech <eric.pouech(a)gmail.com> Date: Mon Feb 21 08:08:20 2022 +0100 user32/tests: Don't outdent todo_wine*. This prevents -Wmisleading-indentation warnings (Mingw GCC11). Signed-off-by: Eric Pouech <eric.pouech(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/user32/tests/scroll.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/user32/tests/scroll.c b/dlls/user32/tests/scroll.c index 57bf759879c..40eb13943fc 100644 --- a/dlls/user32/tests/scroll.c +++ b/dlls/user32/tests/scroll.c @@ -358,7 +358,7 @@ static void scrollbar_test_default( DWORD style) if( !(style & WS_VSCROLL)) { if (bThemeActive || style != WS_HSCROLL) -todo_wine + todo_wine ok( (winstyle & (WS_HSCROLL|WS_VSCROLL)) == ( style | WS_VSCROLL), "unexpected style change %08x/%08x\n", winstyle, style); else @@ -394,7 +394,7 @@ todo_wine if( !(style & WS_HSCROLL)) { if (bThemeActive || style != WS_VSCROLL) -todo_wine + todo_wine ok( (winstyle & (WS_HSCROLL|WS_VSCROLL)) == ( style | WS_HSCROLL), "unexpected style change %08x/%08x\n", winstyle, style); else

1 0

Eric Pouech : shell32/tests: Don't outdent todo_wine*.
by Alexandre Julliard 21 Feb '22

21 Feb '22

Module: wine Branch: master Commit: 0fc80c7db107a9a7bbf21fbdb3b79377b03b168e URL: https://source.winehq.org/git/wine.git/?a=commit;h=0fc80c7db107a9a7bbf21fbd… Author: Eric Pouech <eric.pouech(a)gmail.com> Date: Mon Feb 21 08:08:15 2022 +0100 shell32/tests: Don't outdent todo_wine*. This prevents -Wmisleading-indentation warnings (Mingw GCC11). Signed-off-by: Eric Pouech <eric.pouech(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/shell32/tests/shelldispatch.c | 2 +- dlls/shell32/tests/shlfolder.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/shell32/tests/shelldispatch.c b/dlls/shell32/tests/shelldispatch.c index f243ba053d0..860024ae6fa 100644 --- a/dlls/shell32/tests/shelldispatch.c +++ b/dlls/shell32/tests/shelldispatch.c @@ -1146,7 +1146,7 @@ static void test_ShellWindows(void) IDispatch_Release(app); hr = IWebBrowser2_get_Document(wb, &doc); -todo_wine + todo_wine ok(hr == S_OK, "got 0x%08x\n", hr); if (hr == S_OK) { test_dispatch_typeinfo(doc, viewdual_riids); diff --git a/dlls/shell32/tests/shlfolder.c b/dlls/shell32/tests/shlfolder.c index f593f8b821d..13108f92246 100644 --- a/dlls/shell32/tests/shlfolder.c +++ b/dlls/shell32/tests/shlfolder.c @@ -4423,7 +4423,7 @@ static void test_contextmenu(IContextMenu *menu, BOOL background) cmi.lpVerb = "foobar_wine_test"; hr = IContextMenu_InvokeCommand(menu, &cmi); - todo_wine_if(background) + todo_wine_if(background) ok((hr == E_INVALIDARG) || (hr == E_FAIL /* Win7 */) || (hr == HRESULT_FROM_WIN32(ERROR_NO_ASSOCIATION) /* Vista */), "Unexpected hr %#x.\n", hr);

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

wine-commits February 2022