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

Juan Lang : crypt32: Implement CryptVerifyMessageSignature.
by Alexandre Julliard Aug. 23, 2007

Aug. 23, 2007

Module: wine Branch: master Commit: f5b4806159deb89a5d340145b16ec5fce935b537 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f5b4806159deb89a5d340145b… Author: Juan Lang <juan.lang(a)gmail.com> Date: Wed Aug 22 09:40:43 2007 -0700 crypt32: Implement CryptVerifyMessageSignature. --- dlls/crypt32/main.c | 12 ---- dlls/crypt32/msg.c | 132 +++++++++++++++++++++++++++++++++++++++++++ dlls/crypt32/tests/msg.c | 140 ++++++++++++++++++++++++++++++++++++++++++++++ 3 files changed, 272 insertions(+), 12 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=f5b4806159deb89a5d340…

1 0

Juan Lang : crypt32: Test and correct verifying the signature of a valid signed message.
by Alexandre Julliard Aug. 23, 2007

Aug. 23, 2007

Module: wine Branch: master Commit: 3a9e1d66480c85c98c72c4ebd9710bb297ae3091 URL: http://source.winehq.org/git/wine.git/?a=commit;h=3a9e1d66480c85c98c72c4ebd… Author: Juan Lang <juan.lang(a)gmail.com> Date: Wed Aug 22 09:40:11 2007 -0700 crypt32: Test and correct verifying the signature of a valid signed message. --- dlls/crypt32/msg.c | 14 ++++++++++---- dlls/crypt32/tests/msg.c | 25 ++++++++++++++++++++++++- 2 files changed, 34 insertions(+), 5 deletions(-) diff --git a/dlls/crypt32/msg.c b/dlls/crypt32/msg.c index 3ce2bca..dea6a11 100644 --- a/dlls/crypt32/msg.c +++ b/dlls/crypt32/msg.c @@ -2114,15 +2114,21 @@ static BOOL CDecodeSignedMsg_VerifySignature(CDecodeMsg *msg, PCERT_INFO info) if (ret) { HCRYPTHASH hash; + CRYPT_HASH_BLOB reversedHash; if (msg->u.signed_data.info->rgSignerInfo[i].AuthAttrs.cAttr) hash = msg->u.signed_data.signerHandles[i].authAttrHash; else hash = msg->u.signed_data.signerHandles[i].contentHash; - ret = CryptVerifySignatureW(hash, - msg->u.signed_data.info->rgSignerInfo[i].EncryptedHash.pbData, - msg->u.signed_data.info->rgSignerInfo[i].EncryptedHash.cbData, - key, NULL, 0); + ret = CRYPT_ConstructBlob(&reversedHash, + &msg->u.signed_data.info->rgSignerInfo[i].EncryptedHash); + if (ret) + { + CRYPT_ReverseBytes(&reversedHash); + ret = CryptVerifySignatureW(hash, reversedHash.pbData, + reversedHash.cbData, key, NULL, 0); + CryptMemFree(reversedHash.pbData); + } CryptDestroyKey(key); } } diff --git a/dlls/crypt32/tests/msg.c b/dlls/crypt32/tests/msg.c index 33b29c3..008e60a 100644 --- a/dlls/crypt32/tests/msg.c +++ b/dlls/crypt32/tests/msg.c @@ -1110,6 +1110,12 @@ static const BYTE privKey[] = { 0x69, 0x1c, 0x7a, 0xff, 0x81, 0x9d, 0x53, 0x52, 0x97, 0x9a, 0x76, 0x79, 0xda, 0x93, 0x32, 0x16, 0xec, 0x69, 0x51, 0x1a, 0x4e, 0xc3, 0xf1, 0x72, 0x80, 0x78, 0x5e, 0x66, 0x4a, 0x8d, 0x85, 0x2f, 0x3f, 0xb2, 0xa7 }; +static BYTE pubKey[] = { +0x30,0x48,0x02,0x41,0x00,0xe2,0x54,0x3a,0xa7,0x83,0xb1,0x27,0x14,0x3e,0x59, +0xbb,0xb4,0x53,0xe6,0x1f,0xe7,0x5d,0xf1,0x21,0x68,0xad,0x85,0x53,0xdb,0x6b, +0x1e,0xeb,0x65,0x97,0x03,0x86,0x60,0xde,0xf3,0x6c,0x38,0x75,0xe0,0x4c,0x61, +0xbb,0xbc,0x62,0x17,0xa9,0xcd,0x79,0x3f,0x21,0x4e,0x96,0xcb,0x0e,0xdc,0x61, +0x94,0x30,0x18,0x10,0x6b,0xd0,0x1c,0x10,0x79,0x02,0x03,0x01,0x00,0x01 }; static void test_signed_msg_update(void) { @@ -2424,7 +2430,24 @@ static void test_msg_control(void) ok(!ret && GetLastError() == NTE_BAD_SIGNATURE, "Expected NTE_BAD_SIGNATURE, got %08x\n", GetLastError()); CryptMsgClose(msg); - /* FIXME: need to test with a message with a valid signature and signer */ + /* A message with no data doesn't have a valid signature */ + msg = CryptMsgOpenToDecode(PKCS_7_ASN_ENCODING, 0, 0, 0, NULL, NULL); + CryptMsgUpdate(msg, signedWithCertWithValidPubKeyEmptyContent, + sizeof(signedWithCertWithValidPubKeyEmptyContent), TRUE); + certInfo.SubjectPublicKeyInfo.Algorithm.pszObjId = oid_rsa_rsa; + certInfo.SubjectPublicKeyInfo.PublicKey.cbData = sizeof(pubKey); + certInfo.SubjectPublicKeyInfo.PublicKey.pbData = pubKey; + SetLastError(0xdeadbeef); + ret = CryptMsgControl(msg, 0, CMSG_CTRL_VERIFY_SIGNATURE, &certInfo); + ok(!ret && GetLastError() == NTE_BAD_SIGNATURE, + "Expected NTE_BAD_SIGNATURE, got %08x\n", GetLastError()); + CryptMsgClose(msg); + /* Finally, this succeeds */ + msg = CryptMsgOpenToDecode(PKCS_7_ASN_ENCODING, 0, 0, 0, NULL, NULL); + CryptMsgUpdate(msg, signedWithCertWithValidPubKeyContent, + sizeof(signedWithCertWithValidPubKeyContent), TRUE); + ret = CryptMsgControl(msg, 0, CMSG_CTRL_VERIFY_SIGNATURE, &certInfo); + ok(ret, "CryptMsgControl failed: %08x\n", GetLastError()); } static void test_msg_get_signer_count(void)

