diff options
author | Nicholas Bishop <nbishop@neverware.com> | 2016-09-08 15:55:02 -0400 |
---|---|---|
committer | Emil Velikov <emil.l.velikov@gmail.com> | 2016-09-27 13:37:21 +0100 |
commit | 2d05ba2ca0af336ceae3167432d7df9df3366b74 (patch) | |
tree | 1dc90142959190c9bb0ce477b41e0f9788954e04 /src/gbm/backends | |
parent | a5993022275c20061ac025d9adc26c5f9d02afee (diff) | |
download | external_mesa3d-2d05ba2ca0af336ceae3167432d7df9df3366b74.tar.gz external_mesa3d-2d05ba2ca0af336ceae3167432d7df9df3366b74.tar.bz2 external_mesa3d-2d05ba2ca0af336ceae3167432d7df9df3366b74.zip |
gbm: return appropriate error when queryImage() fails
Change gbm_dri_bo_get_fd to check the return value of queryImage and
return -1 (an invalid file descriptor) if an error occurs.
Update the comment for gbm_bo_get_fd to return -1, since (apart from the
above) we've already return -1 on error.
Cc: mesa-stable@lists.freedesktop.org
Signed-off-by: Nicholas Bishop <nbishop@neverware.com>
Reviewed-by: Eric Engestrom <eric.engestrom@imgtec.com> (v1)
[Emil Velikov: Split from larger patch, polish coding style, cc stable]
Signed-off-by: Emil Velikov <emil.velikov@collabora.com>
Diffstat (limited to 'src/gbm/backends')
-rw-r--r-- | src/gbm/backends/dri/gbm_dri.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/gbm/backends/dri/gbm_dri.c b/src/gbm/backends/dri/gbm_dri.c index 0ab67dad21..f863616ab0 100644 --- a/src/gbm/backends/dri/gbm_dri.c +++ b/src/gbm/backends/dri/gbm_dri.c @@ -591,7 +591,8 @@ gbm_dri_bo_get_fd(struct gbm_bo *_bo) if (bo->image == NULL) return -1; - dri->image->queryImage(bo->image, __DRI_IMAGE_ATTRIB_FD, &fd); + if (!dri->image->queryImage(bo->image, __DRI_IMAGE_ATTRIB_FD, &fd)) + return -1; return fd; } |