Skip to content
This repository has been archived by the owner on Jan 29, 2020. It is now read-only.

Set sys.argv from the macho template #1375

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Set sys.argv from the macho template #1375

wants to merge 1 commit into from

Conversation

OJ
Copy link

@OJ OJ commented Sep 26, 2019

When running an agent in a macho binary, we have an issue where any modules that make use of sys.argv (directly or indirectly through dependencies) fail because sys.argv hasn't been set. We get the following error:

image

If this same module is run from a python one-liner the error doesn't appear.

This PR attempts to fix this problem by changing the template so that argc and argv are passed to Python before the agent is run. From here, any references to sys.argv should not result in failure.

Note: I have not compiled this code (no current access to OSX), but I think it should just work out of the box.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant