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

Generate password: Floating point exception (core dumped) #102

Closed
jdhoek opened this issue Sep 23, 2015 · 7 comments
Closed

Generate password: Floating point exception (core dumped) #102

jdhoek opened this issue Sep 23, 2015 · 7 comments

Comments

@jdhoek
Copy link

@jdhoek jdhoek commented Sep 23, 2015

When I click the Generate button for a password QtPass crashes:

Floating point exception (core dumped)

No further information is shown.

QtPass version 1.0.1
Ubuntu 15.04

pwgen is installed and works.

@annejan
Copy link
Member

@annejan annejan commented Sep 23, 2015

This is probably an issue of missing (or mis-configured) pwgen executable #95

Will be fixed ASAP.

As a workaround you can try using the build-in password generator.

Thanks for noticing this bug though.

@jdhoek
Copy link
Author

@jdhoek jdhoek commented Sep 23, 2015

Actually, I just found the Use pwgen switch in the config. If I turn that option on, pwgen generates a password and it works. It seems it is the built-in one that is causing this issue.

@annejan
Copy link
Member

@annejan annejan commented Sep 23, 2015

Was the characters string field empty in your configuration, or the length?
Might be the actual source (should be filled on update to 1.0.1 but might not have happened) . .

@jdhoek
Copy link
Author

@jdhoek jdhoek commented Sep 23, 2015

Yes it was empty. That seems to be the default. QtPass 1.0.1 is the first version of QtPass I have installed, so there is no prior configuration from older versions.

Actually, if I add some characters there, passwords do get generated.

@annejan
Copy link
Member

@annejan annejan commented Sep 23, 2015

Awesome, that gives me a clear source of the bug.
Will add some (much needed) safeguarding there.

@jdhoek
Copy link
Author

@jdhoek jdhoek commented Sep 23, 2015

My pleasure. 👍

@annejan
Copy link
Member

@annejan annejan commented Sep 24, 2015

Fixed in 7e735f3

Also learned you can't modulus zero.
http://stackoverflow.com/questions/7370154/cant-mod-zero

Fixed in master and will be in release 1.0.2

@annejan annejan closed this Sep 24, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants