Skip to content
This repository has been archived by the owner on Feb 12, 2023. It is now read-only.

qTox hangs when login errors or is cancelled (not wrong password) #5781

Closed
anthonybilinski opened this issue Aug 12, 2019 · 2 comments · Fixed by #5783
Closed

qTox hangs when login errors or is cancelled (not wrong password) #5781

anthonybilinski opened this issue Aug 12, 2019 · 2 comments · Fixed by #5783
Labels
C-bug The issue contains a bug report D-easy We estimate that the issue is easy to fix P-low

Comments

@anthonybilinski
Copy link
Member

Brief Description

OS: Ubuntu 19.04
qTox version: v1.16.3+
Commit hash: 2dde934
toxcore: v0.2.10
Qt: 5.12.2
Hardware: Desktop

Reproducible: Rarely

Steps to reproduce
  1. Attempt to open locked profile.

or

  1. As login screen but before the main screen, ctrl+c - I'm not able to consistently repro this but have seen it a couple times, I think.
Observed Behavior

qTox windows goes away, but qTox does not return - the process hangs indefinitely.

$ qtox -p qTest2
[06:17:34.955 UTC] persistence/settings.cpp:123 : Debug: "Loading settings from /home/abilinski/.config/tox/qtox.ini"
[06:17:35.022 UTC] ipc.cpp:61 : Debug: Our global IPC ID is  9129450030786964297
[06:17:35.023 UTC] main.cpp:274 : Debug: commit:  5b4d9dcd7dc04901e31d99758a712932cf43313d
[06:17:35.025 UTC] persistence/profile.cpp:131 : Warning: Failed to lock profile  "qTest2"
[06:17:35.025 UTC] persistence/settings.cpp:287 : Warning: "Could not load new settings (profile change to nullptr)"


^Z
Expected Behavior

qTox exits.

Additional Info

I think this started happening since 0a30a65

@anthonybilinski anthonybilinski added P-low D-easy We estimate that the issue is easy to fix labels Aug 12, 2019
@sudden6
Copy link
Member

sudden6 commented Aug 12, 2019

ping @Kribylet since you were the author of the suspected PR

Kribylet added a commit to Kribylet/qTox that referenced this issue Aug 12, 2019
Kribylet added a commit to Kribylet/qTox that referenced this issue Aug 12, 2019
@sudden6 sudden6 added the C-bug The issue contains a bug report label Aug 13, 2019
@anthonybilinski
Copy link
Member Author

I just hit this again running 93c9eef, which includes the linked PR.

Repro steps were simply to open login window, then hit escape.

The login window closes and qTox doesn't exit.

Kribylet added a commit to Kribylet/qTox that referenced this issue Oct 16, 2019
Qt triggers reject instead of closeEvent when escape is pressed. This commit
overrides ::reject and calls ::closeEvent, so behavior is unified.

Fixes qTox#5781
Kribylet added a commit to Kribylet/qTox that referenced this issue Oct 17, 2019
Qt triggers reject instead of closeEvent when escape is pressed. This commit
overrides ::reject and calls ::closeEvent, so behavior is unified.

Fixes qTox#5781
Kribylet added a commit to Kribylet/qTox that referenced this issue Oct 18, 2019
…ards

This commit makes LoginScreen return QDialog::Rejected (0) and
QDialog::Accepted (1) instead of C standard return values. This should be more
robust with regards to special cases in Qt implementation.

Fixes qTox#5781
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
C-bug The issue contains a bug report D-easy We estimate that the issue is easy to fix P-low
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants