wine-commits January 2007

wine-commits@winehq.org

4 participants
893 discussions

Alexandre Julliard : widl: Add a few more fields in the stub descriptor.
by Alexandre Julliard 26 Jan '07

26 Jan '07

Module: wine Branch: master Commit: 1a25936ca976f381d4588c1750acef4d3a3a8b5e URL: http://source.winehq.org/git/wine.git/?a=commit;h=1a25936ca976f381d4588c175… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Jan 26 12:08:51 2007 +0100 widl: Add a few more fields in the stub descriptor. --- tools/widl/proxy.c | 27 ++++++++++++++++++++------- 1 files changed, 20 insertions(+), 7 deletions(-) diff --git a/tools/widl/proxy.c b/tools/widl/proxy.c index 3268e5d..2f71cd9 100644 --- a/tools/widl/proxy.c +++ b/tools/widl/proxy.c @@ -67,18 +67,31 @@ static int print_proxy( const char *form static void write_stubdescproto(void) { - print_proxy( "extern const MIDL_STUB_DESC Object_StubDesc;\n"); + print_proxy( "static const MIDL_STUB_DESC Object_StubDesc;\n"); print_proxy( "\n"); } static void write_stubdesc(void) { - print_proxy( "const MIDL_STUB_DESC Object_StubDesc = {\n"); - print_proxy( " 0,\n"); - print_proxy( " NdrOleAllocate,\n"); - print_proxy( " NdrOleFree,\n"); - print_proxy( " {0}, 0, 0, 0, 0,\n"); - print_proxy( " 0 /* __MIDL_TypeFormatString.Format */\n"); + print_proxy( "static const MIDL_STUB_DESC Object_StubDesc =\n{\n"); + indent++; + print_proxy( "0,\n"); + print_proxy( "NdrOleAllocate,\n"); + print_proxy( "NdrOleFree,\n"); + print_proxy( "{0}, 0, 0, 0, 0,\n"); + print_proxy( "__MIDL_TypeFormatString.Format,\n"); + print_proxy( "1, /* -error bounds_check flag */\n"); + print_proxy( "0x10001, /* Ndr library version */\n"); + print_proxy( "0,\n"); + print_proxy( "0x50100a4, /* MIDL Version 5.1.164 */\n"); + print_proxy( "0,\n"); + print_proxy( "0,\n"); + print_proxy( "0, /* notify & notify_flag routine table */\n"); + print_proxy( "1, /* Flags */\n"); + print_proxy( "0, /* Reserved3 */\n"); + print_proxy( "0, /* Reserved4 */\n"); + print_proxy( "0 /* Reserved5 */\n"); + indent--; print_proxy( "};\n"); print_proxy( "\n"); }

1 0

Alexandre Julliard : widl: Rename a couple of variables to make the code closer to what midl generates .
by Alexandre Julliard 26 Jan '07

26 Jan '07

