Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

assertion failed: (conversations.has_key(account)) #1505

Open
bodqhrohro opened this issue Oct 29, 2023 · 2 comments · May be fixed by #1541
Open

assertion failed: (conversations.has_key(account)) #1505

bodqhrohro opened this issue Oct 29, 2023 · 2 comments · May be fixed by #1541

Comments

@bodqhrohro
Copy link
Contributor 

libdino:ERROR:/media/d/temp/git/dino/libdino/src/service/conversation_manager.vala:35:dino_conversation_manager_create_conversation: assertion failed: (conversations.has_key(account))
Bail out! libdino:ERROR:/media/d/temp/git/dino/libdino/src/service/conversation_manager.vala:35:dino_conversation_manager_create_conversation: assertion failed: (conversations.has_key(account))
Aborted

IDK how it happened, probably because I switched an account on and off too fast.
@bodqhrohro
Copy link
Contributor Author

There are also lots of

(dino:3215581): libdino-CRITICAL **: 14:12:05.857: dino_connection_manager_connection_get_uuid: assertion 'self != NULL' failed

(dino:3215581): libdino-CRITICAL **: 14:12:05.857: dino_connection_manager_connection_set_last_activity: assertion 'self != NULL' failed

before in the log.

@eerielili
Copy link
Contributor

eerielili commented Jan 21, 2024 •
edited

@fiaxh @bodqhrohro I can confirm I get the same errors and crash if you toggle an account very fast.
I found that it doesn't happen (or is greatly mitigated) by making the switch insensitive to input while the account is connecting.
Gonna push a PR shortly.

eerielili added a commit to eerielili/dino that referenced this issue Jan 21, 2024
@eerielili
Fix crash when toggling an account very fast (dino#1505)
e1d7363
@eerielili eerielili linked a pull request Jan 21, 2024 that will close this issue
Open
eerielili added a commit to eerielili/dino that referenced this issue Jan 21, 2024
@eerielili
Fix crash when toggling an account very fast (dino#1505)
d624a97
eerielili added a commit to eerielili/dino that referenced this issue Jan 21, 2024
@eerielili
Fix crash when toggling an account very fast (dino#1505)
1ca0326 
	- The switch widget in the account managment
	dialog is now not accepting input while the account
	being enabled is connecting.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix crash when toggling on and off an account very fast (#1505) eerielili/dino
2 participants
@bodqhrohro @eerielili