March 2007 - Wine-Commits - list.winehq.org

Leslie Choong : comctl32: updown: Expect macro and spelling errors.
by Alexandre Julliard March 15, 2007

March 15, 2007

Module: wine Branch: master Commit: 4e337ad8d5f392594710691c93769dc60017359d URL: http://source.winehq.org/git/wine.git/?a=commit;h=4e337ad8d5f392594710691c9… Author: Leslie Choong <septikus(a)gmail.com> Date: Tue Mar 13 20:22:03 2007 -0800 comctl32: updown: Expect macro and spelling errors. --- dlls/comctl32/tests/updown.c | 101 +++++++++++++++++++++--------------------- 1 files changed, 51 insertions(+), 50 deletions(-) diff --git a/dlls/comctl32/tests/updown.c b/dlls/comctl32/tests/updown.c index 5546ab5..c8f944f 100644 --- a/dlls/comctl32/tests/updown.c +++ b/dlls/comctl32/tests/updown.c @@ -41,7 +41,6 @@ * - check UDM_GETPOS, for up-down control with a buddy window, position is the caption of the buddy window, so change the * caption of the buddy window then call UDM_GETPOS * - check UDM_SETRANGE, max can be less than min, so clicking the up arrow decreases the current position - * - check UDM_GETRANGE * - more stuff to test */ @@ -53,6 +52,8 @@ #include "wine/test.h" #include "msg.h" +#define expect(EXPECTED,GOT) ok((GOT)==(EXPECTED), "Expected %d, got %d\n", (EXPECTED), (GOT)) + #define NUM_MSG_SEQUENCES 3 #define PARENT_SEQ_INDEX 0 #define EDIT_SEQ_INDEX 1 @@ -281,48 +282,48 @@ static void test_updown_pos(void) /* Set Range from 0 to 100 */ SendMessage(updown, UDM_SETRANGE, 0 , MAKELONG(100,0) ); r = SendMessage(updown, UDM_GETRANGE, 0,0); - ok(LOWORD(r) == 100, "Expected 100, got %d\n", LOWORD(r)); - ok(HIWORD(r) == 0, "Expected 0, got %d\n", HIWORD(r)); + expect(100,LOWORD(r)); + expect(0,HIWORD(r)); /* Set the position to 5, return is not checked as it was set before func call */ SendMessage(updown, UDM_SETPOS, 0 , MAKELONG(5,0) ); /* Since UDM_SETBUDDYINT was not set at creation HIWORD(r) will always be 1 as a return from UDM_GETPOS */ /* Get the position, which should be 5 */ r = SendMessage(updown, UDM_GETPOS, 0 , 0 ); - ok(LOWORD(r) == 5, "Expected 5, got %d\n", LOWORD(r)); - ok(HIWORD(r) == 1, "Expected 1, got %d\n", HIWORD(r)); + expect(5,LOWORD(r)); + expect(1,HIWORD(r)); /* Set the position to 0, return should be 5 */ r = SendMessage(updown, UDM_SETPOS, 0 , MAKELONG(0,0) ); - ok(r == 5, "Expected 5, got %d\n", r); + expect(5,r); /* Get the position, which should be 0 */ r = SendMessage(updown, UDM_GETPOS, 0 , 0 ); - ok(LOWORD(r) == 0, "Expected 0, got %d\n", LOWORD(r)); - ok(HIWORD(r) == 1, "Expected 1, got %d\n", HIWORD(r)); + expect(0,LOWORD(r)); + expect(1,HIWORD(r)); /* Set the position to -1, return should be 0 */ r = SendMessage(updown, UDM_SETPOS, 0 , MAKELONG(-1,0) ); - ok(r == 0, "Expected 0, got %d\n", r); + expect(0,r); /* Get the position, which should be 0 */ r = SendMessage(updown, UDM_GETPOS, 0 , 0 ); - ok(LOWORD(r) == 0, "Expected 0, got %d\n", LOWORD(r)); - ok(HIWORD(r) == 1, "Expected 1, got %d\n", HIWORD(r)); + expect(0,LOWORD(r)); + expect(1,HIWORD(r)); /* Set the position to 100, return should be 0 */ r = SendMessage(updown, UDM_SETPOS, 0 , MAKELONG(100,0) ); - ok(r == 0, "Expected 0, got %d\n", r); + expect(0,r); /* Get the position, which should be 100 */ r = SendMessage(updown, UDM_GETPOS, 0 , 0 ); - ok(LOWORD(r) == 100, "Expected 100, got %d\n", LOWORD(r)); - ok(HIWORD(r) == 1, "Expected 1, got %d\n", HIWORD(r)); + expect(100,LOWORD(r)); + expect(1,HIWORD(r)); /* Set the position to 101, return should be 100 */ r = SendMessage(updown, UDM_SETPOS, 0 , MAKELONG(101,0) ); - ok(r == 100, "Expected 100, got %d\n", r); + expect(100,r); /* Get the position, which should be 100 */ r = SendMessage(updown, UDM_GETPOS, 0 , 0 ); - ok(LOWORD(r) == 100, "Expected 100, got %d\n", LOWORD(r)); - ok(HIWORD(r) == 1, "Expected 1, got %d\n", HIWORD(r)); + expect(100,LOWORD(r)); + expect(1,HIWORD(r)); } static void test_updown_pos32(void) @@ -334,45 +335,45 @@ static void test_updown_pos32(void) SendMessage(updown, UDM_SETRANGE32, 0 , 1000 ); r = SendMessage(updown, UDM_GETRANGE32, (WPARAM) &low , (LPARAM) &high ); - ok(low == 0, "Expected 0, got %d\n", low); - ok(high == 1000, "Expected 1000, got %d\n", high); + expect(0,low); + expect(1000,high); /* Set position to 500, don't check return since it is unset*/ - r = SendMessage(updown, UDM_SETPOS32, 0 , 500 ); + SendMessage(updown, UDM_SETPOS32, 0 , 500 ); /* Since UDM_SETBUDDYINT was not set at creation bRet will always be true as a return from UDM_GETPOS32 */ r = SendMessage(updown, UDM_GETPOS32, 0 , (LPARAM) &high ); - ok(r == 500, "Expected 500, got %d\n", r); - ok(high == 1 , "Expected 0, got %d\n", high); + expect(500,r); + expect(1,high); /* Set position to 0, return should be 500 */ r = SendMessage(updown, UDM_SETPOS32, 0 , 0 ); - ok(r == 500, "Expected 500, got %d\n", r); + expect(500,r); r = SendMessage(updown, UDM_GETPOS32, 0 , (LPARAM) &high ); - ok(r == 0, "Expected 0, got %d\n", r); - ok(high == 1 , "Expected 0, got %d\n", high); + expect(0,r); + expect(1,high); - /* Set position to -1 which sohuld become 0, return should be 0 */ + /* Set position to -1 which should become 0, return should be 0 */ r = SendMessage(updown, UDM_SETPOS32, 0 , -1 ); - ok(r == 0, "Expected 0, got %d\n", r); + expect(0,r); r = SendMessage(updown, UDM_GETPOS32, 0 , (LPARAM) &high ); - ok(r == 0, "Expected 0, got %d\n", r); - ok(high == 1 , "Expected 0, got %d\n", high); + expect(0,r); + expect(1,high); /* Set position to 1000, return should be 0 */ r = SendMessage(updown, UDM_SETPOS32, 0 , 1000 ); - ok(r == 0, "Expected 0, got %d\n", r); + expect(0,r); r = SendMessage(updown, UDM_GETPOS32, 0 , (LPARAM) &high ); - ok(r == 1000, "Expected 1000, got %d\n", r); - ok(high == 1 , "Expected 0, got %d\n", high); + expect(1000,r); + expect(1,high); - /* Set position to 1001 which sohuld become 1000, return should be 1000 */ + /* Set position to 1001 which should become 1000, return should be 1000 */ r = SendMessage(updown, UDM_SETPOS32, 0 , 1001 ); - ok(r == 1000, "Expected 1000, got %d\n", r); + expect(1000,r); r = SendMessage(updown, UDM_GETPOS32, 0 , (LPARAM) &high ); - ok(r == 1000, "Expected 0, got %d\n", r); - ok(high == 1 , "Expected 0, got %d\n", high); + expect(1000,r); + expect(1,high); } static void test_updown_buddy(void) @@ -389,31 +390,31 @@ static void test_updown_base(void) SendMessage(updown, UDM_SETBASE, 10 , 0); r = SendMessage(updown, UDM_GETBASE, 0 , 0); - ok(r == 10, "Expected 10, got %d\n", r); + expect(10,r); /* Set base to an invalid value, should return 0 and stay at 10 */ r = SendMessage(updown, UDM_SETBASE, 80 , 0); - ok(r == 0, "Expected 0, got %d\n", r); + expect(0,r); r = SendMessage(updown, UDM_GETBASE, 0 , 0); - ok(r == 10, "Expected 10, got %d\n", r); + expect(10,r); /* Set base to 16 now, should get 16 as the return */ r = SendMessage(updown, UDM_SETBASE, 16 , 0); - ok(r == 10, "Expected 10, got %d\n", r); + expect(10,r); r = SendMessage(updown, UDM_GETBASE, 0 , 0); - ok(r == 16, "Expected 16, got %d\n", r); + expect(16,r); /* Set base to an invalid value, should return 0 and stay at 16 */ r = SendMessage(updown, UDM_SETBASE, 80 , 0); - ok(r == 0, "Expected 0, got %d\n", r); + expect(0,r); r = SendMessage(updown, UDM_GETBASE, 0 , 0); - ok(r == 16, "Expected 16, got %d\n", r); + expect(16,r); /* Set base back to 10, return should be 16 */ r = SendMessage(updown, UDM_SETBASE, 10 , 0); - ok(r == 16, "Expected 16, got %d\n", r); + expect(16,r); r = SendMessage(updown, UDM_GETBASE, 0 , 0); - ok(r == 10, "Expected 10, got %d\n", r); + expect(10,r); } static void test_updown_unicode(void) @@ -423,19 +424,19 @@ static void test_updown_unicode(void) /* Set it to ANSI, don't check return as we don't know previous state */ SendMessage(updown, UDM_SETUNICODEFORMAT, 0 , 0); r = SendMessage(updown, UDM_GETUNICODEFORMAT, 0 , 0); - ok(r == 0, "Expected 0, got %d\n", r); + expect(0,r); /* Now set it to Unicode format */ r = SendMessage(updown, UDM_SETUNICODEFORMAT, 1 , 0); - ok(r == 0, "Expected 0, got %d\n", r); + expect(0,r); r = SendMessage(updown, UDM_GETUNICODEFORMAT, 0 , 0); - ok(r == 1, "Expected 1, got %d\n", r); + expect(1,r); /* And now set it back to ANSI */ r = SendMessage(updown, UDM_SETUNICODEFORMAT, 0 , 0); - ok(r == 1, "Expected 1, got %d\n", r); + expect(1,r); r = SendMessage(updown, UDM_GETUNICODEFORMAT, 0 , 0); - ok(r == 0, "Expected 0, got %d\n", r); + expect(0,r); }

