October 2017 - Wine-Commits

Martin Storsjo : api-ms-win-core-synch-ansi-l1-1-0: Add dll.
by Alexandre Julliard Oct. 16, 2017

Oct. 16, 2017

Module: wine Branch: master Commit: 91ad8f5a2eefc094b04fc6e4277f91f5c2ddbfb4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=91ad8f5a2eefc094b04fc6e42… Author: Martin Storsjo <martin(a)martin.st> Date: Mon Oct 16 12:21:22 2017 +0300 api-ms-win-core-synch-ansi-l1-1-0: Add dll. Signed-off-by: Martin Storsjo <martin(a)martin.st> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- configure | 2 ++ configure.ac | 1 + dlls/api-ms-win-core-synch-ansi-l1-1-0/Makefile.in | 1 + .../api-ms-win-core-synch-ansi-l1-1-0.spec | 7 +++++++ tools/make_specfiles | 1 + 5 files changed, 12 insertions(+) diff --git a/configure b/configure index fa9b75e..3d798cb 100755 --- a/configure +++ b/configure @@ -976,6 +976,7 @@ enable_api_ms_win_core_string_l1_1_0 enable_api_ms_win_core_string_l2_1_0 enable_api_ms_win_core_string_obsolete_l1_1_0 enable_api_ms_win_core_stringansi_l1_1_0 +enable_api_ms_win_core_synch_ansi_l1_1_0 enable_api_ms_win_core_synch_l1_1_0 enable_api_ms_win_core_synch_l1_2_0 enable_api_ms_win_core_synch_l1_2_1 @@ -18065,6 +18066,7 @@ wine_fn_config_dll api-ms-win-core-string-l1-1-0 enable_api_ms_win_core_string_l wine_fn_config_dll api-ms-win-core-string-l2-1-0 enable_api_ms_win_core_string_l2_1_0 wine_fn_config_dll api-ms-win-core-string-obsolete-l1-1-0 enable_api_ms_win_core_string_obsolete_l1_1_0 wine_fn_config_dll api-ms-win-core-stringansi-l1-1-0 enable_api_ms_win_core_stringansi_l1_1_0 +wine_fn_config_dll api-ms-win-core-synch-ansi-l1-1-0 enable_api_ms_win_core_synch_ansi_l1_1_0 wine_fn_config_dll api-ms-win-core-synch-l1-1-0 enable_api_ms_win_core_synch_l1_1_0 wine_fn_config_dll api-ms-win-core-synch-l1-2-0 enable_api_ms_win_core_synch_l1_2_0 wine_fn_config_dll api-ms-win-core-synch-l1-2-1 enable_api_ms_win_core_synch_l1_2_1 diff --git a/configure.ac b/configure.ac index 9d33627..74bea22 100644 --- a/configure.ac +++ b/configure.ac @@ -2828,6 +2828,7 @@ WINE_CONFIG_DLL(api-ms-win-core-string-l1-1-0) WINE_CONFIG_DLL(api-ms-win-core-string-l2-1-0) WINE_CONFIG_DLL(api-ms-win-core-string-obsolete-l1-1-0) WINE_CONFIG_DLL(api-ms-win-core-stringansi-l1-1-0) +WINE_CONFIG_DLL(api-ms-win-core-synch-ansi-l1-1-0) WINE_CONFIG_DLL(api-ms-win-core-synch-l1-1-0) WINE_CONFIG_DLL(api-ms-win-core-synch-l1-2-0) WINE_CONFIG_DLL(api-ms-win-core-synch-l1-2-1) diff --git a/dlls/api-ms-win-core-synch-ansi-l1-1-0/Makefile.in b/dlls/api-ms-win-core-synch-ansi-l1-1-0/Makefile.in new file mode 100644 index 0000000..acab15b --- /dev/null +++ b/dlls/api-ms-win-core-synch-ansi-l1-1-0/Makefile.in @@ -0,0 +1 @@ +MODULE = api-ms-win-core-synch-ansi-l1-1-0.dll diff --git a/dlls/api-ms-win-core-synch-ansi-l1-1-0/api-ms-win-core-synch-ansi-l1-1-0.spec b/dlls/api-ms-win-core-synch-ansi-l1-1-0/api-ms-win-core-synch-ansi-l1-1-0.spec new file mode 100644 index 0000000..eb75146 --- /dev/null +++ b/dlls/api-ms-win-core-synch-ansi-l1-1-0/api-ms-win-core-synch-ansi-l1-1-0.spec @@ -0,0 +1,7 @@ +@ stdcall CreateSemaphoreA(ptr long long str) kernel32.CreateSemaphoreA +@ stdcall CreateSemaphoreExA(ptr long long str long long) kernel32.CreateSemaphoreExA +@ stdcall CreateWaitableTimerA(ptr long str) kernel32.CreateWaitableTimerA +@ stdcall CreateWaitableTimerExA(ptr str long long) kernel32.CreateWaitableTimerExA +@ stdcall OpenMutexA(long long str) kernel32.OpenMutexA +@ stdcall OpenSemaphoreA(long long str) kernel32.OpenSemaphoreA +@ stdcall OpenWaitableTimerA(long long str) kernel32.OpenWaitableTimerA diff --git a/tools/make_specfiles b/tools/make_specfiles index e1c7e1f..a9ffa29 100755 --- a/tools/make_specfiles +++ b/tools/make_specfiles @@ -201,6 +201,7 @@ my @dll_groups = "api-ms-win-core-sysinfo-l1-1-0", "api-ms-win-core-sysinfo-l1-2-0", "api-ms-win-core-util-l1-1-0", + "api-ms-win-core-synch-ansi-l1-1-0", "api-ms-win-core-synch-l1-1-0", "api-ms-win-core-synch-l1-2-0", "api-ms-win-core-synch-l1-2-1",

