diff options
| author | Jeremy Allison <jra@samba.org> | 2006-03-13 01:08:27 +0000 |
|---|---|---|
| committer | Gerald (Jerry) Carter <jerry@samba.org> | 2007-10-10 11:15:22 -0500 |
| commit | 478cd9dd53550aa33a9046d35bc0784ad75a4a15 (patch) | |
| tree | 554a881c6cb3460b545afdd16ad9091ff465ec72 /source3 | |
| parent | 3b58c82a52c3a1298f867728adb16b9791479063 (diff) | |
| download | samba-478cd9dd53550aa33a9046d35bc0784ad75a4a15.tar.gz samba-478cd9dd53550aa33a9046d35bc0784ad75a4a15.tar.xz samba-478cd9dd53550aa33a9046d35bc0784ad75a4a15.zip | |
r14275: Shut-up coverity false positive (bug #199) by making an assertion
explicit.
Jeremy.
(This used to be commit aeae20a8d9f3658acb8edd373eb601bdf7eab98b)
Diffstat (limited to 'source3')
| -rw-r--r-- | source3/nsswitch/winbindd_pam.c | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/source3/nsswitch/winbindd_pam.c b/source3/nsswitch/winbindd_pam.c index fdc6e071190..becfc37c44b 100644 --- a/source3/nsswitch/winbindd_pam.c +++ b/source3/nsswitch/winbindd_pam.c @@ -1137,6 +1137,13 @@ process_result: DOM_SID user_sid; + /* In all codepaths were result == NT_STATUS_OK info3 must have + been initialized. */ + if (!info3) { + result = NT_STATUS_INTERNAL_ERROR; + goto done; + } + netsamlogon_cache_store(name_user, info3); wcache_invalidate_samlogon(find_domain_from_name(name_domain), info3); |