February 2022 - Wine-commits

Huw Davies : winealsa: Pass a stream ptr to alsa_{read|write}_data().
by Alexandre Julliard 21 Feb '22

21 Feb '22

Module: wine Branch: master Commit: 02d0071724d599be663a77de9fc68090438d934c URL: https://source.winehq.org/git/wine.git/?a=commit;h=02d0071724d599be663a77de… Author: Huw Davies <huw(a)codeweavers.com> Date: Fri Feb 18 08:29:32 2022 +0000 winealsa: Pass a stream ptr to alsa_{read|write}_data(). 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 | 34 ++++++++++++++-------------------- 1 file changed, 14 insertions(+), 20 deletions(-) diff --git a/dlls/winealsa.drv/mmdevdrv.c b/dlls/winealsa.drv/mmdevdrv.c index 14dfbd07acb..c9a7c331158 100644 --- a/dlls/winealsa.drv/mmdevdrv.c +++ b/dlls/winealsa.drv/mmdevdrv.c @@ -1470,9 +1470,8 @@ static BYTE *remap_channels(struct alsa_stream *stream, BYTE *buf, snd_pcm_ufram return stream->remapping_buf; } -static void adjust_buffer_volume(const ACImpl *This, BYTE *buf, snd_pcm_uframes_t frames) +static void adjust_buffer_volume(const struct alsa_stream *stream, BYTE *buf, snd_pcm_uframes_t frames) { - struct alsa_stream *stream = This->stream; BOOL adjust = FALSE; UINT32 i, channels, mute = 0; BYTE *end; @@ -1587,12 +1586,11 @@ static void adjust_buffer_volume(const ACImpl *This, BYTE *buf, snd_pcm_uframes_ } } -static snd_pcm_sframes_t alsa_write_best_effort(ACImpl *This, BYTE *buf, snd_pcm_uframes_t frames) +static snd_pcm_sframes_t alsa_write_best_effort(struct alsa_stream *stream, BYTE *buf, snd_pcm_uframes_t frames) { - struct alsa_stream *stream = This->stream; snd_pcm_sframes_t written; - adjust_buffer_volume(This, buf, frames); + adjust_buffer_volume(stream, buf, frames); /* Mark the frames we've already adjusted */ if (stream->vol_adjusted_frames < frames) @@ -1625,11 +1623,10 @@ static snd_pcm_sframes_t alsa_write_best_effort(ACImpl *This, BYTE *buf, snd_pcm return written; } -static snd_pcm_sframes_t alsa_write_buffer_wrap(ACImpl *This, BYTE *buf, +static snd_pcm_sframes_t alsa_write_buffer_wrap(struct alsa_stream *stream, BYTE *buf, snd_pcm_uframes_t buflen, snd_pcm_uframes_t offs, snd_pcm_uframes_t to_write) { - struct alsa_stream *stream = This->stream; snd_pcm_sframes_t ret = 0; while(to_write){ @@ -1641,7 +1638,7 @@ static snd_pcm_sframes_t alsa_write_buffer_wrap(ACImpl *This, BYTE *buf, else chunk = to_write; - tmp = alsa_write_best_effort(This, buf + offs * stream->fmt->nBlockAlign, chunk); + tmp = alsa_write_best_effort(stream, buf + offs * stream->fmt->nBlockAlign, chunk); if(tmp < 0) return ret; if(!tmp) @@ -1663,9 +1660,8 @@ static UINT buf_ptr_diff(UINT left, UINT right, UINT bufsize) return bufsize - (left - right); } -static UINT data_not_in_alsa(ACImpl *This) +static UINT data_not_in_alsa(struct alsa_stream *stream) { - struct alsa_stream *stream = This->stream; UINT32 diff; diff = buf_ptr_diff(stream->lcl_offs_frames, stream->wri_offs_frames, stream->bufsize_frames); @@ -1692,9 +1688,8 @@ static UINT data_not_in_alsa(ACImpl *This) * * During Stop, we rewind the ALSA buffer */ -static void alsa_write_data(ACImpl *This) +static void alsa_write_data(struct alsa_stream *stream) { - struct alsa_stream *stream = This->stream; snd_pcm_sframes_t written; snd_pcm_uframes_t avail, max_copy_frames, data_frames_played; int err; @@ -1723,13 +1718,13 @@ static void alsa_write_data(ACImpl *This) * continuous rendering. Additional benefit: Force ALSA to start. */ if(stream->data_in_alsa_frames == 0 && stream->held_frames < stream->alsa_period_frames) { - alsa_write_best_effort(This, stream->silence_buf, + alsa_write_best_effort(stream, stream->silence_buf, stream->alsa_period_frames - stream->held_frames); stream->vol_adjusted_frames = 0; } if(stream->started) - max_copy_frames = data_not_in_alsa(This); + max_copy_frames = data_not_in_alsa(stream); else max_copy_frames = 0; @@ -1747,7 +1742,7 @@ static void alsa_write_data(ACImpl *This) to_write = min(avail, max_copy_frames); - written = alsa_write_buffer_wrap(This, stream->local_buffer, + written = alsa_write_buffer_wrap(stream, stream->local_buffer, stream->bufsize_frames, stream->lcl_offs_frames, to_write); if(written <= 0) break; @@ -1763,9 +1758,8 @@ static void alsa_write_data(ACImpl *This) SetEvent(stream->event); } -static void alsa_read_data(ACImpl *This) +static void alsa_read_data(struct alsa_stream *stream) { - struct alsa_stream *stream = This->stream; snd_pcm_sframes_t nread; UINT32 pos = stream->wri_offs_frames, limit = stream->held_frames; unsigned int i; @@ -1833,9 +1827,9 @@ static void CALLBACK alsa_push_buffer_data(void *user, BOOLEAN timer) QueryPerformanceCounter(&stream->last_period_time); if(This->dataflow == eRender) - alsa_write_data(This); + alsa_write_data(stream); else if(This->dataflow == eCapture) - alsa_read_data(This); + alsa_read_data(stream); LeaveCriticalSection(&This->lock); } @@ -1923,7 +1917,7 @@ static HRESULT WINAPI AudioClient_Start(IAudioClient3 *iface) offs = stream->wri_offs_frames - stream->held_frames; /* fill it with data */ - written = alsa_write_buffer_wrap(This, stream->local_buffer, + written = alsa_write_buffer_wrap(stream, stream->local_buffer, stream->bufsize_frames, offs, avail); if(written > 0){

1 0

Huw Davies : winealsa: Keep a copy of the dataflow in the stream.
by Alexandre Julliard 21 Feb '22

21 Feb '22

Module: wine Branch: master Commit: a63ca0eb0b95484ebf7897ece52e79d6b243fa80 URL: https://source.winehq.org/git/wine.git/?a=commit;h=a63ca0eb0b95484ebf7897ec… Author: Huw Davies <huw(a)codeweavers.com> Date: Fri Feb 18 08:29:33 2022 +0000 winealsa: Keep a copy of the dataflow 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 | 15 ++++++++------- dlls/winealsa.drv/unixlib.h | 1 + 2 files changed, 9 insertions(+), 7 deletions(-) diff --git a/dlls/winealsa.drv/mmdevdrv.c b/dlls/winealsa.drv/mmdevdrv.c index c9a7c331158..5bb3bdbcc73 100644 --- a/dlls/winealsa.drv/mmdevdrv.c +++ b/dlls/winealsa.drv/mmdevdrv.c @@ -1030,6 +1030,7 @@ static HRESULT WINAPI AudioClient_Initialize(IAudioClient3 *iface, } stream->alsa_format = format; + stream->flow = This->dataflow; rate = fmt->nSamplesPerSec; if((err = snd_pcm_hw_params_set_rate_near(stream->pcm_handle, stream->hw_params, @@ -1276,7 +1277,7 @@ static HRESULT WINAPI AudioClient_GetStreamLatency(IAudioClient3 *iface, * + mmdevapi_period such that at the end of it, ALSA still has data; * + EXTRA_SAFE (~4ms) to allow for late callback invocation / fluctuation; * + alsa_period such that ALSA always has at least one period to play. */ - if(This->dataflow == eRender) + if(stream->flow == eRender) *latency = MulDiv(stream->hidden_frames, 10000000, stream->fmt->nSamplesPerSec); else *latency = MulDiv(stream->alsa_period_frames, 10000000, stream->fmt->nSamplesPerSec) @@ -1826,9 +1827,9 @@ static void CALLBACK alsa_push_buffer_data(void *user, BOOLEAN timer) QueryPerformanceCounter(&stream->last_period_time); - if(This->dataflow == eRender) + if(stream->flow == eRender) alsa_write_data(stream); - else if(This->dataflow == eCapture) + else if(stream->flow == eCapture) alsa_read_data(stream); LeaveCriticalSection(&This->lock); @@ -1900,7 +1901,7 @@ static HRESULT WINAPI AudioClient_Start(IAudioClient3 *iface) return AUDCLNT_E_NOT_STOPPED; } - if(This->dataflow == eCapture){ + if(stream->flow == eCapture){ /* dump any data that might be leftover in the ALSA capture buffer */ snd_pcm_readi(stream->pcm_handle, stream->local_buffer, stream->bufsize_frames); @@ -1964,7 +1965,7 @@ static HRESULT WINAPI AudioClient_Stop(IAudioClient3 *iface) return S_FALSE; } - if(This->dataflow == eRender) + if(stream->flow == eRender) alsa_rewind_best_effort(This); stream->started = FALSE; @@ -2007,7 +2008,7 @@ static HRESULT WINAPI AudioClient_Reset(IAudioClient3 *iface) if(snd_pcm_prepare(stream->pcm_handle) < 0) WARN("snd_pcm_prepare failed\n"); - if(This->dataflow == eRender){ + if(stream->flow == eRender){ stream->written_frames = 0; stream->last_pos_frames = 0; }else{ @@ -2675,7 +2676,7 @@ static HRESULT WINAPI AudioClock_GetPosition(IAudioClock *iface, UINT64 *pos, snd_pcm_avail_update(stream->pcm_handle); alsa_state = snd_pcm_state(stream->pcm_handle); - if(This->dataflow == eRender){ + if(stream->flow == eRender){ position = stream->written_frames - stream->held_frames; if(stream->started && alsa_state == SND_PCM_STATE_RUNNING && stream->held_frames) diff --git a/dlls/winealsa.drv/unixlib.h b/dlls/winealsa.drv/unixlib.h index f121763952c..c0dc625e27f 100644 --- a/dlls/winealsa.drv/unixlib.h +++ b/dlls/winealsa.drv/unixlib.h @@ -30,6 +30,7 @@ struct alsa_stream WAVEFORMATEX *fmt; DWORD flags; AUDCLNT_SHAREMODE share; + EDataFlow flow; HANDLE event; BOOL need_remapping;

1 0

Santino Mazza : ncrypt: Validate existence of property before creating one.
by Alexandre Julliard 21 Feb '22

21 Feb '22

Module: wine Branch: master Commit: e08c2f35f8c8a1a9947b13bb1a696a918c665868 URL: https://source.winehq.org/git/wine.git/?a=commit;h=e08c2f35f8c8a1a9947b13bb… Author: Santino Mazza <mazzasantino1206(a)gmail.com> Date: Mon Feb 21 12:13:41 2022 +0100 ncrypt: Validate existence of property before creating one. Signed-off-by: Santino Mazza <mazzasantino1206(a)gmail.com> Signed-off-by: Hans Leidekker <hans(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ncrypt/main.c | 27 +++++++++++++++++++-------- 1 file changed, 19 insertions(+), 8 deletions(-) diff --git a/dlls/ncrypt/main.c b/dlls/ncrypt/main.c index 4f9a2f357bd..f8ae2d7a796 100644 --- a/dlls/ncrypt/main.c +++ b/dlls/ncrypt/main.c @@ -144,12 +144,12 @@ SECURITY_STATUS WINAPI NCryptFreeObject(NCRYPT_HANDLE handle) return ret; } -static const struct object_property *get_object_property(struct object *object, const WCHAR *name) +static struct object_property *get_object_property(struct object *object, const WCHAR *name) { unsigned int i; for (i = 0; i < object->num_properties; i++) { - const struct object_property *property = &object->properties[i]; + struct object_property *property = &object->properties[i]; if (!lstrcmpW(property->key, name)) return property; } return NULL; @@ -181,17 +181,16 @@ static struct object *allocate_object(enum object_type type) return ret; } -static SECURITY_STATUS set_object_property(struct object *object, const WCHAR *name, BYTE *value, DWORD value_size) +struct object_property *add_object_property(struct object *object, const WCHAR *name) { struct object_property *property; - FIXME("check duplicates\n"); if (!object->num_properties) { if (!(object->properties = malloc(sizeof(*property)))) { ERR("Error allocating memory.\n"); - return NTE_NO_MEMORY; + return NULL; } property = &object->properties[object->num_properties++]; } @@ -201,7 +200,7 @@ static SECURITY_STATUS set_object_property(struct object *object, const WCHAR *n if (!(tmp = realloc(object->properties, sizeof(*property) * (object->num_properties + 1)))) { ERR("Error allocating memory.\n"); - return NTE_NO_MEMORY; + return NULL; } object->properties = tmp; property = &object->properties[object->num_properties++]; @@ -211,12 +210,22 @@ static SECURITY_STATUS set_object_property(struct object *object, const WCHAR *n if (!(property->key = malloc((lstrlenW(name) + 1) * sizeof(WCHAR)))) { ERR("Error allocating memory.\n"); - return NTE_NO_MEMORY; + return NULL; } lstrcpyW(property->key, name); + return property; +} + +static SECURITY_STATUS set_object_property(struct object *object, const WCHAR *name, BYTE *value, DWORD value_size) +{ + struct object_property *property = get_object_property(object, name); + void *tmp; + + if (!property && !(property = add_object_property(object, name))) return NTE_NO_MEMORY; + property->value_size = value_size; - if (!(property->value = malloc(value_size))) + if (!(tmp = realloc(property->value, value_size))) { ERR("Error allocating memory.\n"); free(property->key); @@ -224,7 +233,9 @@ static SECURITY_STATUS set_object_property(struct object *object, const WCHAR *n return NTE_NO_MEMORY; } + property->value = tmp; memcpy(property->value, value, value_size); + return ERROR_SUCCESS; }

1 0

Santino Mazza : ncrypt: Check null handle when setting or getting properties.
by Alexandre Julliard 21 Feb '22

21 Feb '22

Module: wine Branch: master Commit: ebf4953e90fddc672f55b498cbfb66ba7e8fef15 URL: https://source.winehq.org/git/wine.git/?a=commit;h=ebf4953e90fddc672f55b498… Author: Santino Mazza <mazzasantino1206(a)gmail.com> Date: Mon Feb 21 12:13:42 2022 +0100 ncrypt: Check null handle when setting or getting properties. Signed-off-by: Santino Mazza <mazzasantino1206(a)gmail.com> Signed-off-by: Hans Leidekker <hans(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ncrypt/main.c | 3 +++ dlls/ncrypt/tests/ncrypt.c | 3 +++ 2 files changed, 6 insertions(+) diff --git a/dlls/ncrypt/main.c b/dlls/ncrypt/main.c index f8ae2d7a796..68d9d884618 100644 --- a/dlls/ncrypt/main.c +++ b/dlls/ncrypt/main.c @@ -164,7 +164,9 @@ SECURITY_STATUS WINAPI NCryptGetProperty(NCRYPT_HANDLE handle, const WCHAR *name TRACE("(%#Ix, %s, %p, %lu, %p, %#lx)\n", handle, wine_dbgstr_w(name), output, outsize, result, flags); if (flags) FIXME("flags %#lx not supported\n", flags); + if (!object) return NTE_INVALID_HANDLE; if (!(property = get_object_property(object, name))) return NTE_INVALID_PARAMETER; + *result = property->value_size; if (!output) return ERROR_SUCCESS; if (outsize < property->value_size) return NTE_BUFFER_TOO_SMALL; @@ -376,6 +378,7 @@ SECURITY_STATUS WINAPI NCryptSetProperty(NCRYPT_HANDLE handle, const WCHAR *name TRACE("(%#Ix, %s, %p, %lu, %#lx)\n", handle, wine_dbgstr_w(name), input, insize, flags); if (flags) FIXME("flags %#lx not supported\n", flags); + if (!object) return NTE_INVALID_HANDLE; return set_object_property(object, name, input, insize); } diff --git a/dlls/ncrypt/tests/ncrypt.c b/dlls/ncrypt/tests/ncrypt.c index 891e5b60254..7ca20b2bfa6 100644 --- a/dlls/ncrypt/tests/ncrypt.c +++ b/dlls/ncrypt/tests/ncrypt.c @@ -199,6 +199,9 @@ static void test_get_property(void) ok(ret == ERROR_SUCCESS, "got %#lx\n", ret); ok(keylength == 1024, "got %lu\n", keylength); + ret = NCryptGetProperty(0, NCRYPT_LENGTH_PROPERTY, (BYTE *)&keylength, size, &size, 0); + ok(ret == NTE_INVALID_HANDLE, "got %#lx\n", ret); + NCryptFreeObject(prov); }

1 0

Santino Mazza : ncrypt/tests: Test for NCryptCreatePersistedKey.
by Alexandre Julliard 21 Feb '22

21 Feb '22

Module: wine Branch: master Commit: bac61d050595c0a6522ba583fd8371413d89be73 URL: https://source.winehq.org/git/wine.git/?a=commit;h=bac61d050595c0a6522ba583… Author: Santino Mazza <mazzasantino1206(a)gmail.com> Date: Mon Feb 21 12:13:43 2022 +0100 ncrypt/tests: Test for NCryptCreatePersistedKey. Signed-off-by: Santino Mazza <mazzasantino1206(a)gmail.com> Signed-off-by: Hans Leidekker <hans(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ncrypt/tests/ncrypt.c | 50 ++++++++++++++++++++++++++++++++++++++++++++-- 1 file changed, 48 insertions(+), 2 deletions(-) diff --git a/dlls/ncrypt/tests/ncrypt.c b/dlls/ncrypt/tests/ncrypt.c index 7ca20b2bfa6..58173fd0ff0 100644 --- a/dlls/ncrypt/tests/ncrypt.c +++ b/dlls/ncrypt/tests/ncrypt.c @@ -136,7 +136,7 @@ static void test_key_import_rsa(void) NCryptFreeObject(prov); } -static void test_ncrypt_free_object(void) +static void test_free_object(void) { NCRYPT_PROV_HANDLE prov; NCRYPT_KEY_HANDLE key; @@ -205,9 +205,55 @@ static void test_get_property(void) NCryptFreeObject(prov); } +static void test_create_persisted_key(void) +{ + NCRYPT_PROV_HANDLE prov; + NCRYPT_KEY_HANDLE key; + SECURITY_STATUS ret; + DWORD size, keylength; + WCHAR alggroup[4]; + + ret = NCryptOpenStorageProvider(&prov, NULL, 0); + ok(ret == ERROR_SUCCESS, "got %#lx\n", ret); + + todo_wine { + key = 0; + ret = NCryptCreatePersistedKey(0, &key, BCRYPT_RSA_ALGORITHM, NULL, 0, 0); + ok(ret == NTE_INVALID_HANDLE, "got %#lx\n", ret); + + ret = NCryptCreatePersistedKey(prov, &key, NULL, NULL, 0, 0); + ok(ret == HRESULT_FROM_WIN32(RPC_X_NULL_REF_POINTER) || broken(ret == NTE_FAIL), "got %#lx\n", ret); + + ret = NCryptCreatePersistedKey(prov, &key, BCRYPT_RSA_ALGORITHM, NULL, 0, 0); + ok(ret == ERROR_SUCCESS, "got %#lx\n", ret); + ok(key, "got null handle\n"); + + ret = NCryptGetProperty(key, NCRYPT_ALGORITHM_GROUP_PROPERTY, NULL, 0, &size, 0); + ok(ret == ERROR_SUCCESS, "got %#lx\n", ret); + ok(size == 8, "got %lu\n", size); + + size = 0; + alggroup[0] = 0; + ret = NCryptGetProperty(key, NCRYPT_ALGORITHM_GROUP_PROPERTY, (BYTE *)alggroup, sizeof(alggroup), &size, 0); + ok(ret == ERROR_SUCCESS, "got %#lx\n", ret); + ok(size == 8, "got %lu\n", size); + ok(!lstrcmpW(alggroup, L"RSA"), "The string doesn't match with 'RSA'\n"); + + ret = NCryptGetProperty(key, NCRYPT_LENGTH_PROPERTY, (BYTE *)&keylength, sizeof(keylength), &size, 0); + ok(ret == ERROR_SUCCESS, "got %#lx\n", ret); + ok(size == 4, "got %lu\n", size); + ok(keylength == 1024, "got %lu\n", keylength); + + NCryptFinalizeKey(key, 0); + NCryptFreeObject(key); + NCryptFreeObject(prov); + } +} + START_TEST(ncrypt) { test_key_import_rsa(); - test_ncrypt_free_object(); + test_free_object(); test_get_property(); + test_create_persisted_key(); }

1 0

Santino Mazza : ncrypt/tests: Test for NCryptSetProperty.
by Alexandre Julliard 21 Feb '22

21 Feb '22

Module: wine Branch: master Commit: 53cb28e6d9daa7cbcc190cd02aeaba37c297adc4 URL: https://source.winehq.org/git/wine.git/?a=commit;h=53cb28e6d9daa7cbcc190cd0… Author: Santino Mazza <mazzasantino1206(a)gmail.com> Date: Mon Feb 21 12:13:44 2022 +0100 ncrypt/tests: Test for NCryptSetProperty. Signed-off-by: Santino Mazza <mazzasantino1206(a)gmail.com> Signed-off-by: Hans Leidekker <hans(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ncrypt/tests/ncrypt.c | 46 ++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 46 insertions(+) diff --git a/dlls/ncrypt/tests/ncrypt.c b/dlls/ncrypt/tests/ncrypt.c index 58173fd0ff0..d1bcbbcab2a 100644 --- a/dlls/ncrypt/tests/ncrypt.c +++ b/dlls/ncrypt/tests/ncrypt.c @@ -205,6 +205,51 @@ static void test_get_property(void) NCryptFreeObject(prov); } +static void test_set_property(void) +{ + NCRYPT_PROV_HANDLE prov; + SECURITY_STATUS ret; + NCRYPT_KEY_HANDLE key; + DWORD keylength; + + ret = NCryptOpenStorageProvider(&prov, NULL, 0); + ok(ret == ERROR_SUCCESS, "got %#lx\n", ret); + + ret = NCryptImportKey(prov, 0, BCRYPT_RSAPUBLIC_BLOB, NULL, &key, rsa_key_blob, sizeof(rsa_key_blob), 0); + ok(ret == ERROR_SUCCESS, "got %#lx\n", ret); + + keylength = 2048; + ret = NCryptSetProperty(key, NCRYPT_LENGTH_PROPERTY, (BYTE *)&keylength, sizeof(keylength), 0); + ok(ret == ERROR_SUCCESS || broken(ret == NTE_INVALID_HANDLE), "got %#lx\n", ret); + + ret = NCryptSetProperty(0, NCRYPT_LENGTH_PROPERTY, (BYTE *)&keylength, sizeof(keylength), 0); + ok(ret == NTE_INVALID_HANDLE, "got %#lx\n", ret); + + todo_wine + { + ret = NCryptSetProperty(key, NCRYPT_NAME_PROPERTY, (BYTE *)L"Key name", sizeof(L"Key name"), 0); + ok(ret == NTE_NOT_SUPPORTED, "got %#lx\n", ret); + NCryptFreeObject(key); + + key = 0; + ret = NCryptCreatePersistedKey(prov, &key, BCRYPT_RSA_ALGORITHM, NULL, 0, 0); + ok(ret == ERROR_SUCCESS, "got %#lx\n", ret); + ok(key, "got null handle\n"); + + keylength = 2048; + ret = NCryptSetProperty(key, NCRYPT_LENGTH_PROPERTY, (BYTE *)&keylength, sizeof(keylength), 0); + ok(ret == ERROR_SUCCESS, "got %#lx\n", ret); + + ret = NCryptSetProperty(key, NCRYPT_NAME_PROPERTY, (BYTE *)L"Key name", sizeof(L"Key name"), 0); + ok(ret == NTE_NOT_SUPPORTED, "got %#lx\n", ret); + + ret = NCryptSetProperty(key, L"My Custom Property", (BYTE *)L"value", sizeof(L"value"), 0); + ok(ret == NTE_NOT_SUPPORTED, "got %#lx\n", ret); + } + NCryptFreeObject(key); + NCryptFreeObject(prov); +} + static void test_create_persisted_key(void) { NCRYPT_PROV_HANDLE prov; @@ -255,5 +300,6 @@ START_TEST(ncrypt) test_key_import_rsa(); test_free_object(); test_get_property(); + test_set_property(); test_create_persisted_key(); }

1 0

Eric Pouech : virtdisk: Enable compilation with long types.
by Alexandre Julliard 21 Feb '22

21 Feb '22

Module: wine Branch: master Commit: 7f44de63f78fc8bc63ef044267f9fcfa2230ce54 URL: https://source.winehq.org/git/wine.git/?a=commit;h=7f44de63f78fc8bc63ef0442… Author: Eric Pouech <eric.pouech(a)gmail.com> Date: Mon Feb 21 07:58:44 2022 +0100 virtdisk: Enable compilation with long types. Signed-off-by: Eric Pouech <eric.pouech(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/virtdisk/Makefile.in | 1 - dlls/virtdisk/virtdisk_main.c | 4 ++-- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/dlls/virtdisk/Makefile.in b/dlls/virtdisk/Makefile.in index 0b224bd459b..949065907fc 100644 --- a/dlls/virtdisk/Makefile.in +++ b/dlls/virtdisk/Makefile.in @@ -1,4 +1,3 @@ -EXTRADEFS = -DWINE_NO_LONG_TYPES MODULE = virtdisk.dll EXTRADLLFLAGS = -Wb,--prefer-native diff --git a/dlls/virtdisk/virtdisk_main.c b/dlls/virtdisk/virtdisk_main.c index c4d13241698..bad01a31f46 100644 --- a/dlls/virtdisk/virtdisk_main.c +++ b/dlls/virtdisk/virtdisk_main.c @@ -30,7 +30,7 @@ DWORD WINAPI GetStorageDependencyInformation(HANDLE obj, GET_STORAGE_DEPENDENCY_ { ULONG temp_size = sizeof(STORAGE_DEPENDENCY_INFO); - FIXME("(%p, 0x%x, %u, %p, %p): stub\n", obj, flags, size, info, used); + FIXME("(%p, 0x%x, %lu, %p, %p): stub\n", obj, flags, size, info, used); if (used) *used = temp_size; @@ -61,6 +61,6 @@ DWORD WINAPI OpenVirtualDisk(VIRTUAL_STORAGE_TYPE *type, const WCHAR *path, VIRT DWORD WINAPI DetachVirtualDisk(HANDLE handle, DETACH_VIRTUAL_DISK_FLAG flags, ULONG specific_flags) { - FIXME("(%p, 0x%x, %d): stub\n", handle, flags, specific_flags); + FIXME("(%p, 0x%x, %ld): stub\n", handle, flags, specific_flags); return ERROR_INVALID_PARAMETER; }

