launcher: Drop unecessary checks for drm_fd == -1
authorKristian Høgsberg <krh@bitplanet.net>
Mon, 14 Oct 2013 20:59:53 +0000 (13:59 -0700)
committerKristian Høgsberg <krh@bitplanet.net>
Mon, 14 Oct 2013 21:00:03 +0000 (14:00 -0700)
We never get into this case, and if we do, we just want to pass the invalid
fd through to the underlying ioctl and get the error that way."

src/launcher-util.c

index ea20895..4f77d11 100644 (file)
@@ -69,25 +69,20 @@ struct weston_launcher {
 static int
 drm_drop_master(int drm_fd)
 {
-       if (drm_fd != -1)
-               return drmDropMaster(drm_fd);
-       return -EBADF;
+       return drmDropMaster(drm_fd);
 }
 static int
 drm_set_master(int drm_fd)
 {
-       if (drm_fd != -1)
-               return drmSetMaster(drm_fd);
-       return -EBADF;
+       return drmSetMaster(drm_fd);
 }
 static int
 drm_is_master(int drm_fd)
 {
        drm_magic_t magic;
-       if (drm_fd != -1)
-               return drmGetMagic(drm_fd, &magic) == 0 &&
-                      drmAuthMagic(drm_fd, magic) == 0;
-       return 0;
+
+       return drmGetMagic(drm_fd, &magic) == 0 &&
+               drmAuthMagic(drm_fd, magic) == 0;
 }
 #else
 static int drm_drop_master(int drm_fd) {return 0;}