September 2011 - Wine-commits

Frédéric Delanoy : cmd: Fix echo's special '; ' handling.
by Alexandre Julliard 01 Sep '11

01 Sep '11

Module: wine Branch: master Commit: 02ef576739082235f6bbc4d07c5063339deb81ec URL: http://source.winehq.org/git/wine.git/?a=commit;h=02ef576739082235f6bbc4d07… Author: Frédéric Delanoy <frederic.delanoy(a)gmail.com> Date: Wed Aug 31 22:10:57 2011 +0200 cmd: Fix echo's special ';' handling. --- programs/cmd/builtins.c | 7 +++++-- 1 files changed, 5 insertions(+), 2 deletions(-) diff --git a/programs/cmd/builtins.c b/programs/cmd/builtins.c index 7f003ed..bbf2920 100644 --- a/programs/cmd/builtins.c +++ b/programs/cmd/builtins.c @@ -864,10 +864,13 @@ void WCMD_echo (const WCHAR *command) { int count; const WCHAR *origcommand = command; - if (command[0]==' ' || command[0]=='\t' || command[0]=='.' || command[0]==':') + if ( command[0]==' ' || command[0]=='\t' || command[0]=='.' + || command[0]==':' || command[0]==';') command++; + count = strlenW(command); - if (count == 0 && origcommand[0]!='.' && origcommand[0]!=':') { + if (count == 0 && origcommand[0]!='.' && origcommand[0]!=':' + && origcommand[0]!=';') { if (echo_mode) WCMD_output (WCMD_LoadMessage(WCMD_ECHOPROMPT), onW); else WCMD_output (WCMD_LoadMessage(WCMD_ECHOPROMPT), offW); return;

1 0

Francois Gouget : hhctrl.ocx: Issue a fixme if asked for unsupported toolbar buttons.
by Alexandre Julliard 01 Sep '11

01 Sep '11

Module: wine Branch: master Commit: 083abe3fe98235e8a26539b9210ddc32a089c50e URL: http://source.winehq.org/git/wine.git/?a=commit;h=083abe3fe98235e8a26539b92… Author: Francois Gouget <fgouget(a)free.fr> Date: Thu Sep 1 15:03:44 2011 +0200 hhctrl.ocx: Issue a fixme if asked for unsupported toolbar buttons. --- dlls/hhctrl.ocx/help.c | 9 +++++++++ 1 files changed, 9 insertions(+), 0 deletions(-) diff --git a/dlls/hhctrl.ocx/help.c b/dlls/hhctrl.ocx/help.c index d52939d..7bde256 100644 --- a/dlls/hhctrl.ocx/help.c +++ b/dlls/hhctrl.ocx/help.c @@ -785,6 +785,7 @@ static void TB_AddButtonsFromFlags(HHInfo *pHHInfo, TBBUTTON *pButtons, DWORD dw int nHistBitmaps = 0, nStdBitmaps = 0, nHHBitmaps = 0; HWND hToolbar = pHHInfo->WinType.hwndToolBar; TBADDBITMAP tbAB; + DWORD unsupported; /* Common bitmaps */ tbAB.hInst = HINST_COMMCTRL; @@ -799,6 +800,14 @@ static void TB_AddButtonsFromFlags(HHInfo *pHHInfo, TBBUTTON *pButtons, DWORD dw *pdwNumButtons = 0; + unsupported = dwButtonFlags & (HHWIN_BUTTON_BROWSE_FWD | + HHWIN_BUTTON_BROWSE_BCK | HHWIN_BUTTON_NOTES | HHWIN_BUTTON_CONTENTS | + HHWIN_BUTTON_INDEX | HHWIN_BUTTON_SEARCH | HHWIN_BUTTON_HISTORY | + HHWIN_BUTTON_FAVORITES | HHWIN_BUTTON_JUMP1 | HHWIN_BUTTON_JUMP2 | + HHWIN_BUTTON_ZOOM | HHWIN_BUTTON_TOC_NEXT | HHWIN_BUTTON_TOC_PREV); + if (unsupported) + FIXME("got asked for unsupported buttons: %06x\n", unsupported); + if (dwButtonFlags & HHWIN_BUTTON_EXPAND) { TB_AddButton(pButtons, (*pdwNumButtons)++, IDTB_EXPAND, nHistBitmaps + HIST_VIEWTREE);

