From 6ddedeba011fa32e005112cc4655bddbd7ce1e9f Mon Sep 17 00:00:00 2001 From: "Kelvin M. Klann" Date: Mon, 29 Nov 2021 01:47:56 -0300 Subject: [PATCH] Make nogroups work on nvidia again Remove workaround from commit 623e68216 ("temporary fix for nvidia/nogroups/noroot issue (#3644, #841)", 2020-10-02) and from commit cb460c32c ("more nvidia (#3644)", 2020-10-03). The handling of the "render" and "video" groups is separate from `nogroups` now, so disabling `nogroups` on nvidia shouldn't be necessary anymore. See the previous 2 commits for details. See also the discussion on PR #4632. --- src/firejail/profile.c | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/src/firejail/profile.c b/src/firejail/profile.c index b8bb086d236..756b370aa46 100644 --- a/src/firejail/profile.c +++ b/src/firejail/profile.c @@ -416,13 +416,7 @@ int profile_check_line(char *ptr, int lineno, const char *fname) { return 0; } else if (strcmp(ptr, "nogroups") == 0) { - // nvidia cards require video group; disable nogroups - if (access("/dev/nvidiactl", R_OK) == 0 && arg_no3d == 0) { - fwarning("Warning: NVIDIA card detected, nogroups command disabled\n"); - arg_nogroups = 0; - } - else - arg_nogroups = 1; + arg_nogroups = 1; return 0; } else if (strcmp(ptr, "nosound") == 0) {