1 0

Detlef Riekenberg : winspool: Add always the Win9x and the NT printerdriver .
by Alexandre Julliard Aug. 23, 2007

Aug. 23, 2007

Module: wine Branch: master Commit: 8506125cb96e45f226623663a0ebcfaa81115e7a URL: http://source.winehq.org/git/wine.git/?a=commit;h=8506125cb96e45f226623663a… Author: Detlef Riekenberg <wine.dev(a)web.de> Date: Wed Aug 22 20:24:35 2007 +0200 winspool: Add always the Win9x and the NT printerdriver. --- dlls/winspool.drv/info.c | 38 +++++++++++++++++++++----------------- 1 files changed, 21 insertions(+), 17 deletions(-) diff --git a/dlls/winspool.drv/info.c b/dlls/winspool.drv/info.c index 56203da..1ed05fd 100644 --- a/dlls/winspool.drv/info.c +++ b/dlls/winspool.drv/info.c @@ -470,31 +470,35 @@ static BOOL add_printer_driver(const char *name) { DRIVER_INFO_3A di3a; - static char driver_path[] = "wineps16", - data_file[] = "<datafile?>", - config_file[] = "wineps16", - help_file[] = "<helpfile?>", - dep_file[] = "<dependent files?>\0", - monitor_name[] = "<monitor name?>", + static char driver_9x[] = "wineps16.drv", + driver_nt[] = "wineps.drv", + env_9x[] = "Windows 4.0", + env_nt[] = "Windows NT x86", + data_file[] = "generic.ppd", default_data_type[] = "RAW"; - di3a.cVersion = (GetVersion() & 0x80000000) ? 0 : 3; /* FIXME: add 1, 2 */ + ZeroMemory(&di3a, sizeof(DRIVER_INFO_3A)); + di3a.cVersion = 3; di3a.pName = (char *)name; - di3a.pEnvironment = NULL; /* NULL means auto */ - di3a.pDriverPath = driver_path; + di3a.pEnvironment = env_nt; + di3a.pDriverPath = driver_nt; di3a.pDataFile = data_file; - di3a.pConfigFile = config_file; - di3a.pHelpFile = help_file; - di3a.pDependentFiles = dep_file; - di3a.pMonitorName = monitor_name; + di3a.pConfigFile = driver_nt; di3a.pDefaultDataType = default_data_type; - if (!AddPrinterDriverA(NULL, 3, (LPBYTE)&di3a)) + if (AddPrinterDriverA(NULL, 3, (LPBYTE)&di3a)) { - ERR("Failed adding driver (%d)\n", GetLastError()); - return FALSE; + di3a.cVersion = 0; + di3a.pEnvironment = env_9x; + di3a.pDriverPath = driver_9x; + di3a.pConfigFile = driver_9x; + if (AddPrinterDriverA(NULL, 3, (LPBYTE)&di3a)) + { + return TRUE; + } } - return TRUE; + ERR("Failed adding driver %s: %u\n", debugstr_a(di3a.pDriverPath), GetLastError()); + return FALSE; } #ifdef SONAME_LIBCUPS

1 0

Paul Vriens : atl: Replace SHDeleteKeyW with RegDeleteTreeW and get rid of the import of shlwapi .
by Alexandre Julliard Aug. 23, 2007

Aug. 23, 2007

Module: wine Branch: master Commit: 4f49655a6591f9ef1626e619ddcf1ea4be1316de URL: http://source.winehq.org/git/wine.git/?a=commit;h=4f49655a6591f9ef1626e619d… Author: Paul Vriens <paul.vriens.wine(a)gmail.com> Date: Wed Aug 22 11:05:35 2007 +0200 atl: Replace SHDeleteKeyW with RegDeleteTreeW and get rid of the import of shlwapi. --- dlls/atl/Makefile.in | 2 +- dlls/atl/registrar.c | 5 ++--- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/dlls/atl/Makefile.in b/dlls/atl/Makefile.in index d419a5d..413c50d 100644 --- a/dlls/atl/Makefile.in +++ b/dlls/atl/Makefile.in @@ -4,7 +4,7 @@ SRCDIR = @srcdir@ VPATH = @srcdir@ MODULE = atl.dll IMPORTLIB = libatl.$(IMPLIBEXT) -IMPORTS = ole32 oleaut32 shlwapi user32 gdi32 advapi32 kernel32 +IMPORTS = ole32 oleaut32 user32 gdi32 advapi32 kernel32 EXTRALIBS = -luuid C_SRCS = \ diff --git a/dlls/atl/registrar.c b/dlls/atl/registrar.c index 2bc078d..9c0c73b 100644 --- a/dlls/atl/registrar.c +++ b/dlls/atl/registrar.c @@ -27,7 +27,6 @@ #include "winreg.h" #include "objbase.h" #include "oaidl.h" -#include "shlwapi.h" #define ATL_INITGUID #include "atliface.h" @@ -243,10 +242,10 @@ static HRESULT do_process_key(LPCOLESTR *pstr, HKEY parent_key, strbuf *buf, BOO strbuf_write(buf->str, &name, -1); }else if(key_type == DO_DELETE) { TRACE("Deleting %s\n", debugstr_w(buf->str)); - lres = SHDeleteKeyW(parent_key, buf->str); + lres = RegDeleteTreeW(parent_key, buf->str); }else { if(key_type == FORCE_REMOVE) - SHDeleteKeyW(parent_key, buf->str); + RegDeleteTreeW(parent_key, buf->str); lres = RegCreateKeyW(parent_key, buf->str, &hkey); if(lres != ERROR_SUCCESS) { WARN("Could not create(open) key: %08x\n", lres);

1 0

Huw Davies : gdi32: Quote the printer name prior to passing it to the shell .
by Alexandre Julliard Aug. 23, 2007

Aug. 23, 2007

