compositor-drm: Use signed int for width/height

This makes it sign-compatible with weston_output->{width,height}.

Differential Revision: https://phabricator.freedesktop.org/D1486

Signed-off-by: Daniel Stone <daniels@collabora.com>
Reviewed-by: Quentin Glidic <sardemff7+git@sardemff7.net>
Reviewed-by: Armin Krezović <krezovic.armin@gmail.com>
This commit is contained in:
Daniel Stone 2016-11-14 17:43:57 +00:00
parent 4e5eceb075
commit f214fdca5a

View file

@ -102,8 +102,8 @@ struct drm_backend {
* due to out of bounds dimensions, and then mistakenly set * due to out of bounds dimensions, and then mistakenly set
* sprites_are_broken: * sprites_are_broken:
*/ */
uint32_t min_width, max_width; int min_width, max_width;
uint32_t min_height, max_height; int min_height, max_height;
int no_addfb2; int no_addfb2;
struct wl_list sprite_list; struct wl_list sprite_list;
@ -253,7 +253,7 @@ drm_fb_destroy_callback(struct gbm_bo *bo, void *data)
} }
static struct drm_fb * static struct drm_fb *
drm_fb_create_dumb(struct drm_backend *b, unsigned width, unsigned height, drm_fb_create_dumb(struct drm_backend *b, int width, int height,
uint32_t format) uint32_t format)
{ {
struct drm_fb *fb; struct drm_fb *fb;
@ -371,7 +371,7 @@ drm_fb_get_from_bo(struct gbm_bo *bo,
struct drm_backend *backend, uint32_t format) struct drm_backend *backend, uint32_t format)
{ {
struct drm_fb *fb = gbm_bo_get_user_data(bo); struct drm_fb *fb = gbm_bo_get_user_data(bo);
uint32_t width, height; int width, height;
uint32_t handles[4] = { 0 }, pitches[4] = { 0 }, offsets[4] = { 0 }; uint32_t handles[4] = { 0 }, pitches[4] = { 0 }, offsets[4] = { 0 };
int ret; int ret;
@ -391,7 +391,8 @@ drm_fb_get_from_bo(struct gbm_bo *bo,
fb->size = fb->stride * height; fb->size = fb->stride * height;
fb->fd = backend->drm.fd; fb->fd = backend->drm.fd;
if (backend->min_width > width || width > backend->max_width || if (backend->min_width > width ||
width > backend->max_width ||
backend->min_height > height || backend->min_height > height ||
height > backend->max_height) { height > backend->max_height) {
weston_log("bo geometry out of bounds\n"); weston_log("bo geometry out of bounds\n");