diff options
| author | Simon Ser <contact@emersion.fr> | 2019-01-29 22:00:40 +0000 |
|---|---|---|
| committer | Pekka Paalanen <pekka.paalanen@collabora.com> | 2019-01-30 15:49:04 +0200 |
| commit | 6afb152122f242d39c886bffa02c55d764e20fe8 (patch) | |
| tree | 9bf697825c05d44c0512513fc4b6271acbe83977 /src/connection.c | |
| parent | proto, server: Add internal server error message. (v2) (diff) | |
| download | wayland-6afb152122f242d39c886bffa02c55d764e20fe8.tar wayland-6afb152122f242d39c886bffa02c55d764e20fe8.tar.gz wayland-6afb152122f242d39c886bffa02c55d764e20fe8.tar.bz2 wayland-6afb152122f242d39c886bffa02c55d764e20fe8.tar.lz wayland-6afb152122f242d39c886bffa02c55d764e20fe8.tar.xz wayland-6afb152122f242d39c886bffa02c55d764e20fe8.tar.zst wayland-6afb152122f242d39c886bffa02c55d764e20fe8.zip | |
Print NULL strings as "nil" in wl_closure_print
Calling printf("%s", NULL) is undefined behaviour.
Signed-off-by: Simon Ser <contact@emersion.fr>
Reviewed-by: Pekka Paalanen <pekka.paalanen@collabora.com>
Diffstat (limited to 'src/connection.c')
| -rw-r--r-- | src/connection.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/src/connection.c b/src/connection.c index f965210..474c97b 100644 --- a/src/connection.c +++ b/src/connection.c @@ -1278,7 +1278,10 @@ wl_closure_print(struct wl_closure *closure, struct wl_object *target, int send) wl_fixed_to_double(closure->args[i].f)); break; case 's': - fprintf(stderr, "\"%s\"", closure->args[i].s); + if (closure->args[i].s) + fprintf(stderr, "\"%s\"", closure->args[i].s); + else + fprintf(stderr, "nil"); break; case 'o': if (closure->args[i].o) |