December 2008 - Wine-Commits

Vincent Povirk : shell32: Display error messages in ShellExecuteEx.
by Alexandre Julliard Dec. 19, 2008

Dec. 19, 2008

Module: wine Branch: master Commit: 5c24f70c8bcee41e7a4dd3c0142d170b2042f59c URL: http://source.winehq.org/git/wine.git/?a=commit;h=5c24f70c8bcee41e7a4dd3c01… Author: Vincent Povirk <vincent(a)codeweavers.com> Date: Mon Dec 15 10:10:07 2008 -0600 shell32: Display error messages in ShellExecuteEx. --- dlls/shell32/shlexec.c | 17 ++++++++++++++++- 1 files changed, 16 insertions(+), 1 deletions(-) diff --git a/dlls/shell32/shlexec.c b/dlls/shell32/shlexec.c index 4dc3f12..8a65636 100644 --- a/dlls/shell32/shlexec.c +++ b/dlls/shell32/shlexec.c @@ -1447,6 +1447,16 @@ static UINT_PTR SHELL_execute_url( LPCWSTR lpFile, LPCWSTR wFile, LPCWSTR wcmd, return retval; } +static void do_error_dialog( UINT_PTR retval, HWND hwnd ) +{ + WCHAR msg[2048]; + int error_code=GetLastError(); + + FormatMessageW(FORMAT_MESSAGE_FROM_SYSTEM, NULL, error_code, 0, msg, sizeof(msg)/sizeof(WCHAR), NULL); + + MessageBoxW(hwnd, msg, NULL, MB_ICONERROR); +} + /************************************************************************* * SHELL_execute [Internal] */ @@ -1458,7 +1468,7 @@ BOOL SHELL_execute( LPSHELLEXECUTEINFOW sei, SHELL_ExecuteW32 execfunc ) static const WCHAR wHttp[] = {'h','t','t','p',':','/','/',0}; static const DWORD unsupportedFlags = SEE_MASK_INVOKEIDLIST | SEE_MASK_ICON | SEE_MASK_HOTKEY | - SEE_MASK_CONNECTNETDRV | SEE_MASK_FLAG_DDEWAIT | SEE_MASK_FLAG_NO_UI | + SEE_MASK_CONNECTNETDRV | SEE_MASK_FLAG_DDEWAIT | SEE_MASK_UNICODE | SEE_MASK_ASYNCOK | SEE_MASK_HMONITOR; WCHAR parametersBuffer[1024], dirBuffer[MAX_PATH], wcmdBuffer[1024]; @@ -1592,6 +1602,8 @@ BOOL SHELL_execute( LPSHELLEXECUTEINFOW sei, SHELL_ExecuteW32 execfunc ) { retval = SHELL_execute_class( wszApplicationName, &sei_tmp, sei, execfunc ); + if (retval <= 32 && !(sei_tmp.fMask & SEE_MASK_FLAG_NO_UI)) + do_error_dialog(retval, sei_tmp.hwnd); HeapFree(GetProcessHeap(), 0, wszApplicationName); if (wszParameters != parametersBuffer) HeapFree(GetProcessHeap(), 0, wszParameters); @@ -1815,6 +1827,9 @@ BOOL SHELL_execute( LPSHELLEXECUTEINFOW sei, SHELL_ExecuteW32 execfunc ) HeapFree(GetProcessHeap(), 0, wcmd); sei->hInstApp = (HINSTANCE)(retval > 32 ? 33 : retval); + + if (retval <= 32 && !(sei_tmp.fMask & SEE_MASK_FLAG_NO_UI)) + do_error_dialog(retval, sei_tmp.hwnd); return retval > 32; }

1 0

Vincent Povirk : shell32: Use ShellExecuteEx instead of ShellExecute in the run dialog.
by Alexandre Julliard Dec. 19, 2008

Dec. 19, 2008