1 0

Dmitry Timoshkov : gdi32: Add a buch of CreateScalableFontResource() tests.
by Alexandre Julliard 01 Sep '11

01 Sep '11

Module: wine Branch: master Commit: dfb4d2e96005b3a7fc582a2575ef10f145e0f614 URL: http://source.winehq.org/git/wine.git/?a=commit;h=dfb4d2e96005b3a7fc582a257… Author: Dmitry Timoshkov <dmitry(a)baikal.ru> Date: Tue Aug 30 11:59:42 2011 +0900 gdi32: Add a buch of CreateScalableFontResource() tests. --- dlls/gdi32/tests/Makefile.in | 2 + dlls/gdi32/tests/font.c | 193 ++++++++++++++++++++++++++++++++++++++++ dlls/gdi32/tests/resource.rc | 24 +++++ dlls/gdi32/tests/wine_test.sfd | 180 +++++++++++++++++++++++++++++++++++++ dlls/gdi32/tests/wine_test.ttf | Bin 0 -> 1552 bytes 5 files changed, 399 insertions(+), 0 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=dfb4d2e96005b3a7fc582…

1 0

Andrew Talbot : msvcp90: Avoid a forward declaration.
by Alexandre Julliard 01 Sep '11

01 Sep '11

Module: wine Branch: master Commit: c8cb13f949d42dd49dc4d5cf9f389592e966f190 URL: http://source.winehq.org/git/wine.git/?a=commit;h=c8cb13f949d42dd49dc4d5cf9… Author: Andrew Talbot <andrew.talbot(a)talbotville.com> Date: Wed Aug 31 22:04:17 2011 +0100 msvcp90: Avoid a forward declaration. --- dlls/msvcp90/locale.c | 64 +++++++++++++++++++++++------------------------- 1 files changed, 31 insertions(+), 33 deletions(-) diff --git a/dlls/msvcp90/locale.c b/dlls/msvcp90/locale.c index c838744..6ebfb51 100644 --- a/dlls/msvcp90/locale.c +++ b/dlls/msvcp90/locale.c @@ -84,7 +84,37 @@ int locale_id__Id_cnt = 0; /* ?_Clocptr(a)_Locimp@locale(a)std@@0PEAV123(a)EA */ locale__Locimp *locale__Locimp__Clocptr = NULL; -static const vtable_ptr MSVCP_locale_facet_vtable[]; +/* ??1facet(a)locale@std@@UAE(a)XZ */ +/* ??1facet(a)locale@std@@UEAA(a)XZ */ +DEFINE_THISCALL_WRAPPER(locale_facet_dtor, 4) +void __thiscall locale_facet_dtor(locale_facet *this) +{ + TRACE("(%p)\n", this); +} + +DEFINE_THISCALL_WRAPPER(MSVCP_locale_facet_vector_dtor, 8) +locale_facet* __thiscall MSVCP_locale_facet_vector_dtor(locale_facet *this, unsigned int flags) +{ + TRACE("(%p %x)\n", this, flags); + if(flags & 2) { + /* we have an array, with the number of elements stored before the first object */ + int i, *ptr = (int *)this-1; + + for(i=*ptr-1; i>=0; i--) + locale_facet_dtor(this+i); + MSVCRT_operator_delete(ptr); + } else { + locale_facet_dtor(this); + if(flags & 1) + MSVCRT_operator_delete(this); + } + + return this; +} + +static const vtable_ptr MSVCP_locale_facet_vtable[] = { + (vtable_ptr)THISCALL_NAME(MSVCP_locale_facet_vector_dtor) +}; /* ??0id(a)locale@std@@QAE(a)I@Z */ /* ??0id(a)locale@std@@QEAA(a)_K@Z */ @@ -156,34 +186,6 @@ locale_facet* __thiscall locale_facet_ctor_refs(locale_facet *this, MSVCP_size_t return this; } -/* ??1facet(a)locale@std@@UAE(a)XZ */ -/* ??1facet(a)locale@std@@UEAA(a)XZ */ -DEFINE_THISCALL_WRAPPER(locale_facet_dtor, 4) -void __thiscall locale_facet_dtor(locale_facet *this) -{ - TRACE("(%p)\n", this); -} - -DEFINE_THISCALL_WRAPPER(MSVCP_locale_facet_vector_dtor, 8) -locale_facet* __thiscall MSVCP_locale_facet_vector_dtor(locale_facet *this, unsigned int flags) -{ - TRACE("(%p %x)\n", this, flags); - if(flags & 2) { - /* we have an array, with the number of elements stored before the first object */ - int i, *ptr = (int *)this-1; - - for(i=*ptr-1; i>=0; i--) - locale_facet_dtor(this+i); - MSVCRT_operator_delete(ptr); - } else { - locale_facet_dtor(this); - if(flags & 1) - MSVCRT_operator_delete(this); - } - - return this; -} - /* ?_Incref(a)facet@locale(a)std@@QAEXXZ */ /* ?_Incref(a)facet@locale(a)std@@QEAAXXZ */ DEFINE_THISCALL_WRAPPER(locale_facet__Incref, 4) @@ -226,10 +228,6 @@ MSVCP_size_t __cdecl locale_facet__Getcat(const locale_facet **facet, const loca return -1; } -static const vtable_ptr MSVCP_locale_facet_vtable[] = { - (vtable_ptr)THISCALL_NAME(MSVCP_locale_facet_vector_dtor) -}; - /* ??_F_Locimp(a)locale@std@@QAEXXZ */ /* ??_F_Locimp(a)locale@std@@QEAAXXZ */ DEFINE_THISCALL_WRAPPER(locale__Locimp_ctor, 4)

