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.
System and Service Manager
Details
Most documentation is available on systemd's web site.
Assorted, older, general information about systemd can be found in the systemd Wiki.
Information about build requirements is provided in the README file.
Consult our NEWS file for information about what's new in the most recent systemd versions.
Please see the Code Map for information about this repository's layout and content.
Please see the Hacking guide for information on how to hack on systemd and test your modifications.
Please see our Contribution Guidelines for more information about filing GitHub Issues and posting GitHub Pull Requests.
When preparing patches for systemd, please follow our Coding Style Guidelines.
If you are looking for support, please contact our mailing list or join our IRC channel.
Stable branches with backported patches are available in the stable repo.
