search

Project-ARTist / ArtistGui

ARTist's deployment application, for code-injection on an Android device.
https://artist.cispa.saarland
Other
37 stars 12 forks source link

Replace chmod 777 with reasonable permissions #58

Open schrnz opened 6 years ago

schrnz commented 6 years ago

chmod'ing everything to 777 is an obsolete and dangerous relic from the early times of the project where it was all about getting things done quickly. Now that we want to use ARTist in production systems, we should take care of assigning proper access rights to executable files, such as the resulting instrumented oat file.

The objective is to either dynamically find the correct access rights or find fixed ones that work on a multitude of devices. While the dynamic way is preferable, #11 has shown that it might not always be possible to implement this (easily).

I would assume this is a good first issue if you want to get started with ARTist since you do not have to understand much about the ARTist ecosystem. It is just about finding a way to assign proper access rights to those files that are currently chmod'ed to 777.

schrnz commented 6 years ago

Postponed, will not be part of the beta.