diff options
| author | Lukas Slebodnik <lslebodn@redhat.com> | 2016-04-13 21:44:37 +0200 |
|---|---|---|
| committer | Lukas Slebodnik <lslebodn@redhat.com> | 2016-04-18 18:15:38 +0200 |
| commit | aeb3cef10158a1c621d58c6d982e53569e40c6c0 (patch) | |
| tree | ffdcc25997395ca4af2b883fddefc293d2599b94 /src/python | |
| parent | 60e54ce78177b72ae82b8986e37dc4d09fce5681 (diff) | |
| download | sssd-aeb3cef10158a1c621d58c6d982e53569e40c6c0.tar.gz sssd-aeb3cef10158a1c621d58c6d982e53569e40c6c0.tar.xz sssd-aeb3cef10158a1c621d58c6d982e53569e40c6c0.zip | |
SYSDB_SUDO: Remove useless test
The function sysdb_search_custom cannot return EOK
and together set output argument count to zero.
This case is already handled in function sysdb_search_entry
which is used inside sysdb_search_custom.
Such useless test can just cause read of unitialized variable
in case of other errors returned from sysdb_search_custom.
Error: UNINIT (CWE-457): [#def1]
sssd-1.13.4/src/db/sysdb_sudo.c:678:
var_decl: Declaring variable "count" without initializer.
sssd-1.13.4/src/db/sysdb_sudo.c:698:
uninit_use: Using uninitialized value "count".
# 696| SUDORULE_SUBDIR, attrs,
# 697| &count, &msgs);
# 698|-> if (ret == ENOENT || count == 0) {
# 699| DEBUG(SSSDBG_TRACE_FUNC, "No rules matched\n");
# 700| ret = EOK;
Reviewed-by: Pavel Březina <pbrezina@redhat.com>
Diffstat (limited to 'src/python')
0 files changed, 0 insertions, 0 deletions