1 0

Octavian Voicu : shell32: Fix an incorrect pointer/ integer cast on 64-bit and fix check for resource id.
by Alexandre Julliard 01 Sep '11

01 Sep '11

Module: wine Branch: master Commit: 776e27834d97eb6886e855fd5c90585eafdcf83f URL: http://source.winehq.org/git/wine.git/?a=commit;h=776e27834d97eb6886e855fd5… Author: Octavian Voicu <octavian.voicu(a)gmail.com> Date: Thu Sep 1 18:10:05 2011 +0300 shell32: Fix an incorrect pointer/integer cast on 64-bit and fix check for resource id. --- dlls/shell32/recyclebin.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/shell32/recyclebin.c b/dlls/shell32/recyclebin.c index 39b662a..eec9e00 100644 --- a/dlls/shell32/recyclebin.c +++ b/dlls/shell32/recyclebin.c @@ -238,9 +238,9 @@ static HRESULT WINAPI RecycleBinMenu_InvokeCommand(IContextMenu2 *iface, { RecycleBinMenu *This = impl_from_IContextMenu2(iface); LPCSTR verb = pici->lpVerb; - if(!HIWORD(verb)) + if(IS_INTRESOURCE(verb)) { - switch((UINT)verb) + switch(LOWORD(verb)) { case IDM_RECYCLEBIN_ERASE: DoErase(This);

1 0

Jacek Caban : jscript: Added more SetInterfaceSafetyOptions tests and fixes .
by Alexandre Julliard 01 Sep '11

01 Sep '11

