December 2009 - Wine-commits

Juan Lang : cryptnet: Add stub for verifying revocation via OCSP.
by Alexandre Julliard 18 Dec '09

18 Dec '09

Module: wine Branch: master Commit: 41b3b8e4eb78fdf211b059f005cafbdc728ea83a URL: http://source.winehq.org/git/wine.git/?a=commit;h=41b3b8e4eb78fdf211b059f00… Author: Juan Lang <juan.lang(a)gmail.com> Date: Thu Dec 3 11:51:09 2009 -0800 cryptnet: Add stub for verifying revocation via OCSP. --- dlls/cryptnet/cryptnet_main.c | 40 ++++++++++++++++++++++++++++++++++++++++ 1 files changed, 40 insertions(+), 0 deletions(-) diff --git a/dlls/cryptnet/cryptnet_main.c b/dlls/cryptnet/cryptnet_main.c index 10863a2..d4d0e2c 100644 --- a/dlls/cryptnet/cryptnet_main.c +++ b/dlls/cryptnet/cryptnet_main.c @@ -1654,6 +1654,42 @@ static DWORD verify_cert_revocation_from_dist_points_ext( return error; } +static DWORD verify_cert_revocation_from_aia_ext( + const CRYPT_DATA_BLOB *value, PCCERT_CONTEXT cert, DWORD index, + FILETIME *pTime, DWORD dwFlags, PCERT_REVOCATION_PARA pRevPara, + PCERT_REVOCATION_STATUS pRevStatus) +{ + BOOL ret; + DWORD error, size; + CERT_AUTHORITY_INFO_ACCESS *aia; + + ret = CryptDecodeObjectEx(X509_ASN_ENCODING, X509_AUTHORITY_INFO_ACCESS, + value->pbData, value->cbData, CRYPT_DECODE_ALLOC_FLAG, NULL, &aia, &size); + if (ret) + { + DWORD i; + + for (i = 0; i < aia->cAccDescr; i++) + if (!strcmp(aia->rgAccDescr[i].pszAccessMethod, + szOID_PKIX_OCSP)) + { + if (aia->rgAccDescr[i].AccessLocation.dwAltNameChoice == + CERT_ALT_NAME_URL) + FIXME("OCSP URL = %s\n", + debugstr_w(aia->rgAccDescr[i].AccessLocation.u.pwszURL)); + else + FIXME("unsupported AccessLocation type %d\n", + aia->rgAccDescr[i].AccessLocation.dwAltNameChoice); + } + LocalFree(aia); + /* FIXME: lie and pretend OCSP validated the cert */ + error = ERROR_SUCCESS; + } + else + error = GetLastError(); + return error; +} + static DWORD verify_cert_revocation(PCCERT_CONTEXT cert, DWORD index, FILETIME *pTime, DWORD dwFlags, PCERT_REVOCATION_PARA pRevPara, PCERT_REVOCATION_STATUS pRevStatus) @@ -1665,6 +1701,10 @@ static DWORD verify_cert_revocation(PCCERT_CONTEXT cert, DWORD index, cert->pCertInfo->cExtension, cert->pCertInfo->rgExtension))) error = verify_cert_revocation_from_dist_points_ext(&ext->Value, cert, index, pTime, dwFlags, pRevPara, pRevStatus); + else if ((ext = CertFindExtension(szOID_AUTHORITY_INFO_ACCESS, + cert->pCertInfo->cExtension, cert->pCertInfo->rgExtension))) + error = verify_cert_revocation_from_aia_ext(&ext->Value, cert, + index, pTime, dwFlags, pRevPara, pRevStatus); else { if (pRevPara && pRevPara->hCrlStore && pRevPara->pIssuerCert)

1 0

Henri Verbeet : wined3d: Add some error handling to create_primary_opengl_context().
by Alexandre Julliard 18 Dec '09

18 Dec '09

Module: wine Branch: master Commit: 23f05028f8d5709c6d19e0598dc911a18f1f5f2d URL: http://source.winehq.org/git/wine.git/?a=commit;h=23f05028f8d5709c6d19e0598… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Thu Dec 17 19:14:35 2009 +0100 wined3d: Add some error handling to create_primary_opengl_context(). --- dlls/wined3d/device.c | 62 ++++++++++++++++++++++++++++++++---------------- 1 files changed, 41 insertions(+), 21 deletions(-) diff --git a/dlls/wined3d/device.c b/dlls/wined3d/device.c index 3dfc4e9..bb9e097 100644 --- a/dlls/wined3d/device.c +++ b/dlls/wined3d/device.c @@ -6293,45 +6293,65 @@ void delete_opengl_contexts(IWineD3DDevice *iface, IWineD3DSwapChain *swapchain_ HRESULT create_primary_opengl_context(IWineD3DDevice *iface, IWineD3DSwapChain *swapchain_iface) { IWineD3DDeviceImpl *This = (IWineD3DDeviceImpl *) iface; IWineD3DSwapChainImpl *swapchain = (IWineD3DSwapChainImpl *) swapchain_iface; + struct wined3d_context *context; HRESULT hr; IWineD3DSurfaceImpl *target; /* Recreate the primary swapchain's context */ swapchain->context = HeapAlloc(GetProcessHeap(), 0, sizeof(*swapchain->context)); - if(swapchain->backBuffer) { - target = (IWineD3DSurfaceImpl *) swapchain->backBuffer[0]; - } else { - target = (IWineD3DSurfaceImpl *) swapchain->frontBuffer; + if (!swapchain->context) + { + ERR("Failed to allocate memory for swapchain context array.\n"); + return E_OUTOFMEMORY; } - swapchain->context[0] = context_create(This, target, swapchain->win_handle, FALSE, &swapchain->presentParms); - swapchain->num_contexts = 1; - create_dummy_textures(This); + target = (IWineD3DSurfaceImpl *)(swapchain->backBuffer ? swapchain->backBuffer[0] : swapchain->frontBuffer); + context = context_create(This, target, swapchain->win_handle, FALSE, &swapchain->presentParms); + if (!context) + { + WARN("Failed to create context.\n"); + HeapFree(GetProcessHeap(), 0, swapchain->context); + return E_FAIL; + } - context_release(swapchain->context[0]); + swapchain->context[0] = context; + swapchain->num_contexts = 1; + create_dummy_textures(This); + context_release(context); hr = This->shader_backend->shader_alloc_private(iface); - if(FAILED(hr)) { - ERR("Failed to recreate shader private data\n"); - goto err_out; + if (FAILED(hr)) + { + ERR("Failed to allocate shader private data, hr %#x.\n", hr); + goto err; } + hr = This->frag_pipe->alloc_private(iface); - if(FAILED(hr)) { - TRACE("Fragment pipeline private data couldn't be allocated\n"); - goto err_out; + if (FAILED(hr)) + { + ERR("Failed to allocate fragment pipe private data, hr %#x.\n", hr); + This->shader_backend->shader_free_private(iface); + goto err; } + hr = This->blitter->alloc_private(iface); - if(FAILED(hr)) { - TRACE("Blitter private data couldn't be allocated\n"); - goto err_out; + if (FAILED(hr)) + { + ERR("Failed to allocate blitter private data, hr %#x.\n", hr); + This->frag_pipe->free_private(iface); + This->shader_backend->shader_free_private(iface); + goto err; } return WINED3D_OK; -err_out: - This->blitter->free_private(iface); - This->frag_pipe->free_private(iface); - This->shader_backend->shader_free_private(iface); +err: + context_acquire(This, NULL, CTXUSAGE_RESOURCELOAD); + destroy_dummy_textures(This, context->gl_info); + context_release(context); + context_destroy(This, context); + HeapFree(GetProcessHeap(), 0, swapchain->context); + swapchain->num_contexts = 0; return hr; }

1 0

Juan Lang : cryptnet: Make sure an issuer certificate can sign CRLs before looking for a CRL.
by Alexandre Julliard 18 Dec '09

18 Dec '09

Module: wine Branch: master Commit: 0bfb7951a48372ea37c7bf77583a97469a978863 URL: http://source.winehq.org/git/wine.git/?a=commit;h=0bfb7951a48372ea37c7bf775… Author: Juan Lang <juan.lang(a)gmail.com> Date: Wed Dec 2 16:24:48 2009 -0800 cryptnet: Make sure an issuer certificate can sign CRLs before looking for a CRL. --- dlls/cryptnet/cryptnet_main.c | 50 +++++++++++++++++++++++++++++++++++----- 1 files changed, 43 insertions(+), 7 deletions(-) diff --git a/dlls/cryptnet/cryptnet_main.c b/dlls/cryptnet/cryptnet_main.c index 9e29daa..ef326f7 100644 --- a/dlls/cryptnet/cryptnet_main.c +++ b/dlls/cryptnet/cryptnet_main.c @@ -1584,15 +1584,51 @@ static DWORD verify_cert_revocation(PCCERT_CONTEXT cert, DWORD index, { if (pRevPara && pRevPara->hCrlStore && pRevPara->pIssuerCert) { - PCCRL_CONTEXT crl; + PCERT_EXTENSION ext; + PCCRL_CONTEXT crl = NULL; + BOOL canSignCRLs; - /* If the caller was helpful enough to tell us where to find a CRL - * for the cert, look for one and check it. + /* If the caller told us about the issuer, make sure the issuer + * can sign CRLs before looking for one. */ - crl = CertFindCRLInStore(pRevPara->hCrlStore, - cert->dwCertEncodingType, - CRL_FIND_ISSUED_BY_SIGNATURE_FLAG | CRL_FIND_ISSUED_BY_AKI_FLAG, - CRL_FIND_ISSUED_BY, pRevPara->pIssuerCert, NULL); + if ((ext = CertFindExtension(szOID_KEY_USAGE, + pRevPara->pIssuerCert->pCertInfo->cExtension, + pRevPara->pIssuerCert->pCertInfo->rgExtension))) + { + CRYPT_BIT_BLOB usage; + DWORD size = sizeof(usage); + + if (!CryptDecodeObjectEx(cert->dwCertEncodingType, X509_BITS, + ext->Value.pbData, ext->Value.cbData, + CRYPT_DECODE_NOCOPY_FLAG, NULL, &usage, &size)) + canSignCRLs = FALSE; + else if (usage.cbData > 2) + { + /* The key usage extension only defines 9 bits => no more + * than 2 bytes are needed to encode all known usages. + */ + canSignCRLs = FALSE; + } + else + { + BYTE usageBits = usage.pbData[usage.cbData - 1]; + + canSignCRLs = usageBits & CERT_CRL_SIGN_KEY_USAGE; + } + } + else + canSignCRLs = TRUE; + if (canSignCRLs) + { + /* If the caller was helpful enough to tell us where to find a + * CRL for the cert, look for one and check it. + */ + crl = CertFindCRLInStore(pRevPara->hCrlStore, + cert->dwCertEncodingType, + CRL_FIND_ISSUED_BY_SIGNATURE_FLAG | + CRL_FIND_ISSUED_BY_AKI_FLAG, + CRL_FIND_ISSUED_BY, pRevPara->pIssuerCert, NULL); + } if (crl) { error = verify_cert_revocation_with_crl(cert, crl, index,

1 0

Alexandre Julliard : user32: Move the 16-bit menu functions to user16.c.
by Alexandre Julliard 17 Dec '09

17 Dec '09

Module: wine Branch: master Commit: 15000f32b6a57393de41d6a59b23c345cf36ce8f URL: http://source.winehq.org/git/wine.git/?a=commit;h=15000f32b6a57393de41d6a59… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Dec 17 15:04:21 2009 +0100 user32: Move the 16-bit menu functions to user16.c. --- dlls/user32/menu.c | 90 +++++-------------------------------------- dlls/user32/user16.c | 104 ++++++++++++++++++++++++++++++++++++++++++++++++++ dlls/user32/win.c | 9 ---- 3 files changed, 114 insertions(+), 89 deletions(-) diff --git a/dlls/user32/menu.c b/dlls/user32/menu.c index 230e412..2f0ffc2 100644 --- a/dlls/user32/menu.c +++ b/dlls/user32/menu.c @@ -51,9 +51,6 @@ #include "winbase.h" #include "wingdi.h" #include "winnls.h" -#include "wine/winbase16.h" -#include "wine/winuser16.h" -#include "wownt32.h" #include "wine/server.h" #include "wine/unicode.h" #include "wine/exception.h" @@ -2132,10 +2129,10 @@ static MENUITEM *MENU_InsertItem( HMENU hMenu, UINT pos, UINT flags ) * * NOTE: flags is equivalent to the mtOption field */ -static LPCSTR MENU_ParseResource( LPCSTR res, HMENU hMenu, BOOL unicode ) +static LPCSTR MENU_ParseResource( LPCSTR res, HMENU hMenu ) { WORD flags, id = 0; - LPCSTR str; + LPCWSTR str; BOOL end_flag; do @@ -2150,23 +2147,18 @@ static LPCSTR MENU_ParseResource( LPCSTR res, HMENU hMenu, BOOL unicode ) id = GET_WORD(res); res += sizeof(WORD); } - str = res; - if (!unicode) res += strlen(str) + 1; - else res += (strlenW((LPCWSTR)str) + 1) * sizeof(WCHAR); + str = (LPCWSTR)res; + res += (strlenW(str) + 1) * sizeof(WCHAR); if (flags & MF_POPUP) { HMENU hSubMenu = CreatePopupMenu(); if (!hSubMenu) return NULL; - if (!(res = MENU_ParseResource( res, hSubMenu, unicode ))) - return NULL; - if (!unicode) AppendMenuA( hMenu, flags, (UINT_PTR)hSubMenu, str ); - else AppendMenuW( hMenu, flags, (UINT_PTR)hSubMenu, (LPCWSTR)str ); + if (!(res = MENU_ParseResource( res, hSubMenu ))) return NULL; + AppendMenuW( hMenu, flags, (UINT_PTR)hSubMenu, str ); } else /* Not a popup */ { - if (!unicode) AppendMenuA( hMenu, flags, id, *str ? str : NULL ); - else AppendMenuW( hMenu, flags, id, - *(LPCWSTR)str ? (LPCWSTR)str : NULL ); + AppendMenuW( hMenu, flags, id, *str ? str : NULL ); } } while (!end_flag); return res; @@ -3791,7 +3783,7 @@ static void MENU_mnu2mnuii( UINT flags, UINT_PTR id, LPCWSTR str, pmii->dwTypeData = (LPWSTR)str; } else if( flags & MFT_BITMAP){ pmii->fMask |= MIIM_BITMAP | MIIM_STRING; - pmii->hbmpItem = HBITMAP_32(LOWORD(str)); + pmii->hbmpItem = ULongToHandle(LOWORD(str)); } if( flags & MF_OWNERDRAW){ pmii->fMask |= MIIM_DATA; @@ -4326,39 +4318,6 @@ BOOL WINAPI EndMenu(void) } -/*********************************************************************** - * LookupMenuHandle (USER.217) - */ -HMENU16 WINAPI LookupMenuHandle16( HMENU16 hmenu, INT16 id ) -{ - HMENU hmenu32 = HMENU_32(hmenu); - UINT id32 = id; - if (!MENU_FindItem( &hmenu32, &id32, MF_BYCOMMAND )) return 0; - else return HMENU_16(hmenu32); -} - - -/********************************************************************** - * LoadMenu (USER.150) - */ -HMENU16 WINAPI LoadMenu16( HINSTANCE16 instance, LPCSTR name ) -{ - HRSRC16 hRsrc; - HGLOBAL16 handle; - HMENU16 hMenu; - - if (HIWORD(name) && name[0] == '#') name = ULongToPtr(atoi( name + 1 )); - if (!name) return 0; - - instance = GetExePtr( instance ); - if (!(hRsrc = FindResource16( instance, name, (LPSTR)RT_MENU ))) return 0; - if (!(handle = LoadResource16( instance, hRsrc ))) return 0; - hMenu = LoadMenuIndirect16(LockResource16(handle)); - FreeResource16( handle ); - return hMenu; -} - - /***************************************************************** * LoadMenuA (USER32.@) */ @@ -4382,35 +4341,6 @@ HMENU WINAPI LoadMenuW( HINSTANCE instance, LPCWSTR name ) /********************************************************************** - * LoadMenuIndirect (USER.220) - */ -HMENU16 WINAPI LoadMenuIndirect16( LPCVOID template ) -{ - HMENU hMenu; - WORD version, offset; - LPCSTR p = template; - - TRACE("(%p)\n", template ); - version = GET_WORD(p); - p += sizeof(WORD); - if (version) - { - WARN("version must be 0 for Win16\n" ); - return 0; - } - offset = GET_WORD(p); - p += sizeof(WORD) + offset; - if (!(hMenu = CreateMenu())) return 0; - if (!MENU_ParseResource( p, hMenu, FALSE )) - { - DestroyMenu( hMenu ); - return 0; - } - return HMENU_16(hMenu); -} - - -/********************************************************************** * LoadMenuIndirectW (USER32.@) */ HMENU WINAPI LoadMenuIndirectW( LPCVOID template ) @@ -4428,7 +4358,7 @@ HMENU WINAPI LoadMenuIndirectW( LPCVOID template ) offset = GET_WORD(p); p += sizeof(WORD) + offset; if (!(hMenu = CreateMenu())) return 0; - if (!MENU_ParseResource( p, hMenu, TRUE )) + if (!MENU_ParseResource( p, hMenu )) { DestroyMenu( hMenu ); return 0; @@ -4784,7 +4714,7 @@ static BOOL MENU_NormalizeMenuItemInfoStruct( const MENUITEMINFOW *pmii_in, pmii_out->fMask |= MIIM_STRING; } else if( (pmii_out->fType) & MFT_BITMAP){ pmii_out->fMask |= MIIM_BITMAP; - pmii_out->hbmpItem = HBITMAP_32(LOWORD(pmii_out->dwTypeData)); + pmii_out->hbmpItem = UlongToHandle(LOWORD(pmii_out->dwTypeData)); } } return TRUE; diff --git a/dlls/user32/user16.c b/dlls/user32/user16.c index 34133e5..1c5adb4 100644 --- a/dlls/user32/user16.c +++ b/dlls/user32/user16.c @@ -190,6 +190,15 @@ BOOL16 WINAPI GetCursorPos16( POINT16 *pt ) } +/******************************************************************* + * AnyPopup (USER.52) + */ +BOOL16 WINAPI AnyPopup16(void) +{ + return AnyPopup(); +} + + /*********************************************************************** * SetCursor (USER.69) */ @@ -511,6 +520,27 @@ INT16 WINAPI GetClipboardFormatName16( UINT16 id, LPSTR buffer, INT16 maxlen ) /********************************************************************** + * LoadMenu (USER.150) + */ +HMENU16 WINAPI LoadMenu16( HINSTANCE16 instance, LPCSTR name ) +{ + HRSRC16 hRsrc; + HGLOBAL16 handle; + HMENU16 hMenu; + + if (HIWORD(name) && name[0] == '#') name = ULongToPtr(atoi( name + 1 )); + if (!name) return 0; + + instance = GetExePtr( instance ); + if (!(hRsrc = FindResource16( instance, name, (LPSTR)RT_MENU ))) return 0; + if (!(handle = LoadResource16( instance, hRsrc ))) return 0; + hMenu = LoadMenuIndirect16(LockResource16(handle)); + FreeResource16( handle ); + return hMenu; +} + + +/********************************************************************** * CreateMenu (USER.151) */ HMENU16 WINAPI CreateMenu16(void) @@ -869,6 +899,80 @@ DWORD WINAPI UserSeeUserDo16(WORD wReqType, WORD wParam1, WORD wParam2, WORD wPa } +/*********************************************************************** + * LookupMenuHandle (USER.217) + */ +HMENU16 WINAPI LookupMenuHandle16( HMENU16 hmenu, INT16 id ) +{ + FIXME( "%04x %04x: stub\n", hmenu, id ); + return hmenu; +} + + +static LPCSTR parse_menu_resource( LPCSTR res, HMENU hMenu ) +{ + WORD flags, id = 0; + LPCSTR str; + BOOL end_flag; + + do + { + flags = GET_WORD(res); + end_flag = flags & MF_END; + /* Remove MF_END because it has the same value as MF_HILITE */ + flags &= ~MF_END; + res += sizeof(WORD); + if (!(flags & MF_POPUP)) + { + id = GET_WORD(res); + res += sizeof(WORD); + } + str = res; + res += strlen(str) + 1; + if (flags & MF_POPUP) + { + HMENU hSubMenu = CreatePopupMenu(); + if (!hSubMenu) return NULL; + if (!(res = parse_menu_resource( res, hSubMenu ))) return NULL; + AppendMenuA( hMenu, flags, (UINT_PTR)hSubMenu, str ); + } + else /* Not a popup */ + { + AppendMenuA( hMenu, flags, id, *str ? str : NULL ); + } + } while (!end_flag); + return res; +} + +/********************************************************************** + * LoadMenuIndirect (USER.220) + */ +HMENU16 WINAPI LoadMenuIndirect16( LPCVOID template ) +{ + HMENU hMenu; + WORD version, offset; + LPCSTR p = template; + + TRACE("(%p)\n", template ); + version = GET_WORD(p); + p += sizeof(WORD); + if (version) + { + WARN("version must be 0 for Win16\n" ); + return 0; + } + offset = GET_WORD(p); + p += sizeof(WORD) + offset; + if (!(hMenu = CreateMenu())) return 0; + if (!parse_menu_resource( p, hMenu )) + { + DestroyMenu( hMenu ); + return 0; + } + return HMENU_16(hMenu); +} + + /************************************************************************* * ScrollDC (USER.221) */ diff --git a/dlls/user32/win.c b/dlls/user32/win.c index 8f43e92..9712beb 100644 --- a/dlls/user32/win.c +++ b/dlls/user32/win.c @@ -3082,15 +3082,6 @@ BOOL WINAPI EnumChildWindows( HWND parent, WNDENUMPROC func, LPARAM lParam ) /******************************************************************* - * AnyPopup (USER.52) - */ -BOOL16 WINAPI AnyPopup16(void) -{ - return AnyPopup(); -} - - -/******************************************************************* * AnyPopup (USER32.@) */ BOOL WINAPI AnyPopup(void)

1 0

Alexandre Julliard : user32: Move EnumProps16 to wnd16.c.
by Alexandre Julliard 17 Dec '09

17 Dec '09

Module: wine Branch: master Commit: 6b78e24a098a10ce91b53eeb26eca36a85e66aa6 URL: http://source.winehq.org/git/wine.git/?a=commit;h=6b78e24a098a10ce91b53eeb2… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Dec 17 17:24:16 2009 +0100 user32: Move EnumProps16 to wnd16.c. --- dlls/user32/property.c | 45 +---------------------------------- dlls/user32/wnd16.c | 62 ++++++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 63 insertions(+), 44 deletions(-) diff --git a/dlls/user32/property.c b/dlls/user32/property.c index d5b1bfd..52f389e 100644 --- a/dlls/user32/property.c +++ b/dlls/user32/property.c @@ -26,9 +26,8 @@ #include "windef.h" #include "winbase.h" -#include "wownt32.h" +#include "winuser.h" #include "wine/unicode.h" -#include "wine/winuser16.h" #include "wine/server.h" /* size of buffer needed to store an atom string */ @@ -252,45 +251,3 @@ INT WINAPI EnumPropsExW(HWND hwnd, PROPENUMPROCEXW func, LPARAM lParam) } return ret; } - - -/*********************************************************************** - * EnumProps (USER.27) - */ -INT16 WINAPI EnumProps16( HWND16 hwnd, PROPENUMPROC16 func ) -{ - int ret = -1, i, count; - property_data_t *list = get_properties( HWND_32(hwnd), &count ); - - if (list) - { - char string[ATOM_BUFFER_SIZE]; - SEGPTR segptr = MapLS( string ); - WORD args[4]; - DWORD result; - - for (i = 0; i < count; i++) - { - if (list[i].string) /* it was a string originally */ - { - if (!GlobalGetAtomNameA( list[i].atom, string, ATOM_BUFFER_SIZE )) continue; - args[3] = hwnd; - args[2] = SELECTOROF(segptr); - args[1] = OFFSETOF(segptr); - args[0] = LOWORD(list[i].data); - } - else - { - args[3] = hwnd; - args[2] = 0; - args[1] = list[i].atom; - args[0] = LOWORD(list[i].data); - } - WOWCallback16Ex( (DWORD)func, WCB16_PASCAL, sizeof(args), args, &result ); - if (!(ret = LOWORD(result))) break; - } - UnMapLS( segptr ); - HeapFree( GetProcessHeap(), 0, list ); - } - return ret; -} diff --git a/dlls/user32/wnd16.c b/dlls/user32/wnd16.c index 4c2f2f2..4873be7 100644 --- a/dlls/user32/wnd16.c +++ b/dlls/user32/wnd16.c @@ -22,6 +22,10 @@ #include "wownt32.h" #include "win.h" #include "user_private.h" +#include "wine/server.h" + +/* size of buffer needed to store an atom string */ +#define ATOM_BUFFER_SIZE 256 /* handle <--> handle16 conversions */ #define HANDLE_16(h32) (LOWORD(h32)) @@ -158,6 +162,64 @@ BOOL16 WINAPI SetProp16( HWND16 hwnd, LPCSTR str, HANDLE16 handle ) } +/*********************************************************************** + * EnumProps (USER.27) + */ +INT16 WINAPI EnumProps16( HWND16 hwnd, PROPENUMPROC16 func ) +{ + int ret = -1, i, count, total = 32; + property_data_t *list; + + while (total) + { + if (!(list = HeapAlloc( GetProcessHeap(), 0, total * sizeof(*list) ))) break; + count = 0; + SERVER_START_REQ( get_window_properties ) + { + req->window = wine_server_user_handle( HWND_32(hwnd) ); + wine_server_set_reply( req, list, total * sizeof(*list) ); + if (!wine_server_call( req )) count = reply->total; + } + SERVER_END_REQ; + + if (count && count <= total) + { + char string[ATOM_BUFFER_SIZE]; + SEGPTR segptr = MapLS( string ); + WORD args[4]; + DWORD result; + + for (i = 0; i < count; i++) + { + if (list[i].string) /* it was a string originally */ + { + if (!GlobalGetAtomNameA( list[i].atom, string, ATOM_BUFFER_SIZE )) continue; + args[3] = hwnd; + args[2] = SELECTOROF(segptr); + args[1] = OFFSETOF(segptr); + args[0] = LOWORD(list[i].data); + } + else + { + args[3] = hwnd; + args[2] = 0; + args[1] = list[i].atom; + args[0] = LOWORD(list[i].data); + } + WOWCallback16Ex( (DWORD)func, WCB16_PASCAL, sizeof(args), args, &result ); + if (!(ret = LOWORD(result))) break; + } + UnMapLS( segptr ); + HeapFree( GetProcessHeap(), 0, list ); + break; + } + HeapFree( GetProcessHeap(), 0, list ); + total = count; /* restart with larger buffer */ + } + return ret; +} + + /************************************************************************** * ClientToScreen (USER.28) */

