diff options
author | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2021-07-09 14:38:23 +0200 |
---|---|---|
committer | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2021-07-09 15:12:18 +0200 |
commit | 1421705d9a8233a09f6bd7c25436ed48e640a368 (patch) | |
tree | e25d0ec27efd49e1d49595ad652d5c2900122eff /src/creds | |
parent | 2744c7bb0176dc6b86a69acd4c449ea9e269e097 (diff) | |
download | systemd-1421705d9a8233a09f6bd7c25436ed48e640a368.tar.gz |
core: drop unnecessary initialization
cunescape() sets output on success, so initialization is not necessary. There
was no comment, but I think they may have been added because the compiler
wasn't convinced that the return value is non-negative on success. It could
have been confused by the int return type on escape*(), which was changed by
the one of preceeding commits to ssize_t, or by the length calculation, so add
an assert to help the compiler.
For some reason coverity thinks the output can be leaked here (CID #1458111).
I don't see how.
Diffstat (limited to 'src/creds')
0 files changed, 0 insertions, 0 deletions