On 05/07/2015 06:04 PM, Martin Babinsky wrote:
On 04/24/2015 01:31 PM, Petr Vobornik wrote:
On 04/24/2015 07:19 AM, Jan Cholasta wrote:
Dne 23.4.2015 v 22:18 Nathaniel McCallum napsal(a):
On Thu, 2015-04-23 at 14:12 +0200, Petr Vobornik wrote:
On 04/23/2015 12:24 PM, Petr Vobornik wrote:
If unbind was called when disconnected it raised:
    AttributeError: 'NoneType' object has no attribute 'unbind_s'

AttributeError is not a public error and therefore it prevented
ldap2.destroy_connection() to be called multiple times.



After discussion with Honza, the approach was changed.

Also I've added patch which removes unnecessary incorrect code which
revealed the regression.

Additional testing shows that these patches actually don't fix the
original issue of #4991. See
https://fedorahosted.org/freeipa/ticket/4991#comment:4

0823 - ACK
0824 - ACK

Nathaniel


I would prefer if the connection was closed manually in patch 824, IMO
it is a good practice to release resources once you are done with them
just in time, and I don't think you can always trust the automatic
disconnect at the end of request.


Changed (also in user-status command).



ACK


823-2: Pushed to master: 7d10547ae3098b96762846ff36e813042a503d59
824-2: Pushed to master: 375eb7583334f231c27420c80b1b074e0cc554fe
--
Petr Vobornik

--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to