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 POC for code execution #34

Closed
wants to merge 1 commit into from
Closed

Conversation

jakiki6
Copy link

@jakiki6 jakiki6 commented Feb 18, 2021

It's not that smart to use pickle for saving data. I included a new command which shows that you can include your own code in an encoded file.

Have fun with it!

@Theelx
Copy link
Collaborator

Theelx commented Feb 18, 2021

Sorry, why would you want to make a malicious file with this? If that's not the intent, can you name the function and argument help something else?

@Theelx
Copy link
Collaborator

Theelx commented Feb 18, 2021

@AlfredoSequeida Do you think this violates the GitHub TOS in that it allows for creating malicious files, or do you think this is okay to merge?

@jakiki6
Copy link
Author

jakiki6 commented Feb 18, 2021

It's just a proof of concept.
I wouldn't merge it.

@Theelx
Copy link
Collaborator

Theelx commented Feb 18, 2021

Then why make it a pull request here?

@jakiki6
Copy link
Author

jakiki6 commented Feb 18, 2021

I just wanted to show that it's vulnerable and you should be careful

@AlfredoSequeida
Copy link
Owner

Yeah I have seen this come up before and I can kind of understand the concern. If I get some time I'll try to look into what we can do about that. If anyone has any suggestions regarding this issue I'm open to hearing them.

@Theelx
Copy link
Collaborator

Theelx commented Feb 18, 2021

#35 I made an issue for this if anyone wants to help

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

Successfully merging this pull request may close these issues.

None yet

3 participants