diff options
| author | Jeremy Allison <jra@samba.org> | 2006-03-09 22:49:38 +0000 |
|---|---|---|
| committer | Gerald (Jerry) Carter <jerry@samba.org> | 2007-10-10 11:11:13 -0500 |
| commit | acf0c6fb66b4182a0e5e7610d5063649ca237311 (patch) | |
| tree | 5d2b9787a885ab95a8d58098ad1f57115e1d8041 /source3/libads | |
| parent | b19d871a1af88d58e41af7b5ac0fd74ba8a43fc7 (diff) | |
| download | samba-acf0c6fb66b4182a0e5e7610d5063649ca237311.tar.gz | |
r14118: Fix coverity bug #24. Missing return statement meant
a possible NULL ptr deref.
Jeremy.
(This used to be commit 78ac3f9cbdabc1df9480f75fb3910a3a108a0e91)
Diffstat (limited to 'source3/libads')
| -rw-r--r-- | source3/libads/ldap.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/source3/libads/ldap.c b/source3/libads/ldap.c index babbabdd80e..dd49c706f46 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); |