1 0

Nikolay Sivov : gdiplus/metafile: Implement EmfPlusRecordTypeDrawImagePoints record playback.
by Alexandre Julliard Oct. 16, 2017

Oct. 16, 2017

Module: wine Branch: master Commit: 440efc8244da18cb8a9bbd441645468fdbcba087 URL: http://source.winehq.org/git/wine.git/?a=commit;h=440efc8244da18cb8a9bbd441… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Mon Oct 16 10:12:22 2017 +0300 gdiplus/metafile: Implement EmfPlusRecordTypeDrawImagePoints record playback. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Vincent Povirk <vincent(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/gdiplus/metafile.c | 89 +++++++++++++++++++++++++++++++++++++++++++------ 1 file changed, 79 insertions(+), 10 deletions(-) diff --git a/dlls/gdiplus/metafile.c b/dlls/gdiplus/metafile.c index 2fc741e..fa4bafd 100644 --- a/dlls/gdiplus/metafile.c +++ b/dlls/gdiplus/metafile.c @@ -328,6 +328,18 @@ typedef struct EmfPlusRectF float Height; } EmfPlusRectF; +typedef struct EmfPlusPointR7 +{ + BYTE X; + BYTE Y; +} EmfPlusPointR7; + +typedef struct EmfPlusPoint +{ + short X; + short Y; +} EmfPlusPoint; + typedef struct EmfPlusPointF { float X; @@ -343,10 +355,10 @@ typedef struct EmfPlusDrawImagePoints DWORD count; union { - /*EmfPlusPointR pointR; - EmfPlusPoint point;*/ - EmfPlusPointF pointF; - } PointData[3]; + EmfPlusPointR7 pointsR[3]; + EmfPlusPoint points[3]; + EmfPlusPointF pointsF[3]; + } PointData; } EmfPlusDrawImagePoints; typedef struct EmfPlusDrawPath @@ -2084,6 +2096,68 @@ GpStatus WINGDIPAPI GdipPlayMetafileRecord(GDIPCONST GpMetafile *metafile, { return METAFILE_PlaybackObject(real_metafile, flags, dataSize, data); } + case EmfPlusRecordTypeDrawImagePoints: + { + EmfPlusDrawImagePoints *draw = (EmfPlusDrawImagePoints *)header; + static const UINT fixed_part_size = FIELD_OFFSET(EmfPlusDrawImagePoints, PointData) - + FIELD_OFFSET(EmfPlusDrawImagePoints, ImageAttributesID); + BYTE image = flags & 0xff; + GpPointF points[3]; + unsigned int i; + UINT size; + + if (image >= EmfPlusObjectTableSize || real_metafile->objtable[image].type != ObjectTypeImage) + return InvalidParameter; + + if (dataSize <= fixed_part_size) + return InvalidParameter; + dataSize -= fixed_part_size; + + if (draw->ImageAttributesID >= EmfPlusObjectTableSize || + real_metafile->objtable[draw->ImageAttributesID].type != ObjectTypeImageAttributes) + return InvalidParameter; + + if (draw->count != 3) + return InvalidParameter; + + if ((flags >> 13) & 1) /* E */ + FIXME("image effects are not supported.\n"); + + if ((flags >> 11) & 1) /* P */ + size = sizeof(EmfPlusPointR7) * draw->count; + else if ((flags >> 14) & 1) /* C */ + size = sizeof(EmfPlusPoint) * draw->count; + else + size = sizeof(EmfPlusPointF) * draw->count; + + if (dataSize != size) + return InvalidParameter; + + if ((flags >> 11) & 1) /* P */ + { + points[0].X = draw->PointData.pointsR[0].X; + points[0].Y = draw->PointData.pointsR[0].Y; + for (i = 1; i < 3; i++) + { + points[i].X = points[i-1].X + draw->PointData.pointsR[i].X; + points[i].Y = points[i-1].Y + draw->PointData.pointsR[i].Y; + } + } + else if ((flags >> 14) & 1) /* C */ + { + for (i = 0; i < 3; i++) + { + points[i].X = draw->PointData.points[i].X; + points[i].Y = draw->PointData.points[i].Y; + } + } + else + memcpy(points, draw->PointData.pointsF, sizeof(points)); + + return GdipDrawImagePointsRect(real_metafile->playback_graphics, real_metafile->objtable[image].u.image, + points, 3, draw->SrcRect.X, draw->SrcRect.Y, draw->SrcRect.Width, draw->SrcRect.Height, draw->SrcUnit, + real_metafile->objtable[draw->ImageAttributesID].u.image_attributes, NULL, NULL); + } default: FIXME("Not implemented for record type %x\n", recordType); return NotImplemented; @@ -2982,12 +3056,7 @@ GpStatus METAFILE_DrawImagePointsRect(GpMetafile *metafile, GpImage *image, draw_image_record->SrcRect.Width = units_to_pixels(srcwidth, srcUnit, metafile->image.xres); draw_image_record->SrcRect.Height = units_to_pixels(srcheight, srcUnit, metafile->image.yres); draw_image_record->count = 3; - draw_image_record->PointData[0].pointF.X = points[0].X; - draw_image_record->PointData[0].pointF.Y = points[0].Y; - draw_image_record->PointData[1].pointF.X = points[1].X; - draw_image_record->PointData[1].pointF.Y = points[1].Y; - draw_image_record->PointData[2].pointF.X = points[2].X; - draw_image_record->PointData[2].pointF.Y = points[2].Y; + memcpy(draw_image_record->PointData.pointsF, points, 3 * sizeof(*points)); METAFILE_WriteRecords(metafile); return Ok; }