Module: wine Branch: master Commit: ff0cea33d04e2282dc47a32c225b5a9071c3cb44 URL: http://source.winehq.org/git/wine.git/?a=commit;h=ff0cea33d04e2282dc47a32c2… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu Sep 1 17:47:41 2011 +0200 jscript: Added more SetInterfaceSafetyOptions tests and fixes. --- dlls/jscript/jscript.c | 2 +- dlls/jscript/tests/jscript.c | 32 ++++++++++++++++++++++++++++++++ 2 files changed, 33 insertions(+), 1 deletions(-) diff --git a/dlls/jscript/jscript.c b/dlls/jscript/jscript.c index fb7fd8c..be225b7 100644 --- a/dlls/jscript/jscript.c +++ b/dlls/jscript/jscript.c @@ -943,7 +943,7 @@ static HRESULT WINAPI JScriptSafety_SetInterfaceSafetyOptions(IObjectSafety *ifa if(dwOptionSetMask & ~SUPPORTED_OPTIONS) return E_FAIL; - This->safeopt = dwEnabledOptions & dwEnabledOptions; + This->safeopt = (dwEnabledOptions & dwOptionSetMask) | (This->safeopt & ~dwOptionSetMask) | INTERFACE_USES_DISPEX; return S_OK; } diff --git a/dlls/jscript/tests/jscript.c b/dlls/jscript/tests/jscript.c index 67067b9..eda867d 100644 --- a/dlls/jscript/tests/jscript.c +++ b/dlls/jscript/tests/jscript.c @@ -329,6 +329,38 @@ static void test_safety(IUnknown *unk) ok(enabled == (INTERFACESAFE_FOR_UNTRUSTED_DATA|INTERFACE_USES_DISPEX|INTERFACE_USES_SECURITY_MANAGER), "enabled=%x\n", enabled); + hres = IObjectSafety_SetInterfaceSafetyOptions(safety, &IID_IActiveScriptParse, INTERFACESAFE_FOR_UNTRUSTED_DATA, 0); + ok(hres == S_OK, "SetInterfaceSafetyOptions failed: %08x\n", hres); + + supported = enabled = 0xdeadbeef; + hres = IObjectSafety_GetInterfaceSafetyOptions(safety, &IID_IActiveScriptParse, &supported, &enabled); + ok(hres == S_OK, "GetInterfaceSafetyOptions failed: %08x\n", hres); + ok(supported == (INTERFACESAFE_FOR_UNTRUSTED_DATA|INTERFACE_USES_DISPEX|INTERFACE_USES_SECURITY_MANAGER), + "supported=%x\n", supported); + ok(enabled == (INTERFACE_USES_DISPEX|INTERFACE_USES_SECURITY_MANAGER), "enabled=%x\n", enabled); + + hres = IObjectSafety_SetInterfaceSafetyOptions(safety, &IID_IActiveScriptParse, + INTERFACESAFE_FOR_UNTRUSTED_DATA|INTERFACE_USES_DISPEX|INTERFACE_USES_SECURITY_MANAGER, 0); + ok(hres == S_OK, "SetInterfaceSafetyOptions failed: %08x\n", hres); + + supported = enabled = 0xdeadbeef; + hres = IObjectSafety_GetInterfaceSafetyOptions(safety, &IID_IActiveScriptParse, &supported, &enabled); + ok(hres == S_OK, "GetInterfaceSafetyOptions failed: %08x\n", hres); + ok(supported == (INTERFACESAFE_FOR_UNTRUSTED_DATA|INTERFACE_USES_DISPEX|INTERFACE_USES_SECURITY_MANAGER), + "supported=%x\n", supported); + ok(enabled == INTERFACE_USES_DISPEX, "enabled=%x\n", enabled); + + hres = IObjectSafety_SetInterfaceSafetyOptions(safety, &IID_IActiveScriptParse, + INTERFACE_USES_DISPEX, 0); + ok(hres == S_OK, "SetInterfaceSafetyOptions failed: %08x\n", hres); + + supported = enabled = 0xdeadbeef; + hres = IObjectSafety_GetInterfaceSafetyOptions(safety, &IID_IActiveScriptParse, &supported, &enabled); + ok(hres == S_OK, "GetInterfaceSafetyOptions failed: %08x\n", hres); + ok(supported == (INTERFACESAFE_FOR_UNTRUSTED_DATA|INTERFACE_USES_DISPEX|INTERFACE_USES_SECURITY_MANAGER), + "supported=%x\n", supported); + ok(enabled == INTERFACE_USES_DISPEX, "enabled=%x\n", enabled); + IObjectSafety_Release(safety); }

1 0

Andrew Eikum : winecfg: Remove driver selection from Audio tab.
by Alexandre Julliard 01 Sep '11

01 Sep '11

1 0

Dan Kegel : configure: AC_C_CONST is obsolete.
by Alexandre Julliard 01 Sep '11

