On Tue, Sep 04, 2012 at 06:34:23PM +0200, Jakub Hrozek wrote:
The sdap_id_release_conn_data function is already called inside
sdap_id_op_hook_conn_data() so the net result is a double free. I can't
think of a reason when you'd want to call release_conn_data but not
op_hook_conn_data..
This hunk was changed during the work on the primary servers. I verified
that primary/backup server reconnection logic still works and the
connection to the backup server is properly closed.
Jan, what was the reason behind the hunk?
ACK
bye,
Sumit