wine-commits November 2022

wine-commits@winehq.org

1 participants
1171 discussions

Fabian Maurer : ntdll/tests: Fix uninitialized read in ok (Coverity).
by Alexandre Julliard 02 Nov '22

02 Nov '22

Module: wine Branch: master Commit: 57e37c2a9133c7bcb771f99882a1666c4758bfcf URL: https://gitlab.winehq.org/wine/wine/-/commit/57e37c2a9133c7bcb771f99882a166… Author: Fabian Maurer <dark.shadow4(a)web.de> Date: Wed Nov 2 02:32:38 2022 +0100 ntdll/tests: Fix uninitialized read in ok (Coverity). Signed-off-by: Fabian Maurer <dark.shadow4(a)web.de> --- dlls/ntdll/tests/pipe.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/ntdll/tests/pipe.c b/dlls/ntdll/tests/pipe.c index 6f26655b118..0e373dcf7ea 100644 --- a/dlls/ntdll/tests/pipe.c +++ b/dlls/ntdll/tests/pipe.c @@ -2920,13 +2920,13 @@ static void subtest_pipe_name(const struct pipe_name_test *pnt) return; } - ok(pipe != NULL, "expected non-NULL handle, got %p\n", client); + ok(pipe != NULL, "expected non-NULL handle\n"); client = NULL; status = NtCreateFile(&client, SYNCHRONIZE, &attr, &iosb, NULL, 0, FILE_SHARE_READ | FILE_SHARE_WRITE, FILE_OPEN, 0, NULL, 0); ok(status == STATUS_SUCCESS, "Expected success, got %#lx\n", status); - ok(client != NULL, "expected non-NULL handle, got %p\n", client); + ok(client != NULL, "expected non-NULL handle\n"); NtClose(client); if (pnt->no_open_name)

1 0

Fabian Maurer : win32u: Prevent use after free (Coverity).
by Alexandre Julliard 02 Nov '22

02 Nov '22

Module: wine Branch: master Commit: df90a32511d44246930b426ee057318ccaa1a874 URL: https://gitlab.winehq.org/wine/wine/-/commit/df90a32511d44246930b426ee05731… Author: Fabian Maurer <dark.shadow4(a)web.de> Date: Wed Nov 2 04:57:33 2022 +0100 win32u: Prevent use after free (Coverity). Signed-off-by: Fabian Maurer <dark.shadow4(a)web.de> --- dlls/win32u/message.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/dlls/win32u/message.c b/dlls/win32u/message.c index caffd837c8b..346ae8bede7 100644 --- a/dlls/win32u/message.c +++ b/dlls/win32u/message.c @@ -1368,7 +1368,6 @@ static LRESULT call_window_proc( HWND hwnd, UINT msg, WPARAM wparam, LPARAM lpar call_hooks( WH_CALLWNDPROC, HC_ACTION, same_thread, (LPARAM)&cwp, sizeof(cwp) ); dispatch_win_proc_params( params, sizeof(*params) + size ); - if (params != &p) free( params ); /* and finally the WH_CALLWNDPROCRET hook */ cwpret.lResult = result; @@ -1376,6 +1375,9 @@ static LRESULT call_window_proc( HWND hwnd, UINT msg, WPARAM wparam, LPARAM lpar cwpret.wParam = wparam; cwpret.message = msg; cwpret.hwnd = params->hwnd; + + if (params != &p) free( params ); + call_hooks( WH_CALLWNDPROCRET, HC_ACTION, same_thread, (LPARAM)&cwpret, sizeof(cwpret) ); return result; }

1 0

Fabian Maurer : shell32: Prevent double free (Coverity).
by Alexandre Julliard 02 Nov '22

02 Nov '22

Module: wine Branch: master Commit: 04589492638ed14c1c6cbdd09f84eb1df419a0a6 URL: https://gitlab.winehq.org/wine/wine/-/commit/04589492638ed14c1c6cbdd09f84eb… Author: Fabian Maurer <dark.shadow4(a)web.de> Date: Wed Nov 2 04:39:06 2022 +0100 shell32: Prevent double free (Coverity). If we get an error, we might get a double free otherwise. Signed-off-by: Fabian Maurer <dark.shadow4(a)web.de> --- dlls/shell32/dde.c | 1 + 1 file changed, 1 insertion(+) diff --git a/dlls/shell32/dde.c b/dlls/shell32/dde.c index 24482d2c33f..96b5a243f93 100644 --- a/dlls/shell32/dde.c +++ b/dlls/shell32/dde.c @@ -403,6 +403,7 @@ static DWORD parse_dde_command(HSZ hszTopic, WCHAR *command) } free(opcode); + opcode = NULL; for (i = 0; i < argc; i++) free(argv[i]); free(argv);

1 0

Fabian Maurer : wbemproxy/tests: Prevent double free (Coverity).
by Alexandre Julliard 02 Nov '22

02 Nov '22

Module: wine Branch: master Commit: f2459fa8695a9e508648249d5332dab0f49d0f56 URL: https://gitlab.winehq.org/wine/wine/-/commit/f2459fa8695a9e508648249d5332da… Author: Fabian Maurer <dark.shadow4(a)web.de> Date: Wed Nov 2 02:44:26 2022 +0100 wbemproxy/tests: Prevent double free (Coverity). Signed-off-by: Fabian Maurer <dark.shadow4(a)web.de> --- dlls/wbemprox/tests/query.c | 1 - 1 file changed, 1 deletion(-) diff --git a/dlls/wbemprox/tests/query.c b/dlls/wbemprox/tests/query.c index e86f7fd5b62..4503ace0b68 100644 --- a/dlls/wbemprox/tests/query.c +++ b/dlls/wbemprox/tests/query.c @@ -2162,7 +2162,6 @@ static void test_Win32_LogicalDisk( IWbemServices *services ) ok( hr == S_OK, "got %#lx\n", hr ); ok( count == 1, "got %lu\n", count ); IWbemClassObject_Release( obj ); - SysFreeString( query ); IEnumWbemClassObject_Release( result ); SysFreeString( query ); SysFreeString( wql );

1 0

Piotr Caban : msvcrt: Fix errno set on write to read-only file.
by Alexandre Julliard 02 Nov '22

02 Nov '22

Module: wine Branch: master Commit: 7f5f904cfa7e062012960283e98229806c3b27c9 URL: https://gitlab.winehq.org/wine/wine/-/commit/7f5f904cfa7e062012960283e98229… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Wed Nov 2 11:08:02 2022 +0100 msvcrt: Fix errno set on write to read-only file. --- dlls/msvcrt/file.c | 4 ++++ dlls/msvcrt/tests/file.c | 6 ++++++ 2 files changed, 10 insertions(+) diff --git a/dlls/msvcrt/file.c b/dlls/msvcrt/file.c index 81416deed5a..affa2fbdd43 100644 --- a/dlls/msvcrt/file.c +++ b/dlls/msvcrt/file.c @@ -3590,6 +3590,8 @@ int CDECL _write(int fd, const void* buf, unsigned int count) TRACE("WriteFile (fd %d, hand %p) failed-last error (%ld)\n", fd, hand, GetLastError()); msvcrt_set_errno(GetLastError()); + if (GetLastError() == ERROR_ACCESS_DENIED) + *_errno() = EBADF; num_written = -1; } @@ -3718,6 +3720,8 @@ int CDECL _write(int fd, const void* buf, unsigned int count) TRACE("WriteFile/WriteConsoleW (fd %d, hand %p) failed-last error (%ld)\n", fd, hand, GetLastError()); msvcrt_set_errno(GetLastError()); + if (GetLastError() == ERROR_ACCESS_DENIED) + *_errno() = EBADF; release_ioinfo(info); return -1; } diff --git a/dlls/msvcrt/tests/file.c b/dlls/msvcrt/tests/file.c index 91b057ea72e..7a5466968b5 100644 --- a/dlls/msvcrt/tests/file.c +++ b/dlls/msvcrt/tests/file.c @@ -311,6 +311,12 @@ static void test_readmode( BOOL ascii_mode ) ok(l == pl+fp,"line 2 ftell got %ld should be %d in %s\n", l, pl+fp, IOMODE); ok(lstrlenA(buffer) == 2+ao,"line 2 fgets got size %d should be %d in %s\n", lstrlenA(buffer), 2+ao, IOMODE); + + errno = 0xdeadbeef; + _doserrno = 0xdeadbeef; + ok(write(fd, buffer, 1) == -1, "read succeeded on write-only file\n"); + ok(errno == EBADF, "errno = %d\n", errno); + ok(_doserrno == ERROR_ACCESS_DENIED, "doserrno = %ld\n", _doserrno); /* test fread across buffer boundary */ rewind(file);

1 0

Piotr Caban : msvcrt: Fix errno set on read from write-only file.
by Alexandre Julliard 02 Nov '22

02 Nov '22

Module: wine Branch: master Commit: 3fd47b4663af18d815f85f50c9f343bb6aca7066 URL: https://gitlab.winehq.org/wine/wine/-/commit/3fd47b4663af18d815f85f50c9f343… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Wed Nov 2 11:05:07 2022 +0100 msvcrt: Fix errno set on read from write-only file. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=53677 --- dlls/msvcrt/file.c | 6 ++++++ dlls/msvcrt/tests/file.c | 7 +++++++ 2 files changed, 13 insertions(+) diff --git a/dlls/msvcrt/file.c b/dlls/msvcrt/file.c index a3140589481..81416deed5a 100644 --- a/dlls/msvcrt/file.c +++ b/dlls/msvcrt/file.c @@ -2756,6 +2756,8 @@ static int read_utf8(ioinfo *fdinfo, wchar_t *buf, unsigned int count) return 0; }else { msvcrt_set_errno(GetLastError()); + if (GetLastError() == ERROR_ACCESS_DENIED) + *_errno() = EBADF; return -1; } }else if(!num_read) { @@ -2813,6 +2815,8 @@ static int read_utf8(ioinfo *fdinfo, wchar_t *buf, unsigned int count) return 0; }else { msvcrt_set_errno(GetLastError()); + if (GetLastError() == ERROR_ACCESS_DENIED) + *_errno() = EBADF; if (readbuf != min_buf) free(readbuf); return -1; } @@ -3041,6 +3045,8 @@ static int read_i(int fd, ioinfo *fdinfo, void *buf, unsigned int count) { TRACE(":failed-last error (%ld)\n", GetLastError()); msvcrt_set_errno(GetLastError()); + if (GetLastError() == ERROR_ACCESS_DENIED) + *_errno() = EBADF; return -1; } } diff --git a/dlls/msvcrt/tests/file.c b/dlls/msvcrt/tests/file.c index 78ed8931d7a..91b057ea72e 100644 --- a/dlls/msvcrt/tests/file.c +++ b/dlls/msvcrt/tests/file.c @@ -258,6 +258,13 @@ static void test_readmode( BOOL ascii_mode ) write (fd, &padbuffer[i], 1); write (fd, nlbuffer, strlen(nlbuffer)); write (fd, outbuffer, sizeof (outbuffer)); + + errno = 0xdeadbeef; + _doserrno = 0xdeadbeef; + ok(read(fd, buffer, 1) == -1, "read succeeded on write-only file\n"); + ok(errno == EBADF, "errno = %d\n", errno); + ok(_doserrno == ERROR_ACCESS_DENIED, "doserrno = %ld\n", _doserrno); + close (fd); if (ascii_mode) {

1 0

Brendan Shanks : winemac: Remove pre-Mac OS X 10.8 workarounds.
by Alexandre Julliard 02 Nov '22

02 Nov '22

