[Asterisk-code-review] res config ldap: Fix erroneous LDAP MOD REPLACE in LDAP modify (asterisk[13])

George Joseph asteriskteam at digium.com
Sun Feb 19 12:08:58 CST 2017


George Joseph has posted comments on this change. ( https://gerrit.asterisk.org/4994 )

Change subject: res_config_ldap: Fix erroneous LDAP_MOD_REPLACE in LDAP modify
......................................................................


Patch Set 4:

Lots of changes in this patch making it a little hard to review.  Would it be possible to split the patch up so the cosmetic changes to the messages and the changes to malloc/free are in separate reviews leaving the actual functional fixes in this one?

-- 
To view, visit https://gerrit.asterisk.org/4994
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I0a534a60c7a5d79876bc75eff6588ce7db0d8c2e
Gerrit-PatchSet: 4
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Owner: Sean Bright <sean.bright at gmail.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-HasComments: No



More information about the asterisk-code-review mailing list