Module: wine Branch: master Commit: a247ca53081e1b871320cb47232aaa766a962971 URL: http://source.winehq.org/git/wine.git/?a=commit;h=a247ca53081e1b871320cb472… Author: Huw Davies <huw(a)codeweavers.com> Date: Wed Aug 22 14:39:46 2007 +0100 gdi32: Quote the printer name prior to passing it to the shell. --- dlls/gdi32/printdrv.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/gdi32/printdrv.c b/dlls/gdi32/printdrv.c index 0cca2ad..9444536 100644 --- a/dlls/gdi32/printdrv.c +++ b/dlls/gdi32/printdrv.c @@ -468,7 +468,7 @@ static int CreateSpoolFile(LPCSTR pszOutput) RegCloseKey(hkey); } if (!psCmd[0] && !strncmp("LPR:",pszOutput,4)) - sprintf(psCmd,"|lpr -P%s",pszOutput+4); + sprintf(psCmd,"|lpr -P'%s'",pszOutput+4); TRACE("Got printerSpoolCommand '%s' for output device '%s'\n", psCmd, pszOutput);

1 0

Juan Lang : advapi32: Add more tests for CryptVerifySignatureW, and correct its parameter checking.
by Alexandre Julliard Aug. 23, 2007

Aug. 23, 2007