Module: wine Branch: master Commit: c69f19552be4b2b61ae25c2fe8cd4ca1791c435c URL: https://gitlab.winehq.org/wine/wine/-/commit/c69f19552be4b2b61ae25c2fe8cd4c… Author: Brendan Shanks <bshanks(a)codeweavers.com> Date: Sat Oct 8 09:42:35 2022 -0500 winemac: Remove pre-Mac OS X 10.8 workarounds. --- dlls/winemac.drv/cocoa_app.m | 16 ++---- dlls/winemac.drv/display.c | 113 +++++++++++++++++-------------------------- 2 files changed, 47 insertions(+), 82 deletions(-) diff --git a/dlls/winemac.drv/cocoa_app.m b/dlls/winemac.drv/cocoa_app.m index b13bc678c6d..d9d56da20f8 100644 --- a/dlls/winemac.drv/cocoa_app.m +++ b/dlls/winemac.drv/cocoa_app.m @@ -760,13 +760,8 @@ static NSString* WineLocalizedString(unsigned int stringID) if (CGDisplayModeGetWidth(mode1) != CGDisplayModeGetWidth(mode2)) return FALSE; if (CGDisplayModeGetHeight(mode1) != CGDisplayModeGetHeight(mode2)) return FALSE; - -#if defined(MAC_OS_X_VERSION_10_8) && MAC_OS_X_VERSION_MAX_ALLOWED >= MAC_OS_X_VERSION_10_8 - if (&CGDisplayModeGetPixelWidth != NULL && - CGDisplayModeGetPixelWidth(mode1) != CGDisplayModeGetPixelWidth(mode2)) return FALSE; - if (&CGDisplayModeGetPixelHeight != NULL && - CGDisplayModeGetPixelHeight(mode1) != CGDisplayModeGetPixelHeight(mode2)) return FALSE; -#endif + if (CGDisplayModeGetPixelWidth(mode1) != CGDisplayModeGetPixelWidth(mode2)) return FALSE; + if (CGDisplayModeGetPixelHeight(mode1) != CGDisplayModeGetPixelHeight(mode2)) return FALSE; encoding1 = [(NSString*)CGDisplayModeCopyPixelEncoding(mode1) autorelease]; encoding2 = [(NSString*)CGDisplayModeCopyPixelEncoding(mode2) autorelease]; @@ -791,12 +786,7 @@ static NSString* WineLocalizedString(unsigned int stringID) - (NSArray*)modesMatchingMode:(CGDisplayModeRef)mode forDisplay:(CGDirectDisplayID)displayID { NSMutableArray* ret = [NSMutableArray array]; - NSDictionary* options = nil; - -#if defined(MAC_OS_X_VERSION_10_8) && MAC_OS_X_VERSION_MAX_ALLOWED >= MAC_OS_X_VERSION_10_8 - options = [NSDictionary dictionaryWithObject:[NSNumber numberWithBool:TRUE] - forKey:(NSString*)kCGDisplayShowDuplicateLowResolutionModes]; -#endif + NSDictionary* options = @{ (NSString*)kCGDisplayShowDuplicateLowResolutionModes: @YES }; NSArray *modes = [(NSArray*)CGDisplayCopyAllDisplayModes(displayID, (CFDictionaryRef)options) autorelease]; for (id candidateModeObject in modes) diff --git a/dlls/winemac.drv/display.c b/dlls/winemac.drv/display.c index 68069df7cc5..f24ada839ab 100644 --- a/dlls/winemac.drv/display.c +++ b/dlls/winemac.drv/display.c @@ -176,18 +176,12 @@ static BOOL write_display_settings(HKEY parent_hkey, CGDirectDisplayID displayID val = CGDisplayModeGetIOFlags(display_mode); if (!set_setting_value(display_hkey, "IOFlags", val)) goto fail; - -#if defined(MAC_OS_X_VERSION_10_8) && MAC_OS_X_VERSION_MAX_ALLOWED >= MAC_OS_X_VERSION_10_8 - if (&CGDisplayModeGetPixelWidth != NULL && &CGDisplayModeGetPixelHeight != NULL) - { - val = CGDisplayModeGetPixelWidth(display_mode); - if (!set_setting_value(display_hkey, "PixelWidth", val)) - goto fail; - val = CGDisplayModeGetPixelHeight(display_mode); - if (!set_setting_value(display_hkey, "PixelHeight", val)) - goto fail; - } -#endif + val = CGDisplayModeGetPixelWidth(display_mode); + if (!set_setting_value(display_hkey, "PixelWidth", val)) + goto fail; + val = CGDisplayModeGetPixelHeight(display_mode); + if (!set_setting_value(display_hkey, "PixelHeight", val)) + goto fail; pixel_encoding = CGDisplayModeCopyPixelEncoding(display_mode); len = CFStringGetLength(pixel_encoding); @@ -370,17 +364,8 @@ static BOOL display_mode_matches_descriptor(CGDisplayModeRef mode, const struct if (fabs(desc->refresh - mode_refresh) > 0.1) return FALSE; -#if defined(MAC_OS_X_VERSION_10_8) && MAC_OS_X_VERSION_MAX_ALLOWED >= MAC_OS_X_VERSION_10_8 - if (&CGDisplayModeGetPixelWidth != NULL && &CGDisplayModeGetPixelHeight != NULL) - { - if (CGDisplayModeGetPixelWidth(mode) != desc->pixel_width || - CGDisplayModeGetPixelHeight(mode) != desc->pixel_height) - return FALSE; - } - else -#endif - if (CGDisplayModeGetWidth(mode) != desc->pixel_width || - CGDisplayModeGetHeight(mode) != desc->pixel_height) + if (CGDisplayModeGetPixelWidth(mode) != desc->pixel_width || + CGDisplayModeGetPixelHeight(mode) != desc->pixel_height) return FALSE; mode_pixel_encoding = CGDisplayModeCopyPixelEncoding(mode); @@ -420,7 +405,6 @@ static int get_default_bpp(void) } -#if defined(MAC_OS_X_VERSION_10_8) && MAC_OS_X_VERSION_MAX_ALLOWED >= MAC_OS_X_VERSION_10_8 static CFDictionaryRef create_mode_dict(CGDisplayModeRef display_mode, BOOL is_original) { CFDictionaryRef ret; @@ -542,7 +526,6 @@ static BOOL mode_is_preferred(CGDisplayModeRef new_mode, CGDisplayModeRef old_mo /* Otherwise, prefer the mode with the smaller pixel size. */ return new_width_pixels < old_width_pixels && new_height_pixels < old_height_pixels; } -#endif static CFComparisonResult mode_compare(const void *p1, const void *p2, void *context) @@ -605,56 +588,48 @@ static CFComparisonResult mode_compare(const void *p1, const void *p2, void *con static CFArrayRef copy_display_modes(CGDirectDisplayID display, BOOL include_unsupported) { CFArrayRef modes = NULL; + CFDictionaryRef options; + struct display_mode_descriptor* desc; + CFMutableDictionaryRef modes_by_size; + CFIndex i, count; + CGDisplayModeRef* mode_array; -#if defined(MAC_OS_X_VERSION_10_8) && MAC_OS_X_VERSION_MAX_ALLOWED >= MAC_OS_X_VERSION_10_8 - if (&CGDisplayModeGetPixelWidth != NULL && &CGDisplayModeGetPixelHeight != NULL) - { - CFDictionaryRef options; - struct display_mode_descriptor* desc; - CFMutableDictionaryRef modes_by_size; - CFIndex i, count; - CGDisplayModeRef* mode_array; - - options = CFDictionaryCreate(NULL, (const void**)&kCGDisplayShowDuplicateLowResolutionModes, - (const void**)&kCFBooleanTrue, 1, &kCFTypeDictionaryKeyCallBacks, - &kCFTypeDictionaryValueCallBacks); - - modes = CGDisplayCopyAllDisplayModes(display, options); - if (options) - CFRelease(options); - if (!modes) - return NULL; - - desc = create_original_display_mode_descriptor(display); - - modes_by_size = CFDictionaryCreateMutable(NULL, 0, &kCFTypeDictionaryKeyCallBacks, &kCFTypeDictionaryValueCallBacks); - count = CFArrayGetCount(modes); - for (i = 0; i < count; i++) - { - CGDisplayModeRef new_mode = (CGDisplayModeRef)CFArrayGetValueAtIndex(modes, i); - BOOL new_is_original = display_mode_matches_descriptor(new_mode, desc); - CFDictionaryRef key = create_mode_dict(new_mode, new_is_original); - CGDisplayModeRef old_mode = (CGDisplayModeRef)CFDictionaryGetValue(modes_by_size, key); + options = CFDictionaryCreate(NULL, (const void**)&kCGDisplayShowDuplicateLowResolutionModes, + (const void**)&kCFBooleanTrue, 1, &kCFTypeDictionaryKeyCallBacks, + &kCFTypeDictionaryValueCallBacks); - if (mode_is_preferred(new_mode, old_mode, desc, include_unsupported)) - CFDictionarySetValue(modes_by_size, key, new_mode); + modes = CGDisplayCopyAllDisplayModes(display, options); + if (options) + CFRelease(options); + if (!modes) + return NULL; - CFRelease(key); - } + desc = create_original_display_mode_descriptor(display); - free_display_mode_descriptor(desc); - CFRelease(modes); + modes_by_size = CFDictionaryCreateMutable(NULL, 0, &kCFTypeDictionaryKeyCallBacks, &kCFTypeDictionaryValueCallBacks); + count = CFArrayGetCount(modes); + for (i = 0; i < count; i++) + { + CGDisplayModeRef new_mode = (CGDisplayModeRef)CFArrayGetValueAtIndex(modes, i); + BOOL new_is_original = display_mode_matches_descriptor(new_mode, desc); + CFDictionaryRef key = create_mode_dict(new_mode, new_is_original); + CGDisplayModeRef old_mode = (CGDisplayModeRef)CFDictionaryGetValue(modes_by_size, key); + + if (mode_is_preferred(new_mode, old_mode, desc, include_unsupported)) + CFDictionarySetValue(modes_by_size, key, new_mode); - count = CFDictionaryGetCount(modes_by_size); - mode_array = malloc(count * sizeof(mode_array[0])); - CFDictionaryGetKeysAndValues(modes_by_size, NULL, (const void **)mode_array); - modes = CFArrayCreate(NULL, (const void **)mode_array, count, &kCFTypeArrayCallBacks); - free(mode_array); - CFRelease(modes_by_size); + CFRelease(key); } - else -#endif - modes = CGDisplayCopyAllDisplayModes(display, NULL); + + free_display_mode_descriptor(desc); + CFRelease(modes); + + count = CFDictionaryGetCount(modes_by_size); + mode_array = malloc(count * sizeof(mode_array[0])); + CFDictionaryGetKeysAndValues(modes_by_size, NULL, (const void **)mode_array); + modes = CFArrayCreate(NULL, (const void **)mode_array, count, &kCFTypeArrayCallBacks); + free(mode_array); + CFRelease(modes_by_size); if (modes) {

