dec05eba/12to11
1
0
Fork 0
forked from 12to11/12to11
Code Pull requests Activity

Minor code pruning

Browse source 
* dmabuf.c (struct _BufferParams): Prune unused code.
* shm.c (CreateBuffer): Fix offset checks.
* xdg_toplevel.c (SetAppId): Don't allow setting the app ID on a
mapped surface.
This commit is contained in:
oldosfan 2022-09-15 07:04:20 +00:00
parent 007ba43dba
commit 266c3e5c1e
3 changed files with 11 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
dmabuf.c
View file

@ -102,9 +102,6 @@ struct _BufferParams
/* The width and height of the buffer that will be created. */
int width, height;
/* The buffer ID that will be used to create the buffer. */
uint32_t buffer_id;
/* The DRM format. */
uint32_t drm_format;
};

10
shm.c
View file

@ -263,17 +263,18 @@ CreateBuffer (struct wl_client *client, struct wl_resource *resource,
if (!depth)
{
wl_resource_post_error (resource, WL_SHM_ERROR_INVALID_FORMAT,
"The specified format is not supported");
"the specified format is not supported");
return;
}
pool = wl_resource_get_user_data (resource);
if (pool->size < offset || stride != width * 4
|| offset + stride * height > pool->size)
|| offset + stride * height > pool->size
|| offset < 0)
{
wl_resource_post_error (resource, WL_SHM_ERROR_INVALID_STRIDE,
"Invalid offset or stride, or pool too small");
"invalid offset or stride, or pool too small");
return;
}
@ -288,7 +289,8 @@ CreateBuffer (struct wl_client *client, struct wl_resource *resource,
{
/* X doesn't support smaller drawables. */
wl_resource_post_error (resource, WL_SHM_ERROR_INVALID_STRIDE,
"Invalid size, X server does not support zero-width drawables");
"invalid size, this server does not support"
" zero-width drawables");
return;
}

5
xdg_toplevel.c
View file

@ -1622,6 +1622,11 @@ SetAppId (struct wl_client *client, struct wl_resource *resource,
if (!toplevel->role)
return;
if (toplevel->state & StateIsMapped)
/* The toplevel is already mapped. Setting class hints in this
situation is not possible under X. */
return;
class_hints.res_name = (char *) app_id;
class_hints.res_class = (char *) app_id;