summaryrefslogtreecommitdiff
path: root/source3/nsswitch/winbindd_pam.c
diff options
context:
space:
mode:
authorJeremy Allison <jra@samba.org>2006-03-13 01:08:27 +0000
committerGerald (Jerry) Carter <jerry@samba.org>2007-10-10 11:15:22 -0500
commit478cd9dd53550aa33a9046d35bc0784ad75a4a15 (patch)
tree554a881c6cb3460b545afdd16ad9091ff465ec72 /source3/nsswitch/winbindd_pam.c
parent3b58c82a52c3a1298f867728adb16b9791479063 (diff)
downloadsamba-478cd9dd53550aa33a9046d35bc0784ad75a4a15.tar.gz
samba-478cd9dd53550aa33a9046d35bc0784ad75a4a15.tar.bz2
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/nsswitch/winbindd_pam.c')
-rw-r--r--source3/nsswitch/winbindd_pam.c7
1 files changed, 7 insertions, 0 deletions
diff --git a/source3/nsswitch/winbindd_pam.c b/source3/nsswitch/winbindd_pam.c
index fdc6e07119..becfc37c44 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);