1 0

Zhiyi Zhang : comctl32/button: Initialize dtFlags in PB_Paint().
by Alexandre Julliard 02 Nov '22

02 Nov '22

Module: wine Branch: master Commit: 398cb9595a01df26ee380a850c5084a453cb72a6 URL: https://gitlab.winehq.org/wine/wine/-/commit/398cb9595a01df26ee380a850c5084… Author: Zhiyi Zhang <zzhang(a)codeweavers.com> Date: Wed Nov 2 16:02:52 2022 +0800 comctl32/button: Initialize dtFlags in PB_Paint(). Fix an uninitialized variable warning. --- dlls/comctl32/button.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/comctl32/button.c b/dlls/comctl32/button.c index 089534b2b9d..6c04c458a6a 100644 --- a/dlls/comctl32/button.c +++ b/dlls/comctl32/button.c @@ -1833,7 +1833,7 @@ static void BUTTON_DrawThemedLabel(const BUTTON_INFO *info, HDC hdc, UINT text_f static void PB_Paint( const BUTTON_INFO *infoPtr, HDC hDC, UINT action ) { RECT rc, labelRect, imageRect, textRect; - UINT dtFlags, uState; + UINT dtFlags = (UINT)-1, uState; HPEN hOldPen, hpen; HBRUSH hOldBrush; INT oldBkMode;

