-
Notifications
You must be signed in to change notification settings - Fork 7
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
Please do a graceful/normal shutdown #13
Comments
I can't agree with this. If there is some sort of attack, I prefer shutdown to be as quick as possible. I would rather see some battery stats issue after reboot than giving attacker more opportunity to continue with attack. There could be e.g. some exploit found in graceful shutdown code in the future (yes, it is unlikely, but like I said I prefer shutdown to be as quick as possible). |
But they don't have any opportunity as nothing can be done to prevent the shutdown or to unlock the device when it is shutting down AFAIK. |
I wonder if using |
At least it is a try, is not it? |
I've checked logcat and |
To do a regular reboot (same as if reboot is done from the Power button menu), the command is |
I've found some info on the safety of |
@gene-pavlovsky thanks for all the interesting research, you did! so what shutdown-procedure do you suggest to use? special android-commands? with sync before or not, and which command for actual shutdown? @xmikos may i suggest to simply give the possibility to change the command (maybe with corresponding warning, readme, links to recourses) by advanced users like Wrong PIN Shutdown does. |
In the end I started using Automagic for various things, and the functionality of SnooperStopper is easily replicated as an Automagic flow. I'm just using "Reboot" Automagic action (requires root). |
well... i want to neither use non-FOSS software, nor have playstore installed! - so this is not an option for me. if i understand this right, you are now using a predefined reboot-function of a proprietary app, where you don't even have an idea which command(s) are used...? 😕 |
I see and respect your point. Wish I was that strong principled on this.
You might want to increase the sleep duration, personally I wouldn't worry too much about this stuff, considering my phone spuriously reboots all by itself for unknown reasons once a month or so, and I didn't notice any harm done or loss of data in my apps :) |
Instead of - somekind of - "unplug" the device from battery could you just do a graceful/normal shutdown?
Otherwise it not only looks like a system crash it also seems to cause issues like this: #12
And I see no reasons for such a drastic shutdown method. During shutdown an attacker cannot unlock the device anyway...
The text was updated successfully, but these errors were encountered: