diff options
| author | file <file@f38db490-d61c-443f-a65b-d21fe96a405b> | 2008-04-21 14:35:53 +0000 |
|---|---|---|
| committer | file <file@f38db490-d61c-443f-a65b-d21fe96a405b> | 2008-04-21 14:35:53 +0000 |
| commit | 86d9c6a80221de4fe60333346b6dbdc2635ccd6f (patch) | |
| tree | 5a0b84b3f5afabd41ef441afc1a7d3ab109fd1ec /res/res_config_ldap.c | |
| parent | 51a9df4df010ccf7332b29bc936b661eb7fd6a01 (diff) | |
Merged revisions 114320 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk
........
r114320 | file | 2008-04-21 11:34:06 -0300 (Mon, 21 Apr 2008) | 6 lines
Only print out the error message if ldap_modify_ext_s actually returns an error, and not success.
(closes issue #12438)
Reported by: gservat
Patches:
res_config_ldap.c-patch-code uploaded by gservat (license 466)
........
git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.6.0@114321 f38db490-d61c-443f-a65b-d21fe96a405b
Diffstat (limited to 'res/res_config_ldap.c')
| -rw-r--r-- | res/res_config_ldap.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/res/res_config_ldap.c b/res/res_config_ldap.c index 376f9aff8..cb1bd78cb 100644 --- a/res/res_config_ldap.c +++ b/res/res_config_ldap.c @@ -1285,7 +1285,7 @@ static int update_ldap(const char *basedn, const char *table_name, const char *a for (i = 0; ldap_entry; i++) { dn = ldap_get_dn(ldapConn, ldap_entry); - if (!(error = ldap_modify_ext_s(ldapConn, dn, ldap_mods, NULL, NULL))) + if ((error = ldap_modify_ext_s(ldapConn, dn, ldap_mods, NULL, NULL)) != LDAP_SUCCESS) ast_log(LOG_ERROR, "Couldn't modify dn:%s because %s", dn, ldap_err2string(error)); ldap_entry = ldap_next_entry(ldapConn, ldap_entry); |