1 0

Alexandre Julliard : makedep: Always generate a fake module for the native architecture.
by Alexandre Julliard 02 Nov '22

02 Nov '22

Module: wine Branch: master Commit: 1e58882e77178562723166502f9b80d6682bcda9 URL: https://gitlab.winehq.org/wine/wine/-/commit/1e58882e77178562723166502f9b80… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Nov 2 11:26:22 2022 +0100 makedep: Always generate a fake module for the native architecture. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=53861 --- tools/makedep.c | 70 ++++++++++++++++++++++++++++++--------------------------- 1 file changed, 37 insertions(+), 33 deletions(-) diff --git a/tools/makedep.c b/tools/makedep.c index 913b3af7620..ca1efed4533 100644 --- a/tools/makedep.c +++ b/tools/makedep.c @@ -3274,6 +3274,38 @@ static char *get_unix_lib_name( struct makefile *make ) } +/******************************************************************* + * output_fake_module + */ +static void output_fake_module( struct makefile *make ) +{ + unsigned int arch = 0; /* fake modules are always native */ + const char *spec_file = NULL, *name = strmake( "%s%s", arch_pe_dirs[arch], make->module ); + + if (!make->is_exe) spec_file = src_dir_path( make, replace_extension( make->module, ".dll", ".spec" )); + + strarray_add( &make->all_targets[arch], name ); + add_install_rule( make, make->module, arch, name, strmake( "d$(dlldir)/%s", name )); + + output( "%s:", obj_dir_path( make, name )); + if (spec_file) output_filename( spec_file ); + output_filenames_obj_dir( make, make->res_files[arch] ); + output_filename( tools_path( make, "winebuild" )); + output_filename( tools_path( make, "winegcc" )); + output( "\n" ); + output_winegcc_command( make, arch ); + output_filename( "-Wb,--fake-module" ); + if (spec_file) + { + output_filename( "-shared" ); + output_filename( spec_file ); + } + output_filenames( make->extradllflags ); + output_filenames_obj_dir( make, make->res_files[arch] ); + output( "\n" ); +} + + /******************************************************************* * output_module */ @@ -3348,37 +3380,8 @@ static void output_module( struct makefile *make, unsigned int arch ) output_filenames( all_libs ); output_filename( arch_make_variable( "LDFLAGS", arch )); output( "\n" ); -} - -/******************************************************************* - * output_fake_module - */ -static void output_fake_module( struct makefile *make, unsigned int arch ) -{ - const char *spec_file = NULL, *name = strmake( "%s%s", arch_pe_dirs[arch], make->module ); - - if (!make->is_exe) spec_file = src_dir_path( make, replace_extension( make->module, ".dll", ".spec" )); - - strarray_add( &make->all_targets[arch], name ); - add_install_rule( make, make->module, arch, name, strmake( "d$(dlldir)/%s", name )); - - output( "%s:", obj_dir_path( make, name )); - if (spec_file) output_filename( spec_file ); - output_filenames_obj_dir( make, make->res_files[arch] ); - output_filename( tools_path( make, "winebuild" )); - output_filename( tools_path( make, "winegcc" )); - output( "\n" ); - output_winegcc_command( make, arch ); - output_filename( "-Wb,--fake-module" ); - if (spec_file) - { - output_filename( "-shared" ); - output_filename( spec_file ); - } - output_filenames( make->extradllflags ); - output_filenames_obj_dir( make, make->res_files[arch] ); - output( "\n" ); + if (!make->data_only && !arch && *dll_ext) output_fake_module( make ); } @@ -3842,10 +3845,11 @@ static void output_sources( struct makefile *make ) } else if (make->module) { - for (arch = 0; arch < archs.count; arch++) + if (!make->use_msvcrt) output_module( make, 0 ); + else { - if (!is_multiarch( arch ) == !make->use_msvcrt) output_module( make, arch ); - else if (is_multiarch( arch ) && *dll_ext) output_fake_module( make, arch ); + for (arch = 0; arch < archs.count; arch++) + if (is_multiarch( arch )) output_module( make, arch ); } if (make->unixlib) output_unix_lib( make ); if (make->importlib) for (arch = 0; arch < archs.count; arch++) output_import_lib( make, arch );