1 0

Nikolay Sivov : gdiplus/metafile: Implement EmfPlusImage deserialization for bitmaps.
by Alexandre Julliard Oct. 16, 2017

Oct. 16, 2017

Module: wine Branch: master Commit: 81459d652629b29eb19e4c14ba240034a0f3c4dc URL: http://source.winehq.org/git/wine.git/?a=commit;h=81459d652629b29eb19e4c14b… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Mon Oct 16 10:12:21 2017 +0300 gdiplus/metafile: Implement EmfPlusImage deserialization for bitmaps. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Vincent Povirk <vincent(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/gdiplus/gdiplus_private.h | 1 + dlls/gdiplus/metafile.c | 117 +++++++++++++++++++++++++++++++++++++++++ 2 files changed, 118 insertions(+) diff --git a/dlls/gdiplus/gdiplus_private.h b/dlls/gdiplus/gdiplus_private.h index 95445e8..f659ccf 100644 --- a/dlls/gdiplus/gdiplus_private.h +++ b/dlls/gdiplus/gdiplus_private.h @@ -394,6 +394,7 @@ struct emfplus_object { EmfPlusObjectType type; union { GpBrush *brush; + GpImage *image; GpImageAttributes *image_attributes; void *object; } u; diff --git a/dlls/gdiplus/metafile.c b/dlls/gdiplus/metafile.c index de208a8..2fc741e 100644 --- a/dlls/gdiplus/metafile.c +++ b/dlls/gdiplus/metafile.c @@ -43,6 +43,8 @@ WINE_DEFAULT_DEBUG_CHANNEL(gdiplus); +HRESULT WINAPI WICCreateImagingFactory_Proxy(UINT, IWICImagingFactory**); + typedef struct EmfPlusARGB { BYTE Blue; @@ -246,6 +248,13 @@ typedef struct EmfPlusRegion BYTE RegionNode[1]; } EmfPlusRegion; +typedef struct EmfPlusPalette +{ + DWORD PaletteStyleFlags; + DWORD PaletteCount; + BYTE PaletteEntries[1]; +} EmfPlusPalette; + typedef enum { BitmapDataTypePixel, @@ -367,6 +376,9 @@ static void metafile_free_object_table_entry(GpMetafile *metafile, BYTE id) case ObjectTypeBrush: GdipDeleteBrush(object->u.brush); break; + case ObjectTypeImage: + GdipDisposeImage(object->u.image); + break; case ObjectTypeImageAttributes: GdipDisposeImageAttributes(object->u.image_attributes); break; @@ -1446,6 +1458,108 @@ static void metafile_set_object_table_entry(GpMetafile *metafile, BYTE id, BYTE metafile->objtable[id].u.object = object; } +static GpStatus metafile_deserialize_image(const BYTE *record_data, UINT data_size, GpImage **image) +{ + EmfPlusImage *data = (EmfPlusImage *)record_data; + GpStatus status; + + *image = NULL; + + if (data_size < FIELD_OFFSET(EmfPlusImage, ImageData)) + return InvalidParameter; + data_size -= FIELD_OFFSET(EmfPlusImage, ImageData); + + switch (data->Type) + { + case ImageDataTypeBitmap: + { + EmfPlusBitmap *bitmapdata = &data->ImageData.bitmap; + + if (data_size <= FIELD_OFFSET(EmfPlusBitmap, BitmapData)) + return InvalidParameter; + data_size -= FIELD_OFFSET(EmfPlusBitmap, BitmapData); + + switch (bitmapdata->Type) + { + case BitmapDataTypePixel: + { + ColorPalette *palette; + BYTE *scan0; + + if (bitmapdata->PixelFormat & PixelFormatIndexed) + { + EmfPlusPalette *palette_obj = (EmfPlusPalette *)bitmapdata->BitmapData; + UINT palette_size = FIELD_OFFSET(EmfPlusPalette, PaletteEntries); + + if (data_size <= palette_size) + return InvalidParameter; + palette_size += palette_obj->PaletteCount * sizeof(EmfPlusARGB); + + if (data_size < palette_size) + return InvalidParameter; + data_size -= palette_size; + + palette = (ColorPalette *)bitmapdata->BitmapData; + scan0 = (BYTE *)bitmapdata->BitmapData + palette_size; + } + else + { + palette = NULL; + scan0 = bitmapdata->BitmapData; + } + + if (data_size < bitmapdata->Height * bitmapdata->Stride) + return InvalidParameter; + + status = GdipCreateBitmapFromScan0(bitmapdata->Width, bitmapdata->Height, bitmapdata->Stride, + bitmapdata->PixelFormat, scan0, (GpBitmap **)image); + if (status == Ok && palette) + { + status = GdipSetImagePalette(*image, palette); + if (status != Ok) + { + GdipDisposeImage(*image); + *image = NULL; + } + } + break; + } + case BitmapDataTypeCompressed: + { + IWICImagingFactory *factory; + IWICStream *stream; + HRESULT hr; + + if (WICCreateImagingFactory_Proxy(WINCODEC_SDK_VERSION, &factory) != S_OK) + return GenericError; + + hr = IWICImagingFactory_CreateStream(factory, &stream); + IWICImagingFactory_Release(factory); + if (hr != S_OK) + return GenericError; + + if (IWICStream_InitializeFromMemory(stream, bitmapdata->BitmapData, data_size) == S_OK) + status = GdipCreateBitmapFromStream((IStream *)stream, (GpBitmap **)image); + else + status = GenericError; + + IWICStream_Release(stream); + break; + } + default: + WARN("Invalid bitmap type %d.\n", bitmapdata->Type); + return InvalidParameter; + } + break; + } + default: + FIXME("image type %d not supported.\n", data->Type); + return NotImplemented; + } + + return status; +} + static GpStatus metafile_deserialize_brush(const BYTE *record_data, UINT data_size, GpBrush **brush) { static const UINT header_size = FIELD_OFFSET(EmfPlusBrush, BrushData); @@ -1488,6 +1602,9 @@ static GpStatus METAFILE_PlaybackObject(GpMetafile *metafile, UINT flags, UINT d case ObjectTypeBrush: status = metafile_deserialize_brush(record_data, data_size, (GpBrush **)&object); break; + case ObjectTypeImage: + status = metafile_deserialize_image(record_data, data_size, (GpImage **)&object); + break; case ObjectTypeImageAttributes: { EmfPlusImageAttributes *data = (EmfPlusImageAttributes *)record_data;