Module: wine Branch: master Commit: 47298e25a1d22711ed5019ca5270818cee5759d4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=47298e25a1d22711ed5019ca5… Author: Vincent Povirk <vincent(a)codeweavers.com> Date: Thu Dec 18 15:18:47 2008 -0600 shell32: Use ShellExecuteEx instead of ShellExecute in the run dialog. --- dlls/shell32/dialogs.c | 16 ++++++++++++---- 1 files changed, 12 insertions(+), 4 deletions(-) diff --git a/dlls/shell32/dialogs.c b/dlls/shell32/dialogs.c index 0fdc951..a7f056a 100644 --- a/dlls/shell32/dialogs.c +++ b/dlls/shell32/dialogs.c @@ -211,19 +211,27 @@ static INT_PTR CALLBACK RunDlgProc (HWND hwnd, UINT message, WPARAM wParam, LPAR if ((ic = GetWindowTextLengthW (htxt))) { WCHAR *psz, *parent=NULL ; - LPCWSTR working_dir ; + SHELLEXECUTEINFOW sei ; + + ZeroMemory (&sei, sizeof(sei)) ; + sei.cbSize = sizeof(sei) ; psz = HeapAlloc( GetProcessHeap(), 0, (ic + 1)*sizeof(WCHAR) ); GetWindowTextW (htxt, psz, ic + 1) ; /* according to http://www.codeproject.com/KB/shell/runfiledlg.aspx we should send a * WM_NOTIFY before execution */ + sei.hwnd = hwnd; + sei.nShow = SW_SHOWNORMAL; + sei.lpFile = psz; + sei.fMask = SEE_MASK_FLAG_NO_UI; + if (prfdp->lpstrDirectory) - working_dir = prfdp->lpstrDirectory; + sei.lpDirectory = prfdp->lpstrDirectory; else - working_dir = parent = RunDlg_GetParentDir(psz); + sei.lpDirectory = parent = RunDlg_GetParentDir(sei.lpFile); - if (ShellExecuteW(hwnd, NULL, psz, NULL, working_dir, SW_SHOWNORMAL) < (HINSTANCE)33) + if (ShellExecuteExW( &sei ) < 33) { char *pszSysMsg = NULL ; char szMsg[256];

1 0

Vincent Povirk : shell32: Fix a memory leak in the run dialog code.
by Alexandre Julliard Dec. 19, 2008

Dec. 19, 2008

Module: wine Branch: master Commit: 061fb815d6fa8180898d770818478173bd10e70a URL: http://source.winehq.org/git/wine.git/?a=commit;h=061fb815d6fa8180898d77081… Author: Vincent Povirk <vincent(a)codeweavers.com> Date: Thu Dec 18 15:52:52 2008 -0600 shell32: Fix a memory leak in the run dialog code. --- dlls/shell32/dialogs.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/dlls/shell32/dialogs.c b/dlls/shell32/dialogs.c index 844b8f4..0fdc951 100644 --- a/dlls/shell32/dialogs.c +++ b/dlls/shell32/dialogs.c @@ -240,6 +240,7 @@ static INT_PTR CALLBACK RunDlgProc (HWND hwnd, UINT message, WPARAM wParam, LPAR MessageBoxA (hwnd, szMsg, "Nix", MB_OK | MB_ICONEXCLAMATION) ; HeapFree(GetProcessHeap(), 0, psz); + HeapFree(GetProcessHeap(), 0, parent); SendMessageA (htxt, CB_SETEDITSEL, 0, MAKELPARAM (0, -1)) ; return TRUE ; }

1 0

Vincent Povirk : shell32: Use the NO_UI flag in ShellExecute.
by Alexandre Julliard Dec. 19, 2008

Dec. 19, 2008

Module: wine Branch: master Commit: 0a2ffa08f556805f9c14be98b81f3bd9382f77e6 URL: http://source.winehq.org/git/wine.git/?a=commit;h=0a2ffa08f556805f9c14be98b… Author: Vincent Povirk <vincent(a)codeweavers.com> Date: Thu Dec 18 15:01:58 2008 -0600 shell32: Use the NO_UI flag in ShellExecute. In my testing on Windows, I wasn't able to make ShellExecute display any error dialogs. --- dlls/shell32/shlexec.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/shell32/shlexec.c b/dlls/shell32/shlexec.c index 515bb48..4dc3f12 100644 --- a/dlls/shell32/shlexec.c +++ b/dlls/shell32/shlexec.c @@ -1831,7 +1831,7 @@ HINSTANCE WINAPI ShellExecuteA(HWND hWnd, LPCSTR lpOperation,LPCSTR lpFile, debugstr_a(lpParameters), debugstr_a(lpDirectory), iShowCmd); sei.cbSize = sizeof(sei); - sei.fMask = 0; + sei.fMask = SEE_MASK_FLAG_NO_UI; sei.hwnd = hWnd; sei.lpVerb = lpOperation; sei.lpFile = lpFile; @@ -1917,7 +1917,7 @@ HINSTANCE WINAPI ShellExecuteW(HWND hwnd, LPCWSTR lpOperation, LPCWSTR lpFile, TRACE("\n"); sei.cbSize = sizeof(sei); - sei.fMask = 0; + sei.fMask = SEE_MASK_FLAG_NO_UI; sei.hwnd = hwnd; sei.lpVerb = lpOperation; sei.lpFile = lpFile;

1 0

Juan Lang : cryptui: Refresh tabs in response to cert properties changes being applied.
by Alexandre Julliard Dec. 19, 2008

Dec. 19, 2008

Module: wine Branch: master Commit: 96a95bba25f2a738118ed2384e18de9d34ff4560 URL: http://source.winehq.org/git/wine.git/?a=commit;h=96a95bba25f2a738118ed2384… Author: Juan Lang <juan.lang(a)gmail.com> Date: Thu Dec 18 15:34:25 2008 -0800 cryptui: Refresh tabs in response to cert properties changes being applied. --- dlls/cryptui/main.c | 31 ++++++++++++++++++++++++++++++- 1 files changed, 30 insertions(+), 1 deletions(-) diff --git a/dlls/cryptui/main.c b/dlls/cryptui/main.c index c90bd34..83fffb1 100644 --- a/dlls/cryptui/main.c +++ b/dlls/cryptui/main.c @@ -2004,6 +2004,15 @@ static void set_cert_string_property(PCCERT_CONTEXT cert, DWORD prop, CertSetCertificateContextProperty(cert, prop, 0, NULL); } +#define WM_REFRESH_VIEW WM_USER + 0 + +static BOOL CALLBACK refresh_propsheet_pages(HWND hwnd, LPARAM lParam) +{ + if ((GetClassLongW(hwnd, GCW_ATOM) == WC_DIALOG)) + SendMessageW(hwnd, WM_REFRESH_VIEW, 0, 0); + return TRUE; +} + #define MAX_FRIENDLY_NAME 40 #define MAX_DESCRIPTION 255 @@ -2070,6 +2079,7 @@ static void apply_general_changes(HWND hwnd) CertSetEnhancedKeyUsage(data->pCertViewInfo->pCertContext, &usage); HeapFree(GetProcessHeap(), 0, usage.rgpszUsageIdentifier); } + EnumChildWindows(GetParent(GetParent(hwnd)), refresh_propsheet_pages, 0); if (data->pfPropertiesChanged) *data->pfPropertiesChanged = TRUE; } @@ -2330,6 +2340,9 @@ static LRESULT CALLBACK detail_dlg_proc(HWND hwnd, UINT msg, WPARAM wp, break; } break; + case WM_REFRESH_VIEW: + refresh_details_view(hwnd); + break; } return 0; } @@ -2601,6 +2614,7 @@ static LRESULT CALLBACK hierarchy_dlg_proc(HWND hwnd, UINT msg, WPARAM wp, PROPSHEETPAGEW *page; struct hierarchy_data *data; LRESULT ret = 0; + HWND tree = GetDlgItem(hwnd, IDC_CERTPATH); TRACE("(%p, %08x, %08lx, %08lx)\n", hwnd, msg, wp, lp); @@ -2621,7 +2635,6 @@ static LRESULT CALLBACK hierarchy_dlg_proc(HWND hwnd, UINT msg, WPARAM wp, case TVN_SELCHANGEDW: { NMTREEVIEWW *nm = (NMTREEVIEWW*)lp; - HWND tree = GetDlgItem(hwnd, IDC_CERTPATH); DWORD selection; CRYPT_PROVIDER_SGNR *provSigner; @@ -2653,6 +2666,22 @@ static LRESULT CALLBACK hierarchy_dlg_proc(HWND hwnd, UINT msg, WPARAM wp, break; } break; + case WM_REFRESH_VIEW: + { + TVITEMW item; + + /* Get hierarchy data */ + memset(&item, 0, sizeof(item)); + item.mask = TVIF_HANDLE | TVIF_PARAM; + item.hItem = (HTREEITEM)SendMessageW(tree, TVM_GETNEXTITEM, TVGN_ROOT, + (LPARAM)NULL); + data = get_hierarchy_data_from_tree_item(tree, item.hItem); + /* Delete the contents of the tree */ + SendMessageW(tree, TVM_DELETEITEM, 0, (LPARAM)TVI_ROOT); + /* Reinitialize the tree */ + show_cert_hierarchy(hwnd, data); + break; + } } return ret; }

1 0

Juan Lang : cryptui: Apply changes to enabled usages in response to applying the edit cert properties dialog .
by Alexandre Julliard Dec. 19, 2008

Dec. 19, 2008

Module: wine Branch: master Commit: 11e770169677b7d92d7440617f27a0b4356d1ef0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=11e770169677b7d92d7440617… Author: Juan Lang <juan.lang(a)gmail.com> Date: Thu Dec 18 14:07:33 2008 -0800 cryptui: Apply changes to enabled usages in response to applying the edit cert properties dialog. --- dlls/cryptui/main.c | 50 +++++++++++++++++++++++++++++++++++++++++++++++++- 1 files changed, 49 insertions(+), 1 deletions(-) diff --git a/dlls/cryptui/main.c b/dlls/cryptui/main.c index 2c0d453..c90bd34 100644 --- a/dlls/cryptui/main.c +++ b/dlls/cryptui/main.c @@ -2021,7 +2021,55 @@ static void apply_general_changes(HWND hwnd) sizeof(buf) / sizeof(buf[0]), (LPARAM)buf); set_cert_string_property(data->pCertViewInfo->pCertContext, CERT_DESCRIPTION_PROP_ID, buf); - FIXME("apply usage state\n"); + if (IsDlgButtonChecked(hwnd, IDC_ENABLE_ALL_PURPOSES)) + { + /* Setting a NULL usage removes the enhanced key usage property. */ + CertSetEnhancedKeyUsage(data->pCertViewInfo->pCertContext, NULL); + } + else if (IsDlgButtonChecked(hwnd, IDC_DISABLE_ALL_PURPOSES)) + { + CERT_ENHKEY_USAGE usage = { 0, NULL }; + + CertSetEnhancedKeyUsage(data->pCertViewInfo->pCertContext, &usage); + } + else if (IsDlgButtonChecked(hwnd, IDC_ENABLE_SELECTED_PURPOSES)) + { + HWND lv = GetDlgItem(hwnd, IDC_CERTIFICATE_USAGES); + CERT_ENHKEY_USAGE usage = { 0, NULL }; + int purposes = SendMessageW(lv, LVM_GETITEMCOUNT, 0, 0), i; + LVITEMW item; + + item.mask = LVIF_STATE | LVIF_PARAM; + item.iSubItem = 0; + item.stateMask = LVIS_STATEIMAGEMASK; + for (i = 0; i < purposes; i++) + { + item.iItem = i; + if (SendMessageW(lv, LVM_GETITEMW, 0, (LPARAM)&item)) + { + int state = item.state >> 12; + + if (state == CheckBitmapIndexChecked) + { + CRYPT_OID_INFO *info = (CRYPT_OID_INFO *)item.lParam; + + if (usage.cUsageIdentifier) + usage.rgpszUsageIdentifier = + HeapReAlloc(GetProcessHeap(), 0, + usage.rgpszUsageIdentifier, + (usage.cUsageIdentifier + 1) * sizeof(LPSTR)); + else + usage.rgpszUsageIdentifier = + HeapAlloc(GetProcessHeap(), 0, sizeof(LPSTR)); + if (usage.rgpszUsageIdentifier) + usage.rgpszUsageIdentifier[usage.cUsageIdentifier++] = + (LPSTR)info->pszOID; + } + } + } + CertSetEnhancedKeyUsage(data->pCertViewInfo->pCertContext, &usage); + HeapFree(GetProcessHeap(), 0, usage.rgpszUsageIdentifier); + } if (data->pfPropertiesChanged) *data->pfPropertiesChanged = TRUE; }

1 0

Juan Lang : cryptui: Set cert friendly name and description in response to applying the edit cert properties dialog .
by Alexandre Julliard Dec. 19, 2008

Dec. 19, 2008

Module: wine Branch: master Commit: 2d2aafbcfa3e1e7b90fc0fd0947b7ee61eb0e942 URL: http://source.winehq.org/git/wine.git/?a=commit;h=2d2aafbcfa3e1e7b90fc0fd09… Author: Juan Lang <juan.lang(a)gmail.com> Date: Thu Dec 18 14:06:53 2008 -0800 cryptui: Set cert friendly name and description in response to applying the edit cert properties dialog. --- dlls/cryptui/main.c | 37 +++++++++++++++++++++++++++++++++++++ 1 files changed, 37 insertions(+), 0 deletions(-) diff --git a/dlls/cryptui/main.c b/dlls/cryptui/main.c index 275526b..2c0d453 100644 --- a/dlls/cryptui/main.c +++ b/dlls/cryptui/main.c @@ -1989,9 +1989,43 @@ static void toggle_usage(HWND hwnd, int iItem) } } +static void set_cert_string_property(PCCERT_CONTEXT cert, DWORD prop, + LPWSTR str) +{ + if (str && strlenW(str)) + { + CRYPT_DATA_BLOB blob; + + blob.pbData = (BYTE *)str; + blob.cbData = (strlenW(str) + 1) * sizeof(WCHAR); + CertSetCertificateContextProperty(cert, prop, 0, &blob); + } + else + CertSetCertificateContextProperty(cert, prop, 0, NULL); +} + #define MAX_FRIENDLY_NAME 40 #define MAX_DESCRIPTION 255 +static void apply_general_changes(HWND hwnd) +{ + WCHAR buf[MAX_DESCRIPTION + 1]; + struct detail_data *data = + (struct detail_data *)GetWindowLongPtrW(hwnd, DWLP_USER); + + SendMessageW(GetDlgItem(hwnd, IDC_FRIENDLY_NAME), WM_GETTEXT, + sizeof(buf) / sizeof(buf[0]), (LPARAM)buf); + set_cert_string_property(data->pCertViewInfo->pCertContext, + CERT_FRIENDLY_NAME_PROP_ID, buf); + SendMessageW(GetDlgItem(hwnd, IDC_DESCRIPTION), WM_GETTEXT, + sizeof(buf) / sizeof(buf[0]), (LPARAM)buf); + set_cert_string_property(data->pCertViewInfo->pCertContext, + CERT_DESCRIPTION_PROP_ID, buf); + FIXME("apply usage state\n"); + if (data->pfPropertiesChanged) + *data->pfPropertiesChanged = TRUE; +} + static LRESULT CALLBACK cert_properties_general_dlg_proc(HWND hwnd, UINT msg, WPARAM wp, LPARAM lp) { @@ -2028,6 +2062,9 @@ static LRESULT CALLBACK cert_properties_general_dlg_proc(HWND hwnd, UINT msg, toggle_usage(hwnd, nm->iItem); SendMessageW(GetParent(hwnd), PSM_CHANGED, (WPARAM)hwnd, 0); break; + case PSN_APPLY: + apply_general_changes(hwnd); + break; } break; }

1 0

Juan Lang : cryptui: Toggle a usage when it's clicked.
by Alexandre Julliard Dec. 19, 2008

Dec. 19, 2008

Module: wine Branch: master Commit: ef8d6d8a966b7829d7969a5fd2b425873d175752 URL: http://source.winehq.org/git/wine.git/?a=commit;h=ef8d6d8a966b7829d7969a5fd… Author: Juan Lang <juan.lang(a)gmail.com> Date: Thu Dec 18 14:06:09 2008 -0800 cryptui: Toggle a usage when it's clicked. --- dlls/cryptui/main.c | 37 +++++++++++++++++++++++++++++++++++++ 1 files changed, 37 insertions(+), 0 deletions(-) diff --git a/dlls/cryptui/main.c b/dlls/cryptui/main.c index 7b70578..275526b 100644 --- a/dlls/cryptui/main.c +++ b/dlls/cryptui/main.c @@ -1967,6 +1967,28 @@ static void set_general_cert_properties(HWND hwnd, struct detail_data *data) show_cert_usages(hwnd, data); } +static void toggle_usage(HWND hwnd, int iItem) +{ + LVITEMW item; + int res; + HWND lv = GetDlgItem(hwnd, IDC_CERTIFICATE_USAGES); + + item.mask = LVIF_STATE; + item.iItem = iItem; + item.iSubItem = 0; + item.stateMask = LVIS_STATEIMAGEMASK; + res = SendMessageW(lv, LVM_GETITEMW, 0, (LPARAM)&item); + if (res) + { + int state = item.state >> 12; + + item.state = INDEXTOSTATEIMAGEMASK( + state == CheckBitmapIndexChecked ? CheckBitmapIndexUnchecked : + CheckBitmapIndexChecked); + SendMessageW(lv, LVM_SETITEMSTATE, iItem, (LPARAM)&item); + } +} + #define MAX_FRIENDLY_NAME 40 #define MAX_DESCRIPTION 255 @@ -1994,6 +2016,21 @@ static LRESULT CALLBACK cert_properties_general_dlg_proc(HWND hwnd, UINT msg, SetWindowLongPtrW(hwnd, DWLP_USER, (LPARAM)data); break; } + case WM_NOTIFY: + { + NMHDR *hdr = (NMHDR *)lp; + NMITEMACTIVATE *nm; + + switch (hdr->code) + { + case NM_CLICK: + nm = (NMITEMACTIVATE *)lp; + toggle_usage(hwnd, nm->iItem); + SendMessageW(GetParent(hwnd), PSM_CHANGED, (WPARAM)hwnd, 0); + break; + } + break; + } case WM_COMMAND: switch (HIWORD(wp)) {

1 0

Juan Lang : cryptui: Add an add purpose dialog.
by Alexandre Julliard Dec. 19, 2008

Dec. 19, 2008

Module: wine Branch: master Commit: 356596561c3ef3cf94fe86a01e512a0665c0b870 URL: http://source.winehq.org/git/wine.git/?a=commit;h=356596561c3ef3cf94fe86a01… Author: Juan Lang <juan.lang(a)gmail.com> Date: Thu Dec 18 14:04:30 2008 -0800 cryptui: Add an add purpose dialog. --- dlls/cryptui/cryptui_En.rc | 11 ++++++++ dlls/cryptui/cryptuires.h | 3 ++ dlls/cryptui/main.c | 59 ++++++++++++++++++++++++++++++++++++++++++++ 3 files changed, 73 insertions(+), 0 deletions(-) diff --git a/dlls/cryptui/cryptui_En.rc b/dlls/cryptui/cryptui_En.rc index 70ed0f1..5dcb98a 100644 --- a/dlls/cryptui/cryptui_En.rc +++ b/dlls/cryptui/cryptui_En.rc @@ -170,3 +170,14 @@ BEGIN 24,100,220,90 PUSHBUTTON "Add &Purpose...", IDC_ADD_PURPOSE,184,194,60,14 END + +IDD_ADD_CERT_PURPOSE DIALOG DISCARDABLE 0,0,200,68 +CAPTION "Add Purpose" +FONT 8, "MS Shell Dlg" +BEGIN + LTEXT "Add the object identifier (OID) for the certificate purpose you wish to add:", + stc1, 6,6,190,28 + EDITTEXT IDC_NEW_PURPOSE, 6,28,190,14, ES_AUTOVSCROLL|ES_MULTILINE|WS_TABSTOP|WS_VSCROLL + PUSHBUTTON "OK", IDOK, 33,48,60,14 + PUSHBUTTON "Cancel", IDCANCEL, 100,48,60,14 +END diff --git a/dlls/cryptui/cryptuires.h b/dlls/cryptui/cryptuires.h index eb26e15..a311531 100644 --- a/dlls/cryptui/cryptuires.h +++ b/dlls/cryptui/cryptuires.h @@ -89,6 +89,7 @@ #define IDD_HIERARCHY 102 #define IDD_USERNOTICE 103 #define IDD_CERT_PROPERTIES_GENERAL 104 +#define IDD_ADD_CERT_PURPOSE 105 #define IDB_SMALL_ICONS 200 #define IDB_CERT 201 @@ -126,4 +127,6 @@ #define IDC_CERTIFICATE_USAGES 2405 #define IDC_ADD_PURPOSE 2406 +#define IDC_NEW_PURPOSE 2500 + #endif /* ndef __CRYPTUIRES_H_ */ diff --git a/dlls/cryptui/main.c b/dlls/cryptui/main.c index dc1c807..e26bf55 100644 --- a/dlls/cryptui/main.c +++ b/dlls/cryptui/main.c @@ -1593,6 +1593,59 @@ static void add_purpose(HWND hwnd, LPCSTR oid) } } +#define MAX_PURPOSE 255 + +static LRESULT CALLBACK add_purpose_dlg_proc(HWND hwnd, UINT msg, + WPARAM wp, LPARAM lp) +{ + LRESULT ret = 0; + char buf[MAX_PURPOSE + 1]; + + switch (msg) + { + case WM_INITDIALOG: + SendMessageW(GetDlgItem(hwnd, IDC_NEW_PURPOSE), EM_SETLIMITTEXT, + MAX_PURPOSE, 0); + ShowScrollBar(GetDlgItem(hwnd, IDC_NEW_PURPOSE), SB_VERT, FALSE); + SetWindowLongPtrW(hwnd, DWLP_USER, lp); + break; + case WM_COMMAND: + switch (HIWORD(wp)) + { + case BN_CLICKED: + switch (LOWORD(wp)) + { + case IDOK: + SendMessageA(GetDlgItem(hwnd, IDC_NEW_PURPOSE), WM_GETTEXT, + sizeof(buf) / sizeof(buf[0]), (LPARAM)buf); + if (!buf[0]) + { + /* An empty purpose is the same as cancelling */ + EndDialog(hwnd, IDCANCEL); + ret = TRUE; + } + else + { + HWND parent = (HWND)GetWindowLongPtrW(hwnd, DWLP_USER); + + FIXME("validate %s\n", debugstr_a(buf)); + add_purpose(parent, buf); + EndDialog(hwnd, wp); + ret = TRUE; + } + break; + case IDCANCEL: + EndDialog(hwnd, wp); + ret = TRUE; + break; + } + break; + } + break; + } + return ret; +} + static WCHAR *get_cert_property_as_string(PCCERT_CONTEXT cert, DWORD prop) { WCHAR *name = NULL; @@ -1862,6 +1915,12 @@ static LRESULT CALLBACK cert_properties_general_dlg_proc(HWND hwnd, UINT msg, case BN_CLICKED: switch (LOWORD(wp)) { + case IDC_ADD_PURPOSE: + if (DialogBoxParamW(hInstance, + MAKEINTRESOURCEW(IDD_ADD_CERT_PURPOSE), hwnd, + add_purpose_dlg_proc, (LPARAM)hwnd) == IDOK) + SendMessageW(GetParent(hwnd), PSM_CHANGED, (WPARAM)hwnd, 0); + break; case IDC_ENABLE_ALL_PURPOSES: case IDC_DISABLE_ALL_PURPOSES: case IDC_ENABLE_SELECTED_PURPOSES:

1 0

Juan Lang : cryptui: Show/hide add purpose OID' s scrollbar depending on how much text it has.
by Alexandre Julliard Dec. 19, 2008

Dec. 19, 2008

Module: wine Branch: master Commit: a0ae811ecc06bfd9f6c1c8c338c1ee91816ad636 URL: http://source.winehq.org/git/wine.git/?a=commit;h=a0ae811ecc06bfd9f6c1c8c33… Author: Juan Lang <juan.lang(a)gmail.com> Date: Thu Dec 18 14:05:17 2008 -0800 cryptui: Show/hide add purpose OID's scrollbar depending on how much text it has. --- dlls/cryptui/main.c | 12 ++++++++++++ 1 files changed, 12 insertions(+), 0 deletions(-) diff --git a/dlls/cryptui/main.c b/dlls/cryptui/main.c index e26bf55..7bd835f 100644 --- a/dlls/cryptui/main.c +++ b/dlls/cryptui/main.c @@ -1612,6 +1612,18 @@ static LRESULT CALLBACK add_purpose_dlg_proc(HWND hwnd, UINT msg, case WM_COMMAND: switch (HIWORD(wp)) { + case EN_CHANGE: + if (LOWORD(wp) == IDC_NEW_PURPOSE) + { + /* Show/hide scroll bar on description depending on how much + * text it has. + */ + HWND description = GetDlgItem(hwnd, IDC_NEW_PURPOSE); + int lines = SendMessageW(description, EM_GETLINECOUNT, 0, 0); + + ShowScrollBar(description, SB_VERT, lines > 1); + } + break; case BN_CLICKED: switch (LOWORD(wp)) {

1 0