01 Sep '11

Module: wine Branch: master Commit: 82954d45de710f3399f56ef0005cb307c3a5f09a URL: http://source.winehq.org/git/wine.git/?a=commit;h=82954d45de710f3399f56ef00… Author: Dan Kegel <dank(a)kegel.com> Date: Wed Aug 31 17:17:44 2011 -0700 configure: AC_C_CONST is obsolete. --- configure | 80 --------------------------------------------------- configure.ac | 1 - include/config.h.in | 3 -- 3 files changed, 0 insertions(+), 84 deletions(-) diff --git a/configure b/configure index c421b2a..7ac2d12 100755 --- a/configure +++ b/configure @@ -13108,86 +13108,6 @@ $as_echo "#define HAVE_SCHED_SETAFFINITY 1" >>confdefs.h fi -{ $as_echo "$as_me:${as_lineno-$LINENO}: checking for an ANSI C-conforming const" >&5 -$as_echo_n "checking for an ANSI C-conforming const... " >&6; } -if ${ac_cv_c_const+:} false; then : - $as_echo_n "(cached) " >&6 -else - cat confdefs.h - <<_ACEOF >conftest.$ac_ext -/* end confdefs.h. */ - -int -main () -{ -/* FIXME: Include the comments suggested by Paul. */ -#ifndef __cplusplus - /* Ultrix mips cc rejects this. */ - typedef int charset[2]; - const charset cs; - /* SunOS 4.1.1 cc rejects this. */ - char const *const *pcpcc; - char **ppc; - /* NEC SVR4.0.2 mips cc rejects this. */ - struct point {int x, y;}; - static struct point const zero = {0,0}; - /* AIX XL C 1.02.0.0 rejects this. - It does not let you subtract one const X* pointer from another in - an arm of an if-expression whose if-part is not a constant - expression */ - const char *g = "string"; - pcpcc = &g + (g ? g-g : 0); - /* HPUX 7.0 cc rejects these. */ - ++pcpcc; - ppc = (char**) pcpcc; - pcpcc = (char const *const *) ppc; - { /* SCO 3.2v4 cc rejects this. */ - char *t; - char const *s = 0 ? (char *) 0 : (char const *) 0; - - *t++ = 0; - if (s) return 0; - } - { /* Someone thinks the Sun supposedly-ANSI compiler will reject this. */ - int x[] = {25, 17}; - const int *foo = &x[0]; - ++foo; - } - { /* Sun SC1.0 ANSI compiler rejects this -- but not the above. */ - typedef const int *iptr; - iptr p = 0; - ++p; - } - { /* AIX XL C 1.02.0.0 rejects this saying - "k.c", line 2.27: 1506-025 (S) Operand must be a modifiable lvalue. */ - struct s { int j; const int *ap[3]; }; - struct s *b; b->j = 5; - } - { /* ULTRIX-32 V3.1 (Rev 9) vcc rejects this */ - const int foo = 10; - if (!foo) return 0; - } - return !cs[0] && !zero.x; -#endif - - ; - return 0; -} -_ACEOF -if ac_fn_c_try_compile "$LINENO"; then : - ac_cv_c_const=yes -else - ac_cv_c_const=no -fi -rm -f core conftest.err conftest.$ac_objext conftest.$ac_ext -fi -{ $as_echo "$as_me:${as_lineno-$LINENO}: result: $ac_cv_c_const" >&5 -$as_echo "$ac_cv_c_const" >&6; } -if test $ac_cv_c_const = no; then - -$as_echo "#define const /**/" >>confdefs.h - -fi - { $as_echo "$as_me:${as_lineno-$LINENO}: checking for inline" >&5 $as_echo_n "checking for inline... " >&6; } if ${ac_cv_c_inline+:} false; then : diff --git a/configure.ac b/configure.ac index ae89108..c824206 100644 --- a/configure.ac +++ b/configure.ac @@ -2055,7 +2055,6 @@ fi dnl **** Check for types **** -AC_C_CONST AC_C_INLINE AC_CHECK_TYPES([mode_t, off_t, pid_t, size_t, ssize_t, long long, fsblkcnt_t, fsfilcnt_t]) AC_CHECK_TYPES([sigset_t],,,[#include <sys/types.h> diff --git a/include/config.h.in b/include/config.h.in index 26528b7..f0016c8 100644 --- a/include/config.h.in +++ b/include/config.h.in @@ -1327,9 +1327,6 @@ convention */ #undef __ASM_STDCALL_FUNC -/* Define to empty if `const' does not conform to ANSI C. */ -#undef const - /* Define to `__inline__' or `__inline' if that's what the C compiler calls it, or to nothing if 'inline' is not supported under any name. */ #ifndef __cplusplus

