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 USB-DriveBy #39

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Add USB-DriveBy #39

wants to merge 2 commits into from

Conversation

90N45-d3v
Copy link

@90N45-d3v 90N45-d3v commented Nov 9, 2023

Use an USB storage device to deploy payloads on-demand while the Packet Squirrel is already set up and running.

Use an USB storage device to deploy payloads on-demand while the Packe Squirrel is already set up and running.

LED FINISH

while true; do
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Using this loop, the payload a appears to be repeatedly triggered. I would add a flag to make sure it only runs once.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's intentional... since no flag is used, you can run multiple “drive-by” payloads from the USB without having to restart the main payload on the squirrel after every single use.
The delay in line 26 provides the user enough time to unplug the USB when the “drive-by” payload finished, so that it is not unintentionally triggered again.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add LED indication that the payload is running so the user knows when to unplug the USB. In its current state it would be difficult to know when the payload has been transferred and is running. I would also recommend using a variable for the payload name to make user configuration easier, just in case someone wants to use something other than payload.txt.

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.

2 participants