1 0

Bartosz Kosiorek : msvcrt: Add _swprintf_p_l implementation.
by Alexandre Julliard 01 Nov '22

01 Nov '22

Module: wine Branch: master Commit: 7be72ce2a708ec88aa2362352f37db30529251c4 URL: https://gitlab.winehq.org/wine/wine/-/commit/7be72ce2a708ec88aa2362352f37db… Author: Bartosz Kosiorek <gang65(a)poczta.onet.pl> Date: Fri Oct 28 15:26:36 2022 +0200 msvcrt: Add _swprintf_p_l implementation. --- dlls/msvcr100/msvcr100.spec | 2 +- dlls/msvcr110/msvcr110.spec | 2 +- dlls/msvcr120/msvcr120.spec | 2 +- dlls/msvcr120_app/msvcr120_app.spec | 2 +- dlls/msvcr80/msvcr80.spec | 2 +- dlls/msvcr90/msvcr90.spec | 2 +- dlls/msvcrt/wcs.c | 16 ++++++++++++++++ 7 files changed, 22 insertions(+), 6 deletions(-) diff --git a/dlls/msvcr100/msvcr100.spec b/dlls/msvcr100/msvcr100.spec index e00d0c181bd..3fcdcb7121b 100644 --- a/dlls/msvcr100/msvcr100.spec +++ b/dlls/msvcr100/msvcr100.spec @@ -1378,7 +1378,7 @@ @ varargs _swprintf(ptr wstr) @ varargs _swprintf_c(ptr long str) @ varargs _swprintf_c_l(ptr long str ptr) -@ stub _swprintf_p +@ varargs _swprintf_p(ptr long wstr) @ varargs _swprintf_p_l(ptr long wstr ptr) @ varargs _swprintf_s_l(ptr long wstr ptr) @ varargs _swscanf_l(wstr wstr ptr) diff --git a/dlls/msvcr110/msvcr110.spec b/dlls/msvcr110/msvcr110.spec index d6e860e18ed..04f496b2fe1 100644 --- a/dlls/msvcr110/msvcr110.spec +++ b/dlls/msvcr110/msvcr110.spec @@ -1735,7 +1735,7 @@ @ varargs _swprintf(ptr wstr) @ varargs _swprintf_c(ptr long str) @ varargs _swprintf_c_l(ptr long str ptr) -@ stub _swprintf_p +@ varargs _swprintf_p(ptr long wstr) @ varargs _swprintf_p_l(ptr long wstr ptr) @ varargs _swprintf_s_l(ptr long wstr ptr) @ varargs _swscanf_l(wstr wstr ptr) diff --git a/dlls/msvcr120/msvcr120.spec b/dlls/msvcr120/msvcr120.spec index 1ad88c3d705..17cf2f87da4 100644 --- a/dlls/msvcr120/msvcr120.spec +++ b/dlls/msvcr120/msvcr120.spec @@ -1753,7 +1753,7 @@ @ varargs _swprintf(ptr wstr) @ varargs _swprintf_c(ptr long str) @ varargs _swprintf_c_l(ptr long str ptr) -@ stub _swprintf_p +@ varargs _swprintf_p(ptr long wstr) @ varargs _swprintf_p_l(ptr long wstr ptr) @ varargs _swprintf_s_l(ptr long wstr ptr) @ varargs _swscanf_l(wstr wstr ptr) diff --git a/dlls/msvcr120_app/msvcr120_app.spec b/dlls/msvcr120_app/msvcr120_app.spec index 10b5afacc0b..b9e8bc0f7c8 100644 --- a/dlls/msvcr120_app/msvcr120_app.spec +++ b/dlls/msvcr120_app/msvcr120_app.spec @@ -1465,7 +1465,7 @@ @ varargs _swprintf(ptr wstr) msvcr120._swprintf @ varargs _swprintf_c(ptr long str) msvcr120._swprintf_c @ varargs _swprintf_c_l(ptr long str ptr) msvcr120._swprintf_c_l -@ stub _swprintf_p +@ varargs _swprintf_p(ptr long wstr) msvcr120._swprintf_p @ varargs _swprintf_p_l(ptr long wstr ptr) msvcr120._swprintf_p_l @ varargs _swprintf_s_l(ptr long wstr ptr) msvcr120._swprintf_s_l @ varargs _swscanf_l(wstr wstr ptr) msvcr120._swscanf_l diff --git a/dlls/msvcr80/msvcr80.spec b/dlls/msvcr80/msvcr80.spec index fe93549517a..eda6232f3a3 100644 --- a/dlls/msvcr80/msvcr80.spec +++ b/dlls/msvcr80/msvcr80.spec @@ -1058,7 +1058,7 @@ @ varargs _swprintf(ptr wstr) @ varargs _swprintf_c(ptr long str) @ varargs _swprintf_c_l(ptr long str ptr) -@ stub _swprintf_p +@ varargs _swprintf_p(ptr long wstr) @ varargs _swprintf_p_l(ptr long wstr ptr) @ varargs _swprintf_s_l(ptr long wstr ptr) @ varargs _swscanf_l(wstr wstr ptr) diff --git a/dlls/msvcr90/msvcr90.spec b/dlls/msvcr90/msvcr90.spec index cebea457e8b..037292a3d58 100644 --- a/dlls/msvcr90/msvcr90.spec +++ b/dlls/msvcr90/msvcr90.spec @@ -1033,7 +1033,7 @@ @ varargs _swprintf(ptr wstr) @ varargs _swprintf_c(ptr long str) @ varargs _swprintf_c_l(ptr long str ptr) -@ stub _swprintf_p +@ varargs _swprintf_p(ptr long wstr) @ varargs _swprintf_p_l(ptr long wstr ptr) @ varargs _swprintf_s_l(ptr long wstr ptr) @ varargs _swscanf_l(wstr wstr ptr) diff --git a/dlls/msvcrt/wcs.c b/dlls/msvcrt/wcs.c index 2c4479f54e0..afc2498dae2 100644 --- a/dlls/msvcrt/wcs.c +++ b/dlls/msvcrt/wcs.c @@ -1886,6 +1886,22 @@ int WINAPIV _sprintf_p(char *buffer, size_t length, const char *format, ...) } #endif +/********************************************************************* + * _swprintf_p (MSVCRT.@) + */ +int WINAPIV _swprintf_p(wchar_t *buffer, size_t length, + const wchar_t *format, ...) +{ + va_list valist; + int r; + + va_start(valist, format); + r = vswprintf_p_l_opt(buffer, length, format, 0, NULL, valist); + va_end(valist); + + return r; +} + /********************************************************************* * _swprintf_p_l (MSVCRT.@) */

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 November 2022