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

add VT_VECTOR support for properties #135

Merged
merged 1 commit into from
Oct 7, 2020

Conversation

mak
Copy link
Contributor

@mak mak commented Jun 19, 2020

No description provided.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.9%) to 56.986% when pulling 8cc753d on MalwareLab-pl:master into d344bc2 on decalage2:master.

@decalage2 decalage2 self-requested a review June 19, 2020 16:28
@decalage2 decalage2 self-assigned this Jun 19, 2020
@decalage2 decalage2 added this to the olefile 0.47 milestone Jun 19, 2020
@decalage2
Copy link
Owner

Thanks a lot @mak, nice addition!
By any chance, do you have sample files containing different kinds of VT_VECTOR, so that I can test it?

@mak
Copy link
Contributor Author

mak commented Jun 19, 2020

Thanks, i used recent zloader lures, for example: 5c7052b45ada75b48b04026e3adb6460. It has vector + string and vector+variant. vector + string should be easy to obtain by creating xls with few sheets

@decalage2 decalage2 merged commit 8cc753d into decalage2:master Oct 7, 2020
decalage2 added a commit that referenced this pull request Oct 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants