Fix a weird edge case with persisted empty event ids

No idea, how that happened and where it came from
4 years ago · c8a547630b
parent 795b8fb7dd
commit c8a547630b
1 changed files with 11 additions and 8 deletions
--- a/src/Cache.cpp
+++ b/src/Cache.cpp
@ -2867,6 +2867,8 @@ Cache::clearTimeline(const std::string &room_id)
                if (passed_pagination_token) {
                        if (obj.count("event_id") != 0) {
                                std::string event_id = obj["event_id"].get<std::string>();
+
+                                if (!event_id.empty()) {
                                        evToOrderDb.del(txn, event_id);
                                        eventsDb.del(txn, event_id);
                                        relationsDb.del(txn, event_id);
@ -2878,6 +2880,7 @@ Cache::clearTimeline(const std::string &room_id)
                                                msg2orderDb.del(txn, event_id);
                                        }
                                }
+                        }
                        lmdb::cursor_del(cursor);
                } else {
                        if (obj.count("prev_batch") != 0)