diff options
| author | meltq <tejasvipin76@gmail.com> | 2024-06-30 22:36:11 +0530 |
|---|---|---|
| committer | Tejas Vipin <tejasvipin76@gmail.com> | 2024-07-11 17:44:04 +0000 |
| commit | 0cecde304f4f78b439d2edaaab6d9b543b30a5c0 (patch) | |
| tree | 6619eae3a6e4106550b90995fa45d23b3fba3b15 /src/wayland-server.c | |
| parent | tests: add enum bitfield test (diff) | |
| download | wayland-0cecde304f4f78b439d2edaaab6d9b543b30a5c0.tar wayland-0cecde304f4f78b439d2edaaab6d9b543b30a5c0.tar.gz wayland-0cecde304f4f78b439d2edaaab6d9b543b30a5c0.tar.bz2 wayland-0cecde304f4f78b439d2edaaab6d9b543b30a5c0.tar.lz wayland-0cecde304f4f78b439d2edaaab6d9b543b30a5c0.tar.xz wayland-0cecde304f4f78b439d2edaaab6d9b543b30a5c0.tar.zst wayland-0cecde304f4f78b439d2edaaab6d9b543b30a5c0.zip | |
src: switch asserts to wl_abort
assert()s can be compiled away by #defining NDEBUG. Some build systems
do this. Using wl_abort gives a human readable error message and it
isn't compiled away. This commit closes issue #230.
Signed-off-by: meltq <tejasvipin76@gmail.com>
Diffstat (limited to 'src/wayland-server.c')
| -rw-r--r-- | src/wayland-server.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/wayland-server.c b/src/wayland-server.c index 2e18563..95f69e7 100644 --- a/src/wayland-server.c +++ b/src/wayland-server.c @@ -1714,7 +1714,8 @@ wl_socket_init_for_display_name(struct wl_socket *s, const char *name) name_size = snprintf(s->addr.sun_path, sizeof s->addr.sun_path, "%s%s%s", runtime_dir, separator, name) + 1; - assert(name_size > 0); + if (!(name_size > 0)) + wl_abort("Error assigning path name for socket address\n"); if (name_size > (int)sizeof s->addr.sun_path) { wl_log("error: socket path \"%s%s%s\" plus null terminator" " exceeds 108 bytes\n", runtime_dir, separator, name); |