wine-commits September 2013

wine-commits@winehq.org

2 participants
645 discussions

Frédéric Delanoy : advapi32: Use BOOL type where appropriate.
by Alexandre Julliard 25 Sep '13

25 Sep '13

Module: wine Branch: master Commit: ce88fd16ae985e1a114298ec10352f674d40bde1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=ce88fd16ae985e1a114298ec1… Author: Frédéric Delanoy <frederic.delanoy(a)gmail.com> Date: Wed Sep 25 15:01:43 2013 +0200 advapi32: Use BOOL type where appropriate. --- dlls/advapi32/registry.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/advapi32/registry.c b/dlls/advapi32/registry.c index e4f7586..e999ce6 100644 --- a/dlls/advapi32/registry.c +++ b/dlls/advapi32/registry.c @@ -79,7 +79,7 @@ static const WCHAR * const root_key_names[] = static HKEY special_root_keys[NB_SPECIAL_ROOT_KEYS]; static BOOL hkcu_cache_disabled; -static const int is_win64 = (sizeof(void *) > sizeof(int)); +static const BOOL is_win64 = (sizeof(void *) > sizeof(int)); /* check if value type needs string conversion (Ansi<->Unicode) */ static inline int is_string( DWORD type )

1 0

Jacek Caban : mshtml: Wine Gecko 2.24 release.
by Alexandre Julliard 25 Sep '13

25 Sep '13

Module: wine Branch: master Commit: 0be56d27d2d4b22367313fa4c6f1e65865c90438 URL: http://source.winehq.org/git/wine.git/?a=commit;h=0be56d27d2d4b22367313fa4c… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Wed Sep 25 14:17:08 2013 +0200 mshtml: Wine Gecko 2.24 release. --- dlls/appwiz.cpl/addons.c | 6 +- dlls/mshtml/htmlelem.c | 24 +++--- dlls/mshtml/htmlwindow.c | 7 ++- dlls/mshtml/nsembed.c | 2 +- dlls/mshtml/nsiface.idl | 180 +++++++++++++++++++++++++++------------------- 5 files changed, 127 insertions(+), 92 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=0be56d27d2d4b22367313…

1 0

Piotr Caban : msvcrt: Call MSVCRT_strtoi64_l in strtoul implementation.
by Alexandre Julliard 25 Sep '13

25 Sep '13

Module: wine Branch: master Commit: a449b2d57b6ad84f11c90346e01b8acef546b4f5 URL: http://source.winehq.org/git/wine.git/?a=commit;h=a449b2d57b6ad84f11c90346e… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Wed Sep 25 14:41:52 2013 +0200 msvcrt: Call MSVCRT_strtoi64_l in strtoul implementation. --- dlls/msvcrt/string.c | 47 ++++++++++++++++--------------------------- dlls/msvcrt/tests/string.c | 20 ++++++++++++++++++ 2 files changed, 38 insertions(+), 29 deletions(-) diff --git a/dlls/msvcrt/string.c b/dlls/msvcrt/string.c index cdce4a2..861389b 100644 --- a/dlls/msvcrt/string.c +++ b/dlls/msvcrt/string.c @@ -726,35 +726,6 @@ int CDECL __STRINGTOLD( MSVCRT__LDOUBLE *value, char **endptr, const char *str, return 0; } -/****************************************************************** - * strtoul (MSVCRT.@) - */ -MSVCRT_ulong CDECL MSVCRT_strtoul(const char* nptr, char** end, int base) -{ - /* wrapper to forward libc error code to msvcrt's error codes */ - unsigned long ret; - - errno = 0; - ret = strtoul(nptr, end, base); - switch (errno) - { - case ERANGE: *MSVCRT__errno() = MSVCRT_ERANGE; break; - case EINVAL: *MSVCRT__errno() = MSVCRT_EINVAL; break; - default: - /* cope with the fact that we may use 64bit long integers on libc - * while msvcrt always uses 32bit long integers - */ - if (ret > MSVCRT_ULONG_MAX) - { - ret = MSVCRT_ULONG_MAX; - *MSVCRT__errno() = MSVCRT_ERANGE; - } - break; - } - - return ret; -} - /********************************************************************* * strlen (MSVCRT.@) */ @@ -918,6 +889,24 @@ MSVCRT_long CDECL MSVCRT_strtol(const char* nptr, char** end, int base) return ret; } +/****************************************************************** + * strtoul (MSVCRT.@) + */ +MSVCRT_ulong CDECL MSVCRT_strtoul(const char* nptr, char** end, int base) +{ + __int64 ret = MSVCRT_strtoi64_l(nptr, end, base, NULL); + + if(ret > MSVCRT_ULONG_MAX) { + ret = MSVCRT_ULONG_MAX; + *MSVCRT__errno() = MSVCRT_ERANGE; + }else if(ret < -(__int64)MSVCRT_ULONG_MAX) { + ret = 1; + *MSVCRT__errno() = MSVCRT_ERANGE; + } + + return ret; +} + /********************************************************************* * _strtoui64_l (MSVCRT.@) * diff --git a/dlls/msvcrt/tests/string.c b/dlls/msvcrt/tests/string.c index dc90b38..3877d18 100644 --- a/dlls/msvcrt/tests/string.c +++ b/dlls/msvcrt/tests/string.c @@ -1341,6 +1341,26 @@ static void test_strtol(void) ul = strtoul("9223372036854775807L", &e, 0); ok(ul==4294967295ul, "wrong value %u\n", ul); ok(errno == ERANGE, "wrong errno %d\n", errno); + + errno = 0; + ul = strtoul("-2", NULL, 0); + ok(ul == -2, "wrong value %u\n", ul); + ok(errno == 0, "wrong errno %d\n", errno); + + errno = 0; + ul = strtoul("-4294967294", NULL, 0); + ok(ul == 2, "wrong value %u\n", ul); + ok(errno == 0, "wrong errno %d\n", errno); + + errno = 0; + ul = strtoul("-4294967295", NULL, 0); + ok(ul==1, "wrong value %u\n", ul); + ok(errno == 0, "wrong errno %d\n", errno); + + errno = 0; + ul = strtoul("-4294967296", NULL, 0); + ok(ul == 1, "wrong value %u\n", ul); + ok(errno == ERANGE, "wrong errno %d\n", errno); } static void test_strnlen(void)

1 0

Piotr Caban : msvcrt: Call MSVCRT_strtoi64_l in strtol implementation.
by Alexandre Julliard 25 Sep '13

25 Sep '13

Module: wine Branch: master Commit: c3c44544ff0e2446979117daebddb70124a0afe1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=c3c44544ff0e2446979117dae… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Wed Sep 25 14:41:40 2013 +0200 msvcrt: Call MSVCRT_strtoi64_l in strtol implementation. --- dlls/msvcrt/string.c | 52 +++++++++++++++++-------------------------------- 1 files changed, 18 insertions(+), 34 deletions(-) diff --git a/dlls/msvcrt/string.c b/dlls/msvcrt/string.c index d3bde66..cdce4a2 100644 --- a/dlls/msvcrt/string.c +++ b/dlls/msvcrt/string.c @@ -727,40 +727,6 @@ int CDECL __STRINGTOLD( MSVCRT__LDOUBLE *value, char **endptr, const char *str, } /****************************************************************** - * strtol (MSVCRT.@) - */ -MSVCRT_long CDECL MSVCRT_strtol(const char* nptr, char** end, int base) -{ - /* wrapper to forward libc error code to msvcrt's error codes */ - long ret; - - errno = 0; - ret = strtol(nptr, end, base); - switch (errno) - { - case ERANGE: *MSVCRT__errno() = MSVCRT_ERANGE; break; - case EINVAL: *MSVCRT__errno() = MSVCRT_EINVAL; break; - default: - /* cope with the fact that we may use 64bit long integers on libc - * while msvcrt always uses 32bit long integers - */ - if (ret > MSVCRT_LONG_MAX) - { - ret = MSVCRT_LONG_MAX; - *MSVCRT__errno() = MSVCRT_ERANGE; - } - else if (ret < -MSVCRT_LONG_MAX - 1) - { - ret = -MSVCRT_LONG_MAX - 1; - *MSVCRT__errno() = MSVCRT_ERANGE; - } - break; - } - - return ret; -} - -/****************************************************************** * strtoul (MSVCRT.@) */ MSVCRT_ulong CDECL MSVCRT_strtoul(const char* nptr, char** end, int base) @@ -934,6 +900,24 @@ int __cdecl MSVCRT_atoi(const char *str) return minus ? -ret : ret; } +/****************************************************************** + * strtol (MSVCRT.@) + */ +MSVCRT_long CDECL MSVCRT_strtol(const char* nptr, char** end, int base) +{ + __int64 ret = MSVCRT_strtoi64_l(nptr, end, base, NULL); + + if(ret > MSVCRT_LONG_MAX) { + ret = MSVCRT_LONG_MAX; + *MSVCRT__errno() = MSVCRT_ERANGE; + } else if(ret < MSVCRT_LONG_MIN) { + ret = MSVCRT_LONG_MIN; + *MSVCRT__errno() = MSVCRT_ERANGE; + } + + return ret; +} + /********************************************************************* * _strtoui64_l (MSVCRT.@) *

1 0

Piotr Caban : msvcrt: Fix internal LONG_MAX and ULONG_MAX definitions.
by Alexandre Julliard 25 Sep '13

25 Sep '13

Module: wine Branch: master Commit: b01f8881293cc9661d3d5ba919b13f1a2fbc9687 URL: http://source.winehq.org/git/wine.git/?a=commit;h=b01f8881293cc9661d3d5ba91… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Wed Sep 25 14:41:30 2013 +0200 msvcrt: Fix internal LONG_MAX and ULONG_MAX definitions. --- dlls/msvcrt/msvcrt.h | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/msvcrt/msvcrt.h b/dlls/msvcrt/msvcrt.h index 98922af..4e41a7d 100644 --- a/dlls/msvcrt/msvcrt.h +++ b/dlls/msvcrt/msvcrt.h @@ -41,9 +41,9 @@ #include "windef.h" #include "winbase.h" -#define MSVCRT_LONG_MAX 0x7fffffffL +#define MSVCRT_LONG_MAX 0x7fffffff #define MSVCRT_LONG_MIN (-MSVCRT_LONG_MAX-1) -#define MSVCRT_ULONG_MAX 0xffffffffUL +#define MSVCRT_ULONG_MAX 0xffffffff #define MSVCRT_I64_MAX (((__int64)0x7fffffff << 32) | 0xffffffff) #define MSVCRT_I64_MIN (-MSVCRT_I64_MAX-1) #define MSVCRT_UI64_MAX (((unsigned __int64)0xffffffff << 32) | 0xffffffff)

1 0

Lu Jiashu : winmm: Clone the whole adwInfo array in MMIO_Open.
by Alexandre Julliard 25 Sep '13

25 Sep '13

Module: wine Branch: master Commit: a0ed65f5937e6eb13f6b2b345d8d27fbd4616c32 URL: http://source.winehq.org/git/wine.git/?a=commit;h=a0ed65f5937e6eb13f6b2b345… Author: Lu Jiashu <lujiashu(a)163.com> Date: Tue Sep 24 11:40:54 2013 +0800 winmm: Clone the whole adwInfo array in MMIO_Open. --- dlls/winmm/mmio.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/winmm/mmio.c b/dlls/winmm/mmio.c index 73cad60..c8f3745 100644 --- a/dlls/winmm/mmio.c +++ b/dlls/winmm/mmio.c @@ -664,7 +664,7 @@ static HMMIO MMIO_Open(LPSTR szFileName, MMIOINFO* refmminfo, DWORD dwOpenFlags, wm->bBufferLoaded = TRUE; /* see mmioDosIOProc for that one */ - wm->info.adwInfo[0] = refmminfo->adwInfo[0]; + memcpy( wm->info.adwInfo, refmminfo->adwInfo, sizeof(wm->info.adwInfo) ); /* call IO proc to actually open file */ refmminfo->wErrorRet = send_message(wm->ioProc, &wm->info, MMIOM_OPEN,

1 0

Alistair Leslie-Hughes : oledb32: Correct DataLinks interface for win64.
by Alexandre Julliard 25 Sep '13

25 Sep '13

Module: wine Branch: master Commit: 3d193930ab33ab521fff8028fc309d605432e2f6 URL: http://source.winehq.org/git/wine.git/?a=commit;h=3d193930ab33ab521fff8028f… Author: Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> Date: Mon Sep 16 09:44:39 2013 +1000 oledb32: Correct DataLinks interface for win64. --- dlls/oledb32/dslocator.c | 6 +++--- include/msdasc.idl | 6 ++++-- 2 files changed, 7 insertions(+), 5 deletions(-) diff --git a/dlls/oledb32/dslocator.c b/dlls/oledb32/dslocator.c index dcc2fe3..f8a98f5 100644 --- a/dlls/oledb32/dslocator.c +++ b/dlls/oledb32/dslocator.c @@ -135,7 +135,7 @@ static HRESULT WINAPI dslocator_Invoke(IDataSourceLocator *iface, DISPID dispIdM return E_NOTIMPL; } -static HRESULT WINAPI dslocator_get_hWnd(IDataSourceLocator *iface, LONG *phwndParent) +static HRESULT WINAPI dslocator_get_hWnd(IDataSourceLocator *iface, COMPATIBLE_LONG *phwndParent) { DSLocatorImpl *This = impl_from_IDataSourceLocator(iface); @@ -144,11 +144,11 @@ static HRESULT WINAPI dslocator_get_hWnd(IDataSourceLocator *iface, LONG *phwndP return E_NOTIMPL; } -static HRESULT WINAPI dslocator_put_hWnd(IDataSourceLocator *iface, LONG phwndParent) +static HRESULT WINAPI dslocator_put_hWnd(IDataSourceLocator *iface, COMPATIBLE_LONG hwndParent) { DSLocatorImpl *This = impl_from_IDataSourceLocator(iface); - FIXME("(%p)->(%d)\n",This, phwndParent); + FIXME("(%p)->(%p)\n",This, (HWND)hwndParent); return E_NOTIMPL; } diff --git a/include/msdasc.idl b/include/msdasc.idl index 25610e5..e3efb1d 100644 --- a/include/msdasc.idl +++ b/include/msdasc.idl @@ -26,6 +26,8 @@ import "oledb.idl"; #define vi_progid(str) #endif +typedef LONG_PTR COMPATIBLE_LONG; + [ uuid(2206CEB0-19C1-11D1-89E0-00C04FD7A829), version(1.0) @@ -56,10 +58,10 @@ library MSDASC interface IDataSourceLocator : IDispatch { [id(0x60020000), propget] - HRESULT hWnd([out, retval] long* phwndParent); + HRESULT hWnd([out, retval] COMPATIBLE_LONG* phwndParent); [id(0x60020000), propput] - HRESULT hWnd([in] long phwndParent); + HRESULT hWnd([in] COMPATIBLE_LONG hwndParent); [id(0x60020002), helpstring("PromptNew")] HRESULT PromptNew([out, retval] IDispatch** ppADOConnection);

1 0

Rico Schüller : d3dx9: Use struct d3dx_object for objects.
by Alexandre Julliard 25 Sep '13

25 Sep '13

Module: wine Branch: master Commit: f7ba8e1472d3db3043ed2abf034a92de03640c83 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f7ba8e1472d3db3043ed2abf0… Author: Rico Schüller <kgbricola(a)web.de> Date: Tue Sep 24 22:50:20 2013 +0200 d3dx9: Use struct d3dx_object for objects. --- dlls/d3dx9_36/effect.c | 248 ++++++++++++++++++----------------------------- 1 files changed, 95 insertions(+), 153 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=f7ba8e1472d3db3043ed2…

1 0

Sebastian Lackner : winex11: Added missing release_win_data() to create_foreign_window().
by Alexandre Julliard 25 Sep '13

25 Sep '13

Module: wine Branch: master Commit: 5fb74c48d10e1f511adc43caf44d09e489c5c29e URL: http://source.winehq.org/git/wine.git/?a=commit;h=5fb74c48d10e1f511adc43caf… Author: Sebastian Lackner <sebastian(a)fds-team.de> Date: Tue Sep 24 05:50:33 2013 +0200 winex11: Added missing release_win_data() to create_foreign_window(). --- dlls/winex11.drv/window.c | 5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff --git a/dlls/winex11.drv/window.c b/dlls/winex11.drv/window.c index 429602b..1ff36eb 100644 --- a/dlls/winex11.drv/window.c +++ b/dlls/winex11.drv/window.c @@ -1897,11 +1897,12 @@ HWND create_foreign_window( Display *display, Window xwin ) SetPropA( hwnd, foreign_window_prop, (HANDLE)xwin ); XSaveContext( display, xwin, winContext, (char *)data->hwnd ); - ShowWindow( hwnd, SW_SHOW ); - TRACE( "win %lx parent %p style %08x %s -> hwnd %p\n", xwin, parent, style, wine_dbgstr_rect(&data->window_rect), hwnd ); + release_win_data( data ); + + ShowWindow( hwnd, SW_SHOW ); return hwnd; }

1 0

Dmitry Timoshkov : kernel32/tests: Add DuplicateHandle test to the file access tests.
by Alexandre Julliard 25 Sep '13

25 Sep '13

Module: wine Branch: master Commit: b4e375ecf4a1a21311634fb7a09c7e38dfcc2117 URL: http://source.winehq.org/git/wine.git/?a=commit;h=b4e375ecf4a1a21311634fb7a… Author: Dmitry Timoshkov <dmitry(a)baikal.ru> Date: Wed Sep 25 18:31:50 2013 +0900 kernel32/tests: Add DuplicateHandle test to the file access tests. --- dlls/kernel32/tests/file.c | 51 +++++++++++++++++++++++++++++++++++++++++-- 1 files changed, 48 insertions(+), 3 deletions(-) diff --git a/dlls/kernel32/tests/file.c b/dlls/kernel32/tests/file.c index a428672..dc69193 100644 --- a/dlls/kernel32/tests/file.c +++ b/dlls/kernel32/tests/file.c @@ -3890,11 +3890,27 @@ static void test_SetFileValidData(void) DeleteFile(filename); } +static unsigned file_map_access(unsigned access) +{ + if (access & GENERIC_READ) access |= FILE_GENERIC_READ; + if (access & GENERIC_WRITE) access |= FILE_GENERIC_WRITE; + if (access & GENERIC_EXECUTE) access |= FILE_GENERIC_EXECUTE; + if (access & GENERIC_ALL) access |= FILE_ALL_ACCESS; + return access & ~(GENERIC_READ | GENERIC_WRITE | GENERIC_EXECUTE | GENERIC_ALL); +} + +static BOOL is_access_compatible(unsigned obj_access, unsigned desired_access) +{ + obj_access = file_map_access(obj_access); + desired_access = file_map_access(desired_access); + return (obj_access & desired_access) == desired_access; +} + static void test_file_access(void) { static const struct { - int access, create_error, write_error, read_error; + unsigned access, create_error, write_error, read_error; } td[] = { { GENERIC_READ | GENERIC_WRITE, 0, 0, 0 }, @@ -3910,8 +3926,8 @@ static void test_file_access(void) }; char path[MAX_PATH], fname[MAX_PATH]; unsigned char buf[16]; - HANDLE hfile; - DWORD i, ret, bytes; + HANDLE hfile, hdup; + DWORD i, j, ret, bytes; GetTempPath(MAX_PATH, path); GetTempFileName(path, "foo", 0, fname); @@ -3930,6 +3946,35 @@ static void test_file_access(void) else ok(hfile != INVALID_HANDLE_VALUE, "%d: CreateFile error %d\n", i, GetLastError()); + for (j = 0; j < sizeof(td)/sizeof(td[0]); j++) + { + SetLastError(0xdeadbeef); + ret = DuplicateHandle(GetCurrentProcess(), hfile, GetCurrentProcess(), &hdup, + td[j].access, 0, 0); + if (is_access_compatible(td[i].access, td[j].access)) + ok(ret, "DuplicateHandle(%#x => %#x) error %d\n", td[i].access, td[j].access, GetLastError()); + else + { + /* FIXME: Remove once Wine is fixed */ + if ((td[j].access & (GENERIC_READ | GENERIC_WRITE)) || + (!(td[i].access & (GENERIC_WRITE | FILE_WRITE_DATA)) && (td[j].access & FILE_WRITE_DATA)) || + (!(td[i].access & (GENERIC_READ | FILE_READ_DATA)) && (td[j].access & FILE_READ_DATA)) || + (!(td[i].access & (GENERIC_WRITE)) && (td[j].access & FILE_APPEND_DATA))) + { +todo_wine + ok(!ret, "DuplicateHandle(%#x => %#x) should fail\n", td[i].access, td[j].access); +todo_wine + ok(GetLastError() == ERROR_ACCESS_DENIED, "expected ERROR_ACCESS_DENIED, got %d\n", GetLastError()); + } + else + { + ok(!ret, "DuplicateHandle(%#x => %#x) should fail\n", td[i].access, td[j].access); + ok(GetLastError() == ERROR_ACCESS_DENIED, "expected ERROR_ACCESS_DENIED, got %d\n", GetLastError()); + } + } + if (ret) CloseHandle(hdup); + } + SetLastError(0xdeadbeef); bytes = 0xdeadbeef; ret = WriteFile(hfile, "\x5e\xa7", 2, &bytes, NULL);

1 0

← Newer
1
...
8
9
10
11
12
13
14
...
65
Older →

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 September 2013