1 files changed, 11 insertions, 6 deletions
diff --git a/lib/user.cpp b/lib/user.cpp
index 0dbc444a..f7840c40 100644
--- a/lib/user.cpp
+++ b/lib/user.cpp
@@ -119,12 +119,17 @@ void User::rename(const QString& newName, const Room* r)
         return;
     }
     // #481: take the current state and update it with the new name
-    auto evtC = r->getCurrentState<RoomMemberEvent>(id())->content();
-    Q_ASSERT_X(evtC.membership == Membership::Join, __FUNCTION__,
-               "Attempt to rename a user that's not a room member");
-    evtC.displayName = sanitized(newName);
-    r->setState<RoomMemberEvent>(id(), move(evtC));
-    // The state will be updated locally after it arrives with sync
+    if (const auto& maybeEvt = r->currentState().get<RoomMemberEvent>(id())) {
+        auto content = maybeEvt->content();
+        if (content.membership == Membership::Join) {
+            content.displayName = sanitized(newName);
+            r->setState<RoomMemberEvent>(id(), move(content));
+            // The state will be updated locally after it arrives with sync
+            return;
+        }
+    }
+    qCCritical(MEMBERS)
+        << "Attempt to rename a non-member in a room context - ignored";
 }
 
 template <typename SourceT>