1 0

Rob Shearman : ole32: Print the contents of the IID input to OleLoad.
by Alexandre Julliard March 15, 2007

March 15, 2007

Module: wine Branch: master Commit: f1ced14eed40365de0d393467f53b7b576a69824 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f1ced14eed40365de0d393467… Author: Rob Shearman <rob(a)codeweavers.com> Date: Wed Mar 14 15:53:33 2007 +0000 ole32: Print the contents of the IID input to OleLoad. --- dlls/ole32/ole2.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/ole32/ole2.c b/dlls/ole32/ole2.c index 8c49621..924c6cd 100644 --- a/dlls/ole32/ole2.c +++ b/dlls/ole32/ole2.c @@ -948,7 +948,7 @@ HRESULT WINAPI OleLoad( STATSTG storageInfo; HRESULT hres; - TRACE("(%p,%p,%p,%p)\n", pStg, riid, pClientSite, ppvObj); + TRACE("(%p, %s, %p, %p)\n", pStg, debugstr_guid(riid), pClientSite, ppvObj); *ppvObj = NULL;

1 0

Hwang YunSong(황윤성) : cmd: Updated Korean Resource.
by Alexandre Julliard March 15, 2007

March 15, 2007

Module: wine Branch: master Commit: 323ff79fe7f45385e0f801e43440c8a9ad604fee URL: http://source.winehq.org/git/wine.git/?a=commit;h=323ff79fe7f45385e0f801e43… Author: Hwang YunSong(황윤성) <hys545(a)dreamwiz.com> Date: Wed Mar 14 17:47:18 2007 +0900 cmd: Updated Korean Resource. --- programs/cmd/Ko.rc | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/programs/cmd/Ko.rc b/programs/cmd/Ko.rc index ad3f33a..ebc3116 100644 --- a/programs/cmd/Ko.rc +++ b/programs/cmd/Ko.rc @@ -230,6 +230,6 @@ HELP <��>�� ġ�� \n" WCMD_CONFIRM, "�� Ȯ��մϱ�?" WCMD_YES, "��" WCMD_NO, "�ƴϿ�" - WCMD_NOASSOC, "�� Ȯ�� %s�� Ǯ�׸�� ϴ� \n" - WCMD_NOFTYPE, "No open command associated with file type '%s'\n" + WCMD_NOASSOC, "�� Ȯ�� %s�� Ǯ�׸�� ϴ�. \n" + WCMD_NOFTYPE, "�� '%s'�� �� ɵ� ��ϴ�.'\n" }

1 0

Rob Shearman : user32: Add several more MsgWaitForMultipleObjects tests and split them out from the main message tests .
by Alexandre Julliard March 15, 2007

March 15, 2007

Module: wine Branch: master Commit: c3b30e535102ddfffa8f97a8bed8a94176f95cd0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=c3b30e535102ddfffa8f97a8b… Author: Rob Shearman <rob(a)codeweavers.com> Date: Wed Mar 14 15:46:56 2007 +0000 user32: Add several more MsgWaitForMultipleObjects tests and split them out from the main message tests. --- dlls/user32/tests/msg.c | 60 ++++++++++++++++++++++++++++++++++------------ 1 files changed, 44 insertions(+), 16 deletions(-) diff --git a/dlls/user32/tests/msg.c b/dlls/user32/tests/msg.c index a380883..50b00c8 100644 --- a/dlls/user32/tests/msg.c +++ b/dlls/user32/tests/msg.c @@ -3625,6 +3625,49 @@ static const struct message WmSetIcon_2[] = { { 0 } }; +static void test_MsgWaitForMultipleObjects(HWND hwnd) +{ + DWORD ret; + MSG msg; + + ret = MsgWaitForMultipleObjects(0, NULL, FALSE, 0, QS_POSTMESSAGE); + ok(ret == WAIT_TIMEOUT, "MsgWaitForMultipleObjects returned %x\n", ret); + + PostMessageA(hwnd, WM_USER, 0, 0); + + ret = MsgWaitForMultipleObjects(0, NULL, FALSE, 0, QS_POSTMESSAGE); + ok(ret == WAIT_OBJECT_0, "MsgWaitForMultipleObjects returned %x\n", ret); + + ok(PeekMessageW( &msg, 0, 0, 0, PM_REMOVE ), "PeekMessage should succeed\n"); + ok(msg.message == WM_USER, "got %04x instead of WM_USER\n", msg.message); + + ret = MsgWaitForMultipleObjects(0, NULL, FALSE, 0, QS_POSTMESSAGE); + ok(ret == WAIT_TIMEOUT, "MsgWaitForMultipleObjects returned %x\n", ret); + + PostMessageA(hwnd, WM_USER, 0, 0); + + ret = MsgWaitForMultipleObjects(0, NULL, FALSE, 0, QS_POSTMESSAGE); + ok(ret == WAIT_OBJECT_0, "MsgWaitForMultipleObjects returned %x\n", ret); + + ok(PeekMessageW( &msg, 0, 0, 0, PM_NOREMOVE ), "PeekMessage should succeed\n"); + ok(msg.message == WM_USER, "got %04x instead of WM_USER\n", msg.message); + + /* shows QS_POSTMESSAGE flag is cleared in the PeekMessage call */ + ret = MsgWaitForMultipleObjects(0, NULL, FALSE, 0, QS_POSTMESSAGE); + ok(ret == WAIT_TIMEOUT, "MsgWaitForMultipleObjects returned %x\n", ret); + + PostMessageA(hwnd, WM_USER, 0, 0); + + /* new incoming message causes it to become signaled again */ + ret = MsgWaitForMultipleObjects(0, NULL, FALSE, 0, QS_POSTMESSAGE); + ok(ret == WAIT_OBJECT_0, "MsgWaitForMultipleObjects returned %x\n", ret); + + ok(PeekMessageW( &msg, 0, 0, 0, PM_REMOVE ), "PeekMessage should succeed\n"); + ok(msg.message == WM_USER, "got %04x instead of WM_USER\n", msg.message); + ok(PeekMessageW( &msg, 0, 0, 0, PM_REMOVE ), "PeekMessage should succeed\n"); + ok(msg.message == WM_USER, "got %04x instead of WM_USER\n", msg.message); +} + /* test if we receive the right sequence of messages */ static void test_messages(void) { @@ -3632,7 +3675,6 @@ static void test_messages(void) HWND hchild2, hbutton; HMENU hmenu; MSG msg; - DWORD ret; flush_sequence(); @@ -3921,21 +3963,7 @@ static void test_messages(void) while (PeekMessage( &msg, 0, 0, 0, PM_REMOVE )) DispatchMessage( &msg ); flush_sequence(); - /* MsgWaitForMultipleObjects test */ - ret = MsgWaitForMultipleObjects(0, NULL, FALSE, 0, QS_POSTMESSAGE); - ok(ret == WAIT_TIMEOUT, "MsgWaitForMultipleObjects returned %x\n", ret); - - PostMessageA(hparent, WM_USER, 0, 0); - - ret = MsgWaitForMultipleObjects(0, NULL, FALSE, 0, QS_POSTMESSAGE); - ok(ret == WAIT_OBJECT_0, "MsgWaitForMultipleObjects returned %x\n", ret); - - ok(PeekMessageW( &msg, 0, 0, 0, PM_REMOVE ), "PeekMessage should succeed\n"); - ok(msg.message == WM_USER, "got %04x instead of WM_USER\n", msg.message); - - ret = MsgWaitForMultipleObjects(0, NULL, FALSE, 0, QS_POSTMESSAGE); - ok(ret == WAIT_TIMEOUT, "MsgWaitForMultipleObjects returned %x\n", ret); - /* end of MsgWaitForMultipleObjects test */ + test_MsgWaitForMultipleObjects(hparent); /* the following test causes an exception in user.exe under win9x */ if (!PostMessageW( hparent, WM_USER, 0, 0 ))

1 0

Rob Shearman : include: Add a definition for REGCLS_SURROGATE and add declarations for CoRegisterSurrogate and CoGetObjectContext .
by Alexandre Julliard March 15, 2007

March 15, 2007

Module: wine Branch: master Commit: 2b5524867d427d2b73d19653fe724609b8b40b89 URL: http://source.winehq.org/git/wine.git/?a=commit;h=2b5524867d427d2b73d19653f… Author: Rob Shearman <rob(a)codeweavers.com> Date: Wed Mar 14 15:43:52 2007 +0000 include: Add a definition for REGCLS_SURROGATE and add declarations for CoRegisterSurrogate and CoGetObjectContext. --- include/objbase.h | 5 ++++- 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/include/objbase.h b/include/objbase.h index 79603a0..4d94c05 100644 --- a/include/objbase.h +++ b/include/objbase.h @@ -355,7 +355,8 @@ typedef enum tagREGCLS REGCLS_SINGLEUSE = 0, REGCLS_MULTIPLEUSE = 1, REGCLS_MULTI_SEPARATE = 2, - REGCLS_SUSPENDED = 4 + REGCLS_SUSPENDED = 4, + REGCLS_SURROGATE = 8 } REGCLS; HRESULT WINAPI CoGetClassObject(REFCLSID rclsid, DWORD dwClsContext, COSERVERINFO *pServerInfo, REFIID iid, LPVOID *ppv); @@ -363,6 +364,7 @@ HRESULT WINAPI CoRegisterClassObject(REFCLSID rclsid,LPUNKNOWN pUnk,DWORD dwClsC HRESULT WINAPI CoRevokeClassObject(DWORD dwRegister); HRESULT WINAPI CoGetPSClsid(REFIID riid,CLSID *pclsid); HRESULT WINAPI CoRegisterPSClsid(REFIID riid, REFCLSID rclsid); +HRESULT WINAPI CoRegisterSurrogate(LPSURROGATE pSurrogate); HRESULT WINAPI CoSuspendClassObjects(void); HRESULT WINAPI CoResumeClassObjects(void); ULONG WINAPI CoAddRefServerProcess(void); @@ -400,6 +402,7 @@ HRESULT WINAPI CoRevertToSelf(void); HRESULT WINAPI CoGetTreatAsClass(REFCLSID clsidOld, LPCLSID pClsidNew); HRESULT WINAPI CoTreatAsClass(REFCLSID clsidOld, REFCLSID clsidNew); HRESULT WINAPI CoAllowSetForegroundWindow(IUnknown *pUnk, LPVOID lpvReserved); +HRESULT WINAPI CoGetObjectContext(REFIID riid, LPVOID *ppv); HRESULT WINAPI CoCreateGuid(GUID* pguid); BOOL WINAPI CoIsOle1Class(REFCLSID rclsid);

1 0

Huw Davies : kernel32: Fix GetPrivateProfileSectionNamesA when the file does not exist.
by Alexandre Julliard March 15, 2007

March 15, 2007

Module: wine Branch: master Commit: 3f49239063eac5d91fdd6b75c09cc13d1af609b1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=3f49239063eac5d91fdd6b75c… Author: Huw Davies <huw(a)codeweavers.com> Date: Wed Mar 14 14:55:14 2007 +0000 kernel32: Fix GetPrivateProfileSectionNamesA when the file does not exist. --- dlls/kernel32/profile.c | 2 ++ dlls/kernel32/tests/profile.c | 14 ++++++++++++++ 2 files changed, 16 insertions(+), 0 deletions(-) diff --git a/dlls/kernel32/profile.c b/dlls/kernel32/profile.c index b48f5e9..0abb887 100644 --- a/dlls/kernel32/profile.c +++ b/dlls/kernel32/profile.c @@ -1629,6 +1629,8 @@ DWORD WINAPI GetPrivateProfileSectionNamesA( LPSTR buffer, DWORD size, else ret = ret-1; } + else if(size) + buffer[0] = '\0'; RtlFreeUnicodeString(&filenameW); HeapFree(GetProcessHeap(), 0, bufferW); diff --git a/dlls/kernel32/tests/profile.c b/dlls/kernel32/tests/profile.c index 4379ee2..3ee66eb 100644 --- a/dlls/kernel32/tests/profile.c +++ b/dlls/kernel32/tests/profile.c @@ -145,6 +145,7 @@ static void test_profile_sections(void) static const char content[]="[section1]\r\n[section2]\r\n[section3]\r\n"; static const char testfile3[]=".\\testwine3.ini"; static const WCHAR testfile3W[]={ '.','\\','t','e','s','t','w','i','n','e','3','.','i','n','i',0 }; + static const WCHAR not_here[] = {'.','\\','n','o','t','_','h','e','r','e','.','i','n','i',0}; DeleteFileA( testfile3 ); h = CreateFileA( testfile3, GENERIC_WRITE, 0, NULL, CREATE_ALWAYS, FILE_ATTRIBUTE_NORMAL, NULL); @@ -185,6 +186,19 @@ static void test_profile_sections(void) ok( buf[ret+1] == 0 && buf[ret] == 0, "returned buffer not terminated with double-null\n" ); DeleteFileA( testfile3 ); + + /* Tests on non-existent file */ + memset(buf, 0xcc, sizeof(buf)); + ret = GetPrivateProfileSectionNamesA( buf, 10, ".\\not_here.ini" ); + ok( ret == 0, "expected return size 0, got %d\n", ret ); + ok( buf[0] == 0, "returned buffer not terminated with null\n" ); + ok( buf[1] != 0, "returned buffer terminated with double-null\n" ); + + memset(bufW, 0xcc, sizeof(bufW)); + ret = GetPrivateProfileSectionNamesW( bufW, 10, not_here ); + ok( ret == 0, "expected return size 0, got %d\n", ret ); + ok( bufW[0] == 0, "returned buffer not terminated with null\n" ); + ok( bufW[1] != 0, "returned buffer terminated with double-null\n" ); } START_TEST(profile)

1 0

Rob Shearman : rpcrt4: Unmarshal NULL OLE interfaces properly by handling the case of a 0 stream size .
by Alexandre Julliard March 15, 2007

March 15, 2007

Module: wine Branch: master Commit: 8620d15d3960921a8f316e0653ef0d11ceb55e89 URL: http://source.winehq.org/git/wine.git/?a=commit;h=8620d15d3960921a8f316e065… Author: Rob Shearman <rob(a)codeweavers.com> Date: Wed Mar 14 15:46:16 2007 +0000 rpcrt4: Unmarshal NULL OLE interfaces properly by handling the case of a 0 stream size. --- dlls/rpcrt4/ndr_ole.c | 10 ++++++---- 1 files changed, 6 insertions(+), 4 deletions(-) diff --git a/dlls/rpcrt4/ndr_ole.c b/dlls/rpcrt4/ndr_ole.c index 6c32121..b65e18b 100644 --- a/dlls/rpcrt4/ndr_ole.c +++ b/dlls/rpcrt4/ndr_ole.c @@ -288,12 +288,14 @@ unsigned char * WINAPI NdrInterfacePointerUnmarshall(PMIDL_STUB_MESSAGE pStubMsg *(LPVOID*)ppMemory = NULL; if (pStubMsg->Buffer + sizeof(DWORD) < (unsigned char *)pStubMsg->RpcMsg->Buffer + pStubMsg->BufferLength) { stream = RpcStream_Create(pStubMsg, FALSE); - if (stream) { + if (!stream) RpcRaiseException(E_OUTOFMEMORY); + if (*((RpcStreamImpl *)stream)->size != 0) hr = COM_UnmarshalInterface(stream, &IID_NULL, (LPVOID*)ppMemory); - IStream_Release(stream); - if (FAILED(hr)) + else + hr = S_OK; + IStream_Release(stream); + if (FAILED(hr)) RpcRaiseException(hr); - } } return NULL; }

1 0

Alexandre Julliard : libwine: Added support for Unicode surrogates in utf-8 conversions.
by Alexandre Julliard March 15, 2007

March 15, 2007

Module: wine Branch: master Commit: 62c3d3d6203d45d1d4b75871038c8a1e3ea4ae99 URL: http://source.winehq.org/git/wine.git/?a=commit;h=62c3d3d6203d45d1d4b758710… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Mar 14 15:58:56 2007 +0100 libwine: Added support for Unicode surrogates in utf-8 conversions. Also make the conversions more strict against invalid input data. --- libs/wine/utf8.c | 171 +++++++++++++++++++++++++++++++++++------------------- 1 files changed, 112 insertions(+), 59 deletions(-) diff --git a/libs/wine/utf8.c b/libs/wine/utf8.c index be1c672..61d3d37 100644 --- a/libs/wine/utf8.c +++ b/libs/wine/utf8.c @@ -29,30 +29,60 @@ static const char utf8_length[128] = 0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0, /* 0x90-0x9f */ 0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0, /* 0xa0-0xaf */ 0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0, /* 0xb0-0xbf */ - 1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1, /* 0xc0-0xcf */ + 0,0,1,1,1,1,1,1,1,1,1,1,1,1,1,1, /* 0xc0-0xcf */ 1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1, /* 0xd0-0xdf */ 2,2,2,2,2,2,2,2,2,2,2,2,2,2,2,2, /* 0xe0-0xef */ - 3,3,3,3,3,3,3,3,4,4,4,4,5,5,0,0 /* 0xf0-0xff */ + 3,3,3,3,3,0,0,0,0,0,0,0,0,0,0,0 /* 0xf0-0xff */ }; /* first byte mask depending on UTF-8 sequence length */ -static const unsigned char utf8_mask[6] = { 0x7f, 0x1f, 0x0f, 0x07, 0x03, 0x01 }; +static const unsigned char utf8_mask[4] = { 0x7f, 0x1f, 0x0f, 0x07 }; /* minimum Unicode value depending on UTF-8 sequence length */ -static const unsigned int utf8_minval[6] = { 0x0, 0x80, 0x800, 0x10000, 0x200000, 0x4000000 }; +static const unsigned int utf8_minval[4] = { 0x0, 0x80, 0x800, 0x10000 }; +/* get the next char value taking surrogates into account */ +static inline unsigned int get_surrogate_value( const WCHAR *src, unsigned int srclen ) +{ + if (src[0] >= 0xd800 && src[0] <= 0xdfff) /* surrogate pair */ + { + if (src[0] > 0xdbff || /* invalid high surrogate */ + srclen <= 1 || /* missing low surrogate */ + src[1] < 0xdc00 || src[1] > 0xdfff) /* invalid low surrogate */ + return 0; + return 0x10000 + ((src[0] & 0x3ff) << 10) + (src[1] & 0x3ff); + } + return src[0]; +} + /* query necessary dst length for src string */ -inline static int get_length_wcs_utf8( const WCHAR *src, unsigned int srclen ) +static inline int get_length_wcs_utf8( int flags, const WCHAR *src, unsigned int srclen ) { int len; - for (len = 0; srclen; srclen--, src++, len++) + unsigned int val; + + for (len = 0; srclen; srclen--, src++) { - if (*src >= 0x80) + if (*src < 0x80) /* 0x00-0x7f: 1 byte */ { len++; - if (*src >= 0x800) len++; + continue; + } + if (*src < 0x800) /* 0x80-0x7ff: 2 bytes */ + { + len += 2; + continue; + } + if (!(val = get_surrogate_value( src, srclen ))) + { + if (flags & WC_ERR_INVALID_CHARS) return -2; + continue; } + if (val < 0x10000) /* 0x800-0xffff: 3 bytes */ + len += 3; + else /* 0x10000-0x10ffff: 4 bytes */ + len += 4; } return len; } @@ -63,11 +93,12 @@ int wine_utf8_wcstombs( int flags, const WCHAR *src, int srclen, char *dst, int { int len; - if (!dstlen) return get_length_wcs_utf8( src, srclen ); + if (!dstlen) return get_length_wcs_utf8( flags, src, srclen ); for (len = dstlen; srclen; srclen--, src++) { WCHAR ch = *src; + unsigned int val; if (ch < 0x80) /* 0x00-0x7f: 1 byte */ { @@ -86,53 +117,80 @@ int wine_utf8_wcstombs( int flags, const WCHAR *src, int srclen, char *dst, int continue; } - /* 0x800-0xffff: 3 bytes */ + if (!(val = get_surrogate_value( src, srclen ))) + { + if (flags & WC_ERR_INVALID_CHARS) return -2; + continue; + } - if ((len -= 3) < 0) return -1; /* overflow */ - dst[2] = 0x80 | (ch & 0x3f); - ch >>= 6; - dst[1] = 0x80 | (ch & 0x3f); - ch >>= 6; - dst[0] = 0xe0 | ch; - dst += 3; + if (val < 0x10000) /* 0x800-0xffff: 3 bytes */ + { + if ((len -= 3) < 0) return -1; /* overflow */ + dst[2] = 0x80 | (val & 0x3f); + val >>= 6; + dst[1] = 0x80 | (val & 0x3f); + val >>= 6; + dst[0] = 0xe0 | val; + dst += 3; + } + else /* 0x10000-0x10ffff: 4 bytes */ + { + if ((len -= 4) < 0) return -1; /* overflow */ + dst[3] = 0x80 | (val & 0x3f); + val >>= 6; + dst[2] = 0x80 | (val & 0x3f); + val >>= 6; + dst[1] = 0x80 | (val & 0x3f); + val >>= 6; + dst[0] = 0xf0 | val; + dst += 4; + } } return dstlen - len; } /* query necessary dst length for src string */ -inline static int get_length_mbs_utf8( const unsigned char *src, int srclen ) +inline static int get_length_mbs_utf8( int flags, const char *src, int srclen ) { - int ret; - const unsigned char *srcend = src + srclen; + int len, ret = 0; + unsigned int res; + const char *srcend = src + srclen; - for (ret = 0; src < srcend; ret++) + while (src < srcend) { unsigned char ch = *src++; - if (ch < 0xc0) continue; + if (ch < 0x80) /* special fast case for 7-bit ASCII */ + { + ret++; + continue; + } + len = utf8_length[ch-0x80]; + if (src + len > srcend) goto bad; + res = ch & utf8_mask[len]; - switch(utf8_length[ch-0x80]) + switch(len) { - case 5: - if (src >= srcend) return ret; /* ignore partial char */ - if ((ch = *src ^ 0x80) >= 0x40) continue; - src++; - case 4: - if (src >= srcend) return ret; /* ignore partial char */ - if ((ch = *src ^ 0x80) >= 0x40) continue; - src++; case 3: - if (src >= srcend) return ret; /* ignore partial char */ - if ((ch = *src ^ 0x80) >= 0x40) continue; + if ((ch = *src ^ 0x80) >= 0x40) goto bad; + res = (res << 6) | ch; src++; case 2: - if (src >= srcend) return ret; /* ignore partial char */ - if ((ch = *src ^ 0x80) >= 0x40) continue; + if ((ch = *src ^ 0x80) >= 0x40) goto bad; + res = (res << 6) | ch; src++; case 1: - if (src >= srcend) return ret; /* ignore partial char */ - if ((ch = *src ^ 0x80) >= 0x40) continue; + if ((ch = *src ^ 0x80) >= 0x40) goto bad; + res = (res << 6) | ch; src++; + if (res < utf8_minval[len]) goto bad; + if (res > 0x10ffff) goto bad; + if (res > 0xffff) ret++; + ret++; + continue; } + bad: + if (flags & MB_ERR_INVALID_CHARS) return -2; /* bad char */ + /* otherwise ignore it */ } return ret; } @@ -141,60 +199,55 @@ inline static int get_length_mbs_utf8( const unsigned char *src, int srclen ) /* return -1 on dst buffer overflow, -2 on invalid input char */ int wine_utf8_mbstowcs( int flags, const char *src, int srclen, WCHAR *dst, int dstlen ) { - int len, count; + int len; unsigned int res; const char *srcend = src + srclen; + WCHAR *dstend = dst + dstlen; - if (!dstlen) return get_length_mbs_utf8( (const unsigned char*)src, srclen ); + if (!dstlen) return get_length_mbs_utf8( flags, src, srclen ); - for (count = dstlen; count && (src < srcend); count--, dst++) + while ((dst < dstend) && (src < srcend)) { unsigned char ch = *src++; if (ch < 0x80) /* special fast case for 7-bit ASCII */ { - *dst = ch; + *dst++ = ch; continue; } len = utf8_length[ch-0x80]; + if (src + len > srcend) goto bad; res = ch & utf8_mask[len]; switch(len) { - case 5: - if (src >= srcend) goto done; /* ignore partial char */ - if ((ch = *src ^ 0x80) >= 0x40) goto bad; - res = (res << 6) | ch; - src++; - case 4: - if (src >= srcend) goto done; /* ignore partial char */ - if ((ch = *src ^ 0x80) >= 0x40) goto bad; - res = (res << 6) | ch; - src++; case 3: - if (src >= srcend) goto done; /* ignore partial char */ if ((ch = *src ^ 0x80) >= 0x40) goto bad; res = (res << 6) | ch; src++; case 2: - if (src >= srcend) goto done; /* ignore partial char */ if ((ch = *src ^ 0x80) >= 0x40) goto bad; res = (res << 6) | ch; src++; case 1: - if (src >= srcend) goto done; /* ignore partial char */ if ((ch = *src ^ 0x80) >= 0x40) goto bad; res = (res << 6) | ch; src++; if (res < utf8_minval[len]) goto bad; - if (res >= 0x10000) goto bad; /* FIXME: maybe we should do surrogates here */ - *dst = res; + if (res > 0x10ffff) goto bad; + if (res <= 0xffff) *dst++ = res; + else /* we need surrogates */ + { + if (dst == dstend - 1) return -1; /* overflow */ + res -= 0x10000; + *dst++ = 0xd800 | (res >> 10); + *dst++ = 0xdc00 | (res & 0x3ff); + } continue; } bad: if (flags & MB_ERR_INVALID_CHARS) return -2; /* bad char */ - *dst = (WCHAR)'?'; + /* otherwise ignore it */ } if (src < srcend) return -1; /* overflow */ -done: - return dstlen - count; + return dstlen - (dstend - dst); }

1 0

Aric Stewart : wineps.drv: Values for DC_MINEXTENT and DC_MAXEXTENT are returned, not placed in the data pointer.
by Alexandre Julliard March 15, 2007

March 15, 2007

Module: wine Branch: master Commit: f1172cd4460972cf6cc000b91845e924ecc11c4d URL: http://source.winehq.org/git/wine.git/?a=commit;h=f1172cd4460972cf6cc000b91… Author: Aric Stewart <aric(a)codeweavers.com> Date: Wed Mar 14 07:31:20 2007 -0500 wineps.drv: Values for DC_MINEXTENT and DC_MAXEXTENT are returned, not placed in the data pointer. --- dlls/wineps.drv/driver.c | 18 ++++++++---------- 1 files changed, 8 insertions(+), 10 deletions(-) diff --git a/dlls/wineps.drv/driver.c b/dlls/wineps.drv/driver.c index 30bc0b3..9c3136b 100644 --- a/dlls/wineps.drv/driver.c +++ b/dlls/wineps.drv/driver.c @@ -603,28 +603,26 @@ DWORD PSDRV_DeviceCapabilities(LPSTR lpszDriver, LPCSTR lpszDevice, LPCSTR lpszP if(ps->PaperDimension->y > ptMax.y) ptMax.y = ps->PaperDimension->y; } - *((POINT*)lpszOutput) = ptMax; - return 1; + return MAKELONG(ptMax.x * 254.0 / 72.0, ptMax.y * 254.0 / 72.0 ); } case DC_MINEXTENT: { PAGESIZE *ps; - POINT ptMax; - ptMax.x = ptMax.y = 0; + POINT ptMin; + ptMin.x = ptMin.y = -1; if(lpszOutput == NULL) return -1; LIST_FOR_EACH_ENTRY(ps, &pi->ppd->PageSizes, PAGESIZE, entry) { - if(ps->PaperDimension->x > ptMax.x) - ptMax.x = ps->PaperDimension->x; - if(ps->PaperDimension->y > ptMax.y) - ptMax.y = ps->PaperDimension->y; + if(ptMin.x == -1 || ps->PaperDimension->x < ptMin.x) + ptMin.x = ps->PaperDimension->x; + if(ptMin.y == -1 || ps->PaperDimension->y < ptMin.y) + ptMin.y = ps->PaperDimension->y; } - *((POINT*)lpszOutput) = ptMax; - return 1; + return MAKELONG(ptMin.x * 254.0 / 72.0, ptMin.y * 254.0 / 72.0); } case DC_SIZE:

1 0

Alexandre Julliard : libwine: Update the Unicode tables to version 5.0. 1 of the standard.
by Alexandre Julliard March 15, 2007

March 15, 2007

Module: wine Branch: master Commit: a44d1e5f6e7b3b65a158c89c7e0d0d40c104459d URL: http://source.winehq.org/git/wine.git/?a=commit;h=a44d1e5f6e7b3b65a158c89c7… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Mar 14 16:22:46 2007 +0100 libwine: Update the Unicode tables to version 5.0.1 of the standard. --- libs/wine/casemap.c | 200 ++++++++++++++++++++++++++++++++++----------------- libs/wine/compose.c | 163 ++++++++++++++++++++++++----------------- libs/wine/wctype.c | 158 ++++++++++++++++++++++++---------------- 3 files changed, 324 insertions(+), 197 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=a44d1e5f6e7b3b65a158c…

1 0