diff options
author | Philipp Zabel <[email protected]> | 2017-03-29 09:44:21 +0200 |
---|---|---|
committer | Nicolai Hähnle <[email protected]> | 2017-03-29 18:04:42 +0200 |
commit | 83e9de25f325f3fce78be864a9dd80e6274b8ca8 (patch) | |
tree | 3ffaec7441bc0e57c86c3273e7b79f8f3abf1462 /src/mesa/state_tracker | |
parent | d10172d52791b1fc0b791dd10c823668ecb7a4f8 (diff) |
st/mesa: EGLImageTarget* error handling
Stop trying to specify texture or renderbuffer objects for unsupported
EGL images. Generate the error codes specified in the OES_EGL_image
extension.
EGLImageTargetTexture2D and EGLImageTargetRenderbuffer would call
the pipe driver's create_surface callback without ever checking that
the given EGL image is actually compatible with the chosen target
texture or renderbuffer. This patch adds a call to the pipe driver's
is_format_supported callback and generates an INVALID_OPERATION error
for unsupported EGL images. If the EGL image handle does not describe
a valid EGL image, an INVALID_VALUE error is generated.
v2: fixed get_surface to actually use the usage and error parameters
Signed-off-by: Philipp Zabel <[email protected]>
Reviewed-by: Nicolai Hähnle <[email protected]>
Diffstat (limited to 'src/mesa/state_tracker')
-rw-r--r-- | src/mesa/state_tracker/st_cb_eglimage.c | 23 |
1 files changed, 19 insertions, 4 deletions
diff --git a/src/mesa/state_tracker/st_cb_eglimage.c b/src/mesa/state_tracker/st_cb_eglimage.c index e1fc9bff759..158a824e593 100644 --- a/src/mesa/state_tracker/st_cb_eglimage.c +++ b/src/mesa/state_tracker/st_cb_eglimage.c @@ -25,6 +25,7 @@ * Chia-I Wu <[email protected]> */ +#include "main/errors.h" #include "main/texobj.h" #include "main/teximage.h" #include "util/u_inlines.h" @@ -44,9 +45,11 @@ * FIXME: I think this should operate on resources, not surfaces */ static struct pipe_surface * -st_egl_image_get_surface(struct gl_context *ctx, GLeglImageOES image_handle) +st_egl_image_get_surface(struct gl_context *ctx, GLeglImageOES image_handle, + unsigned usage, const char *error) { struct st_context *st = st_context(ctx); + struct pipe_screen *screen = st->pipe->screen; struct st_manager *smapi = (struct st_manager *) st->iface.st_context_private; struct st_egl_image stimg; @@ -56,8 +59,18 @@ st_egl_image_get_surface(struct gl_context *ctx, GLeglImageOES image_handle) return NULL; memset(&stimg, 0, sizeof(stimg)); - if (!smapi->get_egl_image(smapi, (void *) image_handle, &stimg)) + if (!smapi->get_egl_image(smapi, (void *) image_handle, &stimg)) { + /* image_handle does not refer to a valid EGL image object */ + _mesa_error(ctx, GL_INVALID_VALUE, error); return NULL; + } + + if (!screen->is_format_supported(screen, stimg.format, PIPE_TEXTURE_2D, + stimg.texture->nr_samples, usage)) { + /* unable to specify a texture object using the specified EGL image */ + _mesa_error(ctx, GL_INVALID_OPERATION, error); + return NULL; + } u_surface_default_template(&surf_tmpl, stimg.texture); surf_tmpl.format = stimg.format; @@ -108,7 +121,8 @@ st_egl_image_target_renderbuffer_storage(struct gl_context *ctx, struct st_renderbuffer *strb = st_renderbuffer(rb); struct pipe_surface *ps; - ps = st_egl_image_get_surface(ctx, image_handle); + ps = st_egl_image_get_surface(ctx, image_handle, PIPE_BIND_RENDER_TARGET, + "glEGLImageTargetRenderbufferStorage"); if (ps) { strb->Base.Width = ps->width; strb->Base.Height = ps->height; @@ -192,7 +206,8 @@ st_egl_image_target_texture_2d(struct gl_context *ctx, GLenum target, { struct pipe_surface *ps; - ps = st_egl_image_get_surface(ctx, image_handle); + ps = st_egl_image_get_surface(ctx, image_handle, PIPE_BIND_SAMPLER_VIEW, + "glEGLImageTargetTexture2D"); if (ps) { st_bind_surface(ctx, target, texObj, texImage, ps); pipe_surface_reference(&ps, NULL); |