| Summary: | ASTERISK-11660: [patch] A couple of trivial patches for res_config_ldap | ||
| Reporter: | Tuomas Jormola (tjormola) | Labels: | |
| Date Opened: | 2008-03-17 11:31:32 | Date Closed: | 2008-03-19 15:10:13 |
| Priority: | Minor | Regression? | No |
| Status: | Closed/Complete | Components: | Resources/res_config_ldap |
| Versions: | Frequency of Occurrence | ||
| Related Issues: | |||
| Environment: | Attachments: | ( 0) fix-ldap-bugs.diff | |
| 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". | ||
| Comments: | |||