wine-commits May 2009

wine-commits@winehq.org

4 participants
793 discussions

Jacek Caban : urlmon: Fixed buffer size argument in Read call.
by Alexandre Julliard 19 May '09

19 May '09

Module: wine Branch: master Commit: 23c6cd4241a775d69b3691e6d0269572606ebdb9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=23c6cd4241a775d69b3691e6d… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon May 18 18:29:54 2009 +0200 urlmon: Fixed buffer size argument in Read call. --- dlls/urlmon/bindprot.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/urlmon/bindprot.c b/dlls/urlmon/bindprot.c index 8977cac..ec58436 100644 --- a/dlls/urlmon/bindprot.c +++ b/dlls/urlmon/bindprot.c @@ -615,7 +615,7 @@ static HRESULT WINAPI ProtocolHandler_Read(IInternetProtocol *iface, void *pv, if(read < cb) { ULONG cread = 0; - hres = IInternetProtocol_Read(This->protocol, (BYTE*)pv+read, cb, &cread); + hres = IInternetProtocol_Read(This->protocol, (BYTE*)pv+read, cb-read, &cread); read += cread; }

1 0

Jacek Caban : urlmon: Use internal IInternetProtocol protocol for communication with pluggable protocol in BindProtocol .
by Alexandre Julliard 19 May '09

19 May '09

Module: wine Branch: master Commit: e5f8427abf9867823e44a9aefcf24f36f88b7429 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e5f8427abf9867823e44a9aef… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon May 18 18:29:24 2009 +0200 urlmon: Use internal IInternetProtocol protocol for communication with pluggable protocol in BindProtocol. --- dlls/urlmon/bindprot.c | 253 +++++++++++++++++++++++++++++++++++++----------- 1 files changed, 195 insertions(+), 58 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=e5f8427abf9867823e44a…

1 0

Paul Vriens : kernel32/tests: Fix some test failures on Win9x/WinME.
by Alexandre Julliard 19 May '09

19 May '09

Module: wine Branch: master Commit: 37babfcece3abe4501a623b12308273544186bd8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=37babfcece3abe4501a623b12… Author: Paul Vriens <Paul.Vriens.Wine(a)gmail.com> Date: Mon May 18 19:22:28 2009 +0200 kernel32/tests: Fix some test failures on Win9x/WinME. --- dlls/kernel32/tests/process.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/dlls/kernel32/tests/process.c b/dlls/kernel32/tests/process.c index 83b61ad..d54a4b8 100644 --- a/dlls/kernel32/tests/process.c +++ b/dlls/kernel32/tests/process.c @@ -943,6 +943,7 @@ static void test_CommandLine(void) ret = CreateProcessA(buffer, NULL, NULL, NULL, FALSE, 0L, NULL, NULL, &startup, &info); ok(!ret, "CreateProcessA unexpectedly succeeded\n"); ok(GetLastError() == ERROR_PATH_NOT_FOUND || + broken(GetLastError() == ERROR_FILE_NOT_FOUND) /* Win9x/WinME */ || broken(GetLastError() == ERROR_ACCESS_DENIED) /* Win98 */, "Expected ERROR_PATH_NOT_FOUND, got %d\n", GetLastError()); @@ -953,6 +954,7 @@ static void test_CommandLine(void) ret = CreateProcessA(buffer, buffer2, NULL, NULL, FALSE, 0L, NULL, NULL, &startup, &info); ok(!ret, "CreateProcessA unexpectedly succeeded\n"); ok(GetLastError() == ERROR_PATH_NOT_FOUND || + broken(GetLastError() == ERROR_FILE_NOT_FOUND) /* Win9x/WinME */ || broken(GetLastError() == ERROR_ACCESS_DENIED) /* Win98 */, "Expected ERROR_PATH_NOT_FOUND, got %d\n", GetLastError());

1 0

Paul Vriens : ntdll: Fix a typo.
by Alexandre Julliard 19 May '09

19 May '09

Module: wine Branch: master Commit: 756eea7bdd29ab76d22d342a146596f3905bb4fa URL: http://source.winehq.org/git/wine.git/?a=commit;h=756eea7bdd29ab76d22d342a1… Author: Paul Vriens <Paul.Vriens.Wine(a)gmail.com> Date: Mon May 18 17:05:10 2009 +0200 ntdll: Fix a typo. --- dlls/ntdll/version.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/ntdll/version.c b/dlls/ntdll/version.c index bf9db4d..d43a5b6 100644 --- a/dlls/ntdll/version.c +++ b/dlls/ntdll/version.c @@ -519,7 +519,7 @@ done: user_shared_data->MinorNtVersion = current_version->dwMinorVersion; user_shared_data->SuiteMask = current_version->wSuiteMask; - TRACE( "got %d.%d plaform %d build %x name %s service pack %d.%d product %d\n", + TRACE( "got %d.%d platform %d build %x name %s service pack %d.%d product %d\n", current_version->dwMajorVersion, current_version->dwMinorVersion, current_version->dwPlatformId, current_version->dwBuildNumber, debugstr_w(current_version->szCSDVersion),

1 0

Paul Vriens : wine.inf: Set the correct initial service pack information.
by Alexandre Julliard 19 May '09

19 May '09

Module: wine Branch: master Commit: 4626308a2b063b6d8bdd531dd95626e38962ebaf URL: http://source.winehq.org/git/wine.git/?a=commit;h=4626308a2b063b6d8bdd531dd… Author: Paul Vriens <Paul.Vriens.Wine(a)gmail.com> Date: Mon May 18 16:48:16 2009 +0200 wine.inf: Set the correct initial service pack information. --- tools/wine.inf.in | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/tools/wine.inf.in b/tools/wine.inf.in index f42dea8..25cec58 100644 --- a/tools/wine.inf.in +++ b/tools/wine.inf.in @@ -2933,5 +2933,5 @@ HKLM,%CurrentVersionNT%,"CSDVersion",2,"Service Pack 2" HKLM,%CurrentVersionNT%,"CurrentBuildNumber",2,"2600" HKLM,%CurrentVersionNT%,"CurrentType",2,"Uniprocessor Free" HKLM,%Control%\ProductOptions,"ProductType",2,"WinNT" -HKLM,%Control%\Windows,"CSDVersion",0x10003,0x400 +HKLM,%Control%\Windows,"CSDVersion",0x10003,0x200 HKLM,%Control%\Session Manager\Environment,"OS",2,"Windows_NT"

1 0

Aric Stewart : msctf: Starting implementation of ITfContext:: RequestEditSession and ITextStoreACPSink::OnLockGranted.
by Alexandre Julliard 19 May '09

19 May '09

Module: wine Branch: master Commit: 4bb8e9959a0c3088c5de014dcf59b87edf7549cc URL: http://source.winehq.org/git/wine.git/?a=commit;h=4bb8e9959a0c3088c5de014dc… Author: Aric Stewart <aric(a)codeweavers.com> Date: Mon May 18 08:58:43 2009 -0500 msctf: Starting implementation of ITfContext::RequestEditSession and ITextStoreACPSink::OnLockGranted. --- dlls/msctf/context.c | 68 ++++++++++++++++++++++++++++++++++-- dlls/msctf/tests/inputprocessor.c | 30 +++++++++++----- include/textstor.idl | 2 + 3 files changed, 87 insertions(+), 13 deletions(-) diff --git a/dlls/msctf/context.c b/dlls/msctf/context.c index 42ae71e..1f255c8 100644 --- a/dlls/msctf/context.c +++ b/dlls/msctf/context.c @@ -74,6 +74,7 @@ typedef struct tagContext { ITfContextOwnerCompositionSink *pITfContextOwnerCompositionSink; ITextStoreACPSink *pITextStoreACPSink; + ITfEditSession* currentEditSession; /* kept as separate lists to reduce unnecessary iterations */ struct list pContextKeyEventSink; @@ -206,9 +207,53 @@ static HRESULT WINAPI Context_RequestEditSession (ITfContext *iface, TfClientId tid, ITfEditSession *pes, DWORD dwFlags, HRESULT *phrSession) { + HRESULT hr; Context *This = (Context *)iface; - FIXME("STUB:(%p)\n",This); - return E_NOTIMPL; + DWORD dwLockFlags = 0x0; + TS_STATUS status; + + TRACE("(%p) %i %p %x %p\n",This, tid, pes, dwFlags, phrSession); + + if (!(dwFlags & TF_ES_READ) && !(dwFlags & TF_ES_READWRITE)) + { + *phrSession = E_FAIL; + return E_INVALIDARG; + } + + if (!This->pITextStoreACP) + { + FIXME("No ITextStoreACP avaliable\n"); + *phrSession = E_FAIL; + return E_FAIL; + } + + if (!(dwFlags & TF_ES_ASYNC)) + dwLockFlags &= TS_LF_SYNC; + + if (dwFlags & TF_ES_READ) + dwLockFlags &= TS_LF_READ; + else if ((dwFlags & TF_ES_READWRITE) == TF_ES_READWRITE) + dwLockFlags &= TS_LF_READWRITE; + + /* TODO: cache this */ + ITextStoreACP_GetStatus(This->pITextStoreACP, &status); + + if (((dwFlags & TF_ES_READWRITE) == TF_ES_READWRITE) && (status.dwDynamicFlags & TS_SD_READONLY)) + { + *phrSession = TS_E_READONLY; + return S_OK; + } + + if (FAILED (ITfEditSession_QueryInterface(pes, &IID_ITfEditSession, (LPVOID*)&This->currentEditSession))) + { + *phrSession = E_FAIL; + return E_INVALIDARG; + } + + + hr = ITextStoreACP_RequestLock(This->pITextStoreACP, dwLockFlags, phrSession); + + return hr; } static HRESULT WINAPI Context_InWriteSession (ITfContext *iface, @@ -597,8 +642,23 @@ static HRESULT WINAPI TextStoreACPSink_OnLockGranted(ITextStoreACPSink *iface, DWORD dwLockFlags) { TextStoreACPSink *This = (TextStoreACPSink *)iface; - FIXME("STUB:(%p)\n",This); - return E_NOTIMPL; + HRESULT hr; + + TRACE("(%p) %x\n",This, dwLockFlags); + + if (!This->pContext || !This->pContext->currentEditSession) + { + ERR("OnLockGranted called on a context without a current edit session\nZ"); + return E_FAIL; + } + + /* TODO: generate and use an edit cookie */ + hr = ITfEditSession_DoEditSession(This->pContext->currentEditSession, 0xdeadcafe); + + ITfEditSession_Release(This->pContext->currentEditSession); + This->pContext->currentEditSession = NULL; + + return hr; } static HRESULT WINAPI TextStoreACPSink_OnStartEditTransaction(ITextStoreACPSink *iface) diff --git a/dlls/msctf/tests/inputprocessor.c b/dlls/msctf/tests/inputprocessor.c index a4d9979..318149b 100644 --- a/dlls/msctf/tests/inputprocessor.c +++ b/dlls/msctf/tests/inputprocessor.c @@ -149,8 +149,15 @@ static HRESULT WINAPI TextStoreACP_RequestLock(ITextStoreACP *iface, static HRESULT WINAPI TextStoreACP_GetStatus(ITextStoreACP *iface, TS_STATUS *pdcs) { - ok(test_ACP_GetStatus == SINK_EXPECTED, "Unexpected TextStoreACP_GetStatus\n"); + static UINT count = 0; + count ++; + + if (count == 1) + ok(test_ACP_GetStatus == SINK_EXPECTED, "Unexpected TextStoreACP_GetStatus\n"); + else + todo_wine ok(count == 1,"GetStatus called too many times\n"); test_ACP_GetStatus = SINK_FIRED; + pdcs->dwDynamicFlags = TS_SD_READONLY; return S_OK; } static HRESULT WINAPI TextStoreACP_QueryInsert(ITextStoreACP *iface, @@ -1410,19 +1417,24 @@ static void test_TStoApplicationText(void) hrSession = 0xfeedface; /* Test no premissions flags */ hr = ITfContext_RequestEditSession(cxt, tid, es, TF_ES_SYNC, &hrSession); - todo_wine ok(hr == E_INVALIDARG,"RequestEditSession should have failed with %x not %x\n",E_INVALIDARG,hr); - todo_wine ok(hrSession == E_FAIL,"hrSession should be %x not %x\n",E_FAIL,hrSession); + ok(hr == E_INVALIDARG,"RequestEditSession should have failed with %x not %x\n",E_INVALIDARG,hr); + ok(hrSession == E_FAIL,"hrSession should be %x not %x\n",E_FAIL,hrSession); hrSession = 0xfeedface; test_ACP_GetStatus = SINK_EXPECTED; + hr = ITfContext_RequestEditSession(cxt, tid, es, TF_ES_SYNC|TF_ES_READWRITE, &hrSession); + ok(SUCCEEDED(hr),"ITfContext_RequestEditSession failed\n"); + ok(hrSession == TS_E_READONLY,"Unexpected hrSession (%x)\n",hrSession); + ok(test_ACP_GetStatus == SINK_FIRED," expected GetStatus not fired\n"); + + test_ACP_GetStatus = SINK_UNEXPECTED; test_ACP_RequestLock = SINK_EXPECTED; hrSession = 0xfeedface; - hr = ITfContext_RequestEditSession(cxt, tid, es, TF_ES_SYNC|TF_ES_READWRITE, &hrSession); - todo_wine ok(SUCCEEDED(hr),"ITfContext_RequestEditSession failed\n"); - todo_wine ok(test_ACP_GetStatus == SINK_FIRED," expected GetStatus not fired\n"); - todo_wine ok(test_ACP_RequestLock == SINK_FIRED," expected RequestLock not fired\n"); - todo_wine ok(test_DoEditSession == SINK_FIRED," expected DoEditSession not fired\n"); - todo_wine ok(hrSession == 0xdeadcafe,"Unexpected hrSession\n"); + hr = ITfContext_RequestEditSession(cxt, tid, es, TF_ES_SYNC|TF_ES_READ, &hrSession); + ok(SUCCEEDED(hr),"ITfContext_RequestEditSession failed\n"); + ok(test_ACP_RequestLock == SINK_FIRED," expected RequestLock not fired\n"); + ok(test_DoEditSession == SINK_FIRED," expected DoEditSession not fired\n"); + ok(hrSession == 0xdeadcafe,"Unexpected hrSession (%x)\n",hrSession); ITfContext_Release(cxt); ITfDocumentMgr_Release(dm); diff --git a/include/textstor.idl b/include/textstor.idl index b603198..90e30ca 100644 --- a/include/textstor.idl +++ b/include/textstor.idl @@ -20,6 +20,8 @@ import "oaidl.idl"; #endif +cpp_quote("#define TS_E_READONLY MAKE_HRESULT(SEVERITY_ERROR, FACILITY_ITF, 0x0209)") + const ULONG TS_DEFAULT_SELECTION = ~0u; const DWORD TS_SD_READONLY = 0x001;

1 0

Andrew Nguyen : kernel32/tests: Use named constants for a drive test.
by Alexandre Julliard 18 May '09

18 May '09

Module: wine Branch: master Commit: 94945af5566c73fa3832f7fe70ebc6dede54861d URL: http://source.winehq.org/git/wine.git/?a=commit;h=94945af5566c73fa3832f7fe7… Author: Andrew Nguyen <arethusa26(a)gmail.com> Date: Mon May 18 05:07:22 2009 -0500 kernel32/tests: Use named constants for a drive test. --- dlls/kernel32/tests/drive.c | 6 ++++-- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/dlls/kernel32/tests/drive.c b/dlls/kernel32/tests/drive.c index 6bee4a7..af1cf7f 100644 --- a/dlls/kernel32/tests/drive.c +++ b/dlls/kernel32/tests/drive.c @@ -39,7 +39,8 @@ static void test_GetDriveTypeA(void) for (drive[0] = 'A'; drive[0] <= 'Z'; drive[0]++) { type = GetDriveTypeA(drive); - ok(type > 0 && type <= 6, "not a valid drive %c: type %u\n", drive[0], type); + ok(type > DRIVE_UNKNOWN && type <= DRIVE_RAMDISK, + "not a valid drive %c: type %u\n", drive[0], type); if (!(logical_drives & 1)) ok(type == DRIVE_NO_ROOT_DIR, @@ -67,7 +68,8 @@ static void test_GetDriveTypeW(void) win_skip("GetDriveTypeW is not available on Win9x\n"); return; } - ok(type > 0 && type <= 6, "not a valid drive %c: type %u\n", drive[0], type); + ok(type > DRIVE_UNKNOWN && type <= DRIVE_RAMDISK, + "not a valid drive %c: type %u\n", drive[0], type); if (!(logical_drives & 1)) ok(type == DRIVE_NO_ROOT_DIR,

1 0

Andrew Nguyen : kernel32: Correct the last error of CreateProcessW with an empty command line string .
by Alexandre Julliard 18 May '09

18 May '09

Module: wine Branch: master Commit: 7c2c6a575c74190864f2659f188d609eaab73d05 URL: http://source.winehq.org/git/wine.git/?a=commit;h=7c2c6a575c74190864f2659f1… Author: Andrew Nguyen <arethusa26(a)gmail.com> Date: Mon May 18 05:07:18 2009 -0500 kernel32: Correct the last error of CreateProcessW with an empty command line string. --- dlls/kernel32/process.c | 1 + dlls/kernel32/tests/process.c | 34 ++++++++++++++++++++++++++++++++++ 2 files changed, 35 insertions(+), 0 deletions(-) diff --git a/dlls/kernel32/process.c b/dlls/kernel32/process.c index 9e6aeaa..f00d972 100644 --- a/dlls/kernel32/process.c +++ b/dlls/kernel32/process.c @@ -1824,6 +1824,7 @@ static LPWSTR get_file_name( LPCWSTR appname, LPWSTR cmdline, LPWSTR buffer, sprintfW( ret, quotesW, name ); strcatW( ret, p ); } + else if (!ret) SetLastError( ERROR_FILE_NOT_FOUND ); done: HeapFree( GetProcessHeap(), 0, name ); diff --git a/dlls/kernel32/tests/process.c b/dlls/kernel32/tests/process.c index 479a376..83b61ad 100644 --- a/dlls/kernel32/tests/process.c +++ b/dlls/kernel32/tests/process.c @@ -955,6 +955,40 @@ static void test_CommandLine(void) ok(GetLastError() == ERROR_PATH_NOT_FOUND || broken(GetLastError() == ERROR_ACCESS_DENIED) /* Win98 */, "Expected ERROR_PATH_NOT_FOUND, got %d\n", GetLastError()); + + /* Test empty command line parameter. */ + SetLastError(0xdeadbeef); + ret = CreateProcessA(NULL, buffer2, NULL, NULL, FALSE, 0L, NULL, NULL, &startup, &info); + ok(!ret, "CreateProcessA unexpectedly succeeded\n"); + ok(GetLastError() == ERROR_FILE_NOT_FOUND || + GetLastError() == ERROR_PATH_NOT_FOUND /* NT4 */ || + GetLastError() == ERROR_BAD_PATHNAME /* Win98 */, + "Expected ERROR_FILE_NOT_FOUND, got %d\n", GetLastError()); + + strcpy(buffer, "doesnotexist.exe"); + strcpy(buffer2, "does not exist.exe"); + + /* Test nonexistent application name. */ + SetLastError(0xdeadbeef); + ret = CreateProcessA(buffer, NULL, NULL, NULL, FALSE, 0L, NULL, NULL, &startup, &info); + ok(!ret, "CreateProcessA unexpectedly succeeded\n"); + ok(GetLastError() == ERROR_FILE_NOT_FOUND, "Expected ERROR_FILE_NOT_FOUND, got %d\n", GetLastError()); + + SetLastError(0xdeadbeef); + ret = CreateProcessA(buffer2, NULL, NULL, NULL, FALSE, 0L, NULL, NULL, &startup, &info); + ok(!ret, "CreateProcessA unexpectedly succeeded\n"); + ok(GetLastError() == ERROR_FILE_NOT_FOUND, "Expected ERROR_FILE_NOT_FOUND, got %d\n", GetLastError()); + + /* Test nonexistent command line parameter. */ + SetLastError(0xdeadbeef); + ret = CreateProcessA(NULL, buffer, NULL, NULL, FALSE, 0L, NULL, NULL, &startup, &info); + ok(!ret, "CreateProcessA unexpectedly succeeded\n"); + ok(GetLastError() == ERROR_FILE_NOT_FOUND, "Expected ERROR_FILE_NOT_FOUND, got %d\n", GetLastError()); + + SetLastError(0xdeadbeef); + ret = CreateProcessA(NULL, buffer2, NULL, NULL, FALSE, 0L, NULL, NULL, &startup, &info); + ok(!ret, "CreateProcessA unexpectedly succeeded\n"); + ok(GetLastError() == ERROR_FILE_NOT_FOUND, "Expected ERROR_FILE_NOT_FOUND, got %d\n", GetLastError()); } static void test_Directory(void)

1 0

Andrew Nguyen : kernel32: Correct the last error of CreateProcessW with an empty application name string .
by Alexandre Julliard 18 May '09

18 May '09

Module: wine Branch: master Commit: 645e59c490bcbd57a4a97a076295e171c44a80c1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=645e59c490bcbd57a4a97a076… Author: Andrew Nguyen <arethusa26(a)gmail.com> Date: Mon May 18 05:07:10 2009 -0500 kernel32: Correct the last error of CreateProcessW with an empty application name string. --- dlls/kernel32/process.c | 9 ++------- dlls/kernel32/tests/process.c | 20 ++++++++++++++++++++ 2 files changed, 22 insertions(+), 7 deletions(-) diff --git a/dlls/kernel32/process.c b/dlls/kernel32/process.c index 8f486e0..9e6aeaa 100644 --- a/dlls/kernel32/process.c +++ b/dlls/kernel32/process.c @@ -210,15 +210,10 @@ static HANDLE open_exe_file( const WCHAR *name ) { WCHAR buffer[MAX_PATH]; /* file doesn't exist, check for builtin */ - if (!contains_path( name )) goto error; - if (!get_builtin_path( name, NULL, buffer, sizeof(buffer) )) goto error; - handle = 0; + if (contains_path( name ) && get_builtin_path( name, NULL, buffer, sizeof(buffer) )) + handle = 0; } return handle; - - error: - SetLastError( ERROR_FILE_NOT_FOUND ); - return INVALID_HANDLE_VALUE; } diff --git a/dlls/kernel32/tests/process.c b/dlls/kernel32/tests/process.c index 6aac615..479a376 100644 --- a/dlls/kernel32/tests/process.c +++ b/dlls/kernel32/tests/process.c @@ -935,6 +935,26 @@ static void test_CommandLine(void) ok(GetLastError() == ERROR_INVALID_PARAMETER, "Expected ERROR_INVALID_PARAMETER, got %d\n", GetLastError()); } + + buffer[0] = '\0'; + + /* Test empty application name parameter. */ + SetLastError(0xdeadbeef); + ret = CreateProcessA(buffer, NULL, NULL, NULL, FALSE, 0L, NULL, NULL, &startup, &info); + ok(!ret, "CreateProcessA unexpectedly succeeded\n"); + ok(GetLastError() == ERROR_PATH_NOT_FOUND || + broken(GetLastError() == ERROR_ACCESS_DENIED) /* Win98 */, + "Expected ERROR_PATH_NOT_FOUND, got %d\n", GetLastError()); + + buffer2[0] = '\0'; + + /* Test empty application name and command line parameters. */ + SetLastError(0xdeadbeef); + ret = CreateProcessA(buffer, buffer2, NULL, NULL, FALSE, 0L, NULL, NULL, &startup, &info); + ok(!ret, "CreateProcessA unexpectedly succeeded\n"); + ok(GetLastError() == ERROR_PATH_NOT_FOUND || + broken(GetLastError() == ERROR_ACCESS_DENIED) /* Win98 */, + "Expected ERROR_PATH_NOT_FOUND, got %d\n", GetLastError()); } static void test_Directory(void)