1 0

Dan Kegel : configure: Fix configure failures under -Werror -Wall.
by Alexandre Julliard 01 Sep '11

01 Sep '11

Module: wine Branch: master Commit: 45725be9d5c721710f69a576dbc80cea6738ed13 URL: http://source.winehq.org/git/wine.git/?a=commit;h=45725be9d5c721710f69a576d… Author: Dan Kegel <dank(a)kegel.com> Date: Wed Aug 31 17:35:49 2011 -0700 configure: Fix configure failures under -Werror -Wall. --- configure | 10 +++++----- configure.ac | 10 +++++----- 2 files changed, 10 insertions(+), 10 deletions(-) diff --git a/configure b/configure index 7ac2d12..563117e 100755 --- a/configure +++ b/configure @@ -8425,7 +8425,7 @@ fi int main () { -static typeof(XRRSetScreenConfigAndRate) * func; +static typeof(XRRSetScreenConfigAndRate) * func; if (func) return 0; ; return 0; } @@ -8502,7 +8502,7 @@ fi int main () { -static typeof(XineramaQueryScreens) * func; +static typeof(XineramaQueryScreens) * func; if (func) return 0; ; return 0; } @@ -8691,7 +8691,7 @@ else int main () { -GLenum test = GL_UNSIGNED_SHORT_5_6_5; +GLenum test = GL_UNSIGNED_SHORT_5_6_5; return (test == GL_UNSIGNED_SHORT_5_6_5); ; return 0; } @@ -9520,7 +9520,7 @@ if test "x$ac_cv_header_gnutls_gnutls_h" = xyes; then : int main () { -typeof(gnutls_mac_get_key_size) *pfunc; +static typeof(gnutls_mac_get_key_size) *func; if (func) return 0; ; return 0; } @@ -10535,7 +10535,7 @@ $as_echo_n "checking whether gint64 defined by gst/app/gstappsink.h is indeed 64 int main () { -int a[sizeof(gint64) > 4 ? 1 : -1] +static int a[sizeof(gint64) > 4 ? 1 : -1]; if (a[0]) return 0; ; return 0; } diff --git a/configure.ac b/configure.ac index c824206..ed7a60c 100644 --- a/configure.ac +++ b/configure.ac @@ -1024,7 +1024,7 @@ then if test "$ac_cv_header_X11_extensions_Xrandr_h" = "yes" -a "x$ac_cv_lib_soname_Xrender" != "x" then AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[#include <X11/Xlib.h> -#include <X11/extensions/Xrandr.h>]], [[static typeof(XRRSetScreenConfigAndRate) * func;]])], +#include <X11/extensions/Xrandr.h>]], [[static typeof(XRRSetScreenConfigAndRate) * func; if (func) return 0;]])], [WINE_CHECK_SONAME(Xrandr,XRRQueryExtension,,,[$X_LIBS $XLIB $X_EXTRA_LIBS])]) fi WINE_NOTICE_WITH(xrandr,[test "x$ac_cv_lib_soname_Xrandr" = "x"], @@ -1034,7 +1034,7 @@ then if test "$ac_cv_header_X11_extensions_Xinerama_h" = "yes" then AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[#include <X11/Xlib.h> -#include <X11/extensions/Xinerama.h>]], [[static typeof(XineramaQueryScreens) * func;]])], +#include <X11/extensions/Xinerama.h>]], [[static typeof(XineramaQueryScreens) * func; if (func) return 0;]])], [WINE_CHECK_SONAME(Xinerama,XineramaQueryScreens,,,[$X_LIBS $XLIB $X_EXTRA_LIBS])]) fi WINE_NOTICE_WITH(xinerama,[test "x$ac_cv_lib_soname_Xinerama" = "x"], @@ -1068,7 +1068,7 @@ then then dnl Check for some problems due to old Mesa versions AC_CACHE_CHECK([for up-to-date OpenGL version], wine_cv_opengl_header_version_OK, - AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[#include <GL/gl.h>]], [[GLenum test = GL_UNSIGNED_SHORT_5_6_5;]])], + AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[#include <GL/gl.h>]], [[GLenum test = GL_UNSIGNED_SHORT_5_6_5; return (test == GL_UNSIGNED_SHORT_5_6_5);]])], [wine_cv_opengl_header_version_OK="yes"], [wine_cv_opengl_header_version_OK="no"])) @@ -1235,7 +1235,7 @@ then CPPFLAGS="$CPPFLAGS $ac_gnutls_cflags" fi AC_CHECK_HEADER(gnutls/gnutls.h, - [AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[#include <gnutls/gnutls.h>]], [[typeof(gnutls_mac_get_key_size) *pfunc;]])], + [AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[#include <gnutls/gnutls.h>]], [[static typeof(gnutls_mac_get_key_size) *func; if (func) return 0;]])], [WINE_CHECK_SONAME(gnutls,gnutls_global_init, [AC_SUBST(GNUTLSINCL,"$ac_gnutls_cflags")],,[$ac_gnutls_libs])])]) CPPFLAGS="$ac_save_CPPFLAGS" @@ -1458,7 +1458,7 @@ then [AC_CHECK_HEADER([gst/app/gstappsink.h], [AC_MSG_CHECKING([whether gint64 defined by gst/app/gstappsink.h is indeed 64-bit]) AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[#include <gst/app/gstappsink.h>]], - [[int a[sizeof(gint64) > 4 ? 1 : -1]]])], + [[static int a[sizeof(gint64) > 4 ? 1 : -1]; if (a[0]) return 0;]])], [AC_MSG_RESULT([yes]) AC_CHECK_LIB(gstreamer-0.10,gst_pad_get_caps_reffed, [AC_CHECK_LIB(gstapp-0.10,gst_app_buffer_new,