Module: wine Branch: master Commit: 6c559cafabde73cbb038f0556402f0454c5192a8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=6c559cafabde73cbb038f0556… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Jan 26 12:04:40 2007 +0100 widl: Rename a couple of variables to make the code closer to what midl generates. --- tools/widl/proxy.c | 21 ++++++++++++--------- 1 files changed, 12 insertions(+), 9 deletions(-) diff --git a/tools/widl/proxy.c b/tools/widl/proxy.c index afe998a..3268e5d 100644 --- a/tools/widl/proxy.c +++ b/tools/widl/proxy.c @@ -263,7 +263,7 @@ static void gen_proxy(type_t *iface, con write_type(proxy, def->type, def, def->tname); print_proxy( " _RetVal;\n"); } - print_proxy( "RPC_MESSAGE _Msg;\n" ); + print_proxy( "RPC_MESSAGE _RpcMessage;\n" ); print_proxy( "MIDL_STUB_MESSAGE _StubMsg;\n" ); print_proxy( "\n"); @@ -273,7 +273,7 @@ static void gen_proxy(type_t *iface, con print_proxy( "RpcTryExcept\n" ); print_proxy( "{\n" ); indent++; - print_proxy( "NdrProxyInitialize(This, &_Msg, &_StubMsg, &Object_StubDesc, %d);\n", idx); + print_proxy( "NdrProxyInitialize(This, &_RpcMessage, &_StubMsg, &Object_StubDesc, %d);\n", idx); proxy_check_pointers( cur->args ); print_proxy( "RpcTryFinally\n" ); @@ -290,7 +290,10 @@ static void gen_proxy(type_t *iface, con print_proxy( "NdrProxySendReceive(This, &_StubMsg);\n" ); fprintf(proxy, "\n"); - print_proxy("if ((_Msg.DataRepresentation&0xffff) != NDR_LOCAL_DATA_REPRESENTATION)\n"); + print_proxy( "_StubMsg.BufferStart = _RpcMessage.Buffer;\n" ); + print_proxy( "_StubMsg.BufferEnd = _StubMsg.BufferStart + _RpcMessage.BufferLength;\n\n" ); + + print_proxy("if ((_RpcMessage.DataRepresentation & 0xffff) != NDR_LOCAL_DATA_REPRESENTATION)\n"); indent++; print_proxy("NdrConvert( &_StubMsg, &__MIDL_ProcFormatString.Format[%u]);\n", proc_offset ); indent--; @@ -345,8 +348,8 @@ static void gen_stub(type_t *iface, cons print_proxy( "_Stub(\n"); indent++; print_proxy( "IRpcStubBuffer* This,\n"); - print_proxy( "IRpcChannelBuffer* pRpcChannelBuffer,\n"); - print_proxy( "PRPC_MESSAGE _Msg,\n"); + print_proxy( "IRpcChannelBuffer *_pRpcChannelBuffer,\n"); + print_proxy( "PRPC_MESSAGE _pRpcMessage,\n"); print_proxy( "DWORD* _pdwStubPhase)\n"); indent--; print_proxy( "{\n"); @@ -358,7 +361,7 @@ static void gen_stub(type_t *iface, cons /* FIXME: trace */ - print_proxy("NdrStubInitialize(_Msg, &_StubMsg, &Object_StubDesc, pRpcChannelBuffer);\n"); + print_proxy("NdrStubInitialize(_pRpcMessage, &_StubMsg, &Object_StubDesc, _pRpcChannelBuffer);\n"); fprintf(proxy, "\n"); if (cur->args) @@ -368,7 +371,7 @@ static void gen_stub(type_t *iface, cons print_proxy("RpcTryFinally\n"); print_proxy("{\n"); indent++; - print_proxy("if ((_Msg->DataRepresentation&0xffff) != NDR_LOCAL_DATA_REPRESENTATION)\n"); + print_proxy("if ((_pRpcMessage->DataRepresentation & 0xffff) != NDR_LOCAL_DATA_REPRESENTATION)\n"); indent++; print_proxy("NdrConvert( &_StubMsg, &__MIDL_ProcFormatString.Format[%u]);\n", proc_offset ); indent--; @@ -405,7 +408,7 @@ static void gen_stub(type_t *iface, cons offset = *type_offset; write_remoting_arguments(proxy, indent, cur, &offset, PASS_OUT, PHASE_BUFFERSIZE); - print_proxy("NdrStubGetBuffer(This, pRpcChannelBuffer, &_StubMsg);\n"); + print_proxy("NdrStubGetBuffer(This, _pRpcChannelBuffer, &_StubMsg);\n"); offset = *type_offset; write_remoting_arguments(proxy, indent, cur, &offset, PASS_OUT, PHASE_MARSHAL); @@ -425,7 +428,7 @@ static void gen_stub(type_t *iface, cons print_proxy("}\n"); print_proxy("RpcEndFinally\n"); - print_proxy("_Msg->BufferLength = _StubMsg.Buffer - (unsigned char *)_Msg->Buffer;\n"); + print_proxy("_pRpcMessage->BufferLength = _StubMsg.Buffer - (unsigned char *)_pRpcMessage->Buffer;\n"); indent--; print_proxy("}\n");

1 0

Alexandre Julliard : widl: Initialize local variables in stub functions.
by Alexandre Julliard 26 Jan '07

26 Jan '07

Module: wine Branch: master Commit: 7e4a985269ee09614fa8e98d06bffdf46b9c46cd URL: http://source.winehq.org/git/wine.git/?a=commit;h=7e4a985269ee09614fa8e98d0… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Jan 26 12:03:02 2007 +0100 widl: Initialize local variables in stub functions. --- tools/widl/proxy.c | 4 ++++ 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/tools/widl/proxy.c b/tools/widl/proxy.c index 26e184e..afe998a 100644 --- a/tools/widl/proxy.c +++ b/tools/widl/proxy.c @@ -361,6 +361,10 @@ static void gen_stub(type_t *iface, cons print_proxy("NdrStubInitialize(_Msg, &_StubMsg, &Object_StubDesc, pRpcChannelBuffer);\n"); fprintf(proxy, "\n"); + if (cur->args) + LIST_FOR_EACH_ENTRY( arg, cur->args, const var_t, entry ) + print_proxy("%s = 0;\n", arg->name); + print_proxy("RpcTryFinally\n"); print_proxy("{\n"); indent++;

1 0

Alexandre Julliard : widl: Use the typegen function for marshalling/ unmarshalling in proxies.
by Alexandre Julliard 26 Jan '07

26 Jan '07

Module: wine Branch: master Commit: 9e6acf818f18acc269b48abeb63bc4c7a333f7b8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9e6acf818f18acc269b48abeb… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Jan 26 12:00:42 2007 +0100 widl: Use the typegen function for marshalling/unmarshalling in proxies. --- tools/widl/proxy.c | 536 +++++----------------------------------------------- 1 files changed, 45 insertions(+), 491 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=9e6acf818f18acc269b48…

1 0

Alexandre Julliard : widl: Copy the UserMarshal support into the generic code.
by Alexandre Julliard 26 Jan '07

26 Jan '07

Module: wine Branch: master Commit: 9f1e9ad326de7ab32c02a61f437eab9a2078668f URL: http://source.winehq.org/git/wine.git/?a=commit;h=9f1e9ad326de7ab32c02a61f4… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Jan 26 11:59:58 2007 +0100 widl: Copy the UserMarshal support into the generic code. --- tools/widl/typegen.c | 20 +++++++++++++++++++- 1 files changed, 19 insertions(+), 1 deletions(-) diff --git a/tools/widl/typegen.c b/tools/widl/typegen.c index 4b4f4cf..36a6228 100644 --- a/tools/widl/typegen.c +++ b/tools/widl/typegen.c @@ -159,6 +159,20 @@ void write_formatstringsdecl(FILE *f, in print_file(f, indent, "\n"); } +static int is_user_derived(const var_t *v) +{ + const type_t *type = v->type; + + if (v->attrs && is_attr( v->attrs, ATTR_WIREMARSHAL )) return 1; + + while (type) + { + if (type->attrs && is_attr( type->attrs, ATTR_WIREMARSHAL )) return 1; + type = type->ref; + } + return 0; +} + static inline int is_base_type(unsigned char type) { switch (type) @@ -1835,7 +1849,11 @@ void write_remoting_arguments(FILE *file rtype = type->type; - if (is_string_type(var->attrs, var->ptr_level, var->array)) + if (is_user_derived( var )) + { + print_phase_function(file, indent, "UserMarshal", phase, var->name, *type_offset); + } + else if (is_string_type(var->attrs, var->ptr_level, var->array)) { if (var->array && !is_conformant_array(var->array)) print_phase_function(file, indent, "NonConformantString", phase, var->name, *type_offset);

1 0

lostwages/ include/winehq.conf news/2007012501.xml
by Alexandre Julliard 25 Jan '07

25 Jan '07

ChangeSet ID: 30909 CVSROOT: /opt/cvs-commit Module name: lostwages Changes by: julliard(a)winehq.org 2007/01/25 10:31:17 Modified files: include : winehq.conf Added files: news : 2007012501.xml Log message: Release 0.9.30 Patch: http://cvs.winehq.org/patch.py?id=30909 Old revision New revision Changes Path 1.52 1.53 +1 -1 lostwages/include/winehq.conf Added 1.1 +0 -0 lostwages/news/2007012501.xml Index: lostwages/include/winehq.conf diff -u -p lostwages/include/winehq.conf:1.52 lostwages/include/winehq.conf:1.53 --- lostwages/include/winehq.conf:1.52 25 Jan 2007 16:31:17 -0000 +++ lostwages/include/winehq.conf 25 Jan 2007 16:31:17 -0000 @@ -29,7 +29,7 @@ news_xml_path: news wwn_xml_path: wwn # Snapshot Date -snapshot_date: 0.9.29 +snapshot_date: 0.9.30 # Current RH Packages Snapshot Date snapshot_date_rh: 0.9.2 Index: lostwages/news/2007012501.xml diff -u -p /dev/null lostwages/news/2007012501.xml:1.1 --- /dev/null 25 Jan 2007 16:31:17 -0000 +++ lostwages/news/2007012501.xml 25 Jan 2007 16:31:17 -0000 @@ -0,0 +1,24 @@ +<news> +<date>January 25, 2007</date> +<title>Wine 0.9.30 Released</title> +<body> +<p>Wine 0.9.30 was released today, with the following main changes:</p> + +<ul> +<li>Many improvements to Direct3D shaders and state management.</li> +<li>Support for inter-process memory allocations.</li> +<li>OLE32 marshalling fixes.</li> +<li>Lots of bug fixes.</li> +</ul> + +<p> +Binary packages are in the process of being built and it may take a +few days for them to appear, but the source is +<a href="http://prdownloads.sourceforge.net/wine/wine-0.9.30.tar.bz2">available now</a>. +You can find out more about this release in the +<a href="?announce=0.9.30">announcement</a>. Check out our +<a href="/site/download">download page</a> for packages for your +distribution. +</p> +</body> +</news>

1 0

Tag wine-0.9.30 : Alexandre Julliard : Release 0.9.30
by Alexandre Julliard 25 Jan '07

25 Jan '07

Module: wine Branch: refs/tags/wine-0.9.30 Tag: 026b1929e01e6a6311c28f33f7ee06ebf5cbe57d URL: http://source.winehq.org/git/wine.git/?a=tag;h=026b1929e01e6a6311c28f33f7ee… Tagger: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Jan 25 16:53:50 2007 +0100 Release 0.9.30

1 0

Alexandre Julliard : Release 0.9.30.
by Alexandre Julliard 25 Jan '07

25 Jan '07

Module: wine Branch: master Commit: 790e80b2981b906e09d6c3f610e9a0c710220e89 URL: http://source.winehq.org/git/wine.git/?a=commit;h=790e80b2981b906e09d6c3f61… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Jan 25 16:53:50 2007 +0100 Release 0.9.30. --- ANNOUNCE | 1089 ++++++++++++++++---------------- ChangeLog | 2086 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ VERSION | 2 +- configure | 18 +- 4 files changed, 2650 insertions(+), 545 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=790e80b2981b906e09d6c…

1 0

Vitaliy Margolen : server: Prevent a crash on error while creating a token.
by Alexandre Julliard 25 Jan '07

25 Jan '07

Module: wine Branch: master Commit: 55fdda4c3e5a2a6dc5b054ee7273ce8900de6b4f URL: http://source.winehq.org/git/wine.git/?a=commit;h=55fdda4c3e5a2a6dc5b054ee7… Author: Vitaliy Margolen <wine-patches(a)kievinfo.com> Date: Wed Jan 24 23:43:30 2007 -0700 server: Prevent a crash on error while creating a token. --- server/token.c | 3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/server/token.c b/server/token.c index 28455a5..04d3808 100644 --- a/server/token.c +++ b/server/token.c @@ -424,6 +424,7 @@ static struct token *create_token( unsig list_init( &token->privileges ); list_init( &token->groups ); token->primary = primary; + token->default_dacl = NULL; /* copy user */ token->user = memdup( user, FIELD_OFFSET(SID, SubAuthority[user->SubAuthorityCount]) ); @@ -477,8 +478,6 @@ static struct token *create_token( unsig return NULL; } } - else - token->default_dacl = NULL; token->source = source; }

1 0

Vitaliy Margolen : advapi32: Add more tests for granted access mask. Fix test on Wine.
by Alexandre Julliard 25 Jan '07

25 Jan '07

Module: wine Branch: master Commit: c04e7e7878fae6591f7fe0df0a6adf2336dc5716 URL: http://source.winehq.org/git/wine.git/?a=commit;h=c04e7e7878fae6591f7fe0df0… Author: Vitaliy Margolen <wine-patches(a)kievinfo.com> Date: Wed Jan 24 23:43:18 2007 -0700 advapi32: Add more tests for granted access mask. Fix test on Wine. --- dlls/advapi32/tests/security.c | 138 +++++++++++++++++++++++++++++++++++++++- server/handle.c | 2 +- 2 files changed, 138 insertions(+), 2 deletions(-) diff --git a/dlls/advapi32/tests/security.c b/dlls/advapi32/tests/security.c index 4fc9fad..bb5954c 100644 --- a/dlls/advapi32/tests/security.c +++ b/dlls/advapi32/tests/security.c @@ -31,6 +31,7 @@ #include "sddl.h" #include "ntsecapi.h" #include "lmcons.h" +#include "winternl.h" #include "wine/test.h" @@ -59,8 +60,11 @@ typedef NTSTATUS (WINAPI *fnLsaQueryInfo typedef NTSTATUS (WINAPI *fnLsaClose)(LSA_HANDLE); typedef NTSTATUS (WINAPI *fnLsaFreeMemory)(PVOID); typedef NTSTATUS (WINAPI *fnLsaOpenPolicy)(PLSA_UNICODE_STRING,PLSA_OBJECT_ATTRIBUTES,ACCESS_MASK,PLSA_HANDLE); +static NTSTATUS (WINAPI *pNtQueryObject)(HANDLE,OBJECT_INFORMATION_CLASS,PVOID,ULONG,PULONG); static HMODULE hmod; +static int myARGC; +static char** myARGV; fnBuildTrusteeWithSidA pBuildTrusteeWithSidA; fnBuildTrusteeWithNameA pBuildTrusteeWithNameA; @@ -85,7 +89,12 @@ struct sidRef static void init(void) { + HMODULE hntdll = GetModuleHandleA("ntdll.dll"); + hmod = GetModuleHandle("advapi32.dll"); + myARGC = winetest_get_mainargs( &myARGV ); + + pNtQueryObject = (void *)GetProcAddress( hntdll, "NtQueryObject" ); } static void test_str_sid(const char *str_sid) @@ -673,7 +682,7 @@ static void test_AccessCheck(void) res = InitializeAcl(Acl, 256, ACL_REVISION); if(!res && GetLastError() == ERROR_CALL_NOT_IMPLEMENTED) { - trace("ACLs not implemented - skipping tests\n"); + skip("ACLs not implemented - skipping tests\n"); return; } ok(res, "InitializeAcl failed with error %d\n", GetLastError()); @@ -1347,10 +1356,136 @@ static void test_LookupAccountName(void) HeapFree(GetProcessHeap(), 0, domain); } +#define TEST_GRANTED_ACCESS(a,b) test_granted_access(a,b,__LINE__) +static void test_granted_access(HANDLE handle, ACCESS_MASK access, int line) +{ + OBJECT_BASIC_INFORMATION obj_info; + NTSTATUS status; + + if (!pNtQueryObject) + { + skip_(__FILE__, line)("Not NT platform - skipping tests\n"); + return; + } + + status = pNtQueryObject( handle, ObjectBasicInformation, &obj_info, + sizeof(obj_info), NULL ); + ok_(__FILE__, line)(!status, "NtQueryObject with err: %08x\n", status); + ok_(__FILE__, line)(obj_info.GrantedAccess == access, "Gratned access should " + "be 0x%08x, instead of 0x%08x\n", access, obj_info.GrantedAccess); +} + +static void test_process_security(void) +{ + BOOL res; + PSID AdminSid = NULL, UsersSid = NULL; + PACL Acl = NULL; + SECURITY_DESCRIPTOR *SecurityDescriptor = NULL; + SID_IDENTIFIER_AUTHORITY SIDAuthNT = { SECURITY_NT_AUTHORITY }; + char buffer[MAX_PATH]; + PROCESS_INFORMATION info; + STARTUPINFOA startup; + SECURITY_ATTRIBUTES psa; + + Acl = HeapAlloc(GetProcessHeap(), 0, 256); + res = InitializeAcl(Acl, 256, ACL_REVISION); + if (!res && GetLastError() == ERROR_CALL_NOT_IMPLEMENTED) + { + skip("ACLs not implemented - skipping tests\n"); + return; + } + ok(res, "InitializeAcl failed with error %d\n", GetLastError()); + + res = AllocateAndInitializeSid( &SIDAuthNT, 2, SECURITY_BUILTIN_DOMAIN_RID, + DOMAIN_ALIAS_RID_ADMINS, 0, 0, 0, 0, 0, 0, &AdminSid); + ok(res, "AllocateAndInitializeSid failed with error %d\n", GetLastError()); + res = AllocateAndInitializeSid( &SIDAuthNT, 2, SECURITY_BUILTIN_DOMAIN_RID, + DOMAIN_ALIAS_RID_USERS, 0, 0, 0, 0, 0, 0, &UsersSid); + ok(res, "AllocateAndInitializeSid failed with error %d\n", GetLastError()); + + res = AddAccessDeniedAce(Acl, ACL_REVISION, PROCESS_VM_READ, AdminSid); + ok(res, "AddAccessDeniedAce failed with error %d\n", GetLastError()); + res = AddAccessAllowedAce(Acl, ACL_REVISION, PROCESS_ALL_ACCESS, AdminSid); + ok(res, "AddAccessAllowedAceEx failed with error %d\n", GetLastError()); + + SecurityDescriptor = HeapAlloc(GetProcessHeap(), 0, SECURITY_DESCRIPTOR_MIN_LENGTH); + res = InitializeSecurityDescriptor(SecurityDescriptor, SECURITY_DESCRIPTOR_REVISION); + ok(res, "InitializeSecurityDescriptor failed with error %d\n", GetLastError()); + + /* Set owner and group and dacl */ + res = SetSecurityDescriptorOwner(SecurityDescriptor, AdminSid, FALSE); + ok(res, "SetSecurityDescriptorOwner failed with error %d\n", GetLastError()); + res = SetSecurityDescriptorGroup(SecurityDescriptor, UsersSid, TRUE); + ok(res, "SetSecurityDescriptorGroup failed with error %d\n", GetLastError()); + res = SetSecurityDescriptorDacl(SecurityDescriptor, TRUE, Acl, FALSE); + ok(res, "SetSecurityDescriptorDacl failed with error %d\n", GetLastError()); + + + sprintf(buffer, "%s tests/security.c test", myARGV[0]); + memset(&startup, 0, sizeof(startup)); + startup.cb = sizeof(startup); + startup.dwFlags = STARTF_USESHOWWINDOW; + startup.wShowWindow = SW_SHOWNORMAL; + + psa.nLength = sizeof(psa); + psa.lpSecurityDescriptor = SecurityDescriptor; + psa.bInheritHandle = TRUE; + + /* Doesn't matter what ACL say we should get full access for ourselves */ + ok(CreateProcessA( NULL, buffer, &psa, NULL, FALSE, 0, NULL, NULL, &startup, &info ), + "CreateProcess with err:%d\n", GetLastError()); + TEST_GRANTED_ACCESS( info.hProcess, PROCESS_ALL_ACCESS ); + ok(WaitForSingleObject(info.hProcess, 30000) == WAIT_OBJECT_0, "Child process termination\n"); + + if (AdminSid) FreeSid(AdminSid); + if (UsersSid) FreeSid(UsersSid); + HeapFree(GetProcessHeap(), 0, Acl); + HeapFree(GetProcessHeap(), 0, SecurityDescriptor); +} + +static void test_process_security_child(void) +{ + HANDLE handle, handle1; + + handle = OpenProcess( PROCESS_TERMINATE, FALSE, GetCurrentProcessId() ); + ok(handle != NULL, "OpenProcess(PROCESS_TERMINATE) with err:%d\n", GetLastError()); + TEST_GRANTED_ACCESS( handle, PROCESS_TERMINATE ); + + ok(DuplicateHandle( GetCurrentProcess(), handle, GetCurrentProcess(), + &handle1, PROCESS_ALL_ACCESS, TRUE, DUPLICATE_SAME_ACCESS), + "duplicating handle err:%d\n", GetLastError()); + TEST_GRANTED_ACCESS( handle, PROCESS_TERMINATE ); + + CloseHandle( handle1 ); + CloseHandle( handle ); + + /* These two should fail - they are denied by ACL */ + handle = OpenProcess( PROCESS_VM_READ, FALSE, GetCurrentProcessId() ); + todo_wine + ok(handle == NULL, "OpenProcess(PROCESS_VM_READ) should have failed\n"); + handle = OpenProcess( PROCESS_ALL_ACCESS, FALSE, GetCurrentProcessId() ); + todo_wine + ok(handle == NULL, "OpenProcess(PROCESS_ALL_ACCESS) should have failed\n"); + + /* Documented privilege elevation */ + ok(DuplicateHandle( GetCurrentProcess(), GetCurrentProcess(), GetCurrentProcess(), + &handle, PROCESS_ALL_ACCESS, TRUE, DUPLICATE_SAME_ACCESS), + "duplicating handle err:%d\n", GetLastError()); + TEST_GRANTED_ACCESS( handle, PROCESS_ALL_ACCESS ); + + CloseHandle( handle ); +} + START_TEST(security) { init(); if (!hmod) return; + + if (myARGC >= 3) + { + test_process_security_child(); + return; + } test_sid(); test_trustee(); test_luid(); @@ -1359,4 +1494,5 @@ START_TEST(security) test_token_attr(); test_LookupAccountSid(); test_LookupAccountName(); + test_process_security(); } diff --git a/server/handle.c b/server/handle.c index f0c58a8..4b52280 100644 --- a/server/handle.c +++ b/server/handle.c @@ -459,7 +459,7 @@ obj_handle_t duplicate_handle( struct pr access = entry->access; else /* pseudo-handle, give it full access */ { - access = STANDARD_RIGHTS_ALL | SPECIFIC_RIGHTS_ALL; + access = obj->ops->map_access( obj, GENERIC_ALL ); clear_error(); } }

1 0

← Newer
1
2
3
4
5
6
7
8
...
90
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 January 2007