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

Breach Compilation list index out of range for a specific email #16

Closed
dkramer47 opened this issue Dec 21, 2018 · 5 comments
Closed

Breach Compilation list index out of range for a specific email #16

dkramer47 opened this issue Dec 21, 2018 · 5 comments

Comments

@dkramer47
Copy link

Hello! I noticed after messing around using theharvester to get emails and then using h8mail to check them for breaches using the breach compilation (I'm not doing anything illegal, I promise), that the email "yourname@yahoo.com" caused a list index out of range crash. Now I know it's not someone's real email, but it caused the entire list check to fail.

I'm not too familiar with python, but I found changing line 178 in h8mail.py from "if line:" to "if ":" in line" fixed it from what I can tell, so maybe it's a good solution, or maybe it causes other things to go wrong that I'm not aware of.

@Cyb0rgbytes
Copy link

Hey there! @dkramer47 it's definitely strange to get a crash from such attempt... However, I'm a bit familiar with python I've invested a decent time learning it but not to an extreme level.
according to what you were saying I would really appreciate it if you can take a look at on my #15 issue since you've tried the breach compilation. I'm also not doing anything illegally just testing for a presentation. and I'm short on time.
Cheers

@Cyb0rgbytes
Copy link

This error " List index out of range" appeared to me also when i was trying to run the tool from a target text file. I'm not sure what is the fix, but thought I mention that.

@dkramer47
Copy link
Author

Try changing the line 178 in h8mail.py from 'if line:' -> 'if ":" in line:'

That fixed it for me, and it hasn't had any negative effects that I can see.

@antonvalletas
Copy link

I can confirm that fixed it for me too.

@khast3x
Copy link
Owner

khast3x commented May 10, 2019

Hello,

I am currently refactoring h8mail, and have included your code fix in this commit.
The code is currently in the v2 branch and should be merged to master soon. Meanwhile feel free to try out the v2 branch

@khast3x khast3x closed this as completed May 10, 2019
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

No branches or pull requests

4 participants