[SSSD] [sssd PR#517][comment] Fix two memory leaks in the AD provider

URL: https://github.com/SSSD/sssd/pull/517 Title: #517: Fix two memory leaks in the AD provider jhrozek commented: """ * master: * e6ad16e05f42a1678a8c6cd14eb54ca75b8d775e

[SSSD] [sssd PR#517][comment] Fix two memory leaks in the AD provider

URL: https://github.com/SSSD/sssd/pull/517 Title: #517: Fix two memory leaks in the AD provider jhrozek commented: """ The internal CI is still down, but honestly I see no reason to wait for it, there's nothing that could work here but wouldn't work on other distros. ACK """ See the full

[SSSD] [sssd PR#517][comment] Fix two memory leaks in the AD provider

URL: https://github.com/SSSD/sssd/pull/517 Title: #517: Fix two memory leaks in the AD provider sumit-bose commented: """ Sorry, there were some unrelated changes in the last commit. """ See the full comment at https://github.com/SSSD/sssd/pull/517#issuecomment-366748803

[SSSD] [sssd PR#517][comment] Fix two memory leaks in the AD provider

URL: https://github.com/SSSD/sssd/pull/517 Title: #517: Fix two memory leaks in the AD provider sumit-bose commented: """ oopsy, fixed version pushed. """ See the full comment at https://github.com/SSSD/sssd/pull/517#issuecomment-366703772 ___

[SSSD] [sssd PR#517][comment] Fix two memory leaks in the AD provider

URL: https://github.com/SSSD/sssd/pull/517 Title: #517: Fix two memory leaks in the AD provider jhrozek commented: """ I think the patches look good in general, but the tests don't compile at the moment: ``` /home/remote/jhrozek/devel/sssd/src/tests/cmocka/test_ad_common.c: In function

[SSSD] [sssd PR#517][comment] Fix two memory leaks in the AD provider

URL: https://github.com/SSSD/sssd/pull/517 Title: #517: Fix two memory leaks in the AD provider lslebodn commented: """ retest this please """ See the full comment at https://github.com/SSSD/sssd/pull/517#issuecomment-366440086 ___ sssd-devel