Some additional return value checking

dev
Darxus 14 years ago committed by Kristian Høgsberg
parent e9f5e36730
commit 55973f2f5a
  1. 5
      clients/window.c
  2. 6
      compositor/compositor-drm.c
  3. 20
      compositor/compositor-x11.c

@ -1463,7 +1463,10 @@ display_create(int *argc, char **argv[], const GOptionEntry *option_entries)
return NULL; return NULL;
} }
eglBindAPI(EGL_OPENGL_API); if (!eglBindAPI(EGL_OPENGL_API)) {
fprintf(stderr, "failed to bind api EGL_OPENGL_API\n");
return NULL;
}
d->ctx = eglCreateContext(d->dpy, NULL, EGL_NO_CONTEXT, NULL); d->ctx = eglCreateContext(d->dpy, NULL, EGL_NO_CONTEXT, NULL);
if (d->ctx == NULL) { if (d->ctx == NULL) {

@ -339,7 +339,11 @@ init_egl(struct drm_compositor *ec, struct udev_device *device)
return -1; return -1;
} }
eglBindAPI(EGL_OPENGL_ES_API); if (!eglBindAPI(EGL_OPENGL_ES_API)) {
fprintf(stderr, "failed to bind api EGL_OPENGL_ES_API\n");
return -1;
}
ec->base.context = eglCreateContext(ec->base.display, NULL, ec->base.context = eglCreateContext(ec->base.display, NULL,
EGL_NO_CONTEXT, context_attribs); EGL_NO_CONTEXT, context_attribs);
if (ec->base.context == NULL) { if (ec->base.context == NULL) {

@ -80,19 +80,21 @@ struct x11_input {
}; };
static void static int
x11_input_create(struct x11_compositor *c) x11_input_create(struct x11_compositor *c)
{ {
struct x11_input *input; struct x11_input *input;
input = malloc(sizeof *input); input = malloc(sizeof *input);
if (input == NULL) if (input == NULL)
return; return -1;
memset(input, 0, sizeof *input); memset(input, 0, sizeof *input);
wlsc_input_device_init(&input->base, &c->base); wlsc_input_device_init(&input->base, &c->base);
c->base.input_device = &input->base; c->base.input_device = &input->base;
return 0;
} }
@ -247,7 +249,11 @@ x11_compositor_init_egl(struct x11_compositor *c)
return -1; return -1;
} }
eglBindAPI(EGL_OPENGL_ES_API); if (!eglBindAPI(EGL_OPENGL_ES_API)) {
fprintf(stderr, "failed to bind EGL_OPENGL_ES_API\n");
return -1;
}
c->base.context = eglCreateContext(c->base.display, NULL, c->base.context = eglCreateContext(c->base.display, NULL,
EGL_NO_CONTEXT, context_attribs); EGL_NO_CONTEXT, context_attribs);
if (c->base.context == NULL) { if (c->base.context == NULL) {
@ -661,15 +667,17 @@ x11_compositor_create(struct wl_display *display, int width, int height)
c->base.wl_display = display; c->base.wl_display = display;
if (x11_compositor_init_egl(c) < 0) if (x11_compositor_init_egl(c) < 0)
return NULL; return NULL;
/* Can't init base class until we have a current egl context */ /* Can't init base class until we have a current egl context */
if (wlsc_compositor_init(&c->base, display) < 0) if (wlsc_compositor_init(&c->base, display) < 0)
return NULL; return NULL;
x11_compositor_create_output(c, width, height); if (x11_compositor_create_output(c, width, height) < 0)
return NULL;
x11_input_create(c); if (x11_input_create(c) < 0)
return NULL;
loop = wl_display_get_event_loop(c->base.wl_display); loop = wl_display_get_event_loop(c->base.wl_display);

Loading…
Cancel
Save