Skip to content

consider changing caf and cafrun permissions to 555 #340

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

Closed
mexas opened this issue Mar 10, 2017 · 2 comments
Closed

consider changing caf and cafrun permissions to 555 #340

mexas opened this issue Mar 10, 2017 · 2 comments
Assignees

Comments

@mexas
Copy link

mexas commented Mar 10, 2017

ATM caf and cafrun are installed as 777, which is unsafe.
E.g. FreeBSD refuses to install under /usr/local/bin with such perms.
Consider changing to 555 by default:

-    PERMISSIONS WORLD_EXECUTE WORLD_READ WORLD_WRITE OWNER_EXECUTE OWNER_READ OWNER_WRITE GROUP_EX
ECUTE GROUP_READ GROUP_WRITE
+    PERMISSIONS WORLD_EXECUTE WORLD_READ OWNER_EXECUTE OWNER_READ GROUP_EXECUTE GROUP_READ
     DESTINATION "${CMAKE_INSTALL_BINDIR}"

in

src/mpi/CMakeLists.txt

Thanks

Anton

@zbeekman
Copy link
Collaborator

hmmmm, I thought I had adjusted these... I'll make sure they are 555 in the next release.

@rouson
Copy link
Member

rouson commented Mar 10, 2017

Thanks for reporting this @mexas

zbeekman added a commit that referenced this issue Mar 21, 2017
 - Use 555 perms for scripts and other executables
 - Fixes #340
@rouson rouson closed this as completed in c76ef50 Apr 18, 2017
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

No branches or pull requests

3 participants