Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/15570 )
Change subject: IMPALA-2563: Support LDAP search bind operations ...................................................................... Patch Set 4: (2 comments) http://gerrit.cloudera.org:8080/#/c/15570/3/be/src/util/ldap-util.cc File be/src/util/ldap-util.cc: http://gerrit.cloudera.org:8080/#/c/15570/3/be/src/util/ldap-util.cc@121 PS3, Line 121: user_filter_ = Split(user_filter, ","); > It's best not to mix that fix into this patch. It seems like a low priority I filed IMPALA-9587 so at least we have a record of it. http://gerrit.cloudera.org:8080/#/c/15570/4/be/src/util/ldap-util.cc File be/src/util/ldap-util.cc: http://gerrit.cloudera.org:8080/#/c/15570/4/be/src/util/ldap-util.cc@266 PS4, Line 266: ldap_msgfree(result); > How did you find what I assume is a memory leak here? I happen to get lucky and notice this while looking through LDAP documentation. Since discovering this, I ran some successful/failed authentications against a cluster in a loop overnight to check that there aren't any other memory leaks here. -- To view, visit http://gerrit.cloudera.org:8080/15570 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I7502a96e9a3c16faa67c03ffac54df2bdebbca8c Gerrit-Change-Number: 15570 Gerrit-PatchSet: 4 Gerrit-Owner: Thomas Tauber-Marshall <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Thomas Tauber-Marshall <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]> Gerrit-Comment-Date: Tue, 31 Mar 2020 20:20:35 +0000 Gerrit-HasComments: Yes
