diff options
author | ikerexxe <ipedrosa@redhat.com> | 2020-06-29 11:34:35 +0200 |
---|---|---|
committer | Dmitry V. Levin <ldv@altlinux.org> | 2020-07-01 09:54:22 +0000 |
commit | a6a1b9f788a79b2a09827c72a755f471c2e05100 (patch) | |
tree | c4eb1687acd17160667ec874f4e88c701e0c79db /modules/pam_namespace | |
parent | 655b5e3cf32cb2bd6606cb8ab696b8f00f87051e (diff) | |
download | pam-a6a1b9f788a79b2a09827c72a755f471c2e05100.tar.gz pam-a6a1b9f788a79b2a09827c72a755f471c2e05100.tar.bz2 pam-a6a1b9f788a79b2a09827c72a755f471c2e05100.zip |
pam_rootok: fix use of va_list
CPPCHECK_WARNING (CWE-843):
error[va_end_missing]: va_list 'ap' was opened but not closed by
va_end().
[ldv: According to POSIX documentation, each invocation of va_start()
must be matched by a corresponding invocation of va_end().
According to the GNU libc documentation, "with most C compilers,
calling 'va_end' does nothing. This is always true in the GNU C
compiler. But you might as well call 'va_end' just in case your
program is someday compiled with a peculiar compiler."
The main reason for applying this change is to pacify static analysis
tools like cppcheck that insist on strict POSIX conformance in this
respect.]
Diffstat (limited to 'modules/pam_namespace')
0 files changed, 0 insertions, 0 deletions