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

[Bug] Malwares_GetReport_1_0 #470

Closed
1623664 opened this issue Apr 23, 2019 · 3 comments

Comments

Projects
None yet
4 participants
@1623664
Copy link

commented Apr 23, 2019

Invalid output
Traceback (most recent call last):
File "Malwares/malwares.py", line 163, in
MalwaresAnalyzer().run()
File "Malwares/malwares.py", line 146, in run
hash = hashlib.sha256(open(filepath, 'r').read()).hexdigest();
TypeError: Unicode-objects must be encoded before hashing

@1623664 1623664 added the bug label Apr 23, 2019

@nadouani

This comment has been minimized.

Copy link
Contributor

commented Apr 26, 2019

Hello, can you please provide an example of observable that produces this error?

Thanks

@nadouani nadouani self-assigned this Apr 26, 2019

@1623664

This comment has been minimized.

Copy link
Author

commented May 4, 2019

sorry for so long answer. This happens every time you check the files. Hash is checked without errors.

@gl4ssiest

This comment has been minimized.

Copy link

commented May 7, 2019

Can confirm it:
Invalid output

_File "Malwares/malwares.py", line 163, in
MalwaresAnalyzer().run()
File "Malwares/malwares.py", line 146, in run
hash = hashlib.sha256(open(filepath, 'r').read()).hexdigest();
File "/usr/lib/python3.7/codecs.py", line 322, in decode
(result, consumed) = self.buffer_decode(data, self.errors, final)
UnicodeDecodeError: 'utf-8' codec can't decode byte 0x90 in position 2: invalid start byte

@jeromeleonard jeromeleonard self-assigned this May 10, 2019

@jeromeleonard jeromeleonard added this to the 2.1.0 milestone May 10, 2019

jeromeleonard added a commit that referenced this issue May 10, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.