1 0

Alexandre Julliard : user32: Use W structures in window creation code.
by Alexandre Julliard 17 Dec '09

17 Dec '09

Module: wine Branch: master Commit: 4b3c0e31d0fef4508f70748a4371c5d6af09f91b URL: http://source.winehq.org/git/wine.git/?a=commit;h=4b3c0e31d0fef4508f70748a4… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Dec 17 15:00:38 2009 +0100 user32: Use W structures in window creation code. --- dlls/user32/win.c | 27 ++++++++++++++------------- 1 files changed, 14 insertions(+), 13 deletions(-) diff --git a/dlls/user32/win.c b/dlls/user32/win.c index 18c2332..9b2cc89 100644 --- a/dlls/user32/win.c +++ b/dlls/user32/win.c @@ -882,7 +882,7 @@ void WIN_DestroyThreadWindows( HWND hwnd ) * Fix the coordinates - Helper for WIN_CreateWindowEx. * returns default show mode in sw. */ -static void WIN_FixCoordinates( CREATESTRUCTA *cs, INT *sw) +static void WIN_FixCoordinates( CREATESTRUCTW *cs, INT *sw) { #define IS_DEFAULT(x) ((x) == CW_USEDEFAULT || (x) == CW_USEDEFAULT16) POINT pos[2]; @@ -1072,7 +1072,7 @@ static void dump_window_styles( DWORD style, DWORD exstyle ) * * Implementation of CreateWindowEx(). */ -static HWND WIN_CreateWindowEx( CREATESTRUCTA *cs, LPCWSTR className, HINSTANCE module, UINT flags ) +static HWND WIN_CreateWindowEx( CREATESTRUCTW *cs, LPCWSTR className, HINSTANCE module, UINT flags ) { INT cx, cy, style, sw = SW_SHOW; LRESULT result; @@ -1080,12 +1080,12 @@ static HWND WIN_CreateWindowEx( CREATESTRUCTA *cs, LPCWSTR className, HINSTANCE WND *wndPtr; HWND hwnd, parent, owner, top_child = 0; BOOL unicode = (flags & WIN_ISUNICODE) != 0; - MDICREATESTRUCTA mdi_cs; - CBT_CREATEWNDA cbtc; - CREATESTRUCTA cbcs; + MDICREATESTRUCTW mdi_cs; + CBT_CREATEWNDW cbtc; + CREATESTRUCTW cbcs; TRACE("%s %s ex=%08x style=%08x %d,%d %dx%d parent=%p menu=%p inst=%p params=%p\n", - unicode ? debugstr_w((LPCWSTR)cs->lpszName) : debugstr_a(cs->lpszName), + unicode ? debugstr_w(cs->lpszName) : debugstr_a((LPCSTR)cs->lpszName), debugstr_w(className), cs->dwExStyle, cs->style, cs->x, cs->y, cs->cx, cs->cy, cs->hwndParent, cs->hMenu, cs->hInstance, cs->lpCreateParams ); @@ -1500,7 +1500,7 @@ HWND16 WINAPI CreateWindowEx16( DWORD exStyle, LPCSTR className, if (!MultiByteToWideChar( CP_ACP, 0, className, -1, bufferW, sizeof(bufferW)/sizeof(WCHAR) )) return 0; - return HWND_16( WIN_CreateWindowEx( &cs, bufferW, HINSTANCE_32(instance), 0 )); + return HWND_16( WIN_CreateWindowEx( (CREATESTRUCTW *)&cs, bufferW, HINSTANCE_32(instance), 0 )); } else { @@ -1510,7 +1510,8 @@ HWND16 WINAPI CreateWindowEx16( DWORD exStyle, LPCSTR className, return 0; } cs.lpszClass = buffer; - return HWND_16( WIN_CreateWindowEx( &cs, (LPCWSTR)className, HINSTANCE_32(instance), 0 )); + return HWND_16( WIN_CreateWindowEx( (CREATESTRUCTW *)&cs, (LPCWSTR)className, + HINSTANCE_32(instance), 0 )); } } @@ -1544,9 +1545,11 @@ HWND WINAPI CreateWindowExA( DWORD exStyle, LPCSTR className, WCHAR bufferW[256]; if (!MultiByteToWideChar( CP_ACP, 0, className, -1, bufferW, sizeof(bufferW)/sizeof(WCHAR) )) return 0; - return WIN_CreateWindowEx( &cs, bufferW, instance, WIN_ISWIN32 ); + return WIN_CreateWindowEx( (CREATESTRUCTW *)&cs, bufferW, instance, WIN_ISWIN32 ); } - return WIN_CreateWindowEx( &cs, (LPCWSTR)className, instance, WIN_ISWIN32 ); + /* Note: we rely on the fact that CREATESTRUCTA and */ + /* CREATESTRUCTW have the same layout. */ + return WIN_CreateWindowEx( (CREATESTRUCTW *)&cs, (LPCWSTR)className, instance, WIN_ISWIN32 ); } @@ -1574,9 +1577,7 @@ HWND WINAPI CreateWindowExW( DWORD exStyle, LPCWSTR className, cs.lpszClass = className; cs.dwExStyle = exStyle; - /* Note: we rely on the fact that CREATESTRUCTA and */ - /* CREATESTRUCTW have the same layout. */ - return WIN_CreateWindowEx( (CREATESTRUCTA *)&cs, className, instance, WIN_ISWIN32 | WIN_ISUNICODE ); + return WIN_CreateWindowEx( &cs, className, instance, WIN_ISWIN32 | WIN_ISUNICODE ); }

