/[winpt]/trunk/Src/wptKeyManagerDlg.cpp

Diff of /trunk/Src/wptKeyManagerDlg.cpp

Parent Directory | Revision Log | View Patch Patch

-revision 123 by twoaday,
Wed Dec 14 09:01:45 2005 UTC
+revision 255 by twoaday,
Tue Aug  1 16:37:23 2006 UTC
 Line 1
  /* wptKeyManagerDlg.cpp - WinPT Key Manager
-  *      Copyright (C) 2000-2005 Timo Schulz
+  *      Copyright (C) 2000-2006 Timo Schulz
   *      Copyright (C) 2004 Andreas Jobs
   *
   * This file is part of WinPT.
 Line 41
  #include "wptKeyserver.h"
  #include "wptKeyEdit.h"
  #include "wptRegistry.h"
+ #include "wptUTF8.h"
  /* Name and ID of the separator window. */
- #define KM_SEPARATOR_ID                10000
+ #define KM_SEPARATOR_ID                 10000
- #define WINDOWCLASS_SEPARATOR_CHILD "WINPT_SEP_CHILD"
+ #define WINDOWCLASS_SEPARATOR_CHILD     "WINPT_SEP_CHILD"
  /* Virtual key codes. */
  #ifndef VK_F
-Line 62
+Line 63
  static subclass_s keylist_proc;
+ /* Handle to the global image list. */
  HIMAGELIST glob_imagelist;
