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

praktomat.policy vs java.policy #307

Open
chris21k opened this issue Dec 17, 2019 · 3 comments · Fixed by hso-praktomat/praktomat#19 · May be fixed by #363
Open

praktomat.policy vs java.policy #307

chris21k opened this issue Dec 17, 2019 · 3 comments · Fixed by hso-praktomat/praktomat#19 · May be fixed by #363

Comments

@chris21k
Copy link

Praktomat/src/checker/checker/DejaGnu.py line 120 references not existing file "praktomat.policy" instead of in directory Praktomat/src/checker/scripts/ supplied "java.policy". Thus, 'spawn java' starts a jvm with empty policies via Praktomat/src/checker/scripts/java. A workaround without touching the python code is
$ ln -s java.policy praktomat.policy
in this directory.

@ifrh
Copy link
Contributor

ifrh commented Nov 25, 2021

Just found on https://docs.oracle.com/en/java/javase/17/security/java-security-overview1.html#GUID-BBEC2DC8-BA00-42B1-B52A-A49488FCF8FE

The Security Manager and APIs related to it have been deprecated and are subject to removal in a future release. There is no replacement for the Security Manager. See JEP 411 - https://openjdk.java.net/jeps/411 - for discussion and alternatives.

corresponding information here:
https://bugs.openjdk.java.net/browse/JDK-8264713

@ifrh
Copy link
Contributor

ifrh commented Jan 28, 2022

@chris21k could you please update the issue headline, that one can read, that Security Manager is deprecated?

referencing to an old mail to praktomat mailinglist (8.10.2016 of @nomeata ), perhaps firejail https://firejail.wordpress.com/ could use as sandbox i.e. on systems where Docker containers are "too big".

@ifrh
Copy link
Contributor

ifrh commented Aug 10, 2022

Implementation changed to

environ['POLICY'] = join(script_dir, "java.policy")

via commit bc585cf#diff-e64c26e50d196ca3f4cccef7ec36c569a692481cafc1b02a789b44f321e7e590 merged to master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants