diff options
author | Simon McVittie <smcv@debian.org> | 2023-02-07 11:38:12 +0000 |
---|---|---|
committer | Simon McVittie <smcv@debian.org> | 2023-02-07 11:54:22 +0000 |
commit | a1a330b8720e4bc1c2154f120196372627dc7b2a (patch) | |
tree | 66a6e4aa4afd369012f655c96856897138a999c2 /src/util.h | |
parent | c7c8945460324d0dc879fc452d8ee12dfc61d184 (diff) | |
download | accountsservice-a1a330b8720e4bc1c2154f120196372627dc7b2a.tar.gz |
user: Use correct format strings to print accounts_user_get_uid()
The Uid property is defined in the D-Bus introspection XML to be a
64-bit unsigned integer, so we need to treat it as such when using
varargs. Otherwise, architectures that do not align arguments on the
stack at 64-bit boundaries can parse the stack incorrectly, resulting
in a crash.
For whatever obscure ABI reason, among Debian's supported architectures
this only showed up as a segmentation fault on 32-bit ARM (specifically
ARMv5 softfloat and ARMv7 hardfloat), and not on (for example) i386.
Resolves: https://gitlab.freedesktop.org/accountsservice/accountsservice/-/issues/109
Signed-off-by: Simon McVittie <smcv@debian.org>
Diffstat (limited to 'src/util.h')
0 files changed, 0 insertions, 0 deletions