diff options
| author | Ian Douglas Scott <idscott@system76.com> | 2022-07-12 09:12:33 -0700 |
|---|---|---|
| committer | Ian Douglas Scott <idscott@system76.com> | 2022-07-14 08:10:38 -0700 |
| commit | 13b05c9ed1570765923a28b277ca385001c2b5c6 (patch) | |
| tree | e24e90ad5257b49943c81fb8d5560742a0a8d44b /src/connection.c | |
| parent | Add release.sh (diff) | |
| download | wayland-13b05c9ed1570765923a28b277ca385001c2b5c6.tar wayland-13b05c9ed1570765923a28b277ca385001c2b5c6.tar.gz wayland-13b05c9ed1570765923a28b277ca385001c2b5c6.tar.bz2 wayland-13b05c9ed1570765923a28b277ca385001c2b5c6.tar.lz wayland-13b05c9ed1570765923a28b277ca385001c2b5c6.tar.xz wayland-13b05c9ed1570765923a28b277ca385001c2b5c6.tar.zst wayland-13b05c9ed1570765923a28b277ca385001c2b5c6.zip | |
Do not allow nullable arrays, which were not correctly implemented
Nullable arrays, which are not used anywhere, were marshalled the same
way as an empty non-null array. The demarshalling logic did not
recognize anything as a null array. Given this, it seems better to just
explicitly not support it.
Fixes https://gitlab.freedesktop.org/wayland/wayland/-/issues/306.
Signed-off-by: Ian Douglas Scott <idscott@system76.com>
Diffstat (limited to 'src/connection.c')
| -rw-r--r-- | src/connection.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/connection.c b/src/connection.c index bf97676..594f2e9 100644 --- a/src/connection.c +++ b/src/connection.c @@ -636,7 +636,7 @@ wl_closure_marshal(struct wl_object *sender, uint32_t opcode, closure->args[i].n = object ? object->id : 0; break; case 'a': - if (!arg.nullable && args[i].a == NULL) + if (args[i].a == NULL) goto err_null; break; case 'h': |