1 0

Andrew Nguyen : kernel32: Don' t validate the command line parameter of CreateProcessW.
by Alexandre Julliard 18 May '09

18 May '09

Module: wine Branch: master Commit: 6c27fa103896779169c4c8d5d4a6c1f344c651d7 URL: http://source.winehq.org/git/wine.git/?a=commit;h=6c27fa103896779169c4c8d5d… Author: Andrew Nguyen <arethusa26(a)gmail.com> Date: Mon May 18 05:07:04 2009 -0500 kernel32: Don't validate the command line parameter of CreateProcessW. --- dlls/kernel32/process.c | 6 ------ dlls/kernel32/tests/process.c | 10 ++++++++++ 2 files changed, 10 insertions(+), 6 deletions(-) diff --git a/dlls/kernel32/process.c b/dlls/kernel32/process.c index b10d08a..8f486e0 100644 --- a/dlls/kernel32/process.c +++ b/dlls/kernel32/process.c @@ -1787,12 +1787,6 @@ static LPWSTR get_file_name( LPCWSTR appname, LPWSTR cmdline, LPWSTR buffer, return ret; } - if (!cmdline) - { - SetLastError( ERROR_INVALID_PARAMETER ); - return NULL; - } - /* first check for a quoted file name */ if ((cmdline[0] == '"') && ((p = strchrW( cmdline + 1, '"' )))) diff --git a/dlls/kernel32/tests/process.c b/dlls/kernel32/tests/process.c index 7cc5841..6aac615 100644 --- a/dlls/kernel32/tests/process.c +++ b/dlls/kernel32/tests/process.c @@ -925,6 +925,16 @@ static void test_CommandLine(void) okChildStringWA("Arguments", "CommandLineW", buffer2); release_memory(); assert(DeleteFileA(resfile) != 0); + + if (0) /* Test crashes on NT-based Windows. */ + { + /* Test NULL application name and command line parameters. */ + SetLastError(0xdeadbeef); + ret = CreateProcessA(NULL, NULL, NULL, NULL, FALSE, 0L, NULL, NULL, &startup, &info); + ok(!ret, "CreateProcessA unexpectedly succeeded\n"); + ok(GetLastError() == ERROR_INVALID_PARAMETER, + "Expected ERROR_INVALID_PARAMETER, got %d\n", GetLastError()); + } } static void test_Directory(void)

1 0

← Newer
1
...
34
35
36
37
38
39
40
...
80
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 May 2009