- struct km_info {
+ struct km_info_s {
      /* Window positions */
      int pos_x, pos_y;
      int ypos_sep;
      int ypercent_sep;
+     /* Different kind of windows. */
+     HWND dlg;
      HWND hwnd_sep;
      HWND toolbar;
      HWND statbar;
-Line 77 
 struct km_info {
+Line 81 
 struct km_info {
      listview_ctrl_t lv;
      int             lv_idx;
      int keylist_sortby;
+     int magic;
+     unsigned int enable_groups:1;
  };
+ typedef struct km_info_s *km_info_t;
  /* Toolbar button structure. */
- struct mybuttons {
+ struct tb_button_s {
      long icon;
      long command;
      char *text;
  };
- struct mybuttons myb[] = {
+ /* The buttons in the toolbar. */
+ struct tb_button_s tbb[] = {
      {IMI_KEY_DELETE, ID_KEYMISC_DELETE, "Delete"},
      {IMI_KEY_PROPS,  ID_KEYMISC_PROPS,  "Properties"},
      {IMI_KEY_SIGN,   ID_KEYMISC_SIGN,   "Sign"},
-Line 95 
 struct mybuttons myb[] = {
+Line 104 
 struct mybuttons myb[] = {
  };
+ static void km_gui_import (struct km_info_s *kmi, int cmd_id, void *param);
+ void key_get_clip_info (const char *keyid, char *buf, size_t buflen);
  /* Subclass the keylist listview control to allow extended commands. */
  static BOOL CALLBACK
  keylist_subclass_proc (HWND dlg, UINT msg, WPARAM wparam, LPARAM lparam)
  {
      struct listview_ctrl_s lv;
+     struct km_info_s *kmi;
      switch (msg) {
+     case WM_DROPFILES:
+         kmi = (km_info_s *)keylist_proc.opaque;
+         km_gui_import (kmi, WM_DROPFILES, (void*)wparam);
+         break;
      case WM_LBUTTONDBLCLK:
          send_cmd_id (keylist_proc.dlg, ID_KEYMISC_PROPS);
          break;
-Line 136 
 keylist_subclass_proc (HWND dlg, UINT ms
+Line 155 
 keylist_subclass_proc (HWND dlg, UINT ms
              break;
          case VK_P:
              if (GetAsyncKeyState (VK_CONTROL))
-                 km_clip_import (keylist_proc.dlg);
+                 send_cmd_id (keylist_proc.dlg, ID_KEYMISC_PASTE);
              break;
          case VK_F:
-Line 154 
 keylist_subclass_proc (HWND dlg, UINT ms
+Line 173 
 keylist_subclass_proc (HWND dlg, UINT ms
  }
+ #define ico2idx(ico) imagelist_getindex ((ico))
+ /* Create a new toolbar window with a pre-defined list of buttons. */
  static HWND
- load_toolbar (HWND dlg, struct km_info *kmi)
+ create_toolbar (HWND dlg, struct km_info_s *kmi)
  {
      HWND tbwnd;
      TBSAVEPARAMS tbsp;
      TBBUTTON tb_buttons[] = {
-         /*{imagelist_getindex(IMI_EXIT),       ID_KEYMISC_QUIT,   TBSTATE_ENABLED, TBSTYLE_BUTTON, 0, 0L, 0},*/
+         {ico2idx (IMI_KEY_NEW),    ID_KEYMISC_KEYWIZARD, TBSTATE_ENABLED, TBSTYLE_BUTTON, {0}, 0L},
-         {imagelist_getindex(IMI_KEY_DELETE), ID_KEYMISC_DELETE, TBSTATE_ENABLED, TBSTYLE_BUTTON, {0}, 0L, 0},
+         {ico2idx (IMI_KEY_DELETE), ID_KEYMISC_DELETE, TBSTATE_ENABLED, TBSTYLE_BUTTON, {0}, 0L, 0},
-         {imagelist_getindex(IMI_KEY_PROPS),  ID_KEYMISC_PROPS,  TBSTATE_ENABLED, TBSTYLE_BUTTON, {0}, 0L, 0},
+         {ico2idx (IMI_KEY_PROPS),  ID_KEYMISC_PROPS,  TBSTATE_ENABLED, TBSTYLE_BUTTON, {0}, 0L, 0},
-         {imagelist_getindex(IMI_KEY_SIGN),   ID_KEYMISC_SIGN,   TBSTATE_ENABLED, TBSTYLE_BUTTON, {0}, 0L, 0},
+         {ico2idx (IMI_KEY_SIGN),   ID_KEYMISC_SIGN,   TBSTATE_ENABLED, TBSTYLE_BUTTON, {0}, 0L, 0},
+         {ico2idx (IMI_KEY_SEARCH), ID_KEYMISC_SENDRECV, TBSTATE_ENABLED, TBSTYLE_BUTTON, {0}, 0L, 0},
          {0,                                  0,                 0,               TBSTYLE_SEP,    {0}, 0L, 0},
-         {imagelist_getindex(IMI_KEY_IMPORT), ID_KEYMISC_IMPORT, TBSTATE_ENABLED, TBSTYLE_BUTTON, {0}, 0L, 0},
+         {ico2idx (IMI_KEY_FILE_IMPORT), ID_KEYMISC_IMPORT, TBSTATE_ENABLED, TBSTYLE_BUTTON, {0}, 0L, 0},
-         {imagelist_getindex(IMI_KEY_EXPORT), ID_KEYMISC_EXPORT, TBSTATE_ENABLED, TBSTYLE_BUTTON, {0}, 0L, 0},
+         {ico2idx (IMI_KEY_FILE_EXPORT), ID_KEYMISC_EXPORT, TBSTATE_ENABLED, TBSTYLE_BUTTON, {0}, 0L, 0},
-      };
+         {ico2idx (IMI_KEY_IMPORT), ID_KEYCTX_PASTE, TBSTATE_ENABLED, TBSTYLE_BUTTON, {0}, 0L, 0},
+         {ico2idx (IMI_KEY_EXPORT), ID_KEYCTX_COPY, TBSTATE_ENABLED, TBSTYLE_BUTTON, {0}, 0L, 0},
+     };
      tbwnd = CreateWindowEx (0, TOOLBARCLASSNAME, NULL,
                              WS_CHILD|TBSTYLE_TOOLTIPS|TBSTYLE_FLAT|CCS_ADJUSTABLE,
 , 0, 0, 0, dlg, (HMENU)IDR_WINPT_KMTB, glob_hinst, NULL);
-     if (tbwnd) {
+     if (!tbwnd)
-         SendMessage (tbwnd, TB_BUTTONSTRUCTSIZE, (WPARAM) sizeof(TBBUTTON), 0);
+         BUG (0);
-         SendMessage (tbwnd, TB_SETIMAGELIST, 0, (LPARAM)glob_imagelist);
+     SendMessage (tbwnd, TB_BUTTONSTRUCTSIZE, (WPARAM) sizeof(TBBUTTON), 0);
-                 SendMessage (tbwnd, TB_AUTOSIZE, 0, 0);
+     SendMessage (tbwnd, TB_SETIMAGELIST, 0, (LPARAM)glob_imagelist);
-         ShowWindow (tbwnd, SW_SHOW);
+     SendMessage (tbwnd, TB_AUTOSIZE, 0, 0);
+     ShowWindow (tbwnd, SW_SHOW);
-         /* Restore toolbar */
-         tbsp.hkr = HKEY_CURRENT_USER;
+     /* Restore toolbar */
-         tbsp.pszSubKey = "Software\\WinPT";
+     tbsp.hkr = HKEY_CURRENT_USER;
-         tbsp.pszValueName = "KM_toolbar";
+     tbsp.pszSubKey = "Software\\WinPT";
-         if (SendMessage(tbwnd, TB_SAVERESTORE, FALSE, (LPARAM)&tbsp ) == 0)
+     tbsp.pszValueName = "KM_toolbar";
-             SendMessage (tbwnd, TB_ADDBUTTONS, sizeof(tb_buttons) / sizeof(tb_buttons[0]),
+     if (SendMessage (tbwnd, TB_SAVERESTORE, FALSE, (LPARAM)&tbsp) == 0)
-                          (LONG)&tb_buttons[0]);
+         SendMessage (tbwnd, TB_ADDBUTTONS,
-      }
+                      sizeof (tb_buttons) / sizeof (tb_buttons[0]),
-      return tbwnd;
+                      (LONG)&tb_buttons[0]);
+     return tbwnd;
+ }
+ /* Restore the width of the columns from the registry.
+    If no bitstring was found, the default size is used. */
+ int
+ restore_column_info (struct km_info_s *kmi)
+ {
+     WORD *buf;
+     HKEY root;
+     DWORD type, size, i;
+     LONG ec;
+     size = kmi->lv->cols*sizeof (WORD);
+     ec = RegOpenKeyEx (HKEY_CURRENT_USER, "Software\\WinPT", 0,
+                        KEY_ALL_ACCESS, &root);
+     if (ec != ERROR_SUCCESS)
+         return -1;
+     buf = new WORD[size/2];
+     if (!buf)
+         BUG (NULL);
+     ec = RegQueryValueEx (root, "KMColumnSize", NULL, &type,
+                           (BYTE*)buf, &size);
+     if (ec != ERROR_SUCCESS) {
+         RegCloseKey (root);
+         free_if_alloc (buf);
+         return -1;
+     }
+     /* check for garbled values. */
+     for (i=0; i < size/2; i++) {
+         if (buf[i] == 0 || buf[i] > 512) {
+             free_if_alloc (buf);
+             return -1;
+         }
+     }
+     for (i=0; i < size/2; i++)
+         listview_set_column_width (kmi->lv, i, buf[i]);
+     free_if_alloc (buf);
+     size = sizeof (kmi->keylist_sortby);
+     ec = RegQueryValueEx (root, "KMSortBy", NULL, &type,
+                           (BYTE*)&kmi->keylist_sortby, &size);
+     if (ec != ERROR_SUCCESS)
+         kmi->keylist_sortby = KEY_SORT_USERID;
+     RegCloseKey (root);
+     return 0;
+ }
+ /* Save the current column width to the registry. */
+ int
+ save_column_info (struct km_info_s *kmi)
+ {
+     HKEY root;
+     WORD *buf;
+     LONG ec;
+     int i;
+     buf = new WORD[kmi->lv->cols];
+     if (!buf)
+         BUG (NULL);
+     for (i=0; i < kmi->lv->cols; i++) {
+         LVCOLUMN lvc;
+         memset (&lvc, 0, sizeof (lvc));
+         lvc.mask = LVCF_WIDTH;
+         ListView_GetColumn (kmi->lv->ctrl, i, &lvc);
+         buf[i] = lvc.cx;
+     }
+     ec = RegOpenKeyEx (HKEY_CURRENT_USER, "Software\\WinPT", 0,
+                        KEY_ALL_ACCESS, &root);
+     if (ec != ERROR_SUCCESS) {
+         free_if_alloc (buf);
+         return -1;
+     }
+     ec = RegSetValueEx (root, "KMColumnSize", 0, REG_BINARY,
+                         (const BYTE*)buf, 2*kmi->lv->cols);
+     if (ec == ERROR_SUCCESS) {
+         ec = RegSetValueEx (root, "KMSortBy", 0, REG_DWORD_BIG_ENDIAN,
+                             (const BYTE*)&kmi->keylist_sortby,
+                             sizeof (kmi->keylist_sortby));
+     }
+     RegCloseKey (root);
+     free_if_alloc (buf);
+     return ec == ERROR_SUCCESS? 0 : -1;
+ }
+ /* Restore the original size of the dialog. */
+ static void
+ restore_window_size (HWND dlg, struct km_info_s *kmi)
+ {
+     char *p;
+     int height = 0, width = 0;
+     /* Restore the original window size. */
+     p = get_reg_entry (HKEY_CURRENT_USER, "Software\\WinPT", "KM_Size_Width");
+     if (p) {
+         width = atoi (p);
+         free_if_alloc (p);
+     }
+     p = get_reg_entry (HKEY_CURRENT_USER, "Software\\WinPT", "KM_Size_Height");
+     if (p) {
+         height = atoi (p);
+         free_if_alloc (p);
+     }
+     if (height > 0 && width > 0)
+         SetWindowPos (dlg, HWND_TOP, kmi->pos_x, kmi->pos_y,
+                       height, width, SWP_SHOWWINDOW);
  }
  /* Center window @dlg. */
  static void
- do_center_window (HWND dlg, struct km_info *kmi)
+ do_center_window (HWND dlg, struct km_info_s *kmi)
  {
      RECT rect;
      char *p;
-     int pos_x = 0, pos_y = 0;
+     int pos_x = 0;
+     int pos_y = 0;
      /* Find bottom of keylist */
      GetWindowRect (GetDlgItem(dlg, IDC_KEYMISC_KEYLIST), &rect);
-Line 211 
 do_center_window (HWND dlg, struct km_in
+Line 352 
 do_center_window (HWND dlg, struct km_in
          return;
      }
      else if (p)
-         pos_x = atol (p);
+         pos_x = atoi (p);
      p = get_reg_entry (HKEY_CURRENT_USER, "Software\\WinPT", "KM_Pos_Y");
      if (p && !strcmp (p, " ")) {
-Line 220 
 do_center_window (HWND dlg, struct km_in
+Line 361 
 do_center_window (HWND dlg, struct km_in
          return;
      }
      else if (p)
-         pos_y = atol (p);
+         pos_y = atoi (p);
      if (!pos_y && !pos_x) {
          center_window (dlg, NULL);
-Line 233 
 do_center_window (HWND dlg, struct km_in
+Line 374 
 do_center_window (HWND dlg, struct km_in
          || pos_y > GetSystemMetrics (SM_CYSCREEN)) {
          pos_x = pos_y = 0;
      }
+     kmi->pos_x = pos_x;
+     kmi->pos_y = pos_y;
      GetClientRect (dlg, &rect);
      MoveWindow (dlg, pos_x, pos_y, rect.right, rect.bottom, TRUE);
  }
-Line 240 
 do_center_window (HWND dlg, struct km_in
+Line 384 
 do_center_window (HWND dlg, struct km_in
  /* Resize the key manager window with the information from @kmi. */
  static void
- do_resize_window (HWND dlg, struct km_info *kmi)
+ do_resize_window (HWND dlg, struct km_info_s *kmi)
  {
      HWND h;
      RECT rclient, rect;
-Line 288 
 do_resize_window (HWND dlg, struct km_in
+Line 432 
 do_resize_window (HWND dlg, struct km_in
      /* Place the keylist above the separator */
      h = GetDlgItem (dlg, IDC_KEYMISC_KEYLIST);
      MoveWindow (h, rclient.left, rclient.top, rclient.right - rclient.left,
                  kmi->ypos_sep - rclient.top, bRepaint);
      rclient.top = kmi->ypos_sep + 5 + 8;
-     /* Place the group text and the group box below the separator */
+     if (kmi->enable_groups != 0) {
-     h = GetDlgItem (dlg, IDC_KEYMISC_GTEXT);
+         /* Place the group text and the group box below the separator */
-     MoveWindow (h, rclient.left, rclient.top, 100, 14, bRepaint);
+         h = GetDlgItem (dlg, IDC_KEYMISC_GTEXT);
-     rclient.top += 18;
+         MoveWindow (h, rclient.left, rclient.top, 100, 14, bRepaint);
+         rclient.top += 18;
-     h = GetDlgItem (dlg, IDC_KEYMISC_GROUP);
-     MoveWindow (h, rclient.left, rclient.top, rclient.right - rclient.left,
+         h = GetDlgItem (dlg, IDC_KEYMISC_GROUP);
-                 (rclient.bottom < rclient.top) ?
+         MoveWindow (h, rclient.left, rclient.top,
-: rclient.bottom - rclient.top, bRepaint);
+                     rclient.right - rclient.left,
+                     (rclient.bottom < rclient.top) ?
+: rclient.bottom - rclient.top, bRepaint);
+     }
      /* Repaint the whole thing */
      InvalidateRect (dlg, NULL, TRUE);
-Line 330 
 static void
+Line 477 
 static void
  do_create_minpopup (HWND dlg)
  {
      HMENU hm;
-     MENUITEMINFO mi;
-     char * s;
      POINT p;
      if (gpg_read_only || !clip_contains_pgpkey ())
          return;
      hm = CreatePopupMenu ();
      if (!hm)
-         BUG( NULL );
+         BUG (0);
-     memset (&mi, 0, sizeof mi);
+     insert_menu_item (hm, 0, ID_KEYCTX_PASTE, _("Paste Key from Clipboard"));
-     mi.cbSize = sizeof mi;
-     s = (char *)_("Paste Key from Clipboard");
-     mi.fType = MF_STRING;
-     mi.dwTypeData = s;
-     mi.cch = strlen (s);
-     mi.fMask = MIIM_DATA | MIIM_ID | MIIM_TYPE;
-     mi.wID = ID_KEYCTX_PASTE;
-     InsertMenuItem (hm, 0, FALSE, &mi);
      GetCursorPos (&p);
      TrackPopupMenu (hm, 0, p.x, p.y, 0, dlg, NULL);
      DestroyMenu (hm);
  }
- /* Reload the key cache if requested. */
+ /* Update the default key entry in the status bar for dialog @dlg. */
- static void
+ void
- do_check_cache (listview_ctrl_t lv, HWND dlg, HWND sb)
+ update_default_key_str (HWND dlg)
+ {
+     const char *fmt;
+     char *keyid;
+     char defkeyinf[128];
+     keyid = get_gnupg_default_key ();
+     if (!keyid)
+         return;
+     if ((keyid[0] >= 'A' && keyid[0] <= 'Z') ||
+         (keyid[0] >= 'a' && keyid[0] <= 'z') ||
+         (keyid[0] == '0' && keyid[1] == 'x'))
+         fmt = _("Default Key: %s");
+     else
+         fmt = _("Default Key: 0x%s");
+     _snprintf (defkeyinf, sizeof (defkeyinf) - 1, fmt, keyid);
+     SendMessage (dlg, SB_SETTEXT, 0, (LPARAM)defkeyinf);
+     gpg_keycache_set_default_key (keycache_get_ctx (0), keyid);
+     free_if_alloc (keyid);
+ }
+ /* Count all keys and show from @lv results in the status bar @sb. */
+ void
+ update_status_bar (HWND sb, listview_ctrl_t lv)
  {
-     gpg_keycache_t cache;
+     char txt_sec[128];
+     char txt_pub[128];
+     int nkeys = 0, nsec = 0;
+     int i;
-     if (keycache_get_reload ()) {
+     nkeys = listview_count_items (lv, 0);
-         keycache_reload (dlg);
+     for (i = 0; i < nkeys; i++) {
-         keycache_set_reload (0);
+         if (km_check_for_seckey (lv, i, NULL))
-         cache = keycache_get_ctx (1);
+             nsec++;
-         if (!cache)
-             BUG (dlg);
-         keylist_reload (lv, cache, KEYLIST_LIST, KEY_SORT_USERID);
-         km_update_status_bar (sb, lv);
      }
+     _snprintf (txt_sec, sizeof (txt_sec)-1, _("%d secret keys"), nsec);
+     _snprintf (txt_pub, sizeof (txt_pub)-1, _("%d keys"), nkeys);
+     SendMessage (sb, SB_SETTEXT, 1, (LPARAM)txt_sec);
+     SendMessage (sb, SB_SETTEXT, 2, (LPARAM)txt_pub);
  }
-Line 376 
 long CALLBACK
+Line 540 
 long CALLBACK
  separator_wnd_proc (HWND hwnd, UINT msg, WPARAM wparam, LPARAM lparam)
  {
      static POINT last_mouse_pos;
+     PAINTSTRUCT ps;
+     RECT rect;
+     HPEN hpen;
      if (msg == WM_CREATE)
          SetWindowLong (hwnd, GWL_USERDATA,
-Line 383 
 separator_wnd_proc (HWND hwnd, UINT msg,
+Line 550 
 separator_wnd_proc (HWND hwnd, UINT msg,
      switch (msg) {
      case WM_PAINT:
-         PAINTSTRUCT ps;
-         RECT rect;
-         HPEN hpen;
          GetClientRect (hwnd, &rect);
          BeginPaint (hwnd, &ps);
-Line 394 
 separator_wnd_proc (HWND hwnd, UINT msg,
+Line 557 
 separator_wnd_proc (HWND hwnd, UINT msg,
          FillRect (ps.hdc, &rect, (HBRUSH)(COLOR_3DFACE+1));
          /* The lines from the light into the dark */
-         MoveToEx(ps.hdc, 0,0, NULL);
+         MoveToEx (ps.hdc, 0,0, NULL);
-         if ((hpen = CreatePen (PS_SOLID, 0, GetSysColor(COLOR_3DHILIGHT))) != NULL) {
+         hpen = CreatePen (PS_SOLID, 0, GetSysColor (COLOR_3DHILIGHT));
+         if (hpen != NULL) {
              SelectObject (ps.hdc, (LPVOID)hpen);
              LineTo (ps.hdc, rect.right, 0);
              DeleteObject (hpen);
          }
-         MoveToEx(ps.hdc, 0, 1, NULL);
+         MoveToEx (ps.hdc, 0, 1, NULL);
-         if ((hpen = CreatePen (PS_SOLID, 0, GetSysColor(COLOR_3DLIGHT))) != NULL) {
+         hpen = CreatePen (PS_SOLID, 0, GetSysColor (COLOR_3DLIGHT));
+         if (hpen != NULL) {
               SelectObject (ps.hdc, (LPVOID)hpen);
               LineTo (ps.hdc, rect.right, rect.bottom);
               DeleteObject (hpen);
           }
-         MoveToEx(ps.hdc, 0, rect.bottom-1, NULL);
+         MoveToEx (ps.hdc, 0, rect.bottom-1, NULL);
-         if ((hpen = CreatePen (PS_SOLID, 0, GetSysColor(COLOR_3DSHADOW))) != NULL) {
+         hpen = CreatePen (PS_SOLID, 0, GetSysColor(COLOR_3DSHADOW));
+         if (hpen != NULL) {
              SelectObject (ps.hdc, (LPVOID)hpen);
              LineTo (ps.hdc, rect.right, rect.bottom-1);
              DeleteObject (hpen);
          }
-         MoveToEx(ps.hdc, 0, rect.bottom, NULL);
+         MoveToEx (ps.hdc, 0, rect.bottom, NULL);
-         if ((hpen = CreatePen(PS_SOLID, 0, GetSysColor(COLOR_3DDKSHADOW))) != NULL) {
+         hpen = CreatePen(PS_SOLID, 0, GetSysColor(COLOR_3DDKSHADOW));
+         if (hpen != NULL) {
              SelectObject (ps.hdc, (LPVOID)hpen);
              LineTo (ps.hdc, rect.right, rect.bottom);
              DeleteObject (hpen);
-Line 436 
 separator_wnd_proc (HWND hwnd, UINT msg,
+Line 603 
 separator_wnd_proc (HWND hwnd, UINT msg,
      case WM_MOUSEMOVE:
          if (wparam == MK_LBUTTON) {
-             struct km_info *kmi;
+             struct km_info_s *kmi;
              POINT p;
-             RECT r;
-             kmi = (struct km_info *)GetWindowLong (hwnd, GWL_USERDATA);
+             kmi = (struct km_info_s *)GetWindowLong (hwnd, GWL_USERDATA);
              if (kmi == NULL)
                  break;
              /* Calculate mouse movement */
-             p.x = LOWORD(lparam);
+             p.x = LOWORD (lparam);
-             p.y = HIWORD(lparam);
+             p.y = HIWORD (lparam);
              ClientToScreen (hwnd, &p);
-             GetWindowRect (hwnd, &r);
+             GetWindowRect (hwnd, &rect);
-             r.top += (short)(p.y - last_mouse_pos.y);
+             rect.top += (short)(p.y - last_mouse_pos.y);
-             r.bottom += (short)(p.y - last_mouse_pos.y);
+             rect.bottom += (short)(p.y - last_mouse_pos.y);
              last_mouse_pos.y = p.y;
              /* Apply mouse movement to window. Beware the MoveWindow is relaive
                 to parent NOT screen */
-             MapWindowPoints (NULL, GetParent(hwnd), (POINT*)&r, 2);
+             MapWindowPoints (NULL, GetParent(hwnd), (POINT*)&rect, 2);
-             kmi->ypos_sep = r.top;
+             kmi->ypos_sep = rect.top;
              kmi->ypercent_sep = 0; /* This forces do_resize_window to use abs. position */
-             do_resize_window (GetParent(hwnd), kmi);
+             do_resize_window (GetParent (hwnd), kmi);
              return 0;
          }
      }
-Line 471 
 separator_wnd_proc (HWND hwnd, UINT msg,
+Line 637 
 separator_wnd_proc (HWND hwnd, UINT msg,
  /* Register the separator window with @dlg as the parent window. */
  static HWND
- regist_sep_wnd (HWND dlg, struct km_info * kmi)
+ regist_sep_wnd (HWND dlg, struct km_info_s *kmi)
  {
      WNDCLASS wndclass;
      HWND h;
+     memset (&wndclass, 0, sizeof (wndclass));
      wndclass.style         = CS_DBLCLKS | CS_HREDRAW | CS_VREDRAW;
      wndclass.lpfnWndProc   = separator_wnd_proc;
      wndclass.cbClsExtra    = 0;
-Line 490 
 regist_sep_wnd (HWND dlg, struct km_info
+Line 657 
 regist_sep_wnd (HWND dlg, struct km_info
      h = CreateWindowEx (WS_EX_WINDOWEDGE, WINDOWCLASS_SEPARATOR_CHILD, NULL,
                          WS_CHILD | WS_CLIPCHILDREN | WS_VISIBLE,
 , 400, 2000, 5, dlg, (HMENU) 0, glob_hinst, kmi);
+     if (!h)
+         BUG (NULL);
      ShowWindow (h, SW_SHOW);
      return h;
  }
- #define enable_button(hwnd, cid) \
+ #define enable_button(hwnd, cid, item_selected) \
-     SendMessage ((hwnd), TB_ENABLEBUTTON, (cid), MAKELONG (key_selected, 0))
+     SendMessage ((hwnd), TB_ENABLEBUTTON, (cid), MAKELONG ((item_selected), 0))
  /* Interactive modification of the dialog item which depend if an item
-Line 506 
 regist_sep_wnd (HWND dlg, struct km_info
+Line 675 
 regist_sep_wnd (HWND dlg, struct km_info
  void
  update_ui_items (HWND hwnd, listview_ctrl_t lv)
  {
-     HWND hwnd_child;
+     HWND tb_hwnd;
      HMENU hmenu;
-     int key_selected = 0;
-     int key_has_sec = 0;
-     int key_inv = 0;
      int mult_resids[] = {ID_KEYMISC_PROPS, ID_KEYMISC_SIGN, ID_KEYMISC_EDITKEY,
                           ID_KEYMISC_CHECKSIGS, ID_KEYMISC_REVCERT, 0};
-     int i;
+     int key_selected;
+     int key_has_sec;
+     int key_inv = 0;
+     int i, state=0;
      /* Get some states */
      key_selected = SendMessage (GetDlgItem (hwnd, IDC_KEYMISC_KEYLIST),
                                             LVM_GETSELECTEDCOUNT, 0, 0)
-Line 528 
 update_ui_items (HWND hwnd, listview_ctr
+Line 697 
 update_ui_items (HWND hwnd, listview_ctr
      }
      /* Enable / disable toolbar buttons */
-     hwnd_child = GetDlgItem (hwnd, IDR_WINPT_KMTB);
+     tb_hwnd = GetDlgItem (hwnd, IDR_WINPT_KMTB);
-     enable_button (hwnd_child, ID_KEYMISC_DELETE);
+     enable_button (tb_hwnd, ID_KEYMISC_DELETE, key_selected);
-     enable_button (hwnd_child, ID_KEYMISC_PROPS);
+     enable_button (tb_hwnd, ID_KEYMISC_PROPS, key_selected);
-     enable_button (hwnd_child, ID_KEYMISC_SIGN);
+     enable_button (tb_hwnd, ID_KEYMISC_SIGN, key_selected && !key_inv);
-     enable_button (hwnd_child, ID_KEYMISC_EXPORT);
+     enable_button (tb_hwnd, ID_KEYMISC_EXPORT, key_selected);
+     enable_button (tb_hwnd, ID_KEYCTX_COPY, key_selected);
+     enable_button (tb_hwnd, ID_KEYCTX_PASTE, clip_contains_pgpkey ());
      /* Enable / disable menu items */
+     state = key_selected? MF_ENABLED : MF_DISABLED|MF_GRAYED;
      hmenu = GetMenu (hwnd);
-     set_menu_state (hmenu, ID_KEYMISC_EXPORT, key_selected ? MF_ENABLED : MF_GRAYED);
+     set_menu_state (hmenu, ID_KEYMISC_EXPORT, state);
-     set_menu_state (hmenu, ID_KEYMISC_EXPORT_PRIVKEY, key_has_sec ? MF_ENABLED : MF_GRAYED);
+     set_menu_state (hmenu, ID_KEYMISC_DELETE, state);
-     set_menu_state (hmenu, ID_KEYMISC_REVCERT, key_has_sec ? MF_ENABLED : MF_GRAYED);
+     set_menu_state (hmenu, ID_KEYMISC_PROPS, state);
-     set_menu_state (hmenu, ID_KEYMISC_DELETE, key_selected ? MF_ENABLED : MF_GRAYED);
+     set_menu_state (hmenu, ID_KEYMISC_EDITKEY, state);
-     set_menu_state (hmenu, ID_KEYMISC_PROPS, key_selected ? MF_ENABLED : MF_GRAYED);
+     set_menu_state (hmenu, ID_KEYMISC_CHECKSIGS, state);
-     set_menu_state (hmenu, ID_KEYMISC_SIGN, key_selected && !key_inv ? MF_ENABLED : MF_GRAYED);
+     set_menu_state (hmenu, ID_KEYMISC_SIGN,
-     set_menu_state (hmenu, ID_KEYMISC_EDITKEY, key_selected? MF_ENABLED : MF_GRAYED);
+                     key_selected && !key_inv ? MF_ENABLED : MF_GRAYED);
-     set_menu_state (hmenu, ID_KEYMISC_CHECKSIGS, key_selected? MF_ENABLED : MF_GRAYED);
+     set_menu_state (hmenu, ID_KEYMISC_EXPORT_PRIVKEY,
+                     key_selected && key_has_sec? MF_ENABLED : MF_GRAYED);
+     set_menu_state (hmenu, ID_KEYMISC_REVCERT,
+                     key_selected && key_has_sec? MF_ENABLED : MF_GRAYED);
      /* Disable some menu items when multiple keys are selected. */
      if (listview_count_items (lv, 1) > 1) {
          for (i=0; mult_resids[i] != 0; i++)
              set_menu_state (hmenu, mult_resids[i], MF_GRAYED);
      }
+     /* Disable all secret-key functions when no secret key is available. */
+     if (!secret_key_available ()) {
+         enable_button (tb_hwnd, ID_KEYMISC_SIGN, FALSE);
+         set_menu_state (hmenu, ID_KEYMISC_SIGN, MF_GRAYED);
+     }
  }
-Line 610 
 popup_gpg_readonly (HWND dlg, HMENU hm)
+Line 791 
 popup_gpg_readonly (HWND dlg, HMENU hm)
  static void
  change_edit_menu (listview_ctrl_t lv, HMENU hm, int id)
  {
+     enum item { EDIT_MENU = 1 };
      int no_sel;
-     if (id != 1 /*EDIT_MENU*/)
+     if (id != EDIT_MENU)
          return;
      if (!clip_contains_pgpkey ())
-Line 667 
 change_key_menu (HMENU hm, int id)
+Line 849 
 change_key_menu (HMENU hm, int id)
  }
- /* Return a user-friendly name for a key derrived from
-    name. If @is_secret is 1, a secret key name will be generated. */
- static char*
- gen_export_filename (const char *keyid, int is_secret)
- {
-     gpgme_key_t key;
-     const char *s;
-     char *p;
-     if (get_pubkey (keyid, &key))
-         return m_strdup (keyid);
-     s = key->uids->name;
-     if (!s)
-         return m_strdup (keyid);
-     p = new char[strlen (s) + 8 + 16];
-     if (!p)
-         BUG (0);
-     sprintf (p, "%s%s.asc", s, is_secret? "_sec" : "");
-     for (size_t i=0; i < strlen (p); i++) {
-         if (p[i] == ' ' || p[i] == ':' || p[i] == '?' || p[i] == '|')
-             p[i] = '_';
-     }
-     return p;
- }
  /* Reload a single key in the cache. */
  static void
  update_key (listview_ctrl_t lv, int pos, const char *keyid, int keypair)
  {
-     gpgme_key_t key;
+     struct winpt_key_s key;
      keycache_update (0, keyid);
      if (keypair)
-Line 705 
 update_key (listview_ctrl_t lv, int pos,
+Line 861 
 update_key (listview_ctrl_t lv, int pos,
      /* because we replaced the key item, we need to update the
         address of the pointer in the ListView control. */
-     get_pubkey (keyid, &key);
+     memset (&key, 0, sizeof (key));
-     keylist_upd_key (lv, pos, key);
+     winpt_get_pubkey (keyid, &key);
+     keylist_upd_key (lv, pos, key.ext, key.ctx);
+     keyring_check_last_access ();
  }
-Line 723 
 setup_status_bar (HWND dlg, listview_ctr
+Line 881 
 setup_status_bar (HWND dlg, listview_ctr
      for (i=1; i <= 3; i++)
          partpos[i-1] = r.right/3*i;
      statbar = CreateStatusWindow (WS_CHILD | WS_VISIBLE | CCS_BOTTOM,
-                                   "status bar", dlg, KM_SEPARATOR_ID);
+                                   "WinPT_Status_Bar", dlg, KM_SEPARATOR_ID);
+     if (!statbar)
+         BUG (NULL);
      ShowWindow (statbar, SW_SHOW);
      SendMessage (statbar, SB_SETPARTS, (WPARAM)3, (LPARAM)partpos);
-     km_update_default_key_str (statbar);
+     update_default_key_str (statbar);
-     km_update_status_bar (statbar, lv);
+     update_status_bar (statbar, lv);
      return statbar;
  }
+ /* Remove or add columns which depends on the state of @checked. */
+ void
+ modify_listview_columns (km_info_s *kmi, UINT m_uid, BOOL checked)
+ {
+     UINT resids[] = {
+,
+         ID_KEYMISC_VIEWKEYID,
+         ID_KEYMISC_VIEWTYPE,
+,
+         ID_KEYMISC_VIEWCIPHER,
+,
+,
+         ID_KEYMISC_VIEWCREAT,
+         ID_KEYMISC_VIEWDESC,
+         -1
+     };
+     listview_column_s cols[] = {
+     {0, 240, (char *)_("User ID")},
+     {1, 78, (char *)_("Key ID")},
+     {2, 52, (char *)_("Type")},
+     {3, 66, (char *)_("Size")},
+     {4, 60, (char *)_("Cipher")},
+     {5, 66, (char *)_("Validity")},
+     {6, 58, (char *)_("Trust")},
+     {7, 72, (char *)_("Creation")},
+     {8, 160,(char *)_("Description")},
+     {0, 0, NULL}
+     };
+     UINT pos;
+     for (pos=0; resids[pos] != -1; pos++) {
+         if (m_uid == resids[pos])
+             break;
+     }
+     if (!checked)
+         listview_del_column (kmi->lv, (int)pos);
+     else {
+         listview_add_column (kmi->lv, &cols[pos]);
+         keylist_upd_col (kmi->lv, pos);
+     }
+ }
+ /* Helper to handle the help file. If @check is 1
+    the existence of the file is checked.
+    Otherwise the help file will be loaded. */
+ static bool
+ start_help (HWND dlg, int check)
+ {
+     DWORD n;
+     char path[MAX_PATH+1+32];
+     n = GetModuleFileName (NULL, path, sizeof (path)-1-32);
+     if (!n)
+         return false;
+     path[n] = 0;
+     while (n-- && path[n] != '\\')
+         ;
+     path[n+1] = 0;
+     strcat (path, "winpt.chm");
+     if (!check)
+         ShellExecute (dlg, "open", path, NULL, NULL, SW_SHOW);
+     return file_exist_check (path) == 0? true : false;
+ }
+ /* Translate all menu strings. */
+ static void
+ translate_menu_strings (HWND dlg)
+ {
+     HMENU menu;
+     menu = LoadMenu (glob_hinst, (LPCSTR)IDR_WINPT_KEYMISC);
+     set_menu_text_bypos (menu, 0, _("File"));
+     set_menu_text_bypos (menu, 1, _("Edit"));
+     set_menu_text_bypos (menu, 2, _("View"));
+     set_menu_text_bypos (menu, 3, _("Key"));
+     set_menu_text_bypos (menu, 4, _("Groups"));
+     set_menu_text (menu, ID_KEYMISC_EDITKEY, _("Edit"));
+     set_menu_text (menu, ID_KEYMISC_MAIL, _("Send Mail..."));
+     set_menu_text (menu, ID_KEYMISC_OT, _("Ownertrust")); /* XXX */
+     set_menu_text (menu, ID_KEYMISC_COPY, _("&Copy\tCtrl+C"));
+     set_menu_text (menu, ID_KEYMISC_PASTE, _("&Paste\tCtrl+V"));
+     set_menu_text (menu, ID_KEYMISC_FIND, _("Search...\tCtrl+F"));
+     set_menu_text (menu, ID_KEYMISC_SELALL, _("Select All\tCtrl+A"));
+     set_menu_text (menu, ID_KEYMISC_QUIT, _("&Quit"));
+     set_menu_text (menu, ID_KEYMISC_UID, _("User ID"));
+     set_menu_text (menu, ID_KEYMISC_NEWKEY, _("&Expert"));
+     set_menu_text (menu, ID_KEYMISC_KEYWIZARD, _("&Normal"));
+     set_menu_text (menu, ID_KEYMISC_EDIT, _("Edit"));
+     set_menu_text (menu, ID_KEYMISC_SIGN, _("&Sign"));
+     set_menu_text (menu, ID_KEYMISC_DELETE, _("&Delete"));
+     set_menu_text (menu, ID_KEYMISC_DELETE2, _("&Delete"));
+     set_menu_text (menu, ID_KEYMISC_REVCERT, _("&Revoke Cert"));
+     set_menu_text (menu, ID_KEYMISC_CHECKSIGS, _("&List Signatures"));
+     set_menu_text (menu, ID_KEYMISC_TRUSTPATH, _("List Trust Path"));
+     set_menu_text (menu, ID_KEYMISC_EXPORT, _("&Export..."));
+     set_menu_text (menu, ID_KEYMISC_IMPORT, _("&Import..."));
+     set_menu_text (menu, ID_KEYMISC_PROPS, _("&Properties"));
+     set_menu_text (menu, ID_KEYMISC_GPGOPT, _("GPG &Options"));
+     set_menu_text (menu, ID_KEYMISC_GPGPREFS, _("&GPG Preferences"));
+     set_menu_text (menu, ID_KEYMISC_WINPTPREFS, _("&WinPT Preferences"));
+     set_menu_text (menu, ID_KEYMISC_SENDRECV, _("Keyserver") );
+     set_menu_text (menu, ID_KEYMISC_EXPORT_PRIVKEY, _("E&xport Secret Key"));
+     set_menu_text (menu, ID_KEYMISC_RECACHE, _("Re&load Key Cache"));
+     set_menu_text (menu, ID_KEYMISC_REBUILD, _("R&everify Signatures"));
+     set_menu_text (menu, ID_KEYMISC_REFRESH_KEYS, _("Refresh &Keys (Keyserver)"));
+     set_menu_text (menu, ID_KEYMISC_INFO, _("Info") );
+     set_menu_text (menu, ID_KEYMISC_HELP, _("&Help"));
+     set_menu_text (menu, ID_KEYMISC_VIEWKEYID, _("Key ID"));
+     set_menu_text (menu, ID_KEYMISC_VIEWCIPHER, _("Cipher"));
+     set_menu_text (menu, ID_KEYMISC_VIEWTYPE, _("Type"));
+     set_menu_text (menu, ID_KEYMISC_VIEWCREAT, _("Creation"));
+     if (!start_help (NULL, 1))
+         set_menu_state (menu, ID_KEYMISC_HELP, MF_GRAYED);
+     SetMenu (dlg, menu);
+ }
+ /* Translate popup menu strings. */
+ static void
+ translate_popupmenu_strings (HMENU popup)
+ {
+     set_menu_text (popup, ID_KEYCTX_UID_COPY, _("Copy User ID to Clipboard"));
+     set_menu_text (popup, ID_KEYCTX_KEYID_COPY, _("Copy Key ID to Clipboard"));
+     set_menu_text (popup, ID_KEYCTX_FPR_COPY, _("Copy Fingerprint to Clipboard"));
+     set_menu_text (popup, ID_KEYCTX_KINFO_COPY, _("Copy Key Info to Clipboard"));
+     set_menu_text (popup, ID_KEYCTX_EMAIL_COPY, _("Copy E-Mail to Clipboard"));
+     set_menu_text (popup, ID_KEYCTX_COPY, _("Copy Key to Clipboard"));
+     set_menu_text (popup, ID_KEYCTX_PASTE, _("Paste Key from Clipboard"));
+     set_menu_text (popup, ID_KEYCTX_RECVFROM, _("Refresh from Keyserver"));
+     set_menu_text (popup, ID_KEYCTX_MAXTRUST, _("Set Implicit &Trust"));
+     set_menu_text (popup, ID_KEYCTX_LISTSIGS, _("&List Signatures"));
+     set_menu_text (popup, ID_KEYCTX_PROPS, _("&Properties"));
+     set_menu_text (popup, ID_KEYCTX_EDIT, _("Key Edit"));
+     set_menu_text (popup, ID_KEYCTX_DEL, _("&Delete"));
+     set_menu_text (popup, ID_KEYCTX_REV, _("&Revoke Cert"));
+     set_menu_text (popup, ID_KEYCTX_SIGN, _("&Sign"));
+     set_menu_text (popup, ID_KEYCTX_ENABLE, _("&Enable"));
+     set_menu_text (popup, ID_KEYCTX_DISABLE, _("&Disable"));
+     set_menu_text (popup, ID_KEYCTX_RECVFROM, _("Re&fresh from Keyserver"));
+     set_menu_text (popup, ID_KEYCTX_SETPREFKS, _("Set preferred Keyserver URL"));
+     set_menu_text (popup, ID_KEYCTX_SENDMAIL, _("Send Key to Mail Recipient"));
+     set_menu_text (popup, ID_KEYCTX_SETDEFKEY, _("Set as Default Key"));
+     set_menu_text (popup, ID_KEYCTX_ADDKEY, _("Key..."));
+     set_menu_text (popup, ID_KEYCTX_ADDUID, _("User ID..."));
+     set_menu_text (popup, ID_KEYCTX_ADDPHOTO, _("Photo ID..."));
+     set_menu_text (popup, ID_KEYCTX_ADDREV, _("Revoker..."));
+     /* change popup texts */
+     set_menu_text_bypos (popup, 0, _("Key Attributes"));
+     set_menu_text_bypos (popup, 6, _("Add"));
+     set_menu_text_bypos (popup, 19, _("Send to Keyserver"));
+ }
+ /* Return true if the cache contain marked keys. */
+ static bool
+ updated_keys_avail (void)
+ {
+     gpg_keycache_t kc = keycache_get_ctx (1);
+     struct keycache_s *k;
+     for (k = kc->item; k; k = k->next) {
+         if (k->flags)
+             return true;
+     }
+     return false;
+ }
+ /* Find the index of the key identified by @key. */
+ static int
+ find_keypos (listview_ctrl_t lv, gpgme_key_t key)
+ {
+     char keyid[32+1];
+     int i;
+     for (i=0; i < listview_count_items (lv, 0); i++) {
+         listview_get_item_text (lv, i, KM_COL_KEYID, keyid, sizeof (keyid)-1);
+         if (!strcmp (key->subkeys->keyid+8, keyid+2))
+             return i;
+     }
+     return -1;
+ }
+ /* Add all recently updated keys in the cache to the list
+    and refresh all GUI elements. */
+ static void
+ refresh_keylist (struct km_info_s *kmi)
+ {
+     struct keycache_s *ctx;
+     gpg_keycache_t kc;
+     int status=0, pos;
+     kc = keycache_get_ctx (1);
+     while (!gpg_keycache_next_updated_key (kc, &ctx, &status)) {
+         if (status == KC_FLAG_ADD)
+             keylist_add_key (kmi->lv, KEYLIST_LIST, ctx, ctx->key);
+         else {
+             pos = find_keypos (kmi->lv, ctx->key);
+             if (pos != -1)
+                 keylist_upd_key (kmi->lv, pos, ctx, ctx->key);
+         }
+     }
+     keylist_sort (kmi->lv, kmi->keylist_sortby);
+     update_status_bar (kmi->statbar, kmi->lv);
+     keyring_check_last_access ();
+ }
+ /* Reload the key cache. */
+ static void
+ keycache_reload (HWND dlg)
+ {
+     refresh_cache_s rcs;
+     memset (&rcs, 0, sizeof rcs);
+     rcs.kr_reload = rcs.kr_update = 1;
+     rcs.tr_update = 0;
+     DialogBoxParam (glob_hinst, (LPCSTR)IDD_WINPT_KEYCACHE, dlg,
+                     keycache_dlg_proc, (LPARAM)&rcs);
+ }
+ /* Check if the secret keyring contains at least one
+    key with ultimate trust.
+    Return value: 0 on success. */
+ int
+ check_ultimate_trusted_key (gpg_keycache_t _sec)
+ {
+     struct keycache_s *n;
+     int nkeys = 0;
+     for (n = _sec->item; n; n = n->next) {
+         if (n->pubpart &&
+             n->pubpart->key->owner_trust == GPGME_VALIDITY_ULTIMATE)
+             return 0;
+         nkeys++;
+     }
+     /* if we do not have any secret keys, it does not make sense
+        to return an error. */
+     if (nkeys == 0)
+         return 0;
+     return -1;
+ }
+ static void
+ reload_keylist (struct km_info_s *kmi)
+ {
+     gpg_keycache_t c;
+     c = keycache_get_ctx (1);
+     keycache_reload (kmi->dlg);
+     keylist_reload (kmi->lv, c, KEYLIST_LIST, kmi->keylist_sortby);
+     update_status_bar (kmi->statbar, kmi->lv);
+     keyring_check_last_access ();
+ }
+ /* Reload the entire key cache and update the listview. */
+ static void
+ reload_keycache (struct km_info_s *kmi)
+ {
+     refresh_cache_s rcs = {0};
+     gpg_keycache_t c;
+     rcs.kr_reload = rcs.kr_update = 1;
+     rcs.tr_update = 0;
+     DialogBoxParam (glob_hinst, (LPCSTR)IDD_WINPT_KEYCACHE, kmi->dlg,
+                     keycache_dlg_proc, (LPARAM)&rcs);
+     c = keycache_get_ctx (1);
+     keylist_reload (kmi->lv, c, KEYLIST_LIST, kmi->keylist_sortby);
+     update_status_bar (kmi->statbar, kmi->lv);
+     SetForegroundWindow (kmi->dlg);
+ }
+ /* Handle all import request. */
+ static void
+ km_gui_import (struct km_info_s *kmi, int cmd_id, void *param)
+ {
+     int newkeys=0, newsks=0;
+     int err = 0;
+     switch (cmd_id) {
+     case ID_KEYMISC_PASTE:
+         err = km_clip_import (kmi->dlg, &newkeys, &newsks);
+         break;
+     case ID_KEYMISC_IMPORT:
+         err = km_file_import (kmi->dlg, NULL, &newkeys, &newsks);
+         break;
+     case WM_DROPFILES:
+         err = km_dropped_file_import (kmi->dlg, (HDROP)param,
+                                       &newkeys, &newsks);
+         break;
+     default:
+         err = 1;
+         break;
+     }
+     if (!err && !newsks && (newkeys > 0 && newkeys < KM_PRELOAD_KEYS))
+         refresh_keylist (kmi);
+     else if (!err) /* avoid to spawn too much processes. */
+         reload_keylist (kmi);
+     SetForegroundWindow (kmi->dlg);
+ }
+ /* Copy the requested key information (@what) to the clipboard.
+    If multiple items were selected, each key info is copied to
+    the clipboard. The data is separated by \r\n. */
+ static void
+ copy_key_info (int what, struct km_info_s *kmi)
+ {
+     gpgme_data_t out;
+     gpgme_key_t key;
+     struct keycache_s *kci = NULL;
+     size_t n=0;
+     char *p, tmp[256];
+     int i=0, end = listview_count_items (kmi->lv, 0);
+     bool single = false;
+     /* Do not iterate over the entire list if just one item is selected. */
+     n = listview_count_items (kmi->lv, 1);
+     if (n == 1) {
+         i = kmi->lv_idx;
+         end = i+1;
+         single = true;
+     }
+     gpgme_data_new (&out);
+     for (; i < end; i++) {
+         if (!listview_get_item_state (kmi->lv, i))
+             continue;
+         switch (what) {
+         case ID_KEYCTX_EMAIL_COPY:
+             key = km_get_key_ptr (kmi->lv, i, NULL);
+             gpgme_data_write (out, key->uids->email, strlen (key->uids->email));
+             if (!single) gpgme_data_write (out, "\r\n", 2);
+             break;
+         case ID_KEYCTX_UID_COPY:
+             key = km_get_key_ptr (kmi->lv, i, &kci);
+             gpgme_data_write (out, kci->uids->uid, strlen (kci->uids->uid));
+             if (!single) gpgme_data_write (out, "\r\n", 2);
+             break;
+         case ID_KEYCTX_KEYID_COPY:
+             key = km_get_key_ptr (kmi->lv, i, NULL);
+             gpgme_data_write (out, "0x", 2);
+             gpgme_data_write (out, key->subkeys->keyid+8, 8);
+             if (!single) gpgme_data_write (out, "\r\n", 2);
+             break;
+         case ID_KEYCTX_FPR_COPY:
+             key = km_get_key_ptr (kmi->lv, i, NULL);
+             gpgme_data_write (out, key->subkeys->fpr, strlen (key->subkeys->fpr));
+             if (!single) gpgme_data_write (out, "\r\n", 2);
+             break;
+         case ID_KEYCTX_KINFO_COPY:
+             key = km_get_key_ptr (kmi->lv, i, NULL);
+             key_get_clip_info (key->subkeys->keyid+8, tmp, sizeof (tmp)-1);
+             gpgme_data_write (out, tmp, strlen (tmp));
+             if (!single) gpgme_data_write (out, "\r\n", 2);
+             break;
+         }
+     }
+     gpgme_data_write (out, "\0", 1);
+     p = gpgme_data_release_and_get_mem (out, &n);
+     set_clip_text (NULL, p, n);
+     gpgme_free (p);
+ }
  /* Dialog box procedure for the Key Manager. */
  BOOL CALLBACK
  keymanager_dlg_proc (HWND dlg, UINT msg, WPARAM wparam, LPARAM lparam)
  {
-     struct km_info *kmi;
-     static HMENU menu = NULL;
-     static int refresh_keys = 0;
-     INITCOMMONCONTROLSEX icex;
      HWND kl;
      HMENU hm;
+     RECT r;
      gpg_keycache_t c;
      gpgme_key_t key;
+     km_info_t kmi = NULL;
      struct genkey_s genkey;
      struct winpt_key_s k = {0};
      struct URL_ctx_s *url;
-     refresh_cache_s rcs = {0};
+     char type[32], *name;
-     char keyid[48], uid[128], type[32], *name;
      const char *t, *host;
-     u16 port = 0;
+     WORD port = 0;
      int l_idx = 0, i=0, rc;
      if ((msg != WM_INITDIALOG)
-         && ((kmi = (struct km_info*)GetWindowLong (dlg, GWL_USERDATA)) == NULL))
+         && ((kmi = (km_info_t)GetWindowLong (dlg, GWL_USERDATA)) == NULL))
          return FALSE;
+     /* handle dynamic popup items in the keyserver menu. */
+     if (msg == WM_COMMAND && LOWORD (wparam) >= WM_APP &&
+                              LOWORD (wparam) <= WM_APP+MAX_KEYSERVERS) {
+         l_idx = LOWORD (wparam)-WM_APP;
+         if (l_idx < 0)
+             return TRUE;
+         host = kserver_get_hostname (l_idx, 0, &port);
+         if (host != NULL)
+             km_send_to_keyserver (kmi->lv, dlg, host, port);
+         return TRUE;
+     }
      switch (msg) {
      case WM_INITDIALOG:
-         kmi = new struct km_info;
+         kmi = new struct km_info_s;
-         memset (kmi, 0, sizeof (struct km_info));
+         memset (kmi, 0, sizeof (struct km_info_s));
          kmi->lv_idx = -1;
-         icex.dwSize = sizeof (INITCOMMONCONTROLSEX);
-         icex.dwICC  = ICC_BAR_CLASSES;
-         InitCommonControlsEx (&icex);
          kmi->hwnd_sep = regist_sep_wnd (dlg, kmi);
          imagelist_load (dlg);
+         translate_menu_strings (dlg);
          SetWindowText (dlg, _("Key Manager"));
-         menu = LoadMenu (glob_hinst, (LPCSTR)IDR_WINPT_KEYMISC);
-         set_menu_text_bypos (menu, 0, _("File"));
-         set_menu_text_bypos (menu, 1, _("Edit"));
-         set_menu_text_bypos (menu, 2, _("View"));
-         set_menu_text_bypos (menu, 3, _("Key"));
-         set_menu_text_bypos (menu, 4, _("Groups"));
-         set_menu_text (menu, ID_KEYMISC_EDITKEY, _("Edit"));
-         set_menu_text (menu, ID_KEYMISC_MAIL, _("Send Mail..."));
-         set_menu_text (menu, ID_KEYMISC_OT, _("Ownertrust")); /* XXX */
-         set_menu_text (menu, ID_KEYMISC_COPY, _("&Copy\tCtrl+C"));
-         set_menu_text (menu, ID_KEYMISC_PASTE, _("&Paste\tCtrl+V"));
-         set_menu_text (menu, ID_KEYMISC_FIND, _("Search...\tCtrl+F"));
-         set_menu_text (menu, ID_KEYMISC_SELALL, _("Select All\tCtrl+A"));
-         set_menu_text (menu, ID_KEYMISC_QUIT, _("&Quit"));
-         set_menu_text (menu, ID_KEYMISC_UID, _("User ID"));
-         set_menu_text (menu, ID_KEYMISC_NEWKEY, _("&Expert"));
-         set_menu_text (menu, ID_KEYMISC_KEYWIZARD, _("&Normal"));
-         set_menu_text (menu, ID_KEYMISC_EDIT, _("Edit"));
-         set_menu_text (menu, ID_KEYMISC_SIGN, _("&Sign"));
-         set_menu_text (menu, ID_KEYMISC_DELETE, _("&Delete"));
-         set_menu_text (menu, ID_KEYMISC_DELETE2, _("&Delete"));
-         set_menu_text (menu, ID_KEYMISC_REVCERT, _("&Revoke"));
-         set_menu_text (menu, ID_KEYMISC_CHECKSIGS, _("&List Signatures"));
-         set_menu_text (menu, ID_KEYMISC_TRUSTPATH, _("List Trust Path"));
-         set_menu_text (menu, ID_KEYMISC_EXPORT, _("&Export..."));
-         set_menu_text (menu, ID_KEYMISC_IMPORT, _("&Import..."));
-         set_menu_text (menu, ID_KEYMISC_PROPS, _("&Properties"));
-         set_menu_text (menu, ID_KEYMISC_GPGOPT, _("Options"));
-         set_menu_text (menu, ID_KEYMISC_GPGPREFS, _("Preferences"));
-         set_menu_text (menu, ID_KEYMISC_SENDRECV, _("Keyserver") );
-         set_menu_text (menu, ID_KEYMISC_EXPORT_PRIVKEY, _("E&xport Secret Key"));
-         set_menu_text (menu, ID_KEYMISC_RECACHE, _("Re&load Key Cache"));
-         set_menu_text (menu, ID_KEYMISC_REBUILD, _("R&everify Signatures"));
-         set_menu_text (menu, ID_KEYMISC_REFRESH_KEYS, _("Refresh &Keys (Keyserver)"));
-         set_menu_text (menu, ID_KEYMISC_INFO, _("Info") );
-         set_menu_text (menu, ID_KEYMISC_HELP, _("&Help"));
-         /* XXX: implement help code. */
-         set_menu_state (menu, ID_KEYMISC_HELP, MF_GRAYED);
-         SetMenu (dlg, menu);
-         if (keyring_check_last_access ())
-             keycache_set_reload (1);
-         if (keycache_get_reload ())
-             keycache_reload (dlg);
          c = keycache_get_ctx (KEYCACHE_PUB);
-         if (!c)
+         kl = GetDlgItem (dlg, IDC_KEYMISC_KEYLIST);
-             BUG (NULL);
          kmi->keylist_sortby = KEY_SORT_USERID;
-         Header_SetImageList(ListView_GetHeader (GetDlgItem (dlg, IDC_KEYMISC_KEYLIST)),
+         Header_SetImageList(ListView_GetHeader (kl), glob_imagelist);
-                             glob_imagelist);
          kmi->lv = keylist_load (GetDlgItem (dlg, IDC_KEYMISC_KEYLIST), c,
                                  NULL, KEYLIST_LIST, kmi->keylist_sortby);
+         if (check_ultimate_trusted_key (keycache_get_ctx (0))) {
+             msg_box (dlg, _("No ultimately trusted key found.\n"
+                             "Please set at least one secret key to ultimate trust."),
+                             _("Key Manager"), MB_WARN);
+         }
          /* init subclassing for the listview */
-         kl = GetDlgItem (dlg, IDC_KEYMISC_KEYLIST);
+         keylist_proc.opaque = kmi;
          keylist_proc.dlg = dlg;
          keylist_proc.current = (WNDPROC)keylist_subclass_proc;
-         keylist_proc.old = (WNDPROC)GetWindowLong( kl, GWL_WNDPROC );
+         keylist_proc.old = (WNDPROC)GetWindowLong(kl, GWL_WNDPROC);
          if (keylist_proc.old) {
-             if( !SetWindowLong (kl, GWL_WNDPROC, (LONG)keylist_proc.current)) {
+             if (!SetWindowLong (kl, GWL_WNDPROC, (LONG)keylist_proc.current)) {
-                 msg_box (dlg, _("Could not set keylist window procedure."),
+                 msg_box (dlg, "Could not set keylist window procedure.",
                           _("Key Manager"), MB_ERR);
                  BUG (NULL);
              }
          }
-         #if 0
-         km_groups_new (&gc, GetDlgItem (dlg, IDC_KEYMISC_GROUP));
-         km_groups_load (gc);
-         #endif
          kmi->statbar = setup_status_bar (dlg, kmi->lv);
          SetWindowLong (dlg, GWL_USERDATA, (LONG)kmi);
-         kmi->toolbar = load_toolbar (dlg, kmi);
+         kmi->toolbar = create_toolbar (dlg, kmi);
+         kmi->dlg = dlg;
          do_center_window (dlg, kmi);
          do_resize_window (dlg, kmi);
+         restore_window_size (dlg, kmi);
          update_ui_items (dlg, kmi->lv);
+         restore_column_info (kmi);
+         keylist_sort (kmi->lv, kmi->keylist_sortby);
          SetDlgItemText (dlg, IDC_KEYMISC_GTEXT, _("Groups"));
+         SetDlgItemText (dlg, ID_GROUP_SHOW, _("&Show"));
+         SetDlgItemText (dlg, ID_GROUP_NEW, _("&New..."));
          SetClassLong (dlg, GCL_HICON, (LONG)LoadIcon (glob_hinst,
                        (LPCTSTR)IDI_WINPT));
          SetForegroundWindow (dlg);
+         kmi->magic = SetTimer (dlg, 1, 1000, NULL);
          return TRUE;
+     case WM_TIMER:
+         KillTimer (dlg, kmi->magic);
+         SetForegroundWindow (dlg);
+         break;
      case WM_DESTROY:
+         save_column_info (kmi);
          if (kmi->lv) {
              keylist_delete (kmi->lv);
              kmi->lv = NULL;
-Line 865 
 keymanager_dlg_proc (HWND dlg, UINT msg,
+Line 1387 
 keymanager_dlg_proc (HWND dlg, UINT msg,
          set_reg_entry (HKEY_CURRENT_USER, "Software\\WinPT", "KM_Pos_X", type);
          ltoa (kmi->pos_y, type, 10);
          set_reg_entry (HKEY_CURRENT_USER, "Software\\WinPT", "KM_Pos_Y", type);
-         /* Remove runtime information. This should be the last action taken here. */
-         delete kmi; kmi = NULL;
+         GetWindowRect (dlg, &r);
+         sprintf (type, "%d", (int)(r.bottom-r.top));
+         set_reg_entry (HKEY_CURRENT_USER, "Software\\WinPT", "KM_Size_Width",
+                            type);
+         sprintf (type, "%d", (int)(r.right-r.left));
+         set_reg_entry (HKEY_CURRENT_USER, "Software\\WinPT", "KM_Size_Height",
+                         type);
+         /* Remove runtime information.
+            This should be the last action taken here. */
+         delete kmi;
+         kmi = NULL;
          SetWindowLong (dlg, GWL_USERDATA, 0);
-         keycache_set_reload (refresh_keys);
          return FALSE;
      case WM_MOVE:
-         RECT r;
          GetWindowRect (dlg, &r);
          kmi->pos_x = r.left;
          kmi->pos_y = r.top;
-Line 903 
 keymanager_dlg_proc (HWND dlg, UINT msg,
+Line 1434 
 keymanager_dlg_proc (HWND dlg, UINT msg,
          case TBN_GETBUTTONINFO:
              LPTBNOTIFY lpTbNotify;
              lpTbNotify = (LPTBNOTIFY)lparam;
-             if (lpTbNotify->iItem < (sizeof(myb) / sizeof(mybuttons))) {
+             if (lpTbNotify->iItem < (sizeof(tbb) / sizeof(tb_button_s))) {
-                 lpTbNotify->tbButton.iBitmap = imagelist_getindex(myb[lpTbNotify->iItem].icon);
+                 lpTbNotify->tbButton.iBitmap = imagelist_getindex (tbb[lpTbNotify->iItem].icon);
-                 lpTbNotify->tbButton.idCommand = myb[lpTbNotify->iItem].command;
+                 lpTbNotify->tbButton.idCommand = tbb[lpTbNotify->iItem].command;
                  lpTbNotify->tbButton.fsState = TBSTATE_ENABLED;
                  lpTbNotify->tbButton.fsStyle = TBSTYLE_BUTTON;
                  lpTbNotify->tbButton.iString = 0;
-                 strncpy (lpTbNotify->pszText, myb[lpTbNotify->iItem].text, lpTbNotify->cchText);
+                 strncpy (lpTbNotify->pszText, tbb[lpTbNotify->iItem].text, lpTbNotify->cchText);
                  SetWindowLong(dlg, DWL_MSGRESULT, TRUE);
              }
              return TRUE;
-Line 943 
 keymanager_dlg_proc (HWND dlg, UINT msg,
+Line 1474 
 keymanager_dlg_proc (HWND dlg, UINT msg,
              lpttt->hinst = NULL;
              switch (lpttt->hdr.idFrom) {
+             case ID_KEYMISC_KEYWIZARD:
+                 lpttt->lpszText = (char*)_("Generate new key pair");
+                 break;
+             case ID_KEYMISC_SENDRECV:
+                 lpttt->lpszText = (char*)_("Search for a specific key");
+                 break;
              case ID_KEYMISC_DELETE:
                  lpttt->lpszText = (char *)_("Delete key from keyring");
                  break;
-Line 955 
 keymanager_dlg_proc (HWND dlg, UINT msg,
+Line 1494 
 keymanager_dlg_proc (HWND dlg, UINT msg,
                  lpttt->lpszText = (char *)_("Sign key");
                  break;
+             case ID_KEYCTX_COPY:
+                 lpttt->lpszText = (char *)_("Copy key to clipboard");
+                 break;
+             case ID_KEYCTX_PASTE:
+                 lpttt->lpszText = (char*)_("Paste key from clipboard");
+                 break;
              case ID_KEYMISC_IMPORT:
                  lpttt->lpszText = (char *)_("Import key to keyring");
                  break;
-Line 974 
 keymanager_dlg_proc (HWND dlg, UINT msg,
+Line 1521 
 keymanager_dlg_proc (HWND dlg, UINT msg,
              break;
          case NM_RCLICK:
+             if (!wait_for_progress_cb (0))
+                 break;
              if (notify->idFrom == IDC_KEYMISC_KEYLIST) {
                  l_idx =listview_get_curr_pos (kmi->lv);
                  if (l_idx == -1)
                      return TRUE; /* Popup only when a item was selected */
-                 do_check_cache (kmi->lv, dlg, kmi->statbar);
                  GetCursorPos (&p);
                  hm = LoadMenu (glob_hinst, MAKEINTRESOURCE (IDR_WINPT_KEYMISC_CTX));
                  popup = GetSubMenu (hm, 0);
+                 translate_popupmenu_strings (popup);
-                 set_menu_text (popup, ID_KEYCTX_UID_COPY, _("Copy User ID to Clipboard"));
-                 set_menu_text (popup, ID_KEYCTX_KEYID_COPY, _("Copy Key ID to Clipboard"));
-                 set_menu_text (popup, ID_KEYCTX_FPR_COPY, _("Copy Fingerprint to Clipboard"));
-                 set_menu_text (popup, ID_KEYCTX_KINFO_COPY, _("Copy Key Info to Clipboard"));
-                 set_menu_text (popup, ID_KEYCTX_COPY, _("Copy Key to Clipboard"));
-                 set_menu_text (popup, ID_KEYCTX_PASTE, _("Paste Key from Clipboard"));
-                 set_menu_text (popup, ID_KEYCTX_RECVFROM, _("Refresh from Keyserver"));
-                 set_menu_text (popup, ID_KEYCTX_MAXTRUST, _("Set Implicit &Trust"));
-                 set_menu_text (popup, ID_KEYCTX_LISTSIGS, _("&List Signatures"));
-                 set_menu_text (popup, ID_KEYCTX_PROPS, _("&Key Properties"));
-                 set_menu_text (popup, ID_KEYCTX_EDIT, _("Key Edit"));
-                 set_menu_text (popup, ID_KEYCTX_DEL, _("&Delete"));
-                 set_menu_text (popup, ID_KEYCTX_REV, _("&Revoke"));
-                 set_menu_text (popup, ID_KEYCTX_SIGN, _("&Sign"));
-                 set_menu_text (popup, ID_KEYCTX_ENABLE, _("&Enable"));
-                 set_menu_text (popup, ID_KEYCTX_DISABLE, _("&Disable"));
-                 set_menu_text (popup, ID_KEYCTX_RECVFROM, _("Re&fresh from Keyserver"));
-                 set_menu_text (popup, ID_KEYCTX_SETPREFKS, _("Set preferred Keyserver URL"));
-                 set_menu_text (popup, ID_KEYCTX_SENDMAIL, _("Send Key to Mail Recipient"));
-                 set_menu_text (popup, ID_KEYCTX_SETDEFKEY, _("Set as Default Key"));
-                 set_menu_text (popup, ID_KEYCTX_ADDKEY, _("Key..."));
-                 set_menu_text (popup, ID_KEYCTX_ADDUID, _("User ID..."));
-                 set_menu_text (popup, ID_KEYCTX_ADDPHOTO, _("Photo ID..."));
-                 set_menu_text (popup, ID_KEYCTX_ADDREV, _("Revoker..."));
-                 /* change popup texts */
-                 set_menu_text_bypos (popup, 0, _("Key Attributes"));
-                 set_menu_text_bypos (popup, 6, _("Add"));
-                 set_menu_text_bypos (popup, 19, _("Send to Keyserver"));
                  if (km_check_for_seckey (kmi->lv, l_idx, &i))
                      set_menu_state (popup, ID_KEYCTX_SETDEFKEY, MF_ENABLED);
                  if (i == 0)
                      set_menu_state (popup, ID_KEYCTX_MAXTRUST, MF_ENABLED);
-                 if (!km_check_for_seckey (kmi->lv, l_idx, NULL)) {
+                 if (!km_check_for_seckey (kmi->lv, l_idx, NULL) ||
+                     (km_get_key_status (kmi->lv, l_idx) & KM_FLAG_REVOKED)) {
                      set_menu_state (popup, ID_KEYCTX_REV, MF_DISABLED|MF_GRAYED);
                      set_menu_state (popup, ID_KEYCTX_ADDKEY, MF_DISABLED|MF_GRAYED);
                      set_menu_state (popup, ID_KEYCTX_ADDUID, MF_DISABLED|MF_GRAYED);
-Line 1045 
 keymanager_dlg_proc (HWND dlg, UINT msg,
+Line 1564 
 keymanager_dlg_proc (HWND dlg, UINT msg,
                      set_menu_state (popup, ID_KEYCTX_PASTE, MF_DISABLED|MF_GRAYED);
                  if (mapi_init ())
                      set_menu_state (popup, ID_KEYCTX_SENDMAIL, MF_DISABLED|MF_GRAYED);
+                 if (!secret_key_available ())
+                     set_menu_state (popup, ID_KEYCTX_SIGN, MF_DISABLED|MF_GRAYED);
                  /* Override 'Default Keyserver' with the actual name. */
                  host = kserver_get_hostname (0, -1, &port);
+                 if (!host)
+                     host = DEF_HKP_KEYSERVER;
                  set_menu_text (popup, ID_KEYCTX_KS_DEFAULT, host);
+                 {
+                     HMENU ks = GetSubMenu (popup, 19);
+                     for (i=0; server[i].name != NULL; i++)
+                         insert_menu_item (ks, i+2, WM_APP+i, server[i].name);
+                 }
                  popup_gpg_readonly (dlg, popup);
                  if (listview_count_items (kmi->lv, 1) > 1)
                      popup_multiple (dlg, popup);
-Line 1056 
 keymanager_dlg_proc (HWND dlg, UINT msg,
+Line 1585 
 keymanager_dlg_proc (HWND dlg, UINT msg,
                  DestroyMenu (hm);
                  return TRUE;
              }
-             #if 0 /* XXX */
-             if( notify->idFrom == IDC_KEYMISC_GROUP ) {
-                 HWND tree = GetDlgItem( dlg, IDC_KEYMISC_GROUP );
-                 if( TreeView_GetSelection( tree ) ) {
-                     GetCursorPos( &p );
-                     hm = LoadMenu( glob_hinst, MAKEINTRESOURCE(IDR_WINPT_GROUP_CTX) );
-                     popup = GetSubMenu (hm, 0);
-                     if (km_index == -1) /* XXX */
-                         set_menu_state( popup, ID_GROUP_PASTE, MF_DISABLED|MF_GRAYED );
-                     set_menu_text( popup, ID_GROUP_PASTE, _("Paste into this group") );
-                     set_menu_text( popup, ID_GROUP_DELETE, _("Delete") );
-                     TrackPopupMenu( popup, TPM_RIGHTALIGN, p.x, p.y, 0, dlg, NULL );
-                     DestroyMenu( popup );
-                     DestroyMenu( hm );
-                     return TRUE;
-                 }
-             }
-             #endif
              break;
          case LVN_COLUMNCLICK:
              if (notify->idFrom == IDC_KEYMISC_KEYLIST) {
                  NMLISTVIEW *nft = (LPNMLISTVIEW) lparam;
                  int sortby = 0;
+                 if (!wait_for_progress_cb (0))
+                     break;
                  switch (nft->iSubItem) {
                  case 0:  sortby = KEY_SORT_USERID; break;
                  case 1:  sortby = KEY_SORT_KEYID; break;
-Line 1089 
 keymanager_dlg_proc (HWND dlg, UINT msg,
+Line 1603 
 keymanager_dlg_proc (HWND dlg, UINT msg,
                  case 6:  sortby = KEY_SORT_OTRUST; break;
                  case 7:  sortby = KEY_SORT_CREATED; break;
                  case 8:  sortby = KEY_SORT_ALGO; break;
-                 default: return TRUE; //sortby = KEY_SORT_USERID; break;
+                 default: return TRUE; /*sortby userid.*/
                  }
                  if ((kmi->keylist_sortby & ~KEYLIST_SORT_DESC) == sortby)
-Line 1125 
 keymanager_dlg_proc (HWND dlg, UINT msg,
+Line 1639 
 keymanager_dlg_proc (HWND dlg, UINT msg,
          break;
      case WM_INITMENUPOPUP:
-         if ((UINT)LOWORD (lparam) == 3) {
+         hm = (HMENU)wparam;
-             HMENU h = (HMENU)wparam;
+         if ((UINT)LOWORD (lparam) == 3)
-             set_menu_text_bypos (h, 0, _("New"));
+             set_menu_text_bypos (hm, 0, _("New"));
-         }
+         else if ((UINT)LOWORD (lparam) == 1)
+             set_menu_text_bypos (hm, 8, _("Preferences..."));
          return FALSE;
      case WM_COMMAND:
-Line 1138 
 keymanager_dlg_proc (HWND dlg, UINT msg,
+Line 1653 
 keymanager_dlg_proc (HWND dlg, UINT msg,
                       _("Key Manager"), MB_ERR);
              return FALSE;
          }
-         do_check_cache (kmi->lv, dlg, kmi->statbar);
-         switch( LOWORD (wparam)) {
+         /*wait_for_progress_cb ();*/
+         switch (LOWORD (wparam)) {
          case ID_KEYMISC_QUIT:
              EndDialog (dlg, TRUE);
              return TRUE;
-         case ID_KEYMISC_MAIL:
-             /* XXX
-             DialogBoxParam (glob_hinst, (LPCTSTR)IDD_WINPT_MAIL, GetDesktopWindow (),
-                             winpt_mail_proc, NULL);*/
-             break;
          case ID_KEYMISC_FIND:
              km_find_key (dlg, kmi->lv);
-Line 1157 
 keymanager_dlg_proc (HWND dlg, UINT msg,
+Line 1667 
 keymanager_dlg_proc (HWND dlg, UINT msg,
          case ID_KEYMISC_DELETE:
          case ID_KEYMISC_DELETE2:
              if (!km_delete_keys (kmi->lv, dlg))
-                 km_update_status_bar (kmi->statbar, kmi->lv);
+                 update_status_bar (kmi->statbar, kmi->lv);
              return TRUE;
          case ID_KEYMISC_SIGN:
              if (kmi->lv_idx == -1) {
                  msg_box (dlg, _("Please select a key."),
                           _("Key Manager"), MB_ERR);
-                 return TRUE;;
+                 return TRUE;
              }
              if (km_check_key_status (kmi->lv, kmi->lv_idx))
                  return TRUE;
-             key = (gpgme_key_t)listview_get_item2 (kmi->lv, kmi->lv_idx);
+             km_get_key (kmi->lv, kmi->lv_idx, &k);
-             listview_get_item_text (kmi->lv, kmi->lv_idx, 1, keyid, DIM (keyid)-1);
-             memset (&k, 0, sizeof (k));
-             k.ctx = key;
-             k.keyid = keyid;
              dialog_box_param (glob_hinst, (LPCSTR)IDD_WINPT_KEYSIGN, dlg,
                                keysign_dlg_proc, (LPARAM)&k,
                                _("Key Signing"), IDS_WINPT_KEYSIGN);
              if (k.update)
-                 update_key (kmi->lv, kmi->lv_idx, k.keyid, 0);
+                 update_key (kmi->lv, kmi->lv_idx, k.tmp_keyid, 0);
              return TRUE;
          case ID_KEYMISC_REVCERT:
-Line 1185 
 keymanager_dlg_proc (HWND dlg, UINT msg,
+Line 1691 
 keymanager_dlg_proc (HWND dlg, UINT msg,
                  msg_box (dlg, _("Please select a key."), _("Key Manager"), MB_ERR);
                  return TRUE;
              }
-             listview_get_item_text (kmi->lv, kmi->lv_idx, 0, uid, sizeof uid-1);
-             listview_get_item_text (kmi->lv, kmi->lv_idx, 1, keyid, sizeof keyid-1);
-             if ( !km_check_for_seckey( kmi->lv, kmi->lv_idx, NULL ) ) {
-                 msg_box (dlg, _("There is no secret key available!"),
-                         _("Key Manager"), MB_ERR);
-                 return TRUE;
-             }
              {
                  char state[64];
                  listview_get_item_text (kmi->lv, kmi->lv_idx, 5,
                                          state, sizeof (state) -1);
-                 if( strchr (state, 'R' )) {
+                 if (strchr (state, 'R' )) {
                      msg_box (dlg, _("Key already revoked!"),
                               _("Key Manager"), MB_INFO);
                      return TRUE;
                  }
              }
-             memset (&k, 0, sizeof (k));
+             km_get_key (kmi->lv, kmi->lv_idx, &k);
-             k.key_pair = 1;
+             if (!k.key_pair) {
-             k.keyid = keyid;
+                 msg_box (dlg, _("There is no secret key available!"),
-             k.is_protected = km_check_if_protected (kmi->lv, kmi->lv_idx);
+                         _("Key Manager"), MB_ERR);
+                 return TRUE;
+             }
              dialog_box_param(glob_hinst, (LPCSTR)IDD_WINPT_KEYREVOKE, dlg,
                               key_revoke_dlg_proc, (LPARAM)&k,
-                              _("Key Revocation"), IDS_WINPT_KEYREVOKE);
+                              _("Key Revocation Cert"), IDS_WINPT_KEYREVOKE);
              return TRUE;
          case ID_KEYMISC_TRUSTPATH:
-Line 1218 
 keymanager_dlg_proc (HWND dlg, UINT msg,
+Line 1719 
 keymanager_dlg_proc (HWND dlg, UINT msg,
                  msg_box (dlg, _("Please select a key."), _("Key Manager"), MB_ERR);
                  return TRUE;
              }
-             listview_get_item_text( kmi->lv, kmi->lv_idx, 0, uid, sizeof uid -1 );
+             km_get_key (kmi->lv, kmi->lv_idx, &k);
-             listview_get_item_text( kmi->lv, kmi->lv_idx, 1, keyid, sizeof keyid -1 );
+             dialog_box_param (glob_hinst, (LPCSTR)IDD_WINPT_KEYTRUST, dlg,
-             if( km_check_for_seckey( kmi->lv, kmi->lv_idx, NULL ) ) {
-                 msg_box( dlg, _("It does not make any sense with a key pair!"), _("Key Manager"), MB_OK );
-                 return TRUE;
-             }
-             memset (&k, 0, sizeof (k));
-             k.keyid = keyid;
-             k.uid = uid;
-             dialog_box_param( glob_hinst, (LPCSTR)IDD_WINPT_KEYTRUST, dlg,
                                keytrust_dlg_proc, (LPARAM)&k,
-                               _("List Trust Path"), IDS_WINPT_KEYTRUST );
+                               _("List Trust Path"), IDS_WINPT_KEYTRUST);
              return TRUE;
          case ID_KEYMISC_CHECKSIGS:
-Line 1237 
 keymanager_dlg_proc (HWND dlg, UINT msg,
+Line 1730 
 keymanager_dlg_proc (HWND dlg, UINT msg,
                  msg_box (dlg, _("Please select a key."), _("Key Manager"), MB_ERR);
                  return TRUE;
              }
-             listview_get_item_text (kmi->lv, kmi->lv_idx, 0, uid, DIM (uid)-1);
+             km_get_key (kmi->lv, kmi->lv_idx, &k);
-             listview_get_item_text (kmi->lv, kmi->lv_idx, 1, keyid, DIM (keyid)-1);
+             dialog_box_param (glob_hinst, (LPCSTR)IDD_WINPT_KEYSIG_TREE, dlg,
-             memset (&k, 0, sizeof (k));
+                               sigtree_dlg_proc, (LPARAM)&k,
-             k.keyid = keyid;
+                               _("Key Signature List"), IDS_WINPT_KEYSIG);
-             k.uid = uid;
+             if (k.update)
-             k.ctx = (gpgme_key_t)listview_get_item2 (kmi->lv, kmi->lv_idx);
+                 update_key (kmi->lv, kmi->lv_idx, k.tmp_keyid, 0);
-             dialog_box_param (glob_hinst, (LPCSTR)IDD_WINPT_KEYSIG, dlg,
+             if (updated_keys_avail ())
-                               keysig_dlg_proc, (LPARAM)&k,
+                 refresh_keylist (kmi);
-                               _("Key Signature List" ), IDS_WINPT_KEYSIG);
              return TRUE;
          case ID_KEYMISC_PROPS:
              if (kmi->lv_idx == -1) {
-                 msg_box( dlg, _("Please select a key."), _("Key Manager"), MB_ERR );
+                 msg_box (dlg, _("Please select a key."), _("Key Manager"), MB_ERR);
                  return TRUE;
              }
-             listview_get_item_text (kmi->lv, kmi->lv_idx, 1, keyid, DIM (keyid)-1);
+             km_get_key (kmi->lv, kmi->lv_idx, &k);
-             listview_get_item_text (kmi->lv, kmi->lv_idx, 2, type, DIM (type)-1);
-             memset (&k, 0, sizeof (k));
-             k.key_pair = 0;
-             k.keyid = keyid;
-             if( !strcmp( type, "pub/sec" ) || !strcmp( type, "pub/crd" ) )
-                 k.key_pair = 1;
              k.callback.ctl = kmi->lv;
              k.callback.idx = kmi->lv_idx;
-             k.is_v3 = km_key_is_v3 (kmi->lv, kmi->lv_idx);
+             dialog_box_param (glob_hinst, (LPCSTR)IDD_WINPT_KEYPROPS, dlg,
-             dialog_box_param( glob_hinst, (LPCSTR)IDD_WINPT_KEYPROPS, dlg,
                                keyprops_dlg_proc, (LPARAM)&k,
-                               _("Key Properties"), IDS_WINPT_KEYPROPS );
+                               _("Key Properties"), IDS_WINPT_KEYPROPS);
-             if (k.callback.new_val != 0) {
+             if (k.update)
-                 t = get_key_trust_str (k.callback.new_val);
+                 update_key (kmi->lv, kmi->lv_idx, k.tmp_keyid, k.key_pair);
-                 listview_add_sub_item (kmi->lv, kmi->lv_idx, 6, t);
-             }
              return TRUE;
          case ID_KEYMISC_RECACHE:
-             /* If there is already a reload request, don't bother the user with a message. */
+             if (updated_keys_avail ())
-             if (keycache_get_reload() == 1)
                  l_idx = IDYES;
              else {
-                 char inf[256];
+                 l_idx = log_box (_("Key Manager"), MB_YESNO,
-                 _snprintf (inf, sizeof (inf) -1,
                             _("This is only useful when the keyring has been "
                               "modified (sign a key...).\n"
                               "Do you really want to reload the keycache?"));
-                 l_idx = msg_box (dlg, inf, _("Key Manager"), MB_YESNO);
-             }
-             if (l_idx == IDYES) {
-                 rcs.kr_reload = rcs.kr_update = 1;
-                 rcs.tr_update = 0;
-                 DialogBoxParam (glob_hinst, (LPCSTR)IDD_WINPT_KEYCACHE, dlg,
-                                 keycache_dlg_proc, (LPARAM)&rcs);
-                 c = keycache_get_ctx (1);
-                 if (!c)
-                     BUG (dlg);
-                 keylist_reload (kmi->lv, c, KEYLIST_LIST, KEY_SORT_USERID);
-                 refresh_keys = 0;
              }
+             if (l_idx == IDYES)
+                 reload_keycache (kmi);
              return TRUE;
          case ID_KEYMISC_REBUILD:
-Line 1305 
 keymanager_dlg_proc (HWND dlg, UINT msg,
+Line 1776 
 keymanager_dlg_proc (HWND dlg, UINT msg,
                  show_msg (dlg, 2000, line? name + (line-name)+1 : name);
                  safe_free (name);
              }
+             SetForegroundWindow (dlg);
              return TRUE;
          case ID_KEYMISC_NEWKEY:
-Line 1312 
 keymanager_dlg_proc (HWND dlg, UINT msg,
+Line 1784 
 keymanager_dlg_proc (HWND dlg, UINT msg,
              dialog_box_param (glob_hinst, (LPCSTR)IDD_WINPT_KEYGEN, dlg,
                                keygen_dlg_proc, (LPARAM)&genkey, _("Key Generation"),
                                IDS_WINPT_KEYGEN);
-             if (genkey.newkey != NULL) {
+             if (genkey.cancel == 0)
-                 keylist_add_key (kmi->lv, KEYLIST_LIST, genkey.newkey);
+                 refresh_keylist (kmi);
-                 keylist_sort (kmi->lv, KEY_SORT_USERID);
-             }
              return TRUE;
          case ID_KEYMISC_CARDNEW:
-Line 1327 
 keymanager_dlg_proc (HWND dlg, UINT msg,
+Line 1797 
 keymanager_dlg_proc (HWND dlg, UINT msg,
              dialog_box_param (glob_hinst, (LPCSTR)IDD_WINPT_CARD_KEYGEN, dlg,
                                card_keygen_dlg_proc, 0, _("Card Key Generation"),
                                IDS_WINPT_CARD_KEYGEN);
-             /* XXX: use new code */
+             if (updated_keys_avail ())
-             if (keycache_get_reload ())
                  send_cmd_id (dlg, ID_KEYMISC_RECACHE);
              return TRUE;
-Line 1338 
 keymanager_dlg_proc (HWND dlg, UINT msg,
+Line 1807 
 keymanager_dlg_proc (HWND dlg, UINT msg,
                                keygen_wizard_dlg_proc, (LPARAM)&genkey,
                                _("Key Generation Wizard"),
                                IDS_WINPT_KEYWIZARD);
-             if (genkey.newkey != NULL) {
+             if (genkey.cancel == 0)
-                 keylist_add_key (kmi->lv, KEYLIST_LIST, genkey.newkey);
+                 refresh_keylist (kmi);
-                 keylist_sort (kmi->lv, KEY_SORT_USERID);
-             }
              return TRUE;
          case ID_KEYMISC_SENDRECV:
-             memset (&genkey, 0, sizeof (genkey));
              dialog_box_param (glob_hinst, (LPCSTR)IDD_WINPT_KEYSERVER, dlg,
-                               keyserver_dlg_proc, (LPARAM)&genkey, _("Keyserver Access"),
+                               keyserver_dlg_proc, 0, _("Keyserver Access"),
                                IDS_WINPT_KEYSERVER);
-             if (genkey.newkey != NULL) {
+             refresh_keylist (kmi);
-                 keylist_add_key (kmi->lv, KEYLIST_LIST, genkey.newkey);
-                 keylist_sort (kmi->lv, KEY_SORT_USERID);
-             }
              return TRUE;
+         case ID_KEYMISC_WINPTPREFS:
+             dialog_box_param (glob_hinst, (LPCSTR)IDD_WINPT_PREFS,
+                               dlg, prefs_dlg_proc, 0,
+                               _("WinPT Preferences"), 0);
+             break;
          case ID_KEYMISC_GPGPREFS:
-             dialog_box_param( glob_hinst, (LPCSTR)IDD_WINPT_GPGPREFS, dlg,
+             rc = dialog_box_param (glob_hinst, (LPCSTR)IDD_WINPT_GPGPREFS,
-                               gpgprefs_dlg_proc, 0, _("GnuPG Preferences"),
+                                    dlg, gpgprefs_dlg_proc, 0,
-                               IDS_WINPT_GPGPREFS );
+                                    _("GnuPG Preferences"), IDS_WINPT_GPGPREFS);
+             if (rc == TRUE) {
+                 reload_keycache (kmi);
+                 update_default_key_str (kmi->statbar);
+             }
              return TRUE;
          case ID_KEYMISC_GPGOPT:
-             dialog_box_param( glob_hinst, (LPCSTR)IDD_WINPT_GPGOPT, dlg,
+             dialog_box_param (glob_hinst, (LPCSTR)IDD_WINPT_GPGOPT, dlg,
-                               gpgopt_dlg_proc, 0, _("GnuPG Options" ),
+                               gpgopt_dlg_proc, 0, _("GnuPG Options"),
-                               IDS_WINPT_GPGOPT );
+                               IDS_WINPT_GPGOPT);
              return TRUE;
          case ID_KEYMISC_IMPORT:
-             t = get_fileopen_dlg (dlg, _("Choose Name of the Key File"), NULL, NULL);
+             km_gui_import (kmi, LOWORD (wparam), NULL);
-             if (t)
-                 km_file_import (dlg, t);
              return TRUE;
          case ID_KEYMISC_IMPORT_HTTP:
              url = (struct URL_ctx_s*)get_http_file_dlg (dlg);
-             if (url->cancel == 0)
+             if (url && url->cancel == 0) {
                  km_http_import (dlg, url->url);
-             delete url; url=NULL;
+                 refresh_keylist (kmi);
+             }
+             free_if_alloc (url);
              break;
          case ID_KEYMISC_EXPORT:
              if (kmi->lv_idx == -1) {
-                 msg_box( dlg, _("Please select a key."), _("Key Manager"), MB_ERR );
+                 msg_box (dlg, _("Please select a key."),
+                          _("Key Manager"), MB_ERR);
                  return TRUE;
              }
              if (listview_count_items (kmi->lv, 1) > 1)
                  name = m_strdup ("Exported_GPG_Keys.asc");
              else {
-                 listview_get_item_text (kmi->lv, kmi->lv_idx, 1, keyid, DIM (keyid)-1);
+                 key = km_get_key_ptr (kmi->lv, kmi->lv_idx, NULL);
-                 name = gen_export_filename (keyid, 0);
+                 name = km_gen_export_filename (key->subkeys->keyid+8, 0);
              }
              t = get_filesave_dlg (dlg, _("Choose Name for Key File"), NULL, name);
              free_if_alloc (name);
-Line 1420 
 keymanager_dlg_proc (HWND dlg, UINT msg,
+Line 1894 
 keymanager_dlg_proc (HWND dlg, UINT msg,
                              "to copy the key to a safe place.\n\n"
                              "Do you really want to export the key?"),
                            _("WARNING"), MB_INFO|MB_YESNO);
-             if( i == IDYES ) {
+             if (i == IDYES) {
-                 listview_get_item_text( kmi->lv, kmi->lv_idx, 1, keyid, sizeof (keyid)-8 );
+                 key = km_get_key_ptr (kmi->lv, kmi->lv_idx, NULL);
-                 name = gen_export_filename (keyid, 1);
+                 name = km_gen_export_filename (key->subkeys->keyid+8, 1);
                  t = get_filesave_dlg (dlg, _("Choose Name for Key File"), NULL, name);
                  if (t != NULL)
                      km_privkey_export (dlg, kmi->lv, t);
-Line 1430 
 keymanager_dlg_proc (HWND dlg, UINT msg,
+Line 1904 
 keymanager_dlg_proc (HWND dlg, UINT msg,
              return TRUE;
          case ID_KEYMISC_INFO:
-             dialog_box_param (glob_hinst, (LPCSTR)IDD_WINPT_ABOUT, glob_hwnd,
+             dialog_box_param (glob_hinst, (LPCSTR)IDD_WINPT_ABOUT, dlg,
                                about_winpt_dlg_proc, 0, _("About WinPT"),
                                IDS_WINPT_ABOUT);
              break;
+         case ID_KEYMISC_DEVWWW:
+             ShellExecute (dlg, "open", "http://wald.intevation.org/projects/winpt",
+                           NULL, NULL, SW_SHOW);
+             break;
+         case ID_KEYMISC_WEBSITE:
+             ShellExecute (dlg, "open", "http://www.winpt.org",
+                           NULL, NULL, SW_SHOW);
+             break;
          case ID_KEYMISC_HELP:
-             ShellExecute (dlg, "open", "winpt.chm", NULL, NULL, SW_SHOW);
+             start_help (dlg, 0);
              break;
          case ID_KEYMISC_OT:
-             dialog_box_param( glob_hinst, (LPCTSTR)IDD_WINPT_OWNERTRUST, glob_hwnd,
+             dialog_box_param (glob_hinst, (LPCTSTR)IDD_WINPT_OWNERTRUST,
-                               ownertrust_dlg_proc, 0,
+                               dlg, ownertrust_dlg_proc, 0,
-                               _("Ownertrust"), IDS_WINPT_OWNERTRUST );
+                               _("Ownertrust"), IDS_WINPT_OWNERTRUST);
              break;
          case ID_KEYMISC_EDITKEY:
-             if (kmi->lv_idx == -1)
+             if (km_get_key (kmi->lv, kmi->lv_idx, &k))
                  break;
-             listview_get_item_text (kmi->lv, kmi->lv_idx, 1, keyid, sizeof (keyid)-1);
-             /* XXX: pub/crd = secret key does not work */
-             memset (&k, 0, sizeof (k));
-             k.is_protected = km_check_if_protected (kmi->lv, kmi->lv_idx);
-             k.key_pair = km_check_for_seckey (kmi->lv, kmi->lv_idx, NULL);
-             k.keyid = keyid;
-             k.is_v3 = km_key_is_v3 (kmi->lv, kmi->lv_idx);
-             k.flags = km_get_key_status (kmi->lv, kmi->lv_idx);
              dialog_box_param (glob_hinst, (LPCTSTR)IDD_WINPT_KEYEDIT, dlg,
                                keyedit_main_dlg_proc, (LPARAM)&k,
                                _("Key Edit"), IDS_KEYCTX_EDIT);
              if (k.update)
-                 update_key (kmi->lv,  kmi->lv_idx, keyid, 1);
+                 update_key (kmi->lv,  kmi->lv_idx, k.tmp_keyid, 1);
              break;
          case ID_KEYMISC_COPY:
-Line 1472 
 keymanager_dlg_proc (HWND dlg, UINT msg,
+Line 1948 
 keymanager_dlg_proc (HWND dlg, UINT msg,
              break;
          case ID_KEYMISC_PASTE:
-             km_clip_import (dlg);
+             km_gui_import (kmi, LOWORD (wparam), NULL);
              break;
          case ID_KEYCTX_SETPREFKS:
-             listview_get_item_text (kmi->lv, kmi->lv_idx, 1, keyid, DIM(keyid)-1);
+             if (km_get_key (kmi->lv, kmi->lv_idx, &k))
-             memset (&k, 0, sizeof (k));
+                 break;
-             k.keyid = keyid;
              keyedit_set_pref_keyserver (&k, dlg);
              break;
          case ID_KEYMISC_REFRESH_KEYS:
              if (listview_count_items (kmi->lv, 1) == 0) {
-                 msg_box (dlg, _("No key was selected, select all by default."), _("Key Manager"), MB_INFO);
+                 msg_box (dlg, _("No key was selected, select all by default."),
+                          _("Key Manager"), MB_INFO);
                  listview_select_all (kmi->lv);
              }
-             km_refresh_from_keyserver (kmi->lv, dlg);
+             if (!km_refresh_from_keyserver (kmi->lv, dlg))
+                 msg_box (dlg, _("Keyserver refresh finished."),
+                          _("Key Manager"), MB_OK);
              break;
          /** Context menu **/
-Line 1525 
 keymanager_dlg_proc (HWND dlg, UINT msg,
+Line 2003 
 keymanager_dlg_proc (HWND dlg, UINT msg,
              break;
          case ID_KEYCTX_ADDKEY:
-             listview_get_item_text (kmi->lv, kmi->lv_idx, 1, keyid, DIM (keyid)-1);
+             km_get_key (kmi->lv, kmi->lv_idx, &k);
-             memset (&k, 0, sizeof (k));
-             k.key_pair = km_check_for_seckey (kmi->lv, kmi->lv_idx, NULL);
-             k.is_protected = km_check_if_protected (kmi->lv, kmi->lv_idx);
-             k.keyid = keyid;
              keyedit_add_subkey (&k, dlg, NULL);
              if (k.update)
-                 update_key (kmi->lv, kmi->lv_idx, keyid, 1);
+                 update_key (kmi->lv, kmi->lv_idx, k.tmp_keyid, 1);
              break;
          case ID_KEYCTX_ADDUID:
-             listview_get_item_text( kmi->lv, kmi->lv_idx, 1, keyid, DIM (keyid)-1);
+             km_get_key (kmi->lv, kmi->lv_idx, &k);
-             memset (&k, 0, sizeof (k));
-             k.key_pair = km_check_for_seckey (kmi->lv, kmi->lv_idx, NULL);
-             k.is_protected = km_check_if_protected (kmi->lv, kmi->lv_idx);
-             k.keyid = keyid;
              keyedit_add_userid (&k, dlg, NULL);
              if (k.update)
-                 update_key (kmi->lv, kmi->lv_idx, keyid, 1);
+                 update_key (kmi->lv, kmi->lv_idx, k.tmp_keyid, 1);
              break;
          case ID_KEYCTX_ADDREV:
-             listview_get_item_text (kmi->lv, kmi->lv_idx, 1, keyid, DIM (keyid)-1);
+             km_get_key (kmi->lv, kmi->lv_idx, &k);
-             memset (&k, 0, sizeof (k));
-             k.keyid = keyid;
-             k.is_protected = km_check_if_protected (kmi->lv, kmi->lv_idx);
-             k.key_pair = km_check_for_seckey( kmi->lv, kmi->lv_idx, NULL );
              keyedit_add_revoker (&k, dlg);
              if (k.update)
-                 update_key (kmi->lv, kmi->lv_idx, keyid, 1);
+                 update_key (kmi->lv, kmi->lv_idx, k.tmp_keyid, 1);
              break;
          case ID_KEYCTX_ADDPHOTO:
-             listview_get_item_text (kmi->lv, kmi->lv_idx, 1, keyid, DIM (keyid)-1);
+             km_get_key (kmi->lv, kmi->lv_idx, &k);
-             memset (&k, 0, sizeof (k));
-             k.keyid = keyid;
-             k.is_protected = km_check_if_protected (kmi->lv, kmi->lv_idx);
-             k.key_pair = km_check_for_seckey (kmi->lv, kmi->lv_idx, NULL);
              keyedit_add_photo (&k, dlg);
              if (k.update)
-                 update_key (kmi->lv, kmi->lv_idx, keyid, 1);
+                 update_key (kmi->lv, kmi->lv_idx, k.tmp_keyid, 1);
              break;
-         case ID_KEYCTX_KS_NL:
+         case ID_KEYCTX_RECVFROM:
-         case ID_KEYCTX_KS_PL:
-         case ID_KEYCTX_KS_AT:
-         case ID_KEYCTX_KS_DE:
-         case ID_KEYCTX_KS_DK:
-         case ID_KEYCTX_KS_CZ:
-         case ID_KEYCTX_KS_ES:
-         case ID_KEYCTX_KS_UK:
-             host = kserver_get_hostname (LOWORD (wparam) - 40107, 0, &port);
-             km_send_to_keyserver (kmi->lv, dlg, host, port);
-             break;
-         case ID_KEYCTX_RECVFROM:
              km_refresh_from_keyserver (kmi->lv, dlg);
+             if (updated_keys_avail ())
+                 refresh_keylist (kmi);
              break;
          case ID_KEYCTX_UID_COPY:
-             /* XXX: add generic function to support multiple selection
-                     with a callback */
-             listview_get_item_text( kmi->lv, kmi->lv_idx, 0, uid, sizeof uid-1 );
-             set_clip_text( NULL, uid, strlen( uid ) );
-             break;
          case ID_KEYCTX_KEYID_COPY:
-             listview_get_item_text (kmi->lv, kmi->lv_idx, 1, uid, sizeof uid-1);
-             set_clip_text (NULL, uid, strlen (uid));
-             break;
          case ID_KEYCTX_FPR_COPY:
-             key = (gpgme_key_t) listview_get_item2 (kmi->lv, kmi->lv_idx);
-             if (key) {
-                 t = get_key_fpr (key);
-                 set_clip_text (NULL, t? t : "", t? strlen (t): 0);
-             }
-             break;
          case ID_KEYCTX_KINFO_COPY:
-             listview_get_item_text( kmi->lv, kmi->lv_idx, 1, uid, sizeof uid-1 );
+         case ID_KEYCTX_EMAIL_COPY:
-             km_set_clip_info( uid );
+             copy_key_info (LOWORD (wparam), kmi);
              break;
          case ID_KEYCTX_COPY:
-             km_clip_export (dlg, kmi->lv);
+             send_cmd_id (dlg, ID_KEYMISC_COPY);
              break;
          case ID_KEYCTX_PASTE:
-             km_clip_import (dlg);
+             send_cmd_id (dlg, ID_KEYMISC_PASTE);
              break;
          case ID_KEYCTX_DISABLE:
          case ID_KEYCTX_ENABLE:
              i = LOWORD (wparam) == ID_KEYCTX_ENABLE? 1 : 0;
-             listview_get_item_text (kmi->lv, kmi->lv_idx, KM_COL_KEYID,
+             if (km_get_key (kmi->lv, kmi->lv_idx, &k))
-                                     keyid, DIM (keyid)-1);
+                 break;
              rc = km_enable_disable_key (kmi->lv, dlg, kmi->lv_idx, i);
              if (!rc)
-                 update_key (kmi->lv, kmi->lv_idx, keyid, 0);
+                 update_key (kmi->lv, kmi->lv_idx, k.tmp_keyid, 0);
-             /* XXX: switching a key from disabled -> enabled. does not
-                change the validity field in the KM. */
              break;
          case ID_KEYCTX_LISTSIGS:
-Line 1634 
 keymanager_dlg_proc (HWND dlg, UINT msg,
+Line 2067 
 keymanager_dlg_proc (HWND dlg, UINT msg,
              break;
          case ID_KEYCTX_MAXTRUST:
-             listview_get_item_text (kmi->lv, kmi->lv_idx, 1, keyid, DIM (keyid)-1);
+             if (km_get_key (kmi->lv, kmi->lv_idx, &k))
+                 break;
              rc = km_set_implicit_trust (dlg, kmi->lv, kmi->lv_idx);
              if (!rc)
-                 update_key (kmi->lv, kmi->lv_idx, keyid, 0);
+                 update_key (kmi->lv, kmi->lv_idx, k.tmp_keyid, 0);
              break;
          case ID_KEYCTX_SETDEFKEY:
              if (!km_check_key_status (kmi->lv, kmi->lv_idx)) {
-                 listview_get_item_text (kmi->lv, kmi->lv_idx, 1, keyid, DIM (keyid)-1);
+                 key = km_get_key_ptr (kmi->lv, kmi->lv_idx, NULL);
-                 rc = set_gnupg_default_key (keyid);
+                 rc = set_gnupg_default_key (key->subkeys->keyid+8);
                  if (rc)
                      msg_box (dlg, winpt_strerror (rc), _("Key Manager"), MB_ERR);
-                 km_update_default_key_str (kmi->statbar);
+                 update_default_key_str (kmi->statbar);
              }
              break;
+         case ID_KEYMISC_VIEWKEYID:
+         case ID_KEYMISC_VIEWCIPHER:
+         case ID_KEYMISC_VIEWTYPE:
+         case ID_KEYMISC_VIEWCREAT:
+         case ID_KEYMISC_VIEWDESC:
+             hm = GetMenu (dlg);
+             i = get_menu_state (hm, LOWORD (wparam));
+             set_menu_state (hm, LOWORD (wparam),
+                             i & MFS_CHECKED? MFS_UNCHECKED : MFS_CHECKED);
+             modify_listview_columns (kmi, LOWORD (wparam), !(i & MFS_CHECKED));
+             break;
          case ID_GROUP_NEW:
          case ID_GROUP_PASTE:
          case ID_GROUP_DELETE:
+         case ID_GROUP_SHOW:
              /* XXX: Implement group manager. */
              return TRUE;
          }
-Line 1662 
 keymanager_dlg_proc (HWND dlg, UINT msg,
+Line 2109 
 keymanager_dlg_proc (HWND dlg, UINT msg,
      return FALSE;
  }

 Legend:



Removed from v.123
 


changed lines


 
Added in v.255
 Legend:



Removed from v.123
 


changed lines


 
Added in v.255
-Removed from v.123
+Added in v.255

[email protected]	ViewVC Help
Powered by ViewVC 1.1.26