-
Notifications
You must be signed in to change notification settings - Fork 30
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
this is likely a safer way to trap #133
base: develop
Are you sure you want to change the base?
Conversation
What is the difference between ". ./evilap.sh" & "./evilap.sh"? |
Odd, I had assume you were manually running via the cli since on my end it On Nov 13, 2016 17:05, "AnipTiK" notifications@github.com wrote:
This e-mail and any attachments are for the sole use of the intended |
I tried your modified version of EvilAP and the modified version from here. They aren't working, here is the log. (same for both) |
Corrupted whitespace somehow when you downloaded. You have windows line On Nov 15, 2016 17:11, "AnipTiK" notifications@github.com wrote:
This e-mail and any attachments are for the sole use of the intended |
Ok, removed it. Fixed it replacing Ctrl C is working. (I used here the other modified file) |
This should solve the reported issue but it NEEDS TESTED
#132
When we dropped trapping of sigint and sigterm it did not occur to me that trapping quit would only work for if the shell is quit, not the script. Meaning according to the report ". ./evilap.sh" will trap properly on ^C but "./evilap.sh" won't. I have not tested the reported issue, nor have I tested the fix. But we removed the trap because it was double running cleanup and this prevents that.