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

TypeError on Hermes version 94 #10

Closed
randshell opened this issue Mar 25, 2024 · 5 comments
Closed

TypeError on Hermes version 94 #10

randshell opened this issue Mar 25, 2024 · 5 comments

Comments

@randshell
Copy link

randshell commented Mar 25, 2024

Logs:

[...]
Address 256941: Line 17179891934 - Column 38654917786 - Statement 445715
  Address 256946: Line 17179891934 - Column 38654917784 - Statement 445715
  Address 256946: Line 19327375581 - Column 38654917789 - Statement 445715
  Address 256926: Line 19327375581 - Column 42949885084 - Statement 445715
  Address 256940: Line 19327375581 - Column 42949885115 - Statement 445715
Traceback (most recent call last):
  File "/home/kali/.local/bin/hbc-file-parser", line 8, in <module>
    sys.exit(main())
             ^^^^^^
  File "/home/kali/.local/share/pipx/venvs/hermes-dec/lib/python3.11/site-packages/hermes_dec/parsers/hbc_file_parser.py", line 915, in main
    print_debug_info(hbc_reader.sources_data_storage)
  File "/home/kali/.local/share/pipx/venvs/hermes-dec/lib/python3.11/site-packages/hermes_dec/parsers/debug_info_parser.py", line 64, in print_debug_info
    current_statement += statement_delta
TypeError: unsupported operand type(s) for +=: 'int' and 'NoneType'
p1-mmr added a commit that referenced this issue Mar 26, 2024
@p1-mmr
Copy link
Collaborator

p1-mmr commented Mar 26, 2024

Hello,

I pushed something in commit 7379e9e, should be better this way?

@p1-mmr p1-mmr closed this as completed Mar 26, 2024
@randshell
Copy link
Author

Thank you for the quick update!

It seems to perform the same way, still stops at Address 256940 with TypeError.

@marin-m
Copy link

marin-m commented Mar 27, 2024

With the latest version from Git?
If yes, can you please share your sample at labs at p1sec dot com?

@randshell
Copy link
Author

Hi,
I've rechecked again and noticed that the 7379e9e commit was on the origin/feature/web-ui-with-graph-view--index-socket-comm branch. I've tested this version now and it fixes the issue for me.

Thank you!

p1-mmr added a commit that referenced this issue Apr 7, 2024
@p1-mmr
Copy link
Collaborator

p1-mmr commented Apr 7, 2024

Ah I pushed this commit to the main, sorry. Thanks for noticing.

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

3 participants