[asterisk-bugs] [JIRA] (ASTERISK-26580) [patch] Error during LDAP modify action when user unregisters

[Rusty Newton (JIRA)]

    [ https://issues.asterisk.org/jira/browse/ASTERISK-26580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=233743#comment-233743 ] 

Rusty Newton commented on ASTERISK-26580:
-----------------------------------------

[~aviator] , as you go through our documentation please let us know if there is any room for improvement.

We tend to error on the side of more detail rather than less detail, so our contribution process documentation can seem a little overwhelming.

If you ever have questions and want to ask them in realtime please feel free to drop by #asterisk-dev on irc.freenode.net , the developers hang out there and can help you out.



> [patch] Error during LDAP modify action when user unregisters
> -------------------------------------------------------------
>
>                 Key: ASTERISK-26580
>                 URL: https://issues.asterisk.org/jira/browse/ASTERISK-26580
>             Project: Asterisk
>          Issue Type: Bug
>      Security Level: None
>          Components: Resources/res_config_ldap
>    Affects Versions: 11.24.1
>         Environment: 3.16.0-4-amd64 #1 SMP Debian 3.16.36-1+deb8u1 (2016-09-03) x86_64 GNU/Linux
> slapd 2.4.40+dfsg-1+deb8u2
>            Reporter: Nicholas John Koch
>            Severity: Minor
>         Attachments: res_config_ldap.c-11.24.1.patch
>
>
> When an account has been looked up from the LDAP directory and activated some values within the LDAP object (AsteriskSIPUser) are updated.
> In this special case we are looking at _AstAccountFullContact_. When this account now unregisters the LDAP object will be updated.
> During this process some attributes get deleted. The bug is that the first attribute in the list is trying to be modified without a value but it should be deleted instead.
> Wiresharking the issue showed that the LDAP modification command has an empty value which is in terms of the LDAP syntax a fault.
> Hence this modification command must be a deletion.
> The patch checks if the first new value ist empty (mostly _AstAccountFullContact_) and therefore inserts a deletion command rather than a modification command. A similar
> procedure is already done some lines below but has not been implemented to the first name/value pair.



--
This message was sent by Atlassian JIRA
(v6.2#6252)