1 0

Alexandre Julliard : server: Avoid potential double free (Coverity).
by Alexandre Julliard Oct. 16, 2017

Oct. 16, 2017

Module: wine Branch: master Commit: af238f5653a4dd1656eb2027d1605e8088613772 URL: http://source.winehq.org/git/wine.git/?a=commit;h=af238f5653a4dd1656eb2027d… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Oct 16 15:28:38 2017 +0200 server: Avoid potential double free (Coverity). Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- server/mapping.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/server/mapping.c b/server/mapping.c index f9f50ed..d9bff4b 100644 --- a/server/mapping.c +++ b/server/mapping.c @@ -507,13 +507,13 @@ static int build_shared_mapping( struct mapping *mapping, int fd, } if (pwrite( shared_fd, buffer, file_size, write_pos ) != file_size) goto error; } - free( buffer ); if (!(shared = alloc_object( &shared_map_ops ))) goto error; shared->fd = (struct fd *)grab_object( mapping->fd ); shared->file = file; list_add_head( &shared_map_list, &shared->entry ); mapping->shared = shared; + free( buffer ); return 1; error:

1 0

Huw Davies : dwrite: Avoid using RegQueryInfoKey().
by Alexandre Julliard Oct. 16, 2017

Oct. 16, 2017

Module: wine Branch: master Commit: 625293289c7cf37e052e0407350fcf73c190d487 URL: http://source.winehq.org/git/wine.git/?a=commit;h=625293289c7cf37e052e04073… Author: Huw Davies <huw(a)codeweavers.com> Date: Mon Oct 16 10:11:17 2017 +0100 dwrite: Avoid using RegQueryInfoKey(). Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/dwrite/font.c | 102 ++++++++++++++++++++++++++--------------------------- 1 file changed, 50 insertions(+), 52 deletions(-) diff --git a/dlls/dwrite/font.c b/dlls/dwrite/font.c index 1916bf2..cab220a 100644 --- a/dlls/dwrite/font.c +++ b/dlls/dwrite/font.c @@ -3825,8 +3825,8 @@ struct system_fontfile_enumerator HKEY hkey; int index; - WCHAR *value; - DWORD max_val_count; + WCHAR *filename; + DWORD filename_size; }; static inline struct system_fontfile_enumerator *impl_from_IDWriteFontFileEnumerator(IDWriteFontFileEnumerator* iface) @@ -3863,7 +3863,7 @@ static ULONG WINAPI systemfontfileenumerator_Release(IDWriteFontFileEnumerator * if (!ref) { IDWriteFactory5_Release(enumerator->factory); RegCloseKey(enumerator->hkey); - heap_free(enumerator->value); + heap_free(enumerator->filename); heap_free(enumerator); } @@ -3894,77 +3894,69 @@ static HRESULT create_local_file_reference(IDWriteFactory5 *factory, const WCHAR static HRESULT WINAPI systemfontfileenumerator_GetCurrentFontFile(IDWriteFontFileEnumerator *iface, IDWriteFontFile **file) { struct system_fontfile_enumerator *enumerator = impl_from_IDWriteFontFileEnumerator(iface); - DWORD ret, type, val_count, count; - WCHAR *value, *filename; - HRESULT hr; *file = NULL; - if (enumerator->index < 0) - return E_FAIL; - - ret = RegQueryInfoKeyW(enumerator->hkey, NULL, NULL, NULL, NULL, NULL, NULL, NULL, &val_count, &count, NULL, NULL); - if (ret != ERROR_SUCCESS) + if (enumerator->index < 0 || !enumerator->filename || !*enumerator->filename) return E_FAIL; - val_count++; - value = heap_alloc( val_count * sizeof(value[0]) ); - filename = heap_alloc(count); - if (!value || !filename) { - heap_free(value); - heap_free(filename); - return E_OUTOFMEMORY; - } - - ret = RegEnumValueW(enumerator->hkey, enumerator->index, value, &val_count, NULL, &type, (BYTE*)filename, &count); - if (ret) { - heap_free(value); - heap_free(filename); - return E_FAIL; - } - - hr = create_local_file_reference(enumerator->factory, filename, file); - - heap_free(value); - heap_free(filename); - return hr; + return create_local_file_reference(enumerator->factory, enumerator->filename, file); } static HRESULT WINAPI systemfontfileenumerator_MoveNext(IDWriteFontFileEnumerator *iface, BOOL *current) { struct system_fontfile_enumerator *enumerator = impl_from_IDWriteFontFileEnumerator(iface); + WCHAR name_buf[256], *name = name_buf; + DWORD name_count, max_name_count = sizeof(name_buf) / sizeof(*name_buf), type, data_size; + HRESULT hr = S_OK; + LONG r; *current = FALSE; enumerator->index++; - if (!enumerator->value) { - if (RegQueryInfoKeyW(enumerator->hkey, NULL, NULL, NULL, NULL, NULL, NULL, NULL, - &enumerator->max_val_count, NULL, NULL, NULL)) - return E_FAIL; - - enumerator->max_val_count++; - if (!(enumerator->value = heap_alloc(enumerator->max_val_count * sizeof(*enumerator->value)))) - return E_OUTOFMEMORY; - } - /* iterate until we find next string value */ for (;;) { - DWORD type = 0, count, val_count; + do { + name_count = max_name_count; + data_size = enumerator->filename_size - sizeof(*enumerator->filename); + + r = RegEnumValueW(enumerator->hkey, enumerator->index, name, &name_count, + NULL, &type, (BYTE *)enumerator->filename, &data_size); + if (r == ERROR_MORE_DATA) { + if (name_count >= max_name_count) { + if (name != name_buf) heap_free(name); + max_name_count *= 2; + name = heap_alloc(max_name_count * sizeof(*name)); + if (!name) return E_OUTOFMEMORY; + } + if (data_size > enumerator->filename_size - sizeof(*enumerator->filename)) { + heap_free(enumerator->filename); + enumerator->filename_size = max(data_size + sizeof(*enumerator->filename), enumerator->filename_size * 2); + enumerator->filename = heap_alloc(enumerator->filename_size); + if (!enumerator->filename) { + hr = E_OUTOFMEMORY; + goto err; + } + } + } + } while (r == ERROR_MORE_DATA); - val_count = enumerator->max_val_count; - *enumerator->value = 0; - if (RegEnumValueW(enumerator->hkey, enumerator->index, enumerator->value, &val_count, - NULL, &type, NULL, &count)) + if (r != ERROR_SUCCESS) { + enumerator->filename[0] = 0; break; - if (type == REG_SZ && *enumerator->value && *enumerator->value != '@') { + } + enumerator->filename[data_size / sizeof(*enumerator->filename)] = 0; + if (type == REG_SZ && *name != '@') { *current = TRUE; break; } enumerator->index++; } - TRACE("index = %d, current = %d\n", enumerator->index, *current); - return S_OK; + +err: + if (name != name_buf) heap_free(name); + return hr; } static const struct IDWriteFontFileEnumeratorVtbl systemfontfileenumeratorvtbl = @@ -3995,13 +3987,19 @@ static HRESULT create_system_fontfile_enumerator(IDWriteFactory5 *factory, IDWri enumerator->ref = 1; enumerator->factory = factory; enumerator->index = -1; - enumerator->value = NULL; - enumerator->max_val_count = 0; + enumerator->filename_size = MAX_PATH * sizeof(*enumerator->filename); + enumerator->filename = heap_alloc(enumerator->filename_size); + if (!enumerator->filename) { + heap_free(enumerator); + return E_OUTOFMEMORY; + } + IDWriteFactory5_AddRef(factory); if (RegOpenKeyExW(HKEY_LOCAL_MACHINE, fontslistW, 0, GENERIC_READ, &enumerator->hkey)) { ERR("failed to open fonts list key\n"); IDWriteFactory5_Release(factory); + heap_free(enumerator->filename); heap_free(enumerator); return E_FAIL; }

