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

games-action/minetest: Executable needs PaX flag "m" #70

Open
Columbus240 opened this issue May 21, 2017 · 2 comments · May be fixed by #73
Open

games-action/minetest: Executable needs PaX flag "m" #70

Columbus240 opened this issue May 21, 2017 · 2 comments · May be fixed by #73

Comments

@Columbus240
Copy link
Contributor

On a PaX enabled kernel, depending on whether CONFIG_PAX_MPROTECT is set, minetest crashes on loading a world or opening the mods menu because LuaJIT is blocked by the kernel.
This is fixed by setting the appropriate flag, using setfattr -v m -n user.pax.flags /usr/bin/minetest.
It would be nice if the ebuild would do this by itself.

I have already filed a bug in the minetest repo (minetest/minetest#5761) and got sent here, to the package maintainer.

@Columbus240
Copy link
Contributor Author

Columbus240 commented May 27, 2017

I tried to use setfattr in pkg_preinst() and in pkg_postinst() and both times it didn't work. There are other binaries with PaX flags set. The ebuilds of these binaries should help.

@Columbus240
Copy link
Contributor Author

Columbus240 commented May 27, 2017

If you have a look at app-emulation/wine-9999 from the Gentoo repo, you might notice that it has a line of inherit pax-utils and does pax-mark psmr "${D}"usr/bin/wine in multilib_src_install_all().
So I think this ebuild should also do the inherit part and do something like pax-mark m "${D}"usr/bin/minetest (if ! use dedicated) (and the same for minetestserver if use server || use dedicated), in src_install().

(To check that the flags are set correctly after installation use getfattr -n user.pax.flags /usr/bin/minetest{,server}

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

Successfully merging a pull request may close this issue.

2 participants