diff options
| author | Jeremy Allison <jra@samba.org> | 2006-03-09 19:03:31 +0000 |
|---|---|---|
| committer | Gerald (Jerry) Carter <jerry@samba.org> | 2007-10-10 11:11:11 -0500 |
| commit | 7f3ace5481e55ef845da28b9c0613a2ea0de0de4 (patch) | |
| tree | f3a976489cc0856113a2ea791aa0518ca1afc88f /source/passdb/secrets.c | |
| parent | 88dd4ab48127bb08fdeb0b5c236020e0b910f0d8 (diff) | |
| download | samba-7f3ace5481e55ef845da28b9c0613a2ea0de0de4.tar.gz samba-7f3ace5481e55ef845da28b9c0613a2ea0de0de4.tar.xz samba-7f3ace5481e55ef845da28b9c0613a2ea0de0de4.zip | |
r14088: Fix Coverity bug #20. Don't deref possible null.
Jeremy.
Diffstat (limited to 'source/passdb/secrets.c')
| -rw-r--r-- | source/passdb/secrets.c | 15 |
1 files changed, 9 insertions, 6 deletions
diff --git a/source/passdb/secrets.c b/source/passdb/secrets.c index bb4aad9fac6..927337d947a 100644 --- a/source/passdb/secrets.c +++ b/source/passdb/secrets.c @@ -177,13 +177,16 @@ BOOL secrets_fetch_domain_guid(const char *domain, struct uuid *guid) strupper_m(key); dyn_guid = (struct uuid *)secrets_fetch(key, &size); - if ((!dyn_guid) && (lp_server_role() == ROLE_DOMAIN_PDC)) { - smb_uuid_generate_random(&new_guid); - if (!secrets_store_domain_guid(domain, &new_guid)) - return False; - dyn_guid = (struct uuid *)secrets_fetch(key, &size); - if (dyn_guid == NULL) + if (!dyn_guid) { + if (lp_server_role() == ROLE_DOMAIN_PDC) { + smb_uuid_generate_random(&new_guid); + if (!secrets_store_domain_guid(domain, &new_guid)) + return False; + dyn_guid = (struct uuid *)secrets_fetch(key, &size); + } + if (dyn_guid == NULL) { return False; + } } if (size != sizeof(struct uuid)) { |