Module: wine Branch: master Commit: a5bbed2b95abb5b8983df487c62039c683fe98cb URL: http://source.winehq.org/git/wine.git/?a=commit;h=a5bbed2b95abb5b8983df487c… Author: Juan Lang <juan.lang(a)gmail.com> Date: Wed Aug 22 09:38:44 2007 -0700 advapi32: Add more tests for CryptVerifySignatureW, and correct its parameter checking. --- dlls/advapi32/crypt.c | 7 +--- dlls/advapi32/tests/crypt.c | 76 +++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 77 insertions(+), 6 deletions(-) diff --git a/dlls/advapi32/crypt.c b/dlls/advapi32/crypt.c index 60131bb..63f89f6 100644 --- a/dlls/advapi32/crypt.c +++ b/dlls/advapi32/crypt.c @@ -2134,12 +2134,7 @@ BOOL WINAPI CryptVerifySignatureW (HCRYPTHASH hHash, CONST BYTE *pbSignature, DW TRACE("(0x%lx, %p, %d, 0x%lx, %s, %08x)\n", hHash, pbSignature, dwSigLen, hPubKey, debugstr_w(sDescription), dwFlags); - if (!hash || !key) - { - SetLastError(ERROR_INVALID_HANDLE); - return FALSE; - } - if (!pbSignature || !dwSigLen || + if (!hash || !key || !hash->pProvider || hash->pProvider->dwMagic != MAGIC_CRYPTPROV || !key->pProvider || key->pProvider->dwMagic != MAGIC_CRYPTPROV) { diff --git a/dlls/advapi32/tests/crypt.c b/dlls/advapi32/tests/crypt.c index a8db384..9fa532c 100644 --- a/dlls/advapi32/tests/crypt.c +++ b/dlls/advapi32/tests/crypt.c @@ -349,6 +349,81 @@ static void test_incorrect_api_usage(void) ok (!result && GetLastError() == ERROR_INVALID_PARAMETER, "%d\n", GetLastError()); } +static const BYTE privKey[] = { + 0x07, 0x02, 0x00, 0x00, 0x00, 0x24, 0x00, 0x00, 0x52, 0x53, 0x41, 0x32, 0x00, + 0x02, 0x00, 0x00, 0x01, 0x00, 0x01, 0x00, 0x79, 0x10, 0x1c, 0xd0, 0x6b, 0x10, + 0x18, 0x30, 0x94, 0x61, 0xdc, 0x0e, 0xcb, 0x96, 0x4e, 0x21, 0x3f, 0x79, 0xcd, + 0xa9, 0x17, 0x62, 0xbc, 0xbb, 0x61, 0x4c, 0xe0, 0x75, 0x38, 0x6c, 0xf3, 0xde, + 0x60, 0x86, 0x03, 0x97, 0x65, 0xeb, 0x1e, 0x6b, 0xdb, 0x53, 0x85, 0xad, 0x68, + 0x21, 0xf1, 0x5d, 0xe7, 0x1f, 0xe6, 0x53, 0xb4, 0xbb, 0x59, 0x3e, 0x14, 0x27, + 0xb1, 0x83, 0xa7, 0x3a, 0x54, 0xe2, 0x8f, 0x65, 0x8e, 0x6a, 0x4a, 0xcf, 0x3b, + 0x1f, 0x65, 0xff, 0xfe, 0xf1, 0x31, 0x3a, 0x37, 0x7a, 0x8b, 0xcb, 0xc6, 0xd4, + 0x98, 0x50, 0x36, 0x67, 0xe4, 0xa1, 0xe8, 0x7e, 0x8a, 0xc5, 0x23, 0xf2, 0x77, + 0xf5, 0x37, 0x61, 0x49, 0x72, 0x59, 0xe8, 0x3d, 0xf7, 0x60, 0xb2, 0x77, 0xca, + 0x78, 0x54, 0x6d, 0x65, 0x9e, 0x03, 0x97, 0x1b, 0x61, 0xbd, 0x0c, 0xd8, 0x06, + 0x63, 0xe2, 0xc5, 0x48, 0xef, 0xb3, 0xe2, 0x6e, 0x98, 0x7d, 0xbd, 0x4e, 0x72, + 0x91, 0xdb, 0x31, 0x57, 0xe3, 0x65, 0x3a, 0x49, 0xca, 0xec, 0xd2, 0x02, 0x4e, + 0x22, 0x7e, 0x72, 0x8e, 0xf9, 0x79, 0x84, 0x82, 0xdf, 0x7b, 0x92, 0x2d, 0xaf, + 0xc9, 0xe4, 0x33, 0xef, 0x89, 0x5c, 0x66, 0x99, 0xd8, 0x80, 0x81, 0x47, 0x2b, + 0xb1, 0x66, 0x02, 0x84, 0x59, 0x7b, 0xc3, 0xbe, 0x98, 0x45, 0x4a, 0x3d, 0xdd, + 0xea, 0x2b, 0xdf, 0x4e, 0xb4, 0x24, 0x6b, 0xec, 0xe7, 0xd9, 0x0c, 0x45, 0xb8, + 0xbe, 0xca, 0x69, 0x37, 0x92, 0x4c, 0x38, 0x6b, 0x96, 0x6d, 0xcd, 0x86, 0x67, + 0x5c, 0xea, 0x54, 0x94, 0xa4, 0xca, 0xa4, 0x02, 0xa5, 0x21, 0x4d, 0xae, 0x40, + 0x8f, 0x9d, 0x51, 0x83, 0xf2, 0x3f, 0x33, 0xc1, 0x72, 0xb4, 0x1d, 0x94, 0x6e, + 0x7d, 0xe4, 0x27, 0x3f, 0xea, 0xff, 0xe5, 0x9b, 0xa7, 0x5e, 0x55, 0x8e, 0x0d, + 0x69, 0x1c, 0x7a, 0xff, 0x81, 0x9d, 0x53, 0x52, 0x97, 0x9a, 0x76, 0x79, 0xda, + 0x93, 0x32, 0x16, 0xec, 0x69, 0x51, 0x1a, 0x4e, 0xc3, 0xf1, 0x72, 0x80, 0x78, + 0x5e, 0x66, 0x4a, 0x8d, 0x85, 0x2f, 0x3f, 0xb2, 0xa7 }; + +static void test_verify_sig(void) +{ + BOOL ret; + HCRYPTPROV prov; + HCRYPTKEY key; + HCRYPTHASH hash; + BYTE bogus[] = { 0 }; + + SetLastError(0xdeadbeef); + ret = pCryptVerifySignatureW(0, NULL, 0, 0, NULL, 0); + ok(!ret && GetLastError() == ERROR_INVALID_PARAMETER, + "Expected ERROR_INVALID_PARAMETER, got %08x\n", GetLastError()); + ret = pCryptAcquireContextA(&prov, szKeySet, NULL, PROV_RSA_FULL, + CRYPT_NEWKEYSET); + if (!ret && GetLastError() == NTE_EXISTS) + ret = pCryptAcquireContextA(&prov, szKeySet, NULL, PROV_RSA_FULL, 0); + ret = pCryptImportKey(prov, (LPBYTE)privKey, sizeof(privKey), 0, 0, &key); + ok(ret, "CryptImportKey failed: %08x\n", GetLastError()); + ret = pCryptCreateHash(prov, CALG_MD5, 0, 0, &hash); + ok(ret, "CryptCreateHash failed: %08x\n", GetLastError()); + SetLastError(0xdeadbeef); + ret = pCryptVerifySignatureW(hash, NULL, 0, 0, NULL, 0); + ok(!ret && GetLastError() == ERROR_INVALID_PARAMETER, + "Expected ERROR_INVALID_PARAMETER, got %08x\n", GetLastError()); + SetLastError(0xdeadbeef); + ret = pCryptVerifySignatureW(0, NULL, 0, key, NULL, 0); + ok(!ret && GetLastError() == ERROR_INVALID_PARAMETER, + "Expected ERROR_INVALID_PARAMETER, got %08x\n", GetLastError()); + SetLastError(0xdeadbeef); + ret = pCryptVerifySignatureW(hash, NULL, 0, key, NULL, 0); + ok(!ret && GetLastError() == NTE_BAD_SIGNATURE, + "Expected NTE_BAD_SIGNATURE, got %08x\n", GetLastError()); + SetLastError(0xdeadbeef); + ret = pCryptVerifySignatureW(hash, NULL, sizeof(bogus), key, NULL, 0); + ok(!ret && GetLastError() == NTE_BAD_SIGNATURE, + "Expected NTE_BAD_SIGNATURE, got %08x\n", GetLastError()); + SetLastError(0xdeadbeef); + ret = pCryptVerifySignatureW(hash, bogus, 0, key, NULL, 0); + ok(!ret && GetLastError() == NTE_BAD_SIGNATURE, + "Expected NTE_BAD_SIGNATURE, got %08x\n", GetLastError()); + SetLastError(0xdeadbeef); + ret = pCryptVerifySignatureW(hash, bogus, sizeof(bogus), key, NULL, 0); + ok(!ret && GetLastError() == NTE_BAD_SIGNATURE, + "Expected NTE_BAD_SIGNATURE, got %08x\n", GetLastError()); + pCryptDestroyKey(key); + pCryptDestroyHash(hash); + pCryptReleaseContext(prov, 0); +} + static BOOL FindProvRegVals(DWORD dwIndex, DWORD *pdwProvType, LPSTR *pszProvName, DWORD *pcbProvName, DWORD *pdwProvCount) { @@ -779,6 +854,7 @@ START_TEST(crypt) init_environment(); test_acquire_context(); test_incorrect_api_usage(); + test_verify_sig(); clean_up_environment(); }

