From 826e3e56de3e1309f91f5b09c942bbffdffb2ed3 Mon Sep 17 00:00:00 2001
From: Rom Walton <rwalton@ssl.berkeley.edu>
Date: Tue, 30 Mar 2010 20:44:13 +0000
Subject: [PATCH]     - MGR: Bad things happen when you don't initialize
 default values         for variables under wxWidgets.

    clientgui/
        TermsOfUsePage.cpp

svn path=/trunk/boinc/; revision=21032
---
 checkin_notes                | 7 +++++++
 clientgui/TermsOfUsePage.cpp | 7 ++++---
 2 files changed, 11 insertions(+), 3 deletions(-)

diff --git a/checkin_notes b/checkin_notes
index 35c91dc562..f0d0ce5ade 100644
--- a/checkin_notes
+++ b/checkin_notes
@@ -2271,3 +2271,10 @@ David  30 Mar 2010
         gui_rpc_client_ops.cpp
     client/
         acct_setup.cpp,h
+
+Rom    30 Mar 2010
+    - MGR: Bad things happen when you don't initialize default values
+        for variables under wxWidgets.
+        
+    clientgui/
+        TermsOfUsePage.cpp
diff --git a/clientgui/TermsOfUsePage.cpp b/clientgui/TermsOfUsePage.cpp
index caa36c4538..0a03bc8d6f 100644
--- a/clientgui/TermsOfUsePage.cpp
+++ b/clientgui/TermsOfUsePage.cpp
@@ -84,9 +84,10 @@ bool CTermsOfUsePage::Create( CBOINCBaseWizard* parent )
     m_pTermsOfUseCtrl = NULL;
     m_pAgreeCtrl = NULL;
     m_pDisagreeCtrl = NULL;
-    m_bUserAgrees = false;
 ////@end CTermsOfUsePage member initialisation
- 
+    m_bUserAgrees = false;
+    m_bCredentialsAlreadyAvailable = false;
+
 ////@begin CTermsOfUsePage creation
     wxWizardPageEx::Create( parent, ID_TERMSOFUSEPAGE );
 
@@ -290,7 +291,7 @@ void CTermsOfUsePage::OnTermsOfUseStatusChange( wxCommandEvent& event ) {
         SetUserAgrees(true);
     } else {
         wxLogTrace(wxT("Function Status"), wxT("CTermsOfUsePage::OnTermsOfUseStatusChange - SetUserAgrees(false)"));
-        SetUserAgrees(true);
+        SetUserAgrees(false);
     }
 
     wxLogTrace(wxT("Function Start/End"), wxT("CTermsOfUsePage::OnTermsOfUseStatusChange - Function End"));