1 0

Alexandre Julliard : user32: Move 16-bit window creation functions to 16-bit files.
by Alexandre Julliard 17 Dec '09

17 Dec '09

Module: wine Branch: master Commit: 8494682f2d7bca32a89a82f266a5b97c979d92f3 URL: http://source.winehq.org/git/wine.git/?a=commit;h=8494682f2d7bca32a89a82f26… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Dec 17 15:03:32 2009 +0100 user32: Move 16-bit window creation functions to 16-bit files. --- dlls/user32/controls.h | 1 + dlls/user32/msg16.c | 67 ++++++++++++++++++++++++++++++++++++++ dlls/user32/win.c | 84 +---------------------------------------------- dlls/user32/win.h | 1 + dlls/user32/winproc.c | 1 + dlls/user32/wnd16.c | 13 +++++++ 6 files changed, 85 insertions(+), 82 deletions(-) diff --git a/dlls/user32/controls.h b/dlls/user32/controls.h index 9abc997..e7aa408 100644 --- a/dlls/user32/controls.h +++ b/dlls/user32/controls.h @@ -98,6 +98,7 @@ struct wow_handlers32 LRESULT (*mdiclient_proc)(HWND,UINT,WPARAM,LPARAM,BOOL); LRESULT (*scrollbar_proc)(HWND,UINT,WPARAM,LPARAM,BOOL); LRESULT (*static_proc)(HWND,UINT,WPARAM,LPARAM,BOOL); + HWND (*create_window)(CREATESTRUCTW*,LPCWSTR,HINSTANCE,UINT); WNDPROC (*alloc_winproc)(WNDPROC,WNDPROC); }; diff --git a/dlls/user32/msg16.c b/dlls/user32/msg16.c index 27a47a4..0e57988 100644 --- a/dlls/user32/msg16.c +++ b/dlls/user32/msg16.c @@ -2003,6 +2003,73 @@ BOOL16 WINAPI TranslateMDISysAccel16( HWND16 hwndClient, LPMSG16 msg ) /*********************************************************************** + * CreateWindowEx (USER.452) + */ +HWND16 WINAPI CreateWindowEx16( DWORD exStyle, LPCSTR className, + LPCSTR windowName, DWORD style, INT16 x, + INT16 y, INT16 width, INT16 height, + HWND16 parent, HMENU16 menu, + HINSTANCE16 instance, LPVOID data ) +{ + CREATESTRUCTA cs; + char buffer[256]; + HWND hwnd; + + /* Fix the coordinates */ + + cs.x = (x == CW_USEDEFAULT16) ? CW_USEDEFAULT : (INT)x; + cs.y = (y == CW_USEDEFAULT16) ? CW_USEDEFAULT : (INT)y; + cs.cx = (width == CW_USEDEFAULT16) ? CW_USEDEFAULT : (INT)width; + cs.cy = (height == CW_USEDEFAULT16) ? CW_USEDEFAULT : (INT)height; + + /* Create the window */ + + cs.lpCreateParams = data; + cs.hInstance = HINSTANCE_32(instance); + cs.hMenu = HMENU_32(menu); + cs.hwndParent = WIN_Handle32( parent ); + cs.style = style; + cs.lpszName = windowName; + cs.lpszClass = className; + cs.dwExStyle = exStyle; + + /* map to module handle */ + if (instance) instance = GetExePtr( instance ); + + /* load the menu */ + if (!menu && (style & (WS_CHILD | WS_POPUP)) != WS_CHILD) + { + WNDCLASSA class; + + if (GetClassInfoA( HINSTANCE_32(instance), className, &class )) + cs.hMenu = HMENU_32( LoadMenu16( instance, class.lpszMenuName )); + } + + if (!IS_INTRESOURCE(className)) + { + WCHAR bufferW[256]; + + if (!MultiByteToWideChar( CP_ACP, 0, className, -1, bufferW, sizeof(bufferW)/sizeof(WCHAR) )) + return 0; + hwnd = wow_handlers32.create_window( (CREATESTRUCTW *)&cs, bufferW, + HINSTANCE_32(instance), 0 ); + } + else + { + if (!GlobalGetAtomNameA( LOWORD(className), buffer, sizeof(buffer) )) + { + ERR( "bad atom %x\n", LOWORD(className)); + return 0; + } + cs.lpszClass = buffer; + hwnd = wow_handlers32.create_window( (CREATESTRUCTW *)&cs, (LPCWSTR)className, + HINSTANCE_32(instance), 0 ); + } + return HWND_16( hwnd ); +} + + +/*********************************************************************** * button_proc16 */ static LRESULT button_proc16( HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam, BOOL unicode ) diff --git a/dlls/user32/win.c b/dlls/user32/win.c index 9b2cc89..8f43e92 100644 --- a/dlls/user32/win.c +++ b/dlls/user32/win.c @@ -27,9 +27,6 @@ #include <string.h> #include "windef.h" #include "winbase.h" -#include "wine/winbase16.h" -#include "wine/winuser16.h" -#include "wownt32.h" #include "wine/server.h" #include "wine/unicode.h" #include "win.h" @@ -884,7 +881,7 @@ void WIN_DestroyThreadWindows( HWND hwnd ) */ static void WIN_FixCoordinates( CREATESTRUCTW *cs, INT *sw) { -#define IS_DEFAULT(x) ((x) == CW_USEDEFAULT || (x) == CW_USEDEFAULT16) +#define IS_DEFAULT(x) ((x) == CW_USEDEFAULT || (x) == (SHORT)0x8000) POINT pos[2]; if (cs->dwExStyle & WS_EX_MDICHILD) @@ -1072,7 +1069,7 @@ static void dump_window_styles( DWORD style, DWORD exstyle ) * * Implementation of CreateWindowEx(). */ -static HWND WIN_CreateWindowEx( CREATESTRUCTW *cs, LPCWSTR className, HINSTANCE module, UINT flags ) +HWND WIN_CreateWindowEx( CREATESTRUCTW *cs, LPCWSTR className, HINSTANCE module, UINT flags ) { INT cx, cy, style, sw = SW_SHOW; LRESULT result; @@ -1440,83 +1437,6 @@ failed: /*********************************************************************** - * CreateWindow (USER.41) - */ -HWND16 WINAPI CreateWindow16( LPCSTR className, LPCSTR windowName, - DWORD style, INT16 x, INT16 y, INT16 width, - INT16 height, HWND16 parent, HMENU16 menu, - HINSTANCE16 instance, LPVOID data ) -{ - return CreateWindowEx16( 0, className, windowName, style, - x, y, width, height, parent, menu, instance, data ); -} - - -/*********************************************************************** - * CreateWindowEx (USER.452) - */ -HWND16 WINAPI CreateWindowEx16( DWORD exStyle, LPCSTR className, - LPCSTR windowName, DWORD style, INT16 x, - INT16 y, INT16 width, INT16 height, - HWND16 parent, HMENU16 menu, - HINSTANCE16 instance, LPVOID data ) -{ - CREATESTRUCTA cs; - char buffer[256]; - - /* Fix the coordinates */ - - cs.x = (x == CW_USEDEFAULT16) ? CW_USEDEFAULT : (INT)x; - cs.y = (y == CW_USEDEFAULT16) ? CW_USEDEFAULT : (INT)y; - cs.cx = (width == CW_USEDEFAULT16) ? CW_USEDEFAULT : (INT)width; - cs.cy = (height == CW_USEDEFAULT16) ? CW_USEDEFAULT : (INT)height; - - /* Create the window */ - - cs.lpCreateParams = data; - cs.hInstance = HINSTANCE_32(instance); - cs.hMenu = HMENU_32(menu); - cs.hwndParent = WIN_Handle32( parent ); - cs.style = style; - cs.lpszName = windowName; - cs.lpszClass = className; - cs.dwExStyle = exStyle; - - /* map to module handle */ - if (instance) instance = GetExePtr( instance ); - - /* load the menu */ - if (!menu && (style & (WS_CHILD | WS_POPUP)) != WS_CHILD) - { - WNDCLASSA class; - - if (GetClassInfoA( HINSTANCE_32(instance), className, &class )) - cs.hMenu = HMENU_32( LoadMenu16( instance, class.lpszMenuName )); - } - - if (!IS_INTRESOURCE(className)) - { - WCHAR bufferW[256]; - - if (!MultiByteToWideChar( CP_ACP, 0, className, -1, bufferW, sizeof(bufferW)/sizeof(WCHAR) )) - return 0; - return HWND_16( WIN_CreateWindowEx( (CREATESTRUCTW *)&cs, bufferW, HINSTANCE_32(instance), 0 )); - } - else - { - if (!GlobalGetAtomNameA( LOWORD(className), buffer, sizeof(buffer) )) - { - ERR( "bad atom %x\n", LOWORD(className)); - return 0; - } - cs.lpszClass = buffer; - return HWND_16( WIN_CreateWindowEx( (CREATESTRUCTW *)&cs, (LPCWSTR)className, - HINSTANCE_32(instance), 0 )); - } -} - - -/*********************************************************************** * CreateWindowExA (USER32.@) */ HWND WINAPI CreateWindowExA( DWORD exStyle, LPCSTR className, diff --git a/dlls/user32/win.h b/dlls/user32/win.h index 743c67e..ab37e3e 100644 --- a/dlls/user32/win.h +++ b/dlls/user32/win.h @@ -87,6 +87,7 @@ extern ULONG WIN_SetStyle( HWND hwnd, ULONG set_bits, ULONG clear_bits ) DECLSPE extern BOOL WIN_GetRectangles( HWND hwnd, RECT *rectWindow, RECT *rectClient ) DECLSPEC_HIDDEN; extern LRESULT WIN_DestroyWindow( HWND hwnd ) DECLSPEC_HIDDEN; extern void WIN_DestroyThreadWindows( HWND hwnd ) DECLSPEC_HIDDEN; +extern HWND WIN_CreateWindowEx( CREATESTRUCTW *cs, LPCWSTR className, HINSTANCE module, UINT flags ) DECLSPEC_HIDDEN; extern BOOL WIN_IsWindowDrawable( HWND hwnd, BOOL ) DECLSPEC_HIDDEN; extern HWND *WIN_ListChildren( HWND hwnd ) DECLSPEC_HIDDEN; extern LONG_PTR WIN_SetWindowLong( HWND hwnd, INT offset, UINT size, LONG_PTR newval, BOOL unicode ) DECLSPEC_HIDDEN; diff --git a/dlls/user32/winproc.c b/dlls/user32/winproc.c index 5bfd407..bd30b49 100644 --- a/dlls/user32/winproc.c +++ b/dlls/user32/winproc.c @@ -1135,6 +1135,7 @@ void WINAPI UserRegisterWowHandlers( const struct wow_handlers16 *new, struct wo orig->mdiclient_proc = MDIClientWndProc_common; orig->scrollbar_proc = ScrollBarWndProc_common; orig->static_proc = StaticWndProc_common; + orig->create_window = WIN_CreateWindowEx; orig->alloc_winproc = WINPROC_AllocProc; wow_handlers = *new; diff --git a/dlls/user32/wnd16.c b/dlls/user32/wnd16.c index f3f7e67..4c2f2f2 100644 --- a/dlls/user32/wnd16.c +++ b/dlls/user32/wnd16.c @@ -305,6 +305,19 @@ BOOL16 WINAPI EndPaint16( HWND16 hwnd, const PAINTSTRUCT16* lps ) } +/*********************************************************************** + * CreateWindow (USER.41) + */ +HWND16 WINAPI CreateWindow16( LPCSTR className, LPCSTR windowName, + DWORD style, INT16 x, INT16 y, INT16 width, + INT16 height, HWND16 parent, HMENU16 menu, + HINSTANCE16 instance, LPVOID data ) +{ + return CreateWindowEx16( 0, className, windowName, style, + x, y, width, height, parent, menu, instance, data ); +} + + /************************************************************************** * ShowWindow (USER.42) */

1 0

Alexandre Julliard : user32: Move loading of the 16-bit window menu to CreateWindowEx16.
by Alexandre Julliard 17 Dec '09

17 Dec '09

Module: wine Branch: master Commit: bfcfe437e2eebce2126b3cfb511cd880cc9529ac URL: http://source.winehq.org/git/wine.git/?a=commit;h=bfcfe437e2eebce2126b3cfb5… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Dec 17 14:48:06 2009 +0100 user32: Move loading of the 16-bit window menu to CreateWindowEx16. --- dlls/user32/win.c | 17 +++++++++++------ 1 files changed, 11 insertions(+), 6 deletions(-) diff --git a/dlls/user32/win.c b/dlls/user32/win.c index 51cc743..18c2332 100644 --- a/dlls/user32/win.c +++ b/dlls/user32/win.c @@ -1282,14 +1282,10 @@ static HWND WIN_CreateWindowEx( CREATESTRUCTA *cs, LPCWSTR className, HINSTANCE } else { - LPCSTR menuName = (LPCSTR)GetClassLongPtrA( hwnd, GCLP_MENUNAME ); + LPCWSTR menuName = (LPCWSTR)GetClassLongPtrW( hwnd, GCLP_MENUNAME ); if (menuName) { - if (!cs->hInstance || HIWORD(cs->hInstance)) - cs->hMenu = LoadMenuA(cs->hInstance,menuName); - else - cs->hMenu = HMENU_32(LoadMenu16(HINSTANCE_16(cs->hInstance),menuName)); - + cs->hMenu = LoadMenuW( cs->hInstance, menuName ); if (cs->hMenu) MENU_SetMenu( hwnd, cs->hMenu ); } } @@ -1489,6 +1485,15 @@ HWND16 WINAPI CreateWindowEx16( DWORD exStyle, LPCSTR className, /* map to module handle */ if (instance) instance = GetExePtr( instance ); + /* load the menu */ + if (!menu && (style & (WS_CHILD | WS_POPUP)) != WS_CHILD) + { + WNDCLASSA class; + + if (GetClassInfoA( HINSTANCE_32(instance), className, &class )) + cs.hMenu = HMENU_32( LoadMenu16( instance, class.lpszMenuName )); + } + if (!IS_INTRESOURCE(className)) { WCHAR bufferW[256];

