diff options
| author | Simon Ser <contact@emersion.fr> | 2020-04-14 18:44:40 +0200 |
|---|---|---|
| committer | Drew DeVault <sir@cmpwn.com> | 2020-04-22 22:14:44 +0200 |
| commit | 01d45062531f8da120343a9ce8d75b12eafcc8da (patch) | |
| tree | 1e4c0807c61ca34c2f119e13663903d77cfc7218 /render | |
| parent | 455a9bd0efe9e5b10cb2e9a962b5f62b2db474b0 (diff) | |
render/drm_format_set: disallow DRM_FORMAT_INVALID
It doesn't make sense to add DRM_FORMAT_INVALID to a format set. Adding
an assertion allows us to safely query the format set with
DRM_FORMAT_INVALID. See [1].
[1]: https://github.com/swaywm/wlroots/pull/2021#discussion_r385839668
Diffstat (limited to 'render')
| -rw-r--r-- | render/drm_format_set.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/render/drm_format_set.c b/render/drm_format_set.c index b09a68a4..cbcc0875 100644 --- a/render/drm_format_set.c +++ b/render/drm_format_set.c @@ -1,3 +1,4 @@ +#include <assert.h> #include <drm_fourcc.h> #include <stdbool.h> #include <stdint.h> @@ -57,6 +58,7 @@ bool wlr_drm_format_set_has(const struct wlr_drm_format_set *set, bool wlr_drm_format_set_add(struct wlr_drm_format_set *set, uint32_t format, uint64_t modifier) { + assert(format != DRM_FORMAT_INVALID); struct wlr_drm_format **ptr = format_set_get_ref(set, format); if (ptr) { |