wine-commits October 2008

wine-commits@winehq.org

3 participants
1216 discussions

Aric Stewart : shell32/tests: Add tests for CommandLineToArgvW.
by Alexandre Julliard 17 Oct '08

17 Oct '08

Module: wine Branch: master Commit: 2c9517d36bb2188d38705b65a61ec3f7fcb1e13a URL: http://source.winehq.org/git/wine.git/?a=commit;h=2c9517d36bb2188d38705b65a… Author: Aric Stewart <aric(a)codeweavers.com> Date: Thu Oct 16 09:22:06 2008 -0500 shell32/tests: Add tests for CommandLineToArgvW. --- dlls/shell32/tests/shlexec.c | 69 ++++++++++++++++++++++++++++++++++++++++++ 1 files changed, 69 insertions(+), 0 deletions(-) diff --git a/dlls/shell32/tests/shlexec.c b/dlls/shell32/tests/shlexec.c index 3c62ac2..ac7915a 100644 --- a/dlls/shell32/tests/shlexec.c +++ b/dlls/shell32/tests/shlexec.c @@ -1605,6 +1605,74 @@ static void cleanup_test(void) CoUninitialize(); } +static void test_commandline(void) +{ + static const WCHAR one[] = {'o','n','e',0}; + static const WCHAR two[] = {'t','w','o',0}; + static const WCHAR three[] = {'t','h','r','e','e',0}; + static const WCHAR four[] = {'f','o','u','r',0}; + + static const WCHAR fmt1[] = {'%','s',' ','%','s',' ','%','s',' ','%','s',0}; + static const WCHAR fmt2[] = {' ','%','s',' ','%','s',' ','%','s',' ','%','s',0}; + static const WCHAR fmt3[] = {'%','s','=','%','s',' ','%','s','=','\"','%','s','\"',0}; + static const WCHAR fmt4[] = {'\"','%','s','\"',' ','\"','%','s',' ','%','s','\"',' ','%','s',0}; + static const WCHAR fmt5[] = {'\\','\"','%','s','\"',' ','%','s','=','\"','%','s','\\','\"',' ','\"','%','s','\\','\"',0}; + + static const WCHAR chkfmt1[] = {'%','s','=','%','s',0}; + static const WCHAR chkfmt2[] = {'%','s',' ','%','s',0}; + static const WCHAR chkfmt3[] = {'\\','\"','%','s','\"',0}; + static const WCHAR chkfmt4[] = {'%','s','=','%','s','\"',' ','%','s','\"',0}; + WCHAR cmdline[255]; + LPWSTR *args = (LPWSTR*)0xdeadcafe; + INT numargs = -1; + + wsprintfW(cmdline,fmt1,one,two,three,four); + args=CommandLineToArgvW(cmdline,&numargs); + if (args == NULL && numargs == -1) + { + win_skip("CommandLineToArgvW not implemented, skipping\n"); + return; + } + ok(numargs == 4, "expected 4 args, got %i\n",numargs); + ok(lstrcmpW(args[0],one)==0,"arg0 is not as expected\n"); + ok(lstrcmpW(args[1],two)==0,"arg1 is not as expected\n"); + ok(lstrcmpW(args[2],three)==0,"arg2 is not as expected\n"); + ok(lstrcmpW(args[3],four)==0,"arg3 is not as expected\n"); + + wsprintfW(cmdline,fmt2,one,two,three,four); + args=CommandLineToArgvW(cmdline,&numargs); + ok(numargs == 5, "expected 5 args, got %i\n",numargs); + ok(args[0][0]==0,"arg0 is not as expected\n"); + ok(lstrcmpW(args[1],one)==0,"arg1 is not as expected\n"); + ok(lstrcmpW(args[2],two)==0,"arg2 is not as expected\n"); + ok(lstrcmpW(args[3],three)==0,"arg3 is not as expected\n"); + ok(lstrcmpW(args[4],four)==0,"arg4 is not as expected\n"); + + wsprintfW(cmdline,fmt3,one,two,three,four); + args=CommandLineToArgvW(cmdline,&numargs); + ok(numargs == 2, "expected 2 args, got %i\n",numargs); + wsprintfW(cmdline,chkfmt1,one,two); + ok(lstrcmpW(args[0],cmdline)==0,"arg0 is not as expected\n"); + wsprintfW(cmdline,chkfmt1,three,four); + ok(lstrcmpW(args[1],cmdline)==0,"arg1 is not as expected\n"); + + wsprintfW(cmdline,fmt4,one,two,three,four); + args=CommandLineToArgvW(cmdline,&numargs); + ok(numargs == 3, "expected 3 args, got %i\n",numargs); + ok(lstrcmpW(args[0],one)==0,"arg0 is not as expected\n"); + wsprintfW(cmdline,chkfmt2,two,three); + ok(lstrcmpW(args[1],cmdline)==0,"arg1 is not as expected\n"); + ok(lstrcmpW(args[2],four)==0,"arg2 is not as expected\n"); + + wsprintfW(cmdline,fmt5,one,two,three,four); + args=CommandLineToArgvW(cmdline,&numargs); + ok(numargs == 2, "expected 2 args, got %i\n",numargs); + wsprintfW(cmdline,chkfmt3,one); + todo_wine ok(lstrcmpW(args[0],cmdline)==0,"arg0 is not as expected\n"); + wsprintfW(cmdline,chkfmt4,two,three,four); + todo_wine ok(lstrcmpW(args[1],cmdline)==0,"arg1 is not as expected\n"); +} + START_TEST(shlexec) { @@ -1624,6 +1692,7 @@ START_TEST(shlexec) test_exes_long(); test_dde(); test_dde_default_app(); + test_commandline(); cleanup_test(); }