1 0

Hugh McMaster : reg/tests: Correctly import hex values with no data.
by Alexandre Julliard Oct. 16, 2017

Oct. 16, 2017

Module: wine Branch: master Commit: c02dd08f910fb684e88b02fcb9d75564c8636e5a URL: http://source.winehq.org/git/wine.git/?a=commit;h=c02dd08f910fb684e88b02fcb… Author: Hugh McMaster <hugh.mcmaster(a)outlook.com> Date: Mon Oct 16 10:27:25 2017 +0000 reg/tests: Correctly import hex values with no data. Signed-off-by: Hugh McMaster <hugh.mcmaster(a)outlook.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- programs/reg/import.c | 4 ++++ programs/reg/tests/reg.c | 8 ++++---- 2 files changed, 8 insertions(+), 4 deletions(-) diff --git a/programs/reg/import.c b/programs/reg/import.c index d2b4cdc..8350fa7 100644 --- a/programs/reg/import.c +++ b/programs/reg/import.c @@ -785,6 +785,9 @@ static WCHAR *hex_data_state(struct parser *parser, WCHAR *pos) { WCHAR *line = pos; + if (!*line) + goto set_value; + if (!convert_hex_csv_to_hex(parser, &line)) goto invalid; @@ -796,6 +799,7 @@ static WCHAR *hex_data_state(struct parser *parser, WCHAR *pos) prepare_hex_string_data(parser); +set_value: set_state(parser, SET_VALUE); return line; diff --git a/programs/reg/tests/reg.c b/programs/reg/tests/reg.c index 42e6a97..e0ff5f0 100644 --- a/programs/reg/tests/reg.c +++ b/programs/reg/tests/reg.c @@ -2322,10 +2322,10 @@ static void test_import(void) "\"Wine68i\"=hex(0):\n\n", &r); ok(r == REG_EXIT_SUCCESS, "got exit code %d, expected 0\n", r); verify_reg(hkey, "Wine68a", REG_SZ, NULL, 0, 0); - verify_reg(hkey, "Wine68b", REG_EXPAND_SZ, NULL, 0, TODO_REG_SIZE); + verify_reg(hkey, "Wine68b", REG_EXPAND_SZ, NULL, 0, 0); verify_reg(hkey, "Wine68c", REG_BINARY, NULL, 0, 0); verify_reg(hkey, "Wine68d", REG_DWORD, NULL, 0, 0); - verify_reg(hkey, "Wine68e", REG_MULTI_SZ, NULL, 0, TODO_REG_SIZE); + verify_reg(hkey, "Wine68e", REG_MULTI_SZ, NULL, 0, 0); verify_reg(hkey, "Wine68f", 0x100, NULL, 0, 0); verify_reg(hkey, "Wine68g", 0xabcd, NULL, 0, 0); verify_reg(hkey, "Wine68h", REG_BINARY, NULL, 0, 0); @@ -3830,10 +3830,10 @@ static void test_unicode_import(void) "\"Wine68i\"=hex(0):\n\n", &r); ok(r == REG_EXIT_SUCCESS, "got exit code %d, expected 0\n", r); verify_reg(hkey, "Wine68a", REG_SZ, NULL, 0, 0); - verify_reg(hkey, "Wine68b", REG_EXPAND_SZ, NULL, 0, TODO_REG_SIZE); + verify_reg(hkey, "Wine68b", REG_EXPAND_SZ, NULL, 0, 0); verify_reg(hkey, "Wine68c", REG_BINARY, NULL, 0, 0); verify_reg(hkey, "Wine68d", REG_DWORD, NULL, 0, 0); - verify_reg(hkey, "Wine68e", REG_MULTI_SZ, NULL, 0, TODO_REG_SIZE); + verify_reg(hkey, "Wine68e", REG_MULTI_SZ, NULL, 0, 0); verify_reg(hkey, "Wine68f", 0x100, NULL, 0, 0); verify_reg(hkey, "Wine68g", 0xabcd, NULL, 0, 0); verify_reg(hkey, "Wine68h", REG_BINARY, NULL, 0, 0);

