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

Fix bug in skype #470

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

MyLoginOnGitHub
Copy link
Contributor

Fixed bug in skype.
Added python3 support, but it still does not works: it could not to perform successful dictionary atack even if the right password have been added in the passwords list.

@AlessandroZ
Copy link
Owner

I prefer to merge your PR once the issue with the dictionary attack works.

Could you try again changing all list from: https://github.com/AlessandroZ/LaZagne/blob/master/Windows/lazagne/config/dico.py
replacing all u"string" by b"string"

It should correct your issue.

Thanks a lot for your help.

@MyLoginOnGitHub
Copy link
Contributor Author

MyLoginOnGitHub commented Jan 6, 2020

Please, tell me what am I doing wrong...
I am using python2 and Windows10.
First, I done the change in line

to u"my_real_password_from_skype",. Dict attack fails.
Second, right now I have been tried to change this line to b"rhjdfnm123",. Dict attack also fails.
More than that, the hashes counted in line
hash_ = hashlib.md5('%s\nskyper\n%s' % (login, word)).hexdigest()
are the same for u"..." and b"...", so, as I understood, there is not matter what to use.
Repeat, I am using python2.

@AlessandroZ
Copy link
Owner

In python 2, there are no difference between u"string" and b"string" but for python 3 you should always use b"string".
So I recommend always using b"".

I'm sorry but I cannot help you without debugging. Try to print your md5 password to see the good result.

hashlib.md5('%s\nskyper\nyourpassword' % (login)).hexdigest() 

Remove the content of the dico and add only your password. Print the content of hash_ and md5 to understand why there are different.

@MyLoginOnGitHub
Copy link
Contributor Author

I'm sorry but I cannot help you without debugging.

I understand it :-)
You know, this is not so important for me to find a solution with this, it's just research interest. So, if you tell its work, then I believe you:-) But just for accuracy, recheck it on your free time:)

@Dutt786
Copy link

Dutt786 commented Aug 8, 2020

hey
I have used your python project LaZagne.I want to contribute to this project and add some juicy code.

@AlessandroZ
Copy link
Owner

AlessandroZ commented Sep 2, 2020

Hi Dutt786,

Do not hesitate to do PRs if you want to contribute. Thanks 👍

@dimeciret
Copy link

hi AlessandroZ]
VERY NICE script

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

Successfully merging this pull request may close these issues.

4 participants