[asterisk-bugs] [Asterisk 0012234]: [patch] A couple of trivial patches for res_config_ldap

[noreply at bugs.digium.com]

The following issue has been UPDATED. 
====================================================================== 
http://bugs.digium.com/view.php?id=12234 
====================================================================== 
Reported By:                tjormola
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   12234
Category:                   Resources/res_config_ldap
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     new
Asterisk Version:           1.6.0-beta5 
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             03-17-2008 11:31 CDT
Last Modified:              03-17-2008 11:32 CDT
====================================================================== 
Summary:                    [patch] A couple of trivial patches for
res_config_ldap
Description: 
This patch fixes two simple bugs in res_config_ldap.c. In function
realtime_ldap_base_ap(), due to missing brackets in an if block, the
execution will always stop to the return NULL statement. In fucntcion
update_ldap(), error checking after call to ldap_modify_ext_s() is not
good. It's better to compare the returned value to LDAP_SUCCESS constant.
Also the formatting of the error message is missing "\n".


====================================================================== 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
03-17-08 11:32  file           Summary                  A couple of trivial
patches for res_config_ldap => [patch] A couple of trivial patches for
res_config_ldap
======================================================================