1 0

Paul Vriens : gdiplus/tests: Fix a test failure on XP/W2K3.
by Alexandre Julliard 17 Dec '09

17 Dec '09

Module: wine Branch: master Commit: 60ca760eddf560908eeeff157a226de69c8bef9d URL: http://source.winehq.org/git/wine.git/?a=commit;h=60ca760eddf560908eeeff157… Author: Paul Vriens <Paul.Vriens.Wine(a)gmail.com> Date: Thu Dec 17 11:08:13 2009 +0100 gdiplus/tests: Fix a test failure on XP/W2K3. --- dlls/gdiplus/tests/image.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/dlls/gdiplus/tests/image.c b/dlls/gdiplus/tests/image.c index 486a679..ebdd3f9 100644 --- a/dlls/gdiplus/tests/image.c +++ b/dlls/gdiplus/tests/image.c @@ -529,6 +529,7 @@ static void test_GdipCreateBitmapFromHBITMAP(void) LogPal = GdipAlloc(sizeof(LOGPALETTE)); ok(LogPal != NULL, "unable to allocate LOGPALETTE\n"); LogPal->palVersion = 0x300; + LogPal->palNumEntries = 1; hpal = CreatePalette(LogPal); ok(hpal != NULL, "CreatePalette failed\n"); GdipFree(LogPal);

1 0

Alexandre Julliard : mapi32: Fix a heap buffer overflow.
by Alexandre Julliard 17 Dec '09

17 Dec '09

Module: wine Branch: master Commit: 4e7389e44bcdba1bc0666313bf65128c0b18012a URL: http://source.winehq.org/git/wine.git/?a=commit;h=4e7389e44bcdba1bc0666313b… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Dec 17 14:12:30 2009 +0100 mapi32: Fix a heap buffer overflow. --- dlls/mapi32/util.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/mapi32/util.c b/dlls/mapi32/util.c index 71a5e1a..90cfa6e 100644 --- a/dlls/mapi32/util.c +++ b/dlls/mapi32/util.c @@ -1037,7 +1037,7 @@ void load_mapi_providers(void) TRACE("appName: %s\n", debugstr_w(appName)); appKey = HeapAlloc(GetProcessHeap(), 0, sizeof(WCHAR) * (lstrlenW(regkey_mail) + - lstrlenW(regkey_backslash) + lstrlenW(appName))); + lstrlenW(regkey_backslash) + lstrlenW(appName) + 1)); if (!appKey) goto cleanUp;

1 0