Skip to content

Commit

Permalink
Update goatrider.py
Browse files Browse the repository at this point in the history
  • Loading branch information
hAxe1 authored Nov 26, 2018
1 parent 44da8bb commit c92f98d
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions goatrider.py
Original file line number Diff line number Diff line change
Expand Up @@ -186,7 +186,7 @@ def search_binarybanlist( hostlist ):

for hosts in hostlist:
hosts = hosts.rstrip()
if hosts in bd_banlist:
if hosts in bd_banlist.split():
if hosts != "":
banlist_findings_lock.acquire()
banlist_findings[ 'banlist' ].append( hosts )
Expand All @@ -205,7 +205,7 @@ def search_binarytorlist( hostlist ):

for hosts in hostlist:
hosts = hosts.rstrip()
if hosts in bd_tor:
if hosts in bd_tor.split():
if hosts != "":
banlist_findings_lock.acquire()
banlist_findings[ 'tor' ].append( hosts )
Expand All @@ -225,7 +225,7 @@ def search_otx( hostlist ):
# FORMAT: ALL: 46.4.123.15 # Malicious Host
for hosts in hostlist:
hosts = hosts.rstrip()
if hosts in otx:
if hosts in otx.split():
if hosts != "":
banlist_findings_lock.acquire()
banlist_findings[ 'otx' ].append( hosts )
Expand Down

1 comment on commit c92f98d

@hAxe1
Copy link
Owner Author

@hAxe1 hAxe1 commented on c92f98d Nov 26, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Splitting lists on whitespace, Originally tried splitting on new line "\n" but it failed to work correctly on the OTX list, whitespace seems to work fine on the three lists. As far as I can tell, this should fix BinaryDefense#4

Please sign in to comment.