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

[noreply at bugs.digium.com]

The following issue has been SUBMITTED. 
====================================================================== 
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:31 CDT
====================================================================== 
Summary:                    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:31  tjormola       Asterisk Version          => 1.6.0-beta5     
03-17-08 11:31  tjormola       SVN Branch (only for SVN checkou => N/A          
  
======================================================================