Re: [SSSD] [PATCH] IPA: Check connection validity explicitly

2015-10-07 Thread Sumit Bose
On Tue, Oct 06, 2015 at 12:54:41PM +0200, Jakub Hrozek wrote: > On Mon, Oct 05, 2015 at 04:46:55PM +0200, Jakub Hrozek wrote: > > On Mon, Oct 05, 2015 at 12:33:48PM +0200, Sumit Bose wrote: > > > On Mon, Oct 05, 2015 at 12:08:26PM +0200, Jakub Hrozek wrote: > > > > On Fri, Oct 02, 2015 at

Re: [SSSD] [PATCH] IPA: Check connection validity explicitly

2015-10-07 Thread Jakub Hrozek
On Wed, Oct 07, 2015 at 12:28:33PM +0200, Sumit Bose wrote: > ACK > > bye, > Sumit Thank you for the review: master: 309aa83d16b5919f727af04850bcd0799ba0962f afb21fd06690a0bec288a7970abf74ed2ea7dfdc sssd-1-13: 15a4b34ccfcfbcec2c9ba529d0113adf251abc16

Re: [SSSD] [PATCH] IPA: Check connection validity explicitly

2015-10-06 Thread Jakub Hrozek
On Mon, Oct 05, 2015 at 04:46:55PM +0200, Jakub Hrozek wrote: > On Mon, Oct 05, 2015 at 12:33:48PM +0200, Sumit Bose wrote: > > On Mon, Oct 05, 2015 at 12:08:26PM +0200, Jakub Hrozek wrote: > > > On Fri, Oct 02, 2015 at 10:32:27AM +0200, Jakub Hrozek wrote: > > > > On Thu, Oct 01, 2015 at

Re: [SSSD] [PATCH] IPA: Check connection validity explicitly

2015-10-05 Thread Sumit Bose
On Mon, Oct 05, 2015 at 12:08:26PM +0200, Jakub Hrozek wrote: > On Fri, Oct 02, 2015 at 10:32:27AM +0200, Jakub Hrozek wrote: > > On Thu, Oct 01, 2015 at 01:41:07PM +0200, Jakub Hrozek wrote: > > > I don't have a good idea how to reproduce except simulate the failure in > > > gdb, sorry... at

Re: [SSSD] [PATCH] IPA: Check connection validity explicitly

2015-10-05 Thread Jakub Hrozek
On Fri, Oct 02, 2015 at 10:32:27AM +0200, Jakub Hrozek wrote: > On Thu, Oct 01, 2015 at 01:41:07PM +0200, Jakub Hrozek wrote: > > I don't have a good idea how to reproduce except simulate the failure in > > gdb, sorry... at least I verified that after setting the context to NULL: > > (gdb)

Re: [SSSD] [PATCH] IPA: Check connection validity explicitly

2015-10-02 Thread Jakub Hrozek
On Thu, Oct 01, 2015 at 01:41:07PM +0200, Jakub Hrozek wrote: > I don't have a good idea how to reproduce except simulate the failure in > gdb, sorry... at least I verified that after setting the context to NULL: > (gdb) set clist[1] = 0 > the request runs to completion and SSSD doesn't

[SSSD] [PATCH] IPA: Check connection validity explicitly

2015-10-01 Thread Jakub Hrozek
Hi, the attached patch (hopefully) fixes https://fedorahosted.org/sssd/ticket/2810 (gdb) bt #0 0x7f9e8dde21d7 in ipa_get_ad_acct_send (ar=0x7f9e9cc2ff80, override_attrs=0x0, be_req=, ipa_ctx=, ev=0x7f9e9cbf0670, mem_ctx=) at src/providers/ipa/ipa_subdomains_id.c:648 #1