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

Use XDG_RUNTIME_DIR instead of /tmp. #6

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Use XDG_RUNTIME_DIR instead of /tmp. #6

wants to merge 2 commits into from

Conversation

Celti
Copy link

@Celti Celti commented Aug 8, 2014

In the interests of improved security and compatibility with the modern
Linux desktop environment, use XDG_RUNTIME_DIR/libgl-switcheroo instead
of the globally-accessible /tmp/libgl-switcheroo-USER.

In the interests of improved security and compatibility with the modern
Linux desktop environment, use XDG_RUNTIME_DIR/libgl-switcheroo instead
of the globally-accessible /tmp/libgl-switcheroo-USER.
@amonakov
Copy link
Owner

amonakov commented Aug 9, 2014

There needs to be a fallback in case XDG_RUNTIME_DIR is not set.

Would you like to take care of that?

@Celti
Copy link
Author

Celti commented Aug 9, 2014

Good point, I'll add one.

@Celti
Copy link
Author

Celti commented Aug 9, 2014

There, that should take care of it. This is literally my first foray into writing C/C++ so please let me know if I did something wrong there.

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

Successfully merging this pull request may close these issues.

2 participants