From: Nicolai Hähnle
From: Nicolai Hähnle
Having different types of code all trying to check for elevated privileges
is a bad idea. This implementation is the most thorough one.
Signed-off-by: Nicolai Hähnle
---
hw/xfree86/common/xf86Init.c | 59 +
inc
Sent that a little too soon; please consider "Reviewed-by: Ben Crocker <
bcroc...@redhat.com>" to be
added after each of the "Signed-off-by: Nicolai Haehnle <
nicolai.haeh...@amd.com>" lines.
Also please note that I rebased Nicolai's original patch,
https://p
Antoine, thanks for the suggestion and the additional review. V2 patch
series forthcoming.
-- Ben
On Sat, Mar 10, 2018 at 11:58 PM, Antoine Martin
wrote:
> On 09/03/18 05:11, Ben Crocker wrote:
> > Sent that a little too soon; please consider "Reviewed-by: Ben Crocker
&g
From: Nicolai Hähnle
Signed-off-by: Nicolai Hähnle
Reviewed-by: Ben Crocker
Reviewed-by: Antoine Martin
Tested-by: Ben Crocker
---
hw/xfree86/common/xf86Config.c | 2 +-
hw/xfree86/common/xf86Init.c | 12 +++-
hw/xfree86/common/xf86Priv.h | 2 --
3 files changed, 4 insertions
From: Nicolai Hähnle
Signed-off-by: Nicolai Hähnle
Reviewed-by: Ben Crocker
Reviewed-by: Antoine Martin
Tested-by: Ben Crocker
---
os/utils.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/os/utils.c b/os/utils.c
index 4305dab26..6e3c16869 100644
--- a/os/utils.c
+++ b
From: Nicolai Hähnle
Having different types of code all trying to check for elevated privileges
is a bad idea. This implementation is the most thorough one.
Signed-off-by: Nicolai Hähnle
Reviewed-by: Ben Crocker
Reviewed-by: Antoine Martin
Tested-by: Ben Crocker
---
hw/xfree86/common
Submit Nicolai Hähnle's four-patch series improving the checks for
elevated privileges and culminating in using the runtime environmental
control LIBGL_DRIVERS_PATH when searching for driver .so's, as
Mesa, and hence Glamor, already do.
The fourth patch prevents a conflict that can arise if, for e
From: Nicolai Hähnle
Allow switching to another driver build without a full installation.
Glamor already takes LIBGL_DRIVERS_PATH into account, so this change
makes sure that the same driver is used in both parts of the server.
Signed-off-by: Nicolai Hähnle
Reviewed-by: Ben Crocker
Reviewed
ad.c, #include "xf86Config.h". In xf86readConfigFile,
change the open-code call to calloc to a call to the new
xf86allocateConfig function.
In xf86AutoConfig.c, add a call to the new xf86allocateConfig function
to the beginning of xf86AutoConfig to make sure the XF86ConfigRec struct
is alloca