1 0

Hugh McMaster : regedit: Correctly import hex values with no data.
by Alexandre Julliard Oct. 16, 2017

Oct. 16, 2017

Module: wine Branch: master Commit: 84dda235365fbd0a705a9e7e229fb7744e85b0b7 URL: http://source.winehq.org/git/wine.git/?a=commit;h=84dda235365fbd0a705a9e7e2… Author: Hugh McMaster <hugh.mcmaster(a)outlook.com> Date: Mon Oct 16 10:27:25 2017 +0000 regedit: Correctly import hex values with no data. Signed-off-by: Hugh McMaster <hugh.mcmaster(a)outlook.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- programs/regedit/regproc.c | 4 ++++ programs/regedit/tests/regedit.c | 8 ++++---- 2 files changed, 8 insertions(+), 4 deletions(-) diff --git a/programs/regedit/regproc.c b/programs/regedit/regproc.c index 508a8a3..a70a6d1 100644 --- a/programs/regedit/regproc.c +++ b/programs/regedit/regproc.c @@ -885,6 +885,9 @@ static WCHAR *hex_data_state(struct parser *parser, WCHAR *pos) { WCHAR *line = pos; + if (!*line) + goto set_value; + if (!convert_hex_csv_to_hex(parser, &line)) goto invalid; @@ -896,6 +899,7 @@ static WCHAR *hex_data_state(struct parser *parser, WCHAR *pos) prepare_hex_string_data(parser); +set_value: set_state(parser, SET_VALUE); return line; diff --git a/programs/regedit/tests/regedit.c b/programs/regedit/tests/regedit.c index b1f30b2..334360b 100644 --- a/programs/regedit/tests/regedit.c +++ b/programs/regedit/tests/regedit.c @@ -662,10 +662,10 @@ static void test_basic_import(void) "\"Wine22h\"=hex:\n" "\"Wine22i\"=hex(0):\n\n"); verify_reg(hkey, "Wine22a", REG_SZ, NULL, 0, 0); - verify_reg(hkey, "Wine22b", REG_EXPAND_SZ, NULL, 0, TODO_REG_SIZE); + verify_reg(hkey, "Wine22b", REG_EXPAND_SZ, NULL, 0, 0); verify_reg(hkey, "Wine22c", REG_BINARY, NULL, 0, 0); verify_reg(hkey, "Wine22d", REG_DWORD, NULL, 0, 0); - verify_reg(hkey, "Wine22e", REG_MULTI_SZ, NULL, 0, TODO_REG_SIZE); + verify_reg(hkey, "Wine22e", REG_MULTI_SZ, NULL, 0, 0); verify_reg(hkey, "Wine22f", 0x100, NULL, 0, 0); verify_reg(hkey, "Wine22g", 0xabcd, NULL, 0, 0); verify_reg(hkey, "Wine22h", REG_BINARY, NULL, 0, 0); @@ -1077,10 +1077,10 @@ static void test_basic_import_unicode(void) "\"Wine22h\"=hex:\n" "\"Wine22i\"=hex(0):\n\n"); verify_reg(hkey, "Wine22a", REG_SZ, NULL, 0, 0); - verify_reg(hkey, "Wine22b", REG_EXPAND_SZ, NULL, 0, TODO_REG_SIZE); + verify_reg(hkey, "Wine22b", REG_EXPAND_SZ, NULL, 0, 0); verify_reg(hkey, "Wine22c", REG_BINARY, NULL, 0, 0); verify_reg(hkey, "Wine22d", REG_DWORD, NULL, 0, 0); - verify_reg(hkey, "Wine22e", REG_MULTI_SZ, NULL, 0, TODO_REG_SIZE); + verify_reg(hkey, "Wine22e", REG_MULTI_SZ, NULL, 0, 0); verify_reg(hkey, "Wine22f", 0x100, NULL, 0, 0); verify_reg(hkey, "Wine22g", 0xabcd, NULL, 0, 0); verify_reg(hkey, "Wine22h", REG_BINARY, NULL, 0, 0);

1 0

Nikolay Sivov : dwrite/tests: Fix object instance leak (Valgrind).
by Alexandre Julliard Oct. 16, 2017

Oct. 16, 2017

Module: wine Branch: master Commit: e3a29a6c1d3f4655ccee888588f7449487f1778d URL: http://source.winehq.org/git/wine.git/?a=commit;h=e3a29a6c1d3f4655ccee88858… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Mon Oct 16 11:55:39 2017 +0300 dwrite/tests: Fix object instance leak (Valgrind). Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/dwrite/tests/font.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/dlls/dwrite/tests/font.c b/dlls/dwrite/tests/font.c index 4adb022..c28f27a 100644 --- a/dlls/dwrite/tests/font.c +++ b/dlls/dwrite/tests/font.c @@ -7924,6 +7924,9 @@ static void test_inmemory_file_loader(void) IDWriteFontFileStream_Release(stream); IDWriteFontFace_Release(fontface); + ref = IDWriteInMemoryFontFileLoader_Release(inmemory); + ok(ref == 0, "loader not released, %u.\n", ref); + ref = IDWriteFactory5_Release(factory); ok(ref == 0, "factory not released, %u\n", ref); }

1 0

Nikolay Sivov : qedit/tests: Fix some leaks in tests (Valgrind).
by Alexandre Julliard Oct. 16, 2017

Oct. 16, 2017

Module: wine Branch: master Commit: 81eb827c98fa7a5016f802e92b2d9148a572f62c URL: http://source.winehq.org/git/wine.git/?a=commit;h=81eb827c98fa7a5016f802e92… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Mon Oct 16 16:03:43 2017 +0300 qedit/tests: Fix some leaks in tests (Valgrind). Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/qedit/tests/timeline.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/dlls/qedit/tests/timeline.c b/dlls/qedit/tests/timeline.c index b1011fc..94c018e 100644 --- a/dlls/qedit/tests/timeline.c +++ b/dlls/qedit/tests/timeline.c @@ -106,6 +106,9 @@ static void test_timeline(void) hr = IAMTimelineObj_GetTimelineNoRef(obj, &timeline2); ok(hr == E_NOINTERFACE, "Expected E_NOINTERFACE got %08x\n", hr); ok(!timeline2, "Expected NULL got %p\n", timeline2); + + IAMTimelineObj_Release(obj); + IAMTimeline_Release(timeline); } static void test_timelineobj_interfaces(void)

1 0

Martin Storsjo : api-ms-win-core-libraryloader-l1-2-1: Add dll.
by Alexandre Julliard Oct. 16, 2017

Oct. 16, 2017

