[krbdev.mit.edu #5604] NIM update inconsistency when deleting credentials

Jeffrey Altman via RT kfw-bugs at MIT.EDU
Thu Aug 2 09:48:25 EDT 2007


Kevin Koch via RT wrote:
> Patch to fix Refresh button behavior:
>
> Index: mainwnd.c
> ===================================================================
> --- mainwnd.c   (revision 19739)
> +++ mainwnd.c   (working copy)
> @@ -261,6 +261,7 @@
>          case KHUI_ACTION_VIEW_REFRESH:
>              khm_cred_refresh();
>              InvalidateRect(khm_hwnd_main_cred, NULL, FALSE);
> +            khui_action_trigger(KHUI_ACTION_LAYOUT_RELOAD, NULL);
>              return 0;
>
>          case KHUI_ACTION_PASSWD_ID:
>
As I pointed out yesterday, this is the wrong patch to apply.  What this
patch will do is reload the layout and the credentials as if the user
had toggled from basic to advanced and back.  What you have done is
altered the timing which means you have covered up a race condition but
have not actually solved the real cause of the problem that you are
experiencing in this ticket and perhaps several others.

Find the race, fix the race. 





More information about the kfwdev mailing list