1 0

Aric Stewart : gdiplus: Avoid dereferencing a potential NULL (Coverity 589) .
by Alexandre Julliard 17 Oct '08

17 Oct '08

Module: wine Branch: master Commit: c4e859a678db1440e1357c42f60e620716e383f8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=c4e859a678db1440e1357c42f… Author: Aric Stewart <aric(a)codeweavers.com> Date: Thu Oct 16 09:09:49 2008 -0500 gdiplus: Avoid dereferencing a potential NULL (Coverity 589). --- dlls/gdiplus/image.c | 4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/dlls/gdiplus/image.c b/dlls/gdiplus/image.c index 9516d64..ffd270b 100644 --- a/dlls/gdiplus/image.c +++ b/dlls/gdiplus/image.c @@ -450,7 +450,9 @@ GpStatus WINGDIPAPI GdipCreateBitmapFromScan0(INT width, INT height, INT stride, TRACE("%d %d %d %d %p %p\n", width, height, stride, format, scan0, bitmap); - if(!bitmap || width <= 0 || height <= 0 || (scan0 && (stride % 4))){ + if (!bitmap) return InvalidParameter; + + if(width <= 0 || height <= 0 || (scan0 && (stride % 4))){ *bitmap = NULL; return InvalidParameter; }

1 0

Joris Huizer : shlwapi: Fix ASSOC_GetExecutable not to use uninitialised variable.
by Alexandre Julliard 17 Oct '08

17 Oct '08

Module: wine Branch: master Commit: 8ed05aeb32de6a2cc820521d0603dd06c3551543 URL: http://source.winehq.org/git/wine.git/?a=commit;h=8ed05aeb32de6a2cc820521d0… Author: Joris Huizer <joris_huizer(a)yahoo.com> Date: Wed Oct 15 00:13:44 2008 +0200 shlwapi: Fix ASSOC_GetExecutable not to use uninitialised variable. --- dlls/shlwapi/assoc.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/shlwapi/assoc.c b/dlls/shlwapi/assoc.c index 9e4b036..31a8420 100644 --- a/dlls/shlwapi/assoc.c +++ b/dlls/shlwapi/assoc.c @@ -603,7 +603,7 @@ static HRESULT ASSOC_GetExecutable(IQueryAssociationsImpl *This, LONG ret; WCHAR * pszCommand; WCHAR * pszEnd; - WCHAR * pszExtraFromReg; + WCHAR * pszExtraFromReg = NULL; WCHAR * pszFileType; WCHAR * pszStart; static const WCHAR commandW[] = { 'c','o','m','m','a','n','d',0 };

1 0

Andrey Esin : regedit: Corrections in Russian translation.
by Alexandre Julliard 17 Oct '08

17 Oct '08

Module: wine Branch: master Commit: f0321f85a024af8cd84cf54d8235a1f2fdf5cc6a URL: http://source.winehq.org/git/wine.git/?a=commit;h=f0321f85a024af8cd84cf54d8… Author: Andrey Esin <andrey(a)esin.name> Date: Tue Oct 14 22:32:13 2008 +0400 regedit: Corrections in Russian translation. --- programs/regedit/Ru.rc | 10 +++++----- 1 files changed, 5 insertions(+), 5 deletions(-) diff --git a/programs/regedit/Ru.rc b/programs/regedit/Ru.rc index 02d5886..4be900e 100644 --- a/programs/regedit/Ru.rc +++ b/programs/regedit/Ru.rc @@ -49,7 +49,7 @@ BEGIN MENUITEM "&�� ", ID_EDIT_NEW_STRINGVALUE MENUITEM "&�� ", ID_EDIT_NEW_BINARYVALUE MENUITEM "&DWORD ��", ID_EDIT_NEW_DWORDVALUE - MENUITEM "&��-�� ", ID_EDIT_NEW_MULTI_STRINGVALUE + MENUITEM "&�� ", ID_EDIT_NEW_MULTI_STRINGVALUE END MENUITEM SEPARATOR MENUITEM "&��\tDel", ID_EDIT_DELETE @@ -100,7 +100,7 @@ BEGIN MENUITEM "&�� ", ID_EDIT_NEW_STRINGVALUE MENUITEM "&�� ", ID_EDIT_NEW_BINARYVALUE MENUITEM "&DWORD ��", ID_EDIT_NEW_DWORDVALUE - MENUITEM "&��-�� ", ID_EDIT_NEW_MULTI_STRINGVALUE + MENUITEM "&�� ", ID_EDIT_NEW_MULTI_STRINGVALUE END MENUITEM SEPARATOR MENUITEM "&��\tDel", ID_EDIT_DELETE @@ -211,7 +211,7 @@ END IDD_EDIT_MULTI_STRING DIALOG DISCARDABLE 22, 17, 210, 175 STYLE DS_MODALFRAME | WS_CAPTION | WS_SYSMENU -CAPTION "�� -�� " +CAPTION "�� " FONT 8, "MS Shell Dlg" BEGIN LTEXT "�� :",IDC_STATIC,5,5,119,8 @@ -240,7 +240,7 @@ END STRINGTABLE DISCARDABLE BEGIN - ID_REGISTRY_MENU "�� " + ID_REGISTRY_MENU "�� " ID_EDIT_MENU "�� " ID_VIEW_MENU "�� " ID_FAVORITES_MENU "�� " @@ -255,7 +255,7 @@ BEGIN ID_EDIT_NEW_STRINGVALUE "�� " ID_EDIT_NEW_BINARYVALUE "�� " ID_EDIT_NEW_DWORDVALUE "�� DWORD-��" - ID_EDIT_NEW_MULTI_STRINGVALUE "�� -�� " + ID_EDIT_NEW_MULTI_STRINGVALUE "�� " ID_REGISTRY_IMPORTREGISTRYFILE "�� " ID_REGISTRY_EXPORTREGISTRYFILE "�� "

1 0

Andrew Talbot : mlang: Sign-compare warnings fix.
by Alexandre Julliard 17 Oct '08

17 Oct '08

Module: wine Branch: master Commit: 59a33c9c7d9ae6cb5a2bd872ab2f38552053b137 URL: http://source.winehq.org/git/wine.git/?a=commit;h=59a33c9c7d9ae6cb5a2bd872a… Author: Andrew Talbot <andrew.talbot(a)talbotville.com> Date: Thu Oct 16 12:47:01 2008 +0100 mlang: Sign-compare warnings fix. --- dlls/mlang/mlang.c | 8 ++++---- 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/dlls/mlang/mlang.c b/dlls/mlang/mlang.c index 9686e9a..ae85f6e 100644 --- a/dlls/mlang/mlang.c +++ b/dlls/mlang/mlang.c @@ -503,7 +503,7 @@ static LONG dll_count; static UINT DetectJapaneseCode(LPCSTR input, DWORD count) { UINT code = 0; - int i = 0; + DWORD i = 0; unsigned char c1,c2; while ((code == 0 || code == 51932) && i < count) @@ -615,7 +615,7 @@ static int han2zen(unsigned char *p1, unsigned char *p2) static UINT ConvertJIS2SJIS(LPCSTR input, DWORD count, LPSTR output) { - int i = 0; + DWORD i = 0; int j = 0; unsigned char p2,p; int shifted = FALSE; @@ -683,7 +683,7 @@ static inline int enter_shift(LPSTR out, int c) static UINT ConvertSJIS2JIS(LPCSTR input, DWORD count, LPSTR output) { - int i = 0; + DWORD i = 0; int j = 0; unsigned char p2,p; int shifted = FALSE; @@ -1343,7 +1343,7 @@ static const IClassFactoryVtbl MLANGCF_Vtbl = */ HRESULT WINAPI DllGetClassObject(REFCLSID rclsid, REFIID iid, LPVOID *ppv) { - int i; + unsigned int i; IClassFactoryImpl *factory; TRACE("%s %s %p\n",debugstr_guid(rclsid), debugstr_guid(iid), ppv);

1 0

Andrew Talbot : mcicda: Sign-compare warning fix.
by Alexandre Julliard 17 Oct '08

17 Oct '08

Module: wine Branch: master Commit: 9e297e2cbe5ce747733cbe9450977474fe1e2189 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9e297e2cbe5ce747733cbe945… Author: Andrew Talbot <andrew.talbot(a)talbotville.com> Date: Thu Oct 16 12:46:57 2008 +0100 mcicda: Sign-compare warning fix. --- dlls/mcicda/mcicda.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/mcicda/mcicda.c b/dlls/mcicda/mcicda.c index 6e4a835..2edb919 100644 --- a/dlls/mcicda/mcicda.c +++ b/dlls/mcicda/mcicda.c @@ -270,7 +270,7 @@ static DWORD MCICDA_Open(UINT wDevID, DWORD dwFlags, LPMCI_OPEN_PARMSW lpOpenPar DWORD ret = MCIERR_HARDWARE; WINE_MCICDAUDIO* wmcda = (WINE_MCICDAUDIO*)mciGetDriverData(wDevID); WCHAR root[7], drive = 0; - int count; + unsigned int count; TRACE("(%04X, %08X, %p);\n", wDevID, dwFlags, lpOpenParms);

1 0

Dmitry Timoshkov : user32: Add a test for fullscreen window sizing, make it pass under Wine.
by Alexandre Julliard 17 Oct '08

17 Oct '08

Module: wine Branch: master Commit: 922dc3122476a8b16e7213c9eeb9d4d15dd525b9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=922dc3122476a8b16e7213c9e… Author: Dmitry Timoshkov <dmitry(a)codeweavers.com> Date: Thu Oct 16 19:58:56 2008 +0900 user32: Add a test for fullscreen window sizing, make it pass under Wine. It appears that Windows doesn't always restrict the window size by the monitor work area. The tests pass under XP SP3 and Win98. --- dlls/user32/tests/win.c | 191 +++++++++++++++++++++++++++++++++++++++++++++++ dlls/user32/winpos.c | 21 ++++-- 2 files changed, 206 insertions(+), 6 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=922dc3122476a8b16e721…

1 0

Alexandre Julliard : services.exe: Pass the service name as first argument when auto-starting services.
by Alexandre Julliard 16 Oct '08

16 Oct '08

Module: wine Branch: stable Commit: 2e739275e2c9556ee5ea13fde2291e931b0ab47d URL: http://source.winehq.org/git/wine.git/?a=commit;h=2e739275e2c9556ee5ea13fde… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Jul 2 12:02:11 2008 +0200 services.exe: Pass the service name as first argument when auto-starting services. (cherry picked from commit 7e8ecd2d6437c0ed2ee83fc2b63b77c950a27c31) --- programs/services/services.c | 5 ++++- 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/programs/services/services.c b/programs/services/services.c index 806fbb6..36ed117 100644 --- a/programs/services/services.c +++ b/programs/services/services.c @@ -256,8 +256,11 @@ static void scmdatabase_autostart_services(struct scmdatabase *db) for (i = 0; i < size; i++) { DWORD err; + const WCHAR *argv[2]; service = services_list[i]; - err = service_start(service, 0, NULL); + argv[0] = service->name; + argv[1] = NULL; + err = service_start(service, 1, argv); /* FIXME: do something if the service failed to start */ release_service(service); }

1 0

Massimo Del Fedele : opengl32: glGetString() should return NULL on NULL context.
by Alexandre Julliard 16 Oct '08

16 Oct '08

Module: wine Branch: stable Commit: f5cbd282eb431353197ab2785cf2f58a607ae261 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f5cbd282eb431353197ab2785… Author: Massimo Del Fedele <max(a)veneto.com> Date: Wed Jul 2 16:51:26 2008 +0200 opengl32: glGetString() should return NULL on NULL context. (cherry picked from commit d6a6b88f1904783157245cae0b3f5a8d3de1ec20) --- dlls/opengl32/wgl.c | 5 +++++ 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/dlls/opengl32/wgl.c b/dlls/opengl32/wgl.c index 6fbdeb4..a07fcf3 100644 --- a/dlls/opengl32/wgl.c +++ b/dlls/opengl32/wgl.c @@ -632,6 +632,11 @@ const GLubyte * WINAPI wine_glGetString( GLenum name ) const GLubyte *ret; const char* GL_Extensions = NULL; + /* this is for buggy nvidia driver, crashing if called from a different + thread with no context */ + if(wglGetCurrentContext() == NULL) + return NULL; + if (GL_EXTENSIONS != name) { ENTER_GL(); ret = glGetString(name);

1 0

Aric Stewart : coreaudio: Make sure Port_SendToMessageThread is not NULL before calling CFMessagePortSendRequest .
by Alexandre Julliard 16 Oct '08

16 Oct '08

Module: wine Branch: stable Commit: 3a50f8fcaa2ad77f812ebd482ee25d2c91c86e29 URL: http://source.winehq.org/git/wine.git/?a=commit;h=3a50f8fcaa2ad77f812ebd482… Author: Aric Stewart <aric(a)codeweavers.com> Date: Thu Jul 10 11:28:56 2008 +0900 coreaudio: Make sure Port_SendToMessageThread is not NULL before calling CFMessagePortSendRequest. Prevents crashing when closing audio device during playback. (cherry picked from commit 686ea70c1853aecc235143862b42e6fb09333ed4) --- dlls/winecoreaudio.drv/audio.c | 9 +++++++++ 1 files changed, 9 insertions(+), 0 deletions(-) diff --git a/dlls/winecoreaudio.drv/audio.c b/dlls/winecoreaudio.drv/audio.c index 90d664f..549b69e 100644 --- a/dlls/winecoreaudio.drv/audio.c +++ b/dlls/winecoreaudio.drv/audio.c @@ -349,6 +349,9 @@ static void wodSendNotifyCompletionsMessage(WINE_WAVEOUT* wwo) CFDataRef data; UInt32 buffer; + if (!Port_SendToMessageThread) + return; + buffer = (UInt32) wwo->woID; data = CFDataCreate(kCFAllocatorDefault, (UInt8 *)&buffer, sizeof(buffer)); @@ -368,6 +371,9 @@ static void wodSendNotifyInputCompletionsMessage(WINE_WAVEIN* wwi) CFDataRef data; UInt32 buffer; + if (!Port_SendToMessageThread) + return; + buffer = (UInt32) wwi->wiID; data = CFDataCreate(kCFAllocatorDefault, (UInt8 *)&buffer, sizeof(buffer)); @@ -694,6 +700,9 @@ void CoreAudio_WaveRelease(void) /* Stop CFRunLoop in messageThread */ TRACE("()\n"); + if (!Port_SendToMessageThread) + return; + CFMessagePortSendRequest(Port_SendToMessageThread, kStopLoopMessage, NULL, 0.0, 0.0, NULL, NULL); CFRelease(Port_SendToMessageThread); Port_SendToMessageThread = NULL;

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 October 2008