From d83f18ab4669ae2d9b80ef06c3b16c698e5df4b0 Mon Sep 17 00:00:00 2001
From: Luke Barnard <lukeb@openmarket.com>
Date: Wed, 31 May 2017 10:03:16 +0100
Subject: [PATCH 1/3] Remove cachedPassword from localStorage on_logged_out

Fixes https://github.com/vector-im/riot-web/issues/4101
---
 src/stores/SessionStore.js | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/stores/SessionStore.js b/src/stores/SessionStore.js
index 62868e4fe4..6ec8ae7697 100644
--- a/src/stores/SessionStore.js
+++ b/src/stores/SessionStore.js
@@ -69,7 +69,9 @@ class SessionStore extends Store {
                 });
                 break;
             case 'on_logged_out':
-                this.reset();
+                this._setState({
+                    cachedPassword: null,
+                });
                 break;
         }
     }

From d0e270bd1c24244a5dc7cd09801dac60fd792432 Mon Sep 17 00:00:00 2001
From: Luke Barnard <lukeb@openmarket.com>
Date: Wed, 31 May 2017 15:13:27 +0100
Subject: [PATCH 2/3] Only re-render LoggedInView if MatrixClientPeg.get() is
 truthy

---
 src/components/structures/LoggedInView.js | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/src/components/structures/LoggedInView.js b/src/components/structures/LoggedInView.js
index a297952a2c..67e2fe8856 100644
--- a/src/components/structures/LoggedInView.js
+++ b/src/components/structures/LoggedInView.js
@@ -24,6 +24,7 @@ import PageTypes from '../../PageTypes';
 import sdk from '../../index';
 import dis from '../../dispatcher';
 import sessionStore from '../../stores/SessionStore';
+import MatrixClientPeg from '../../MatrixClientPeg';
 
 /**
  * This is what our MatrixChat shows when we are logged in. The precise view is
@@ -91,6 +92,16 @@ export default React.createClass({
         }
     },
 
+    // Child components assume that the client peg will not be null, so give them some
+    // sort of assurance here by only allowing a re-render if the client is truthy.
+    //
+    // This is required because `LoggedInView` maintains its own state and if this state
+    // updates after the client peg has been made null (during logout), then it will
+    // attempt to re-render and the children will throw errors.
+    shouldComponentUpdate: function() {
+        return Boolean(MatrixClientPeg.get());
+    },
+
     getScrollStateForRoom: function(roomId) {
         return this._scrollStateMap[roomId];
     },

From b3a862c2c2f0fe2c0381ea12936775d46c86d894 Mon Sep 17 00:00:00 2001
From: Luke Barnard <lukeb@openmarket.com>
Date: Wed, 31 May 2017 15:32:55 +0100
Subject: [PATCH 3/3] Remove redundant `reset`

---
 src/stores/SessionStore.js | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/src/stores/SessionStore.js b/src/stores/SessionStore.js
index 6ec8ae7697..c4bd39b72c 100644
--- a/src/stores/SessionStore.js
+++ b/src/stores/SessionStore.js
@@ -76,10 +76,6 @@ class SessionStore extends Store {
         }
     }
 
-    reset() {
-        this._state = Object.assign({}, INITIAL_STATE);
-    }
-
     getCachedPassword() {
         return this._state.cachedPassword;
     }