diff options
author | Carlos Garnacho <carlosg@gnome.org> | 2020-12-26 12:45:39 +0100 |
---|---|---|
committer | Carlos Garnacho <carlosg@gnome.org> | 2020-12-26 14:08:19 +0100 |
commit | 0b25059e853235dc46ffa17d1261d6e3f4d303c9 (patch) | |
tree | a99ce1a260408f602cee10d5c02f141e19cffd6c /tests/functional-tests | |
parent | ee0e1c012ffa04baf5fc1f0ec270187fc6328b26 (diff) | |
download | tracker-0b25059e853235dc46ffa17d1261d6e3f4d303c9.tar.gz |
libtracker-sparql: Fix out parameter in VAPI/Vala files
This argument was made nullable in commit 5ea2e77fa. While this sounds
what we want (the argument is nullable in C API and the vmethod). This
makes Vala introduce a double indirection for the out parameter, so
the argument ends up as "glong**" in the generated C code.
Of course, this breaks the interaction with C code and cursor class
definition expecting a "glong*" there, extra points for the C compiler
error being silenced. Undoing these specific changes from that commit
makes Vala and C code agree again about the levels of indirection.
This broke callers that rely on the string length being meaningful,
a glaring known broken case is the portal, as it breaks the
redirection/serialization of cursors to sandboxed apps.
Fixes: https://gitlab.gnome.org/GNOME/tracker/-/issues/282
Diffstat (limited to 'tests/functional-tests')
0 files changed, 0 insertions, 0 deletions