1 0

Alexandre Julliard : gdi32: Fix a few more places that use the wrong get/ release function for DCs.
by Alexandre Julliard Aug. 23, 2007

Aug. 23, 2007

Module: wine Branch: master Commit: e9a4c2351d4317f214a51aa8de0740851ae45e04 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e9a4c2351d4317f214a51aa8d… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Aug 22 18:20:39 2007 +0200 gdi32: Fix a few more places that use the wrong get/release function for DCs. --- dlls/gdi32/dc.c | 24 +++++++++++------------- dlls/gdi32/gdiobj.c | 14 ++++---------- 2 files changed, 15 insertions(+), 23 deletions(-) diff --git a/dlls/gdi32/dc.c b/dlls/gdi32/dc.c index 4bf19b4..a3689eb 100644 --- a/dlls/gdi32/dc.c +++ b/dlls/gdi32/dc.c @@ -200,7 +200,7 @@ BOOL DC_FreeDCPtr( DC *dc ) */ static BOOL DC_DeleteObject( HGDIOBJ handle, void *obj ) { - DC_ReleaseDCPtr( obj ); + GDI_ReleaseObj( handle ); return DeleteDC( handle ); } @@ -740,25 +740,23 @@ HDC WINAPI CreateICW( LPCWSTR driver, LPCWSTR device, LPCWSTR output, HDC WINAPI CreateCompatibleDC( HDC hdc ) { DC *dc, *origDC; - const DC_FUNCTIONS *funcs; - PHYSDEV physDev; + const DC_FUNCTIONS *funcs = NULL; + PHYSDEV physDev = NULL; GDI_CheckNotLock(); - if ((origDC = GDI_GetObjPtr( hdc, DC_MAGIC ))) + if ((origDC = DC_GetDCPtr( hdc ))) { - funcs = origDC->funcs; - physDev = origDC->physDev; + if (GetObjectType( hdc ) == OBJ_DC) + { + funcs = origDC->funcs; + physDev = origDC->physDev; + } DC_ReleaseDCPtr( origDC ); /* can't hold the lock while loading the driver */ - funcs = DRIVER_get_driver( funcs ); - } - else - { - funcs = DRIVER_load_driver( displayW ); - physDev = NULL; + if (funcs) funcs = DRIVER_get_driver( funcs ); } - if (!funcs) return 0; + if (!funcs && !(funcs = DRIVER_load_driver( displayW ))) return 0; if (!(dc = DC_AllocDC( funcs, MEMORY_DC_MAGIC ))) goto error; diff --git a/dlls/gdi32/gdiobj.c b/dlls/gdi32/gdiobj.c index f2dd4e1..0388c71 100644 --- a/dlls/gdi32/gdiobj.c +++ b/dlls/gdi32/gdiobj.c @@ -528,15 +528,9 @@ static DWORD get_dpi( void ) * * Increment the reference count of a GDI object. */ -static inline void inc_ref_count( HGDIOBJ handle ) +static inline void inc_ref_count( GDIOBJHDR *header ) { - GDIOBJHDR *header; - - if ((header = GDI_GetObjPtr( handle, MAGIC_DONTCARE ))) - { - header->dwCount++; - GDI_ReleaseObj( handle ); - } + header->dwCount++; } @@ -829,7 +823,7 @@ BOOL WINAPI DeleteObject( HGDIOBJ obj ) { if(dc->funcs->pDeleteObject) dc->funcs->pDeleteObject( dc->physDev, obj ); - GDI_ReleaseObj( header->hdcs->hdc ); + DC_ReleaseDCPtr( dc ); } tmp = header->hdcs; header->hdcs = header->hdcs->next; @@ -1159,7 +1153,7 @@ HGDIOBJ WINAPI SelectObject( HDC hdc, HGDIOBJ hObj ) ret = header->funcs->pSelectObject( hObj, header, hdc ); if (ret && ret != hObj && HandleToULong(ret) > COMPLEXREGION) { - inc_ref_count( hObj ); + inc_ref_count( header ); dec_ref_count( ret ); } }

1 0

Maarten Lankhorst : dsound: Transparantly fall back to software mode if hardware mode is unavailable.
by Alexandre Julliard Aug. 23, 2007

Aug. 23, 2007

Module: wine Branch: master Commit: 2ececbeda999dc67ac93f88233cfaa7e535042cd URL: http://source.winehq.org/git/wine.git/?a=commit;h=2ececbeda999dc67ac93f8823… Author: Maarten Lankhorst <m.b.lankhorst(a)gmail.com> Date: Tue Aug 21 21:04:32 2007 +0200 dsound: Transparantly fall back to software mode if hardware mode is unavailable. --- dlls/dsound/primary.c | 257 +++++++++++++++++++------------------------------ 1 files changed, 98 insertions(+), 159 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=2ececbeda999dc67ac93f…

1 0

appdb/. screenshots.php
by WineHQ Aug. 22, 2007

Aug. 22, 2007

ChangeSet ID: 31336 CVSROOT: /opt/cvs-commit Module name: appdb Changes by: wineowner(a)winehq.org 2007/08/22 21:33:57 Modified files: . : screenshots.php Log message: Chris Morgan <cmorgan(a)alum.wpi.edu> Remove logic that was preventing the opening of the version frame. The code to open the version frame also determins if the user maintains the version or not, which decides whether we include the "Delete Image" link. Change fixes missing "Delete Image" link for maintainers. Patch: http://cvs.winehq.org/patch.py?id=31336 Old revision New revision Changes Path 1.49 1.50 +6 -4 appdb/screenshots.php Index: appdb/screenshots.php diff -u -p appdb/screenshots.php:1.49 appdb/screenshots.php:1.50 --- appdb/screenshots.php:1.49 23 Aug 2007 2:33:57 -0000 +++ appdb/screenshots.php 23 Aug 2007 2:33:57 -0000 @@ -62,7 +62,7 @@ $oVersion = new Version($aClean['iVersio if($hResult && query_num_rows($hResult)) { - echo html_frame_start("Screenshot Gallery for ".$oApp->sName." ".$oVersion->sName,500); + echo html_frame_start("Screenshot Gallery for ".$oApp->sName, 500); // display thumbnails $c = 1; @@ -77,7 +77,7 @@ if($hResult && query_num_rows($hResult)) // if the current version changed then update the current version // and close the previous html frame if this isn't the // first frame - if(!$aClean['iVersionId'] && $oRow->versionId != $iCurrentVersionId) + if($oRow->versionId != $iCurrentVersionId) { if($iCurrentVersionId) { @@ -86,13 +86,14 @@ if($hResult && query_num_rows($hResult)) $c=1; } $iCurrentVersionId = $oRow->versionId; - $bUserIsMaintainerOfVersion = $_SESSION['current']->isMaintainer($iCurrentVersionId); + $bUserIsMaintainerOfVersion = $_SESSION['current']->isMaintainer($iCurrentVersionId); echo html_frame_start("Version ".Version::lookup_name($iCurrentVersionId)); echo "<div align=center><table><tr>\n"; } $oScreenshot = new Screenshot($oRow->id); $img = $oScreenshot->get_thumbnail_img(); + // display image echo "<td>\n"; echo $img; @@ -120,7 +121,8 @@ if($hResult && query_num_rows($hResult)) echo html_frame_end(); // close the current version we are displaying echo html_frame_end(); // close the "Screenshot Gallary..." html frame -} else { +} else +{ echo "<p align=\"center\">There are currently no screenshots for the selected version of this application."; echo "<br />Please consider submitting a screenshot for the selected version yourself.</p>"; }

1 0

appdb/. screenshots.php
by WineHQ Aug. 22, 2007

Aug. 22, 2007

ChangeSet ID: 31335 CVSROOT: /opt/cvs-commit Module name: appdb Changes by: wineowner(a)winehq.org 2007/08/22 20:48:45 Modified files: . : screenshots.php Log message: Chris Morgan <cmorgan(a)alum.wpi.edu> Require that users be logged in to submit screenshots. Display a login link on the screenshots page for users that aren't logged in so they know that if they login they'll be able to submit screenshots. Patch: http://cvs.winehq.org/patch.py?id=31335 Old revision New revision Changes Path 1.48 1.49 +7 -2 appdb/screenshots.php Index: appdb/screenshots.php diff -u -p appdb/screenshots.php:1.48 appdb/screenshots.php:1.49 --- appdb/screenshots.php:1.48 23 Aug 2007 1:48:45 -0000 +++ appdb/screenshots.php 23 Aug 2007 1:48:45 -0000 @@ -125,8 +125,9 @@ if($hResult && query_num_rows($hResult)) echo "<br />Please consider submitting a screenshot for the selected version yourself.</p>"; } -// let's show the screenshot uploading box -if($aClean['iVersionId']) +// let's show the screenshot uploading box, but only +// if the user is logged in +if($aClean['iVersionId'] && $_SESSION['current']->isLoggedIn()) { echo "<p align=\"center\">When submitting screenshots please ensure that the focus is on the application running inside Wine."; echo "<br />This means if the application is running in a window then please crop the image so that only the application is shown and not your desktop.</p>"; @@ -144,6 +145,10 @@ if($aClean['iVersionId']) echo '<input type="hidden" name="MAX_FILE_SIZE" value="4000000" />',"\n"; echo '<input type="hidden" name="sCmd" value="screenshot_upload" />',"\n"; echo '<input type="hidden" name="iVersionId" value="'.$aClean['iVersionId'].'"></form />',"\n"; +} else // else let the person know that if they log in they can submit screenshots +{ + echo '<div align="center"><a href="'.login_url().'">'; + echo "Log in</a> to submit screenshots</div>\n"; } echo html_back_link(1);

1 0