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.

Sign up for GitHub

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

Jump to bottom

self.xlm_macros referenced before assignment #433

Closed
Zeyshi opened this issue Apr 15, 2019 · 0 comments
Closed

self.xlm_macros referenced before assignment #433

Zeyshi opened this issue Apr 15, 2019 · 0 comments
Assignees
@decalage2
Labels
🐛 bug olevba
Milestone
oletools 0.54

Comments

@Zeyshi
Copy link

Zeyshi commented Apr 15, 2019 •
edited
Loading

Affected tool:
olevba

Describe the bug
In olevba.py line 3197, "self.xlm_macros" is referenced before assignment

File/Malware sample to reproduce the bug
1d5669aadc41da43f0ff4fc308d5fd98ef1a68dfba609dcf54d08156157573e1.zip
SHA-1: 20bc714d0f50c0dce92792bad8d941416aa9a78f

How To Reproduce the bug

# Example code:
from olevba import VBA_Parser
vba_obj = VBA_Parser(filename)

Version information:

  • OS: Ubuntu Linux
  • OS version: 18.04 64 bits
  • Python version: 3.6 64 bits
  • oletools version: 0.54.1

Suggested Fix
Add self.xlm_macros = 0 to VBA_Parser class __init__
@decalage2 decalage2 self-assigned this Apr 15, 2019
@decalage2 decalage2 added this to the oletools 0.54 milestone Apr 15, 2019
bitsikas pushed a commit to SpamExperts/oletools that referenced this issue May 17, 2019
@decalage2 @bitsikas
olevba: initialize VBA_Parser.xlm_macros (fixes decalage2#433)
fc76312
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@decalage2 decalage2

Labels
🐛 bug olevba
Projects
None yet
Milestone
oletools 0.54
Development

No branches or pull requests
2 participants
@decalage2 @Zeyshi