diff options
author | Alex Richardson <Alexander.Richardson@cl.cam.ac.uk> | 2021-03-19 11:24:05 +0000 |
---|---|---|
committer | Alex Richardson <Alexander.Richardson@cl.cam.ac.uk> | 2021-03-22 14:05:11 +0000 |
commit | 1349d3d15bf7d100d0eae52d976a121d2dcf32ce (patch) | |
tree | 81b6d977e6a674cc54c91e369b7d18555a8522e9 | |
parent | 3bda3d1b4729c8ee7c533520a199611cb841bc8f (diff) | |
download | wayland-1349d3d15bf7d100d0eae52d976a121d2dcf32ce.tar.gz |
Use MAP_FAILED instead of (void *) -1
While MAP_FAILED is generally defined to that value, we should not be
relying on implementation details of system headers.
Signed-off-by: Alex Richardson <Alexander.Richardson@cl.cam.ac.uk>
Reviewed-by: Simon Ser <contact@emersion.fr>
-rw-r--r-- | src/wayland-shm.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/src/wayland-shm.c b/src/wayland-shm.c index b85e5a7..7320bc9 100644 --- a/src/wayland-shm.c +++ b/src/wayland-shm.c @@ -511,10 +511,8 @@ sigbus_handler(int signum, siginfo_t *info, void *context) sigbus_data->fallback_mapping_used = 1; /* This should replace the previous mapping */ - if (mmap(pool->data, pool->size, - PROT_READ | PROT_WRITE, - MAP_PRIVATE | MAP_FIXED | MAP_ANONYMOUS, - 0, 0) == (void *) -1) { + if (mmap(pool->data, pool->size, PROT_READ | PROT_WRITE, + MAP_PRIVATE | MAP_FIXED | MAP_ANONYMOUS, 0, 0) == MAP_FAILED) { reraise_sigbus(); return; } |