summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJeremy Allison <jra@samba.org>2006-03-09 22:49:38 +0000
committerGerald (Jerry) Carter <jerry@samba.org>2007-10-10 11:11:13 -0500
commitacf0c6fb66b4182a0e5e7610d5063649ca237311 (patch)
tree5d2b9787a885ab95a8d58098ad1f57115e1d8041
parentb19d871a1af88d58e41af7b5ac0fd74ba8a43fc7 (diff)
downloadsamba-acf0c6fb66b4182a0e5e7610d5063649ca237311.tar.gz
samba-acf0c6fb66b4182a0e5e7610d5063649ca237311.tar.bz2
samba-acf0c6fb66b4182a0e5e7610d5063649ca237311.zip
r14118: Fix coverity bug #24. Missing return statement meant
a possible NULL ptr deref. Jeremy. (This used to be commit 78ac3f9cbdabc1df9480f75fb3910a3a108a0e91)
-rw-r--r--source3/libads/ldap.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/source3/libads/ldap.c b/source3/libads/ldap.c
index babbabdd80..dd49c706f4 100644
--- a/source3/libads/ldap.c
+++ b/source3/libads/ldap.c
@@ -2692,7 +2692,7 @@ ADS_STATUS ads_workgroup_name(ADS_STRUCT *ads, TALLOC_CTX *mem_ctx, const char *
asprintf(&expr, "(&(objectclass=computer)(dnshostname=%s.%s))",
ads->config.ldap_server_name, ads->config.realm);
if (expr == NULL) {
- ADS_ERROR_NT(NT_STATUS_NO_MEMORY);
+ return ADS_ERROR_NT(NT_STATUS_NO_MEMORY);
}
rc = ads_search(ads, &res, expr, attrs);