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

Error parsing multiple ASNs from reverse lookup #6

Closed
rshipp opened this issue Feb 22, 2018 · 2 comments
Closed

Error parsing multiple ASNs from reverse lookup #6

rshipp opened this issue Feb 22, 2018 · 2 comments
Assignees

Comments

@rshipp
Copy link

rshipp commented Feb 22, 2018

Ran into this issue using GOSINT, originally opened an issue there (ciscocsirt/GOSINT#19), but moving it here as it seems to be a bug in this library.

When the lookup returns multiple ASNs for a given IP, it tries and fails to parse as a number:

2018/02/22 09:43:48 Error doing reverse lookup on %v: %v 103.235.224.237 Could not parse ASN (4808 23724 56282): %!s(MISSING): strconv.Atoi: parsing "4808 23724 56282": invalid syntax
2018/02/22 09:43:52 Error doing reverse lookup on %v: %v 103.75.156.237 Could not parse ASN (134877 135216): %!s(MISSING): strconv.Atoi: parsing "134877 135216": invalid syntax
2018/02/22 09:43:59 Error doing reverse lookup on %v: %v 106.75.101.69 Could not parse ASN (4808 23724): %!s(MISSING): strconv.Atoi: parsing "4808 23724": invalid syntax
2018/02/22 09:43:59 Error doing reverse lookup on %v: %v 106.75.18.132 Could not parse ASN (4808 23724): %!s(MISSING): strconv.Atoi: parsing "4808 23724": invalid syntax

(Note the above output is from GOSINT.)

These IPs are from the CI Army blacklist, http://cinsscore.com/list/ci-badguys.txt, there's about 160 IPs out of the 15k on the list that trigger the error.

@ammario ammario self-assigned this Feb 22, 2018
@ammario
Copy link
Owner

ammario commented Feb 23, 2018

fixed via d231a84

It will now ignore subsequent ASNs.

@ammario ammario closed this as completed Feb 23, 2018
@rshipp
Copy link
Author

rshipp commented Feb 23, 2018

Thanks!

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

2 participants