xf86drm: Add drmIsMaster()
We can't use drmSetMaster to query whether or not a drm fd is master because it requires CAP_SYS_ADMIN, even if the fd *is* a master fd. Pick DRM_IOCTL_MODE_ATTACHMODE as a long-deprecated ioctl that is DRM_MASTER but not DRM_ROOT_ONLY as the probe by which we can detect whether or not the fd is master. This is useful for code that might get master by open()ing the drm device while no other master exists, but can't call drmSetMaster itself because it's not running as root or is in a container, where container-root isn't real-root. v2: Use the AUTH_MAGIC request rather than MODE_ATTACHMODE, as it's more clearly related to master status. v3: [Emil] Don't expose internals, check for -EACCES. Signed-off-by: Christopher James Halse Rogers <christopher.halse.rogers@canonical.com> Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch> (v2) Signed-off-by: Emil Velikov <emil.velikov@collabora.com> Reviewed-by: Emil Velikov <emil.velikov@collabora.com>main
parent
70a1ae89be
commit
17dfe3ac93
18
xf86drm.c
18
xf86drm.c
|
@ -2743,6 +2743,24 @@ drm_public int drmDropMaster(int fd)
|
|||
return drmIoctl(fd, DRM_IOCTL_DROP_MASTER, NULL);
|
||||
}
|
||||
|
||||
drm_public bool drmIsMaster(int fd)
|
||||
{
|
||||
/* Detect master by attempting something that requires master.
|
||||
*
|
||||
* Authenticating magic tokens requires master and 0 is an
|
||||
* internal kernel detail which we could use. Attempting this on
|
||||
* a master fd would fail therefore fail with EINVAL because 0
|
||||
* is invalid.
|
||||
*
|
||||
* A non-master fd will fail with EACCES, as the kernel checks
|
||||
* for master before attempting to do anything else.
|
||||
*
|
||||
* Since we don't want to leak implementation details, use
|
||||
* EACCES.
|
||||
*/
|
||||
return drmAuthMagic(fd, 0) != -EACCES;
|
||||
}
|
||||
|
||||
drm_public char *drmGetDeviceNameFromFd(int fd)
|
||||
{
|
||||
char name[128];
|
||||
|
|
|
@ -37,6 +37,7 @@
|
|||
#include <stdarg.h>
|
||||
#include <sys/types.h>
|
||||
#include <stdint.h>
|
||||
#include <stdbool.h>
|
||||
#include <drm.h>
|
||||
|
||||
#if defined(__cplusplus)
|
||||
|
@ -733,6 +734,7 @@ extern void drmMsg(const char *format, ...) DRM_PRINTFLIKE(1, 2);
|
|||
|
||||
extern int drmSetMaster(int fd);
|
||||
extern int drmDropMaster(int fd);
|
||||
extern bool drmIsMaster(int fd);
|
||||
|
||||
#define DRM_EVENT_CONTEXT_VERSION 4
|
||||
|
||||
|
|
Loading…
Reference in New Issue