Module: wine Branch: master Commit: b739ed68299331db383cb25007c1133897814efe URL: http://source.winehq.org/git/wine.git/?a=commit;h=b739ed68299331db383cb2500… Author: Martin Storsjo <martin(a)martin.st> Date: Mon Oct 16 12:21:23 2017 +0300 api-ms-win-core-libraryloader-l1-2-1: Add dll. Signed-off-by: Martin Storsjo <martin(a)martin.st> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- configure | 2 ++ configure.ac | 1 + dlls/api-ms-win-core-libraryloader-l1-2-1/Makefile.in | 1 + .../api-ms-win-core-libraryloader-l1-2-1.spec | 3 +++ tools/make_specfiles | 1 + 5 files changed, 8 insertions(+) diff --git a/configure b/configure index 3d798cb..e2c35a9a 100755 --- a/configure +++ b/configure @@ -931,6 +931,7 @@ enable_api_ms_win_core_kernel32_private_l1_1_1 enable_api_ms_win_core_libraryloader_l1_1_0 enable_api_ms_win_core_libraryloader_l1_1_1 enable_api_ms_win_core_libraryloader_l1_2_0 +enable_api_ms_win_core_libraryloader_l1_2_1 enable_api_ms_win_core_libraryloader_l1_2_2 enable_api_ms_win_core_localization_l1_1_0 enable_api_ms_win_core_localization_l1_2_0 @@ -18021,6 +18022,7 @@ wine_fn_config_dll api-ms-win-core-kernel32-private-l1-1-1 enable_api_ms_win_cor wine_fn_config_dll api-ms-win-core-libraryloader-l1-1-0 enable_api_ms_win_core_libraryloader_l1_1_0 wine_fn_config_dll api-ms-win-core-libraryloader-l1-1-1 enable_api_ms_win_core_libraryloader_l1_1_1 wine_fn_config_dll api-ms-win-core-libraryloader-l1-2-0 enable_api_ms_win_core_libraryloader_l1_2_0 +wine_fn_config_dll api-ms-win-core-libraryloader-l1-2-1 enable_api_ms_win_core_libraryloader_l1_2_1 wine_fn_config_dll api-ms-win-core-libraryloader-l1-2-2 enable_api_ms_win_core_libraryloader_l1_2_2 wine_fn_config_dll api-ms-win-core-localization-l1-1-0 enable_api_ms_win_core_localization_l1_1_0 wine_fn_config_dll api-ms-win-core-localization-l1-2-0 enable_api_ms_win_core_localization_l1_2_0 diff --git a/configure.ac b/configure.ac index 74bea22..5873716 100644 --- a/configure.ac +++ b/configure.ac @@ -2783,6 +2783,7 @@ WINE_CONFIG_DLL(api-ms-win-core-kernel32-private-l1-1-1) WINE_CONFIG_DLL(api-ms-win-core-libraryloader-l1-1-0) WINE_CONFIG_DLL(api-ms-win-core-libraryloader-l1-1-1) WINE_CONFIG_DLL(api-ms-win-core-libraryloader-l1-2-0) +WINE_CONFIG_DLL(api-ms-win-core-libraryloader-l1-2-1) WINE_CONFIG_DLL(api-ms-win-core-libraryloader-l1-2-2) WINE_CONFIG_DLL(api-ms-win-core-localization-l1-1-0) WINE_CONFIG_DLL(api-ms-win-core-localization-l1-2-0) diff --git a/dlls/api-ms-win-core-libraryloader-l1-2-1/Makefile.in b/dlls/api-ms-win-core-libraryloader-l1-2-1/Makefile.in new file mode 100644 index 0000000..e5c790f --- /dev/null +++ b/dlls/api-ms-win-core-libraryloader-l1-2-1/Makefile.in @@ -0,0 +1 @@ +MODULE = api-ms-win-core-libraryloader-l1-2-1.dll diff --git a/dlls/api-ms-win-core-libraryloader-l1-2-1/api-ms-win-core-libraryloader-l1-2-1.spec b/dlls/api-ms-win-core-libraryloader-l1-2-1/api-ms-win-core-libraryloader-l1-2-1.spec new file mode 100644 index 0000000..95a765f --- /dev/null +++ b/dlls/api-ms-win-core-libraryloader-l1-2-1/api-ms-win-core-libraryloader-l1-2-1.spec @@ -0,0 +1,3 @@ +@ stdcall FindResourceW(long wstr wstr) kernel32.FindResourceW +@ stdcall LoadLibraryA(str) kernel32.LoadLibraryA +@ stdcall LoadLibraryW(wstr) kernel32.LoadLibraryW diff --git a/tools/make_specfiles b/tools/make_specfiles index a9ffa29..fa1e623 100755 --- a/tools/make_specfiles +++ b/tools/make_specfiles @@ -226,6 +226,7 @@ my @dll_groups = "api-ms-win-core-kernel32-legacy-l1-1-0", "api-ms-win-core-kernel32-legacy-l1-1-1", "api-ms-win-core-kernel32-private-l1-1-1", + "api-ms-win-core-libraryloader-l1-2-1", "api-ms-win-core-libraryloader-l1-2-2", "api-ms-win-core-localization-l1-1-0", "api-ms-win-core-localization-l1-2-0",

1 0