1 0

Eric Pouech : vmm.vxd: Enable compilation with long types.
by Alexandre Julliard 21 Feb '22

21 Feb '22

Module: wine Branch: master Commit: 6d376641f5ba26d654d7cd4c44642c9bba69126a URL: https://source.winehq.org/git/wine.git/?a=commit;h=6d376641f5ba26d654d7cd4c… Author: Eric Pouech <eric.pouech(a)gmail.com> Date: Mon Feb 21 07:58:49 2022 +0100 vmm.vxd: Enable compilation with long types. Signed-off-by: Eric Pouech <eric.pouech(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/vmm.vxd/Makefile.in | 1 - dlls/vmm.vxd/vmm.c | 20 ++++++++++---------- 2 files changed, 10 insertions(+), 11 deletions(-) diff --git a/dlls/vmm.vxd/Makefile.in b/dlls/vmm.vxd/Makefile.in index 9d4818c1926..28a5302f532 100644 --- a/dlls/vmm.vxd/Makefile.in +++ b/dlls/vmm.vxd/Makefile.in @@ -1,4 +1,3 @@ -EXTRADEFS = -DWINE_NO_LONG_TYPES MODULE = vmm.vxd C_SRCS = \ diff --git a/dlls/vmm.vxd/vmm.c b/dlls/vmm.vxd/vmm.c index 373e826efe8..699f1444206 100644 --- a/dlls/vmm.vxd/vmm.c +++ b/dlls/vmm.vxd/vmm.c @@ -144,7 +144,7 @@ DWORD WINAPI VMM_VxDCall( DWORD service, CONTEXT *context ) ULONG npages = stack32_pop( context ); ULONG flags = stack32_pop( context ); - TRACE("PageReserve: page: %08x, npages: %08x, flags: %08x partial stub!\n", + TRACE("PageReserve: page: %08lx, npages: %08lx, flags: %08lx partial stub!\n", page, npages, flags ); if ( page == PR_SYSTEM ) { @@ -175,8 +175,8 @@ DWORD WINAPI VMM_VxDCall( DWORD service, CONTEXT *context ) ULONG pagerdata = stack32_pop( context ); ULONG flags = stack32_pop( context ); - TRACE("PageCommit: page: %08x, npages: %08x, hpd: %08x pagerdata: " - "%08x, flags: %08x partial stub\n", + TRACE("PageCommit: page: %08lx, npages: %08lx, hpd: %08lx pagerdata: " + "%08lx, flags: %08lx partial stub\n", page, npages, hpd, pagerdata, flags ); if ( flags & PC_USER ) @@ -202,7 +202,7 @@ DWORD WINAPI VMM_VxDCall( DWORD service, CONTEXT *context ) ULONG npages = stack32_pop( context ); ULONG flags = stack32_pop( context ); - TRACE("PageDecommit: page: %08x, npages: %08x, flags: %08x partial stub\n", + TRACE("PageDecommit: page: %08lx, npages: %08lx, flags: %08lx partial stub\n", page, npages, flags ); address = (LPVOID )( page * page_size ); ret = VirtualFree ( address, npages * page_size, MEM_DECOMMIT ); @@ -223,7 +223,7 @@ DWORD WINAPI VMM_VxDCall( DWORD service, CONTEXT *context ) ULONG permand = stack32_pop ( context ); ULONG permor = stack32_pop ( context ); - TRACE("PageModifyPermissions %08x %08x %08x %08x partial stub\n", + TRACE("PageModifyPermissions %08lx %08lx %08lx %08lx partial stub\n", page, npages, permand, permor ); address = (LPVOID )( page * page_size ); @@ -264,7 +264,7 @@ DWORD WINAPI VMM_VxDCall( DWORD service, CONTEXT *context ) ERR("Can't change page permissions for %p\n", address ); return 0xffffffff; } - TRACE("Returning: %08x\n", pg_old_perm ); + TRACE("Returning: %08lx\n", pg_old_perm ); return pg_old_perm; } @@ -274,7 +274,7 @@ DWORD WINAPI VMM_VxDCall( DWORD service, CONTEXT *context ) LPVOID hmem = (LPVOID) stack32_pop( context ); DWORD flags = stack32_pop( context ); - TRACE("PageFree: hmem: %p, flags: %08x partial stub\n", + TRACE("PageFree: hmem: %p, flags: %08lx partial stub\n", hmem, flags ); ret = VirtualFree ( hmem, 0, MEM_RELEASE ); @@ -413,7 +413,7 @@ DWORD WINAPI VMM_VxDCall( DWORD service, CONTEXT *context ) * implementation of this. */ - FIXME("GetDemandPageInfo(%08x %08x): stub!\n", dinfo, flags); + FIXME("GetDemandPageInfo(%08lx %08lx): stub!\n", dinfo, flags); return 0; } @@ -462,10 +462,10 @@ DWORD WINAPI VMM_VxDCall( DWORD service, CONTEXT *context ) default: if (LOWORD(service) < N_VMM_SERVICE) - FIXME( "Unimplemented service %s (%08x)\n", + FIXME( "Unimplemented service %s (%08lx)\n", VMM_Service_Name[LOWORD(service)], service); else - FIXME( "Unknown service %08x\n", service); + FIXME( "Unknown service %08lx\n", service); return 0xffffffff; /* FIXME */ } }