1 0

Octavian Voicu : vbscript: Fix possible memory leak/ crash caused by race condition in VBScriptParse_InitNew.
by Alexandre Julliard 01 Sep '11

01 Sep '11

Module: wine Branch: master Commit: 2f053965a2dd8f167b0f0189f312b229e5372a69 URL: http://source.winehq.org/git/wine.git/?a=commit;h=2f053965a2dd8f167b0f0189f… Author: Octavian Voicu <octavian.voicu(a)gmail.com> Date: Thu Sep 1 05:23:12 2011 +0300 vbscript: Fix possible memory leak/crash caused by race condition in VBScriptParse_InitNew. --- dlls/vbscript/vbscript.c | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/dlls/vbscript/vbscript.c b/dlls/vbscript/vbscript.c index 084337f..caee4d4 100644 --- a/dlls/vbscript/vbscript.c +++ b/dlls/vbscript/vbscript.c @@ -348,7 +348,7 @@ static ULONG WINAPI VBScriptParse_Release(IActiveScriptParse *iface) static HRESULT WINAPI VBScriptParse_InitNew(IActiveScriptParse *iface) { VBScript *This = impl_from_IActiveScriptParse(iface); - script_ctx_t *ctx; + script_ctx_t *ctx, *old_ctx; TRACE("(%p)\n", This); @@ -359,8 +359,8 @@ static HRESULT WINAPI VBScriptParse_InitNew(IActiveScriptParse *iface) if(!ctx) return E_OUTOFMEMORY; - ctx = InterlockedCompareExchangePointer((void**)&This->ctx, ctx, NULL); - if(ctx) { + old_ctx = InterlockedCompareExchangePointer((void**)&This->ctx, ctx, NULL); + if(old_ctx) { destroy_script(ctx); return E_UNEXPECTED; }

1 0