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.

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

When master password is used and password dialog entry is cancelled, terminals still opens! #5

Closed
ghost opened this issue Apr 12, 2017 · 11 comments

Comments

@ghost
Copy link

ghost commented Apr 12, 2017

This is probably serious security breach - if master password dialog entry is cancelled, terminals app is still opened.
This should not happen.
Also, in this case the startup RDP connections are opened without explicit setting/confirmation/control with the lowest possible display resolution.

@ghost ghost changed the title When master password is used and pasword dialog entry is cancelled, terminals still opens! When master password is used and password dialog entry is cancelled, terminals still opens! Apr 12, 2017
jirkapok added a commit that referenced this issue May 13, 2017
@jirkapok
Copy link
Contributor

Fixed in 4.0.1

@truist
Copy link

truist commented Aug 7, 2017

Was this a security breach? Were stored passwords accessible in prior versions?

@jirkapok
Copy link
Contributor

jirkapok commented Aug 7, 2017

No, it was introduced in 4.0 and fixed in 4.0.1

@ghost
Copy link
Author

ghost commented Aug 7, 2017

Hmm. I didn't check that. I wrote that is a security breach because one could start remote sessions (starting the program itself) without typing the master password - that is going around password-based security.

@truist
Copy link

truist commented Aug 7, 2017

OK, thanks for responding. But to be clear: if you canceled out of the dialog, you'd have access to the app, but if you tried to use a host that had a saved password, it wouldn't work?

@ghost
Copy link
Author

ghost commented Aug 7, 2017 via email

@truist
Copy link

truist commented Aug 8, 2017

My real question is whether it was - or still is - possible to get an unencrypted password without knowing the master password. I expect @jirkapok would have to answer, to be sure. (I've done a little code digging, and it looks like the passwords are encrypted with the master password, but I'm not certain.)

@awojtas
Copy link

awojtas commented Aug 12, 2017

Does this mean a hacker could use a 4.0.0 version to access saved credentials, read passwords that are now decrypted in memory, or connect using saved credentials, if they had the db from a terminals install?

@ghost
Copy link
Author

ghost commented Aug 13, 2017 via email

@ghost
Copy link
Author

ghost commented Aug 13, 2017 via email

@ghost
Copy link
Author

ghost commented Aug 13, 2017 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants