Fix crash in res_config_ldap.
We need to allocate enough room for 2 pointers, not 2 characters. (closes issue #16397) Reported by: bklang Patches: res_config_ldap.patch uploaded by applsplatz (license 949) Tested by: applsplatz git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@239473 65c4cc65-6c06-0410-ace0-fbb531ad65f3
This commit is contained in:
parent
fa0ef8031d
commit
a5688e483b
|
@ -1214,12 +1214,10 @@ static int update_ldap(const char *basedn, const char *table_name, const char *a
|
||||||
ldap_mods[0] = ast_calloc(1, sizeof(LDAPMod));
|
ldap_mods[0] = ast_calloc(1, sizeof(LDAPMod));
|
||||||
|
|
||||||
ldap_mods[0]->mod_op = LDAP_MOD_REPLACE;
|
ldap_mods[0]->mod_op = LDAP_MOD_REPLACE;
|
||||||
ldap_mods[0]->mod_type = ast_calloc(sizeof(char), strlen(newparam) + 1);
|
ldap_mods[0]->mod_type = ast_strdup(newparam);
|
||||||
strcpy(ldap_mods[0]->mod_type, newparam);
|
|
||||||
|
|
||||||
ldap_mods[0]->mod_values = ast_calloc(sizeof(char), 2);
|
ldap_mods[0]->mod_values = ast_calloc(sizeof(char *), 2);
|
||||||
ldap_mods[0]->mod_values[0] = ast_calloc(sizeof(char), strlen(newval) + 1);
|
ldap_mods[0]->mod_values[0] = ast_strdup(newval);
|
||||||
strcpy(ldap_mods[0]->mod_values[0], newval);
|
|
||||||
|
|
||||||
while ((newparam = va_arg(ap, const char *))) {
|
while ((newparam = va_arg(ap, const char *))) {
|
||||||
newparam = convert_attribute_name_to_ldap(table_config, newparam);
|
newparam = convert_attribute_name_to_ldap(table_config, newparam);
|
||||||
|
|
Loading…
Reference in New Issue