1 0

Eric Pouech : vnbt.vxd: Enable compilation with long types.
by Alexandre Julliard 21 Feb '22

21 Feb '22

Module: wine Branch: master Commit: 164e38039506a064c5a1d3918362068a4656be8a URL: https://source.winehq.org/git/wine.git/?a=commit;h=164e38039506a064c5a1d391… Author: Eric Pouech <eric.pouech(a)gmail.com> Date: Mon Feb 21 07:58:55 2022 +0100 vnbt.vxd: Enable compilation with long types. Signed-off-by: Eric Pouech <eric.pouech(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/vnbt.vxd/Makefile.in | 1 - dlls/vnbt.vxd/vnbt.c | 2 +- 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/dlls/vnbt.vxd/Makefile.in b/dlls/vnbt.vxd/Makefile.in index 61d0c786c36..8fc6b3e4a73 100644 --- a/dlls/vnbt.vxd/Makefile.in +++ b/dlls/vnbt.vxd/Makefile.in @@ -1,4 +1,3 @@ -EXTRADEFS = -DWINE_NO_LONG_TYPES MODULE = vnbt.vxd IMPORTS = iphlpapi ws2_32 diff --git a/dlls/vnbt.vxd/vnbt.c b/dlls/vnbt.vxd/vnbt.c index fd1dbdcbe6a..ea9b48f7f06 100644 --- a/dlls/vnbt.vxd/vnbt.c +++ b/dlls/vnbt.vxd/vnbt.c @@ -149,7 +149,7 @@ BOOL WINAPI VNBT_DeviceIoControl(DWORD dwIoControlCode, * to unimplemented... */ default: - FIXME( "Unimplemented control %d for VxD device VNB\n", + FIXME( "Unimplemented control %ld for VxD device VNB\n", dwIoControlCode ); error = ERROR_NOT_SUPPORTED; break;

1 0

Eric Pouech : vnetbios.vxd: Enable compilation with long types.
by Alexandre Julliard 21 Feb '22

21 Feb '22

Module: wine Branch: master Commit: e0a0b6ab6066fb6ea45c8fe4e6cbd72d4017bcae URL: https://source.winehq.org/git/wine.git/?a=commit;h=e0a0b6ab6066fb6ea45c8fe4… Author: Eric Pouech <eric.pouech(a)gmail.com> Date: Mon Feb 21 07:59:01 2022 +0100 vnetbios.vxd: Enable compilation with long types. Signed-off-by: Eric Pouech <eric.pouech(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/vnetbios.vxd/Makefile.in | 1 - dlls/vnetbios.vxd/vnetbios.c | 2 +- 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/dlls/vnetbios.vxd/Makefile.in b/dlls/vnetbios.vxd/Makefile.in index d1f77cb78e8..8e8cb60481c 100644 --- a/dlls/vnetbios.vxd/Makefile.in +++ b/dlls/vnetbios.vxd/Makefile.in @@ -1,4 +1,3 @@ -EXTRADEFS = -DWINE_NO_LONG_TYPES MODULE = vnetbios.vxd IMPORTS = netapi32 diff --git a/dlls/vnetbios.vxd/vnetbios.c b/dlls/vnetbios.vxd/vnetbios.c index 3bbd0f9d809..c3adf615395 100644 --- a/dlls/vnetbios.vxd/vnetbios.c +++ b/dlls/vnetbios.vxd/vnetbios.c @@ -39,7 +39,7 @@ BOOL WINAPI VNETBIOS_DeviceIoControl( DWORD code, LPVOID lpvInBuffer, DWORD cbIn Netbios(lpvInBuffer); return TRUE; default: - FIXME("(%d,%p,%d,%p,%d,%p,%p): stub\n", + FIXME("(%ld,%p,%ld,%p,%ld,%p,%p): stub\n", code, lpvInBuffer,cbInBuffer, lpvOutBuffer,cbOutBuffer, lpcbBytesReturned, lpOverlapped); return FALSE;

1 0