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

Can't create unix socket without 777 permission #11

Open
scaledteam opened this issue Aug 8, 2021 · 5 comments
Open

Can't create unix socket without 777 permission #11

scaledteam opened this issue Aug 8, 2021 · 5 comments
Labels
bug Something isn't working enhancement New feature or request good first issue Good for newcomers

Comments

@scaledteam
Copy link

Can't create unix socket without 777 permission in /usr/local/share/obs/obs-plugins/linux-kmsgrab/ folder.

@w23
Copy link
Owner

w23 commented Aug 10, 2021

Ah yes, we need to figure out a better path for socket.

@w23 w23 added bug Something isn't working enhancement New feature or request good first issue Good for newcomers labels Aug 10, 2021
@scaledteam
Copy link
Author

obs-vkcapture uses this path "/tmp/obs-vkcapture.sock", and author also mentioned "$XDG_RUNTIME_DIR/obs-vkcapture.socket". But i don't sure about better socket location too, i'm new in unix sockets.

https://github.com/nowrep/obs-vkcapture/blob/3806d838a629884203c871602a817b2d9b8699ab/src/vkcapture.c#L76
nowrep/obs-vkcapture#13 (comment)

@w23
Copy link
Owner

w23 commented Aug 10, 2021

But i don't sure about better socket location too, i'm new in unix sockets.

Me too :D

Something something XDG, freedesktop docs, etc.
It should be a secure location only visible to current user, as this is a fairly security sensitive thing.

@adryd325
Copy link

seems like most other user specific sockets are in /run/user/1000 which is $XDG_RUNTIME_DIR

@xnqs
Copy link

xnqs commented Feb 28, 2022

Yeah, my fork actually does that and it works really well. Do note, though, that it uses an older, working version of the plugin that has implicit EGL dependency, and